This repository has been archived by the owner on Sep 21, 2020. It is now read-only.
GPII-3940: Upgrade terrafrom and providers #56
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR:
updates HelmThis mainly fixes node_pool recreation due to metadata config - see GoogleCloudPlatform/magic-modules#1507 for details.
Update:
Helm update was reverted due to an issue with spec validation (see helm/helm#5750). This should be fixed in soon to be 0.14.1, but it was not available at the time of this PR yet.
This should be tagged as
0.6.0-google_gpii.0
once merged.Also:
Upgrade exekube to latest Alpine
Scan exekube docker image for vulnerabilities
location
field for GKE clusters and node-pools, the oldzone
andregion
have been deprecated (see https://www.terraform.io/docs/providers/google/r/container_cluster.html#location). We should abandon the deprecated fields in favor oflocation
, this will also allow us to get rid of the 2 resources split for regional/zonal clusters ingke-cluster
module. Ticket created - GPII-3949Use
location
in cluster and node_pool resources ingke-cluster
exekube moduleUpstream PR has been opened - exekube#124.