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: GCP improvements #54

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Conversation

marcelmamula
Copy link
Contributor

Description:

  • Replaced 90 second pause with wait for ssh connection
  • Added fail if resources were not found (OS Image, VPC, VPC Subnet, DNS) and catch it with rescue block
  • Enable post provisioning steps which were blocked by incorrect variable name
  • Documentation improvements to include detailed breakdown of required permissions and guide for Cloud NAT port blocking issue.

NOTE
Further improvements to HA provisioning might be needed after sap_ha_pacemaker_cluster role is reworked due to haproxy.

- wait for ssh instead of pause
- fail if resources were not found
- enable post provisioning steps
- documentation improvements
@marcelmamula marcelmamula added the enhancement New feature or request label Aug 12, 2024
@marcelmamula marcelmamula self-assigned this Aug 12, 2024
@marcelmamula
Copy link
Contributor Author

@sean-freeman Updated based on comments and added fix for typos in HA code, which tried to call loop var in tasks without loop.

@marcelmamula marcelmamula merged commit afe682f into sap-linuxlab:dev Aug 23, 2024
3 checks passed
@marcelmamula marcelmamula deleted the gcp branch August 23, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants