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

Create separate bare-metal container-linux-install profiles #97

Merged
merged 1 commit into from
Jan 15, 2018
Merged

Create separate bare-metal container-linux-install profiles #97

merged 1 commit into from
Jan 15, 2018

Conversation

dghubble
Copy link
Member

@dghubble dghubble commented Jan 14, 2018

Create separate bare-metal container-linux-install (or cached-container-linux-install) profiles for each node in a cluster. Name the profile after the cluster and node name (should be unique):

  • Fix contention bug around bare-metal "container-linux-install" profile terraform apply. Apply ensures Matchbox profiles and groups are created for each cluster and its nodes. "container-linux-install" is not namespaced, meaning deletion of a cluster requests the profile's deletion, but existing clusters request its creation (or continued existence). This can mean users have to run terraform apply more than once until the constraints are satisfied (i.e. a deletion and recreation occurs).
  • Allows Container Linux installs to vary between clusters instead of using one global container-linux-install profile (e.g. different clusters may now install different Container Linux versions).

* Create separate container-linux-install profiles (and
cached-container-linux-install) for each node in a cluster
* Fix contention bug on bare-metal during `terraform apply`.
With only a global install profile, terraform would create
(or retain) the profile for each cluster and try to delete
it for each cluster being deleted. As a result, in some cases
apply had to be run multiple times before terraform's repr
of constraints was satisfied (profile deleted and recreated)
* Allow Container Linux install properties to vary between
clusters, such as using a different Container Linux channel
or version for different clusters
@dghubble dghubble merged commit 38fa7df into poseidon:master Jan 15, 2018
@dghubble dghubble deleted the separate-bm-install-profiles branch January 15, 2018 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant