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

sap_vm_provision: exec flow amendments from testing #58

Merged
merged 9 commits into from
Sep 13, 2024

Conversation

sean-freeman
Copy link
Member

sap_vm_provision: exec flow amendments from testing

Copy link
Contributor

@marcelmamula marcelmamula left a comment

Choose a reason for hiding this comment

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

LGTM after connectivity typo is resolved @sean-freeman I suppose removal of AWS and GCP from Readme is because Ansible collections are used instead of CLI, right?

@berndfinger
Copy link
Member

LGTM after connectivity typo is resolved @sean-freeman I suppose removal of AWS and GCP from Readme is because Ansible collections are used instead of CLI, right?

@marcelmamula As there has been no reply yet from @sean-freeman, should we just assume that the answer to your question is yes, and just merge and fix the spelling error and the ansible-lint error in a new small PR right after merging?

Copy link
Member

@Wabri Wabri left a comment

Choose a reason for hiding this comment

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

Minor linting problem to fix

@berndfinger berndfinger merged commit 74862b2 into sap-linuxlab:dev Sep 13, 2024
4 checks passed
@sean-freeman sean-freeman deleted the placement_groups branch November 1, 2024 13:35
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