Skip to content
This repository has been archived by the owner on Sep 21, 2020. It is now read-only.

GPII-3940: Exekube upgrade #410

Merged
merged 2 commits into from
May 24, 2019
Merged

Conversation

stepanstipl
Copy link
Contributor

This PR upgrades exekube image to gpii/exekube:0.6.0-google_gpii.0. This upgrades Terraform, Terraform providers and other components, see gpii-ops/exekube#56 for details.

This mainly fixes node_pool recreation due to metadata config (GPII-3940) - see GoogleCloudPlatform/magic-modules#1507 for details.

Also included is a small formatting fix for preferences chart which was causing wrong yaml indentation and deployment spec validation fail with new helm.

@stepanstipl stepanstipl self-assigned this May 24, 2019
@stepanstipl stepanstipl changed the title Exekube upgrade GPII-3940: Exekube upgrade May 24, 2019
Copy link
Contributor

@amatas amatas left a comment

Choose a reason for hiding this comment

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

LGTM

@mrtyler
Copy link
Contributor

mrtyler commented May 24, 2019

LGTM

I'm not sure why you went to 0.6.0. I would have used 0.5.2 since none of these changes have compatibility consequences (and there are no other significant changes).

@stepanstipl
Copy link
Contributor Author

stepanstipl commented May 24, 2019

Thanks for reviews.

I'm not sure why you went to 0.6.0. I would have used 0.5.2 since none of these changes have compatibility consequences (and there are no other significant changes).

I've chosen to bump minor in the spirit of semantic versioning - lots of the tools and providers upgraded bump their minor or even major versions, introduce new features and even some potentially backward incompatible changes. So while for us it might not mean any significant changes, I think in the context of exekube it wouldn't be enough to increase just patch version.

@stepanstipl stepanstipl merged commit f384727 into gpii-ops:master May 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants