Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ma/update runit #1668

Merged
merged 4 commits into from
Jun 5, 2019
Merged

Ma/update runit #1668

merged 4 commits into from
Jun 5, 2019

Conversation

markan
Copy link
Contributor

@markan markan commented May 16, 2019

Description

Update to newest runit and enterprise-chef-common cookbooks. Necessary to be able to move to Chef 15. Still a bit broken, work in progress.

Built on top of ma/remove_drbd, because the new enterprise_chef_common cookbook removes some drbd/keepalived logic

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]

Check List

@markan markan requested a review from a team May 16, 2019 23:42
@@ -290,7 +290,7 @@ def generate_hash
results["private_chef"]["logs"]["log_retention"] = PrivateChef["log_retention"]
results["private_chef"]["logs"]["log_rotation"] = PrivateChef["log_rotation"]
results["private_chef"]["dark_launch"] = PrivateChef["dark_launch"]
results["private_chef"]["opscode-erchef"]["max_request_size"] = PrivateChef["opscode_erchef"]["max_request_size"]
# results["private_chef"]["opscode-erchef"]["max_request_size"] = PrivateChef["opscode_erchef"]["max_request_size"]
Copy link
Contributor

@srenatus srenatus May 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Intentional?
Update: of course! Found the commit. Sorry for the noise 😉

end

link postgresql_data_dir_symlink do
to postgresql_data_dir
not_if { postgresql_data_dir == postgresql_data_dir_symlink }
end

File.open("/dotfiles/node.json", "w") do |f| f.write node.to_json end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this do?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debugging code; I was trying to figure out some details of what was going on internally

@markan markan force-pushed the ma/update_runit branch 7 times, most recently from 6776ae5 to 33e042b Compare June 3, 2019 06:49
@markan
Copy link
Contributor Author

markan commented Jun 3, 2019

@markan markan force-pushed the ma/update_runit branch 2 times, most recently from ceabaaf to 2264337 Compare June 3, 2019 20:43
@markan markan force-pushed the ma/update_runit branch 2 times, most recently from 6f2f323 to d781fee Compare June 4, 2019 00:36
markan added 4 commits June 4, 2019 21:07
Signed-off-by: Mark Anderson <[email protected]>
Update Berksfile to use 5.1.1 runit
Use enterprise-chef-common 0.15.1

Signed-off-by: Mark Anderson <[email protected]>
Signed-off-by: Mark Anderson <[email protected]>
@markan markan force-pushed the ma/update_runit branch from d781fee to bcda7d2 Compare June 5, 2019 04:14
@markan markan merged commit 493d800 into master Jun 5, 2019
@chef-ci chef-ci deleted the ma/update_runit branch June 5, 2019 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants