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

fix: replace deploy playbook with role #626

Closed
wants to merge 1 commit into from

Conversation

mfocko
Copy link
Member

@mfocko mfocko commented Dec 11, 2024

TODO:

  • ansible-lint warnings for role
  • check if it's possible to move the roles/ out to the top-level to reduce the nesting
  • run tests on TF to make sure the changes are compatible

@mfocko mfocko self-assigned this Dec 11, 2024
@mfocko mfocko requested a review from majamassarini December 11, 2024 13:56
Copy link
Contributor

@mfocko mfocko closed this Dec 11, 2024
@mfocko mfocko deleted the chore/remove-deploy branch December 11, 2024 14:00
softwarefactory-project-zuul bot added a commit that referenced this pull request Jan 2, 2025
fix: replace deploy playbook with role

TODO:

 ansible-lint warnings for role

will be handled in a follow up


 check if it's possible to move the roles/ out to the top-level to reduce the nesting
 run tests on TF to make sure the changes are compatible

Supersedes #626

Reviewed-by: Maja Massarini
Reviewed-by: Laura Barcziová
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant