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
upgrade: Move "crowbar_upgrade_step" attribute to node role #1080
Conversation
Similar to the previous "node_upgrade_state" change this is done to prevent a race condition with a chef-client running on the node that might overwrite "crowbar_wall".
This is largely untested and requires changes on the cloud 6 branch as well as crowbar-ha. |
This is a partial backport of crowbar#1080 to stable/3.0 Note: This should not change any pieces related to the upgrade from tex to stable/3.0 to avoid breaking that. That's why the there are still a lot of reference to crowbar_wall["crowbar_upgrade_step"] left.
@@ -24,7 +24,7 @@ | |||
|
|||
return unless node[:platform_family] == "suse" | |||
|
|||
upgrade_step = node["crowbar_wall"]["crowbar_upgrade_step"] || "none" | |||
upgrade_step = node["crowbar_upgrade_step"] || "none" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we need node["crowbar"]
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jsuchome I think you are right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is actualy correct...
This is a partial backport of crowbar#1080 to stable/3.0 Note: This should not change any pieces related to the upgrade from tex to stable/3.0 to avoid breaking that. That's why the there are still a lot of reference to crowbar_wall["crowbar_upgrade_step"] left.
This is a partial backport of crowbar#1080 to stable/3.0 Note: This should not change any pieces related to the upgrade from tex to stable/3.0 to avoid breaking that. That's why the there are still a lot of reference to crowbar_wall["crowbar_upgrade_step"] left.
For the record: that "upgrade" gating job cannot pass right now, as it would require additional 2 PR's (one on Cloud6 side) as well as pending fix for crowbarctl. |
Similar to the previous "node_upgrade_state" change this is done to
prevent a race condition with a chef-client running on the node that
might overwrite "crowbar_wall".