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

Support Puppet-managed LoadBalancers #19

Merged
merged 8 commits into from
Jul 20, 2021
Merged

Support Puppet-managed LoadBalancers #19

merged 8 commits into from
Jul 20, 2021

Conversation

ccremer
Copy link
Contributor

@ccremer ccremer commented Jul 13, 2021

  • Updates cloudscale sdk to v3
  • Integrates puppet-managed loadbalancers, similar to Exoscale counterpart

This is a breaking change. See the upgrade documentation at https://github.com/appuio/component-openshift4-terraform

@ccremer ccremer added the enhancement New feature or request label Jul 13, 2021
@ccremer ccremer force-pushed the hiera branch 2 times, most recently from 007ac13 to 19acdc2 Compare July 16, 2021 14:57
Copy link
Contributor Author

@ccremer ccremer left a comment

Choose a reason for hiding this comment

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

@simu Would you mind having a first look?

files/register-server.sh Show resolved Hide resolved
lb.tf Outdated Show resolved Hide resolved
@ccremer ccremer marked this pull request as ready for review July 19, 2021 11:24
@ccremer ccremer requested review from simu and corvus-ch July 19, 2021 11:24
@ccremer
Copy link
Contributor Author

ccremer commented Jul 19, 2021

Cleaned up the git history a bit

Copy link
Member

@simu simu left a comment

Choose a reason for hiding this comment

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

LGTM overall

@@ -0,0 +1,16 @@
#!/bin/sh

set -eo pipefail
Copy link
Member

Choose a reason for hiding this comment

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

Please make sure that this is backported to https://github.com/appuio/terraform-openshift4-exoscale

lb.tf Outdated Show resolved Hide resolved
lb.tf Outdated Show resolved Hide resolved
ccremer added 8 commits July 20, 2021 10:51
To be aligned with terraform-exoscale
There's a risk of getting into a terraform timeout after 5 minutes, in which case
the state becomes corrupted even if the VM later succeeds with provisioning in the
background
@ccremer ccremer added breaking and removed enhancement New feature or request labels Jul 20, 2021
@ccremer ccremer merged commit 1ef923a into master Jul 20, 2021
@ccremer ccremer deleted the hiera branch July 20, 2021 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants