Skip to content

[ansible] Allow overriding Kubespray version form within ENV_DIR#435

Merged
lucendio merged 1 commit intodevelopfrom
allow-overriding-kubespray-version
Apr 14, 2021
Merged

[ansible] Allow overriding Kubespray version form within ENV_DIR#435
lucendio merged 1 commit intodevelopfrom
allow-overriding-kubespray-version

Conversation

@lucendio
Copy link
Copy Markdown
Contributor

@lucendio lucendio commented Mar 4, 2021

Please note that this is meant to only exist temporarily until we have a
solid release cycle for the platform in place. Regardless, overriding the
Kubespray version, defined by the the submodule pinning, from within ENV_DIR
should be avoided at all costs.

Please note that this is meant to only exist temporarily until we have a
solid release cycle for the platform in place. Regardless, overriding the
Kubespray version, defined by the the submodule pinning, from within ENV_DIR
should be avoided at all costs.
@lucendio lucendio requested a review from arianvp March 4, 2021 12:18
@lucendio lucendio merged commit 1d44b85 into develop Apr 14, 2021
@lucendio lucendio deleted the allow-overriding-kubespray-version branch April 14, 2021 11:25
arianvp pushed a commit that referenced this pull request Apr 16, 2021
Please note that this is meant to only exist temporarily until we have a
solid release cycle for the platform in place. Regardless, overriding the
Kubespray version, defined by the the submodule pinning, from within ENV_DIR
should be avoided at all costs.
lucendio added a commit that referenced this pull request Apr 19, 2021
This changeset adds a couple of targets from #432

* helmfile invocation (prefixed with 'hf-')
* ansible-play-% to invoke single playbooks from ./terraform/
* a target to override roles, specifically to install an older version of
  Kubespray (this didnt get relocated during initial effort of merging the
  Makefiles due to an async merge of #435)

Aside from that, this change set includes:

* simplify make code
* rename create-inventory to generate-inventory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants