Skip to content

Commit 68f51c1

Browse files
davide125facebook-github-bot
authored andcommitted
fb_init_sample: minor updates
Summary: Add fb_dnf, clean up some C6 stuff Differential Revision: D73541337 fbshipit-source-id: b2669426383f7c42f19285a38a5c8c0a68238a64
1 parent e765807 commit 68f51c1

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

cookbooks/fb_init_sample/metadata.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
depends 'fb_chrony'
1919
depends 'fb_collectd'
2020
depends 'fb_cron'
21+
depends 'fb_dnf'
2122
depends 'fb_dnsmasq'
2223
depends 'fb_dracut'
2324
depends 'fb_e2fsprogs'

cookbooks/fb_init_sample/recipes/default.rb

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
include_recipe 'fb_init_sample::site_settings'
3131

3232
if node.centos?
33-
# HERE: yum
33+
include_recipe 'fb_dnf'
3434
include_recipe 'fb_rpm'
3535
end
3636
if node.debian? || node.ubuntu?
@@ -132,18 +132,18 @@
132132
include_recipe 'fb_init_sample::firstboot'
133133
end
134134

135-
unless node.centos6?
136-
# not packaged in C10 and above
135+
if node.centos?
136+
# https://bugzilla.redhat.com/show_bug.cgi?id=2337139
137137
if node.centos_max_version?(9)
138138
include_recipe 'fb_apcupsd'
139139
end
140140
# Turn off dnsmasq as it doesn't play well with travis
141141
node.default['fb_dnsmasq']['enable'] = false
142142
include_recipe 'fb_dnsmasq'
143-
end
144-
# not packaged in C10 and above
145-
unless node.centos10?
146-
include_recipe 'fb_collectd'
143+
# https://bugzilla.redhat.com/show_bug.cgi?id=2345748
144+
if node.centos_max_version?(9)
145+
include_recipe 'fb_collectd'
146+
end
147147
end
148148
include_recipe 'fb_rsync::server'
149149
if node.centos?

0 commit comments

Comments
 (0)