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

(PE-40934) Ensure agent is enabled on new nodes we migrated to #578

Merged
merged 1 commit into from
Apr 9, 2025

Conversation

davidmalloncares
Copy link
Contributor

Summary

When performing manual testing of migrations on Large installations we found that when DR is enabled then after migration the new nodes we migrated to do not have the puppet agent enabled and puppet runs fail. Adding in a command at the end of the migration plan to ensure the puppet agent is enabled on these nodes fixes this issue.

Checklist

  • [] 🟢 Spec tests.
  • [] 🟢 Acceptance tests.

Changes include test coverage?

  • [] Yes
  • [] Not needed

Have you updated the documentation?

  • [] Yes, I've updated the appropriate docs
  • Not needed

@davidmalloncares davidmalloncares requested review from a team as code owners April 4, 2025 10:25
Copy link
Contributor

@B00885424 B00885424 left a comment

Choose a reason for hiding this comment

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

Looks good to me. If there is a conflict with the all hosts between the 2 PRs I'll just do it if needed

@davidmalloncares davidmalloncares merged commit dd7f25a into main Apr 9, 2025
107 checks passed
@davidmalloncares davidmalloncares deleted the PE-40934-enforce-agent-enabled branch April 9, 2025 12:24
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.

2 participants