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

opctl build should build config.yaml and pull the manifests #385

Open
aleksandrmelnikov opened this issue Jun 30, 2020 · 1 comment
Open
Labels
kind/bug Something isn't working

Comments

@aleksandrmelnikov
Copy link
Contributor

aleksandrmelnikov commented Jun 30, 2020

and pull the correct manifests based on params.yaml. The use case here is that a user should be able to download a params.yaml file from git (encrypted) or a key vault service and be able to deploy/update the cluster

Thursday, June 25, 2020 - 01_55_35 PM
For opctl init, we should warn the user if they already have params.yaml existing.

  • It'll warn them that this operation is destructive
  • They have to pass --force to over-write existing params.yaml

Tuesday, June 30, 2020 - 12_18_11 PM
Validate the params.yaml before running build or apply.

  • Version 0.11.0 of CLI will know about application.provider
  • Version 0.10.0 will not

So when apply is run on 0.11.0, it will check params.yaml.
It sees application. It checks for application.fqdn, application.insecure, and application.provider (plus others, prior are examples).
It sees provider missing, it throws an error with the value. Points user to docs.

Add breaking changes log

@issue-label-bot issue-label-bot bot added the kind/bug Something isn't working label Jun 30, 2020
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label kind/bug to this issue, with a confidence of 0.57. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@aleksandrmelnikov aleksandrmelnikov changed the title opctl build should build config.yaml and pull the correct manifests based on params.yaml. The use case here is that a user should be able to download a params.yaml file from git (encrypted) or a key vault service and be able to deploy/update the cluster opctl build should build config.yaml Jun 30, 2020
@rushtehrani rushtehrani added this to the v0.12.0 milestone Jul 1, 2020
@rushtehrani rushtehrani changed the title opctl build should build config.yaml opctl build should build config.yaml and pull the manifests Jul 2, 2020
@rushtehrani rushtehrani modified the milestones: v0.12.0, v0.13.0 Jul 10, 2020
@rushtehrani rushtehrani changed the title opctl build should build config.yaml and pull the manifests opctl build should build params.yaml and pull the manifests Aug 10, 2020
@rushtehrani rushtehrani changed the title opctl build should build params.yaml and pull the manifests opctl build should build config.yaml and pull the manifests Aug 10, 2020
@rushtehrani rushtehrani modified the milestones: v0.13.0, v0.14.0 Aug 21, 2020
@rushtehrani rushtehrani modified the milestones: v0.14.0, v0.15.0 Sep 16, 2020
@rushtehrani rushtehrani modified the milestones: v0.15.0, v0.16.0 Oct 23, 2020
@rushtehrani rushtehrani modified the milestones: v0.16.0, v0.17.0 Nov 25, 2020
@rushtehrani rushtehrani removed this from the v0.17.0 milestone Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants