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

Static files for values #9

Merged
merged 2 commits into from
Apr 15, 2020
Merged

Static files for values #9

merged 2 commits into from
Apr 15, 2020

Conversation

inercia
Copy link
Contributor

@inercia inercia commented Apr 15, 2020

  • Load files for values from some predefined locations. The Operator will use these values files for setting Helm values before installing the Chart.
  • Static values files as ConfigMaps, with different deploymentTool for the manifest and for the Operator Helm chart.
  • Static value file added in the image that is deployed to Azure, with a specific deploymentTool for Azure.

Notes for reviewers

  • the k3d.sh has suffered some re-formatting. Please ignore.
  • one of the commits is just about /vendor being removed. Please ignore it.

@@ -2,8 +2,8 @@

k3d_prov_dir="$(cd "$(dirname "${BASH_SOURCE[0]}")" >/dev/null 2>&1 && pwd)"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignore changes in this file: just formatting.

@inercia
Copy link
Contributor Author

inercia commented Apr 15, 2020

CI will fail until a new release of the Helm chart is available.

* Load files for values from some predefined locations. The Operator
  will use these values files for setting Helm values before installing
  the Chart.
* Static values files as ConfigMaps, with different `deploymentTool`
  for the manifest and for the Operator Helm chart.
* Static value file added in the image that is deployed to Azure, with
  a specific `deploymentTool` for Azure.

Signed-off-by: Alvaro Saurin <alvaro.saurin@gmail.com>
Signed-off-by: Alvaro Saurin <alvaro.saurin@gmail.com>
@inercia
Copy link
Contributor Author

inercia commented Apr 15, 2020

A new Helm chart has been published and CI is passing. Ready for review.

Copy link

@khussey khussey left a comment

Choose a reason for hiding this comment

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

+1

@inercia inercia merged commit 39306e0 into master Apr 15, 2020
@inercia inercia deleted the inercia/extra_labels branch April 15, 2020 21:42
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