-
Notifications
You must be signed in to change notification settings - Fork 48
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
cli: fix upgrade by passing placeholder values for images #2250
Conversation
✅ Deploy Preview for constellation-docs canceled.
|
@@ -23,13 +23,13 @@ func TerraformUpgradeVars(conf *config.Config) (terraform.Variables, error) { | |||
// The image is updates via our operator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update suggestion for the line above:
// Note that we don't pass any real image as imageRef, as we ignore changes to the image in the terraform.
// Still, the terraform variable verification must accept the values.
// For AWS, we enforce some basic constraints on the image variable.
// For Azure, the provider enforces that the source_image_id is not empty.
// For GCP, any placeholder works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
running all E2E upgrades before merging and starting new release |
432745c
to
9321881
Compare
Converted this to a draft until the azure upgrade tests passes. |
Context
Fixes a bug from #2211.
The current TF validation on AWS prevents an empty value set during an upgrade. Similarly, Azure also doesn't allow empty strings.
Proposed change(s)
Checklist