Skip to content

Commit f77d617

Browse files
jaymzhfacebook-github-bot
authored andcommitted
Fix fb_init_sample (facebook#309)
Summary: Commit 68f51c1 accidentally removed a bunch of stuff from Ubuntu and Debian. This fixes the logic to put it back. That fixes current test failures. Signed-off-by: Phil Dibowitz <phil@ipom.com> Pull Request resolved: facebook#309 Differential Revision: D73619852 fbshipit-source-id: 3b8bdc9ed4465aa8c9c28ad5cba1512d09d6211c
1 parent fb437df commit f77d617

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

cookbooks/fb_init_sample/recipes/default.rb

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -134,18 +134,20 @@
134134
include_recipe 'fb_init_sample::firstboot'
135135
end
136136

137-
if node.centos?
138-
# https://bugzilla.redhat.com/show_bug.cgi?id=2337139
139-
if node.centos_max_version?(9)
140-
include_recipe 'fb_apcupsd'
141-
end
142-
# Turn off dnsmasq as it doesn't play well with travis
143-
node.default['fb_dnsmasq']['enable'] = false
144-
include_recipe 'fb_dnsmasq'
145-
# https://bugzilla.redhat.com/show_bug.cgi?id=2345748
146-
if node.centos_max_version?(9)
147-
include_recipe 'fb_collectd'
148-
end
137+
# APCUPSD is not yet in C10+
138+
# https://bugzilla.redhat.com/show_bug.cgi?id=2337139
139+
if !node.rhel_family? || node.el_max_version?(9)
140+
include_recipe 'fb_apcupsd'
141+
end
142+
143+
# Turn off dnsmasq as it doesn't play well with travis
144+
node.default['fb_dnsmasq']['enable'] = false
145+
include_recipe 'fb_dnsmasq'
146+
147+
# Collected is not yet in C10+
148+
# https://bugzilla.redhat.com/show_bug.cgi?id=2345748
149+
if !node.rhel_family? || node.el_max_version?(9)
150+
include_recipe 'fb_collectd'
149151
end
150152
include_recipe 'fb_rsync::server'
151153
if node.centos?

0 commit comments

Comments
 (0)