Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

acs-engine quick-create ... #3378

Closed
khenidak opened this issue Jun 27, 2018 · 6 comments
Closed

acs-engine quick-create ... #3378

khenidak opened this issue Jun 27, 2018 · 6 comments

Comments

@khenidak
Copy link
Contributor

We have talked multiple times with regards to optimized creation process (cc: @jessfraz @jackfrancis @cpuguy83, @CecileRobertMichon ) - The current process is:

  1. Lengthy : copy api-model, modify values, do acs-engine generate then deploy. Each step is prone to errors (spelling issues, copy/paste etc).
  2. The current process is not optimized the best practices (turn on MSI, use premium disks etc).

Proposal:

  1. Establish a well know golden api-model (as part of the binary). That golden model encompasses all the best practices that we learned and learning. The api-model will also be updated as part of the release as needed.

  2. Create a new CLI command acs-engine quick-create --resource-group --location --master-count {n} --agentpool-count {n} --node-per-pool {n} --overrides [{key:val}] --output-directory {path} calling this command will:

  3. Generate the needed api-model + ARM template

  4. Create resource-group

  5. Perform deployment

It is assumed that some of the advanced feature (such as BYO VNET) will not supported by quick-create

@cpuguy83 is willing to carry this forward.

comments/questions?

@cpuguy83
Copy link
Member

Not sure how I feel about the --overrides flag. It's feels a bit too loosey-goosey.
What if it can generate a golden template that users can edit for overrides?

@khenidak
Copy link
Contributor Author

khenidak commented Jun 27, 2018 via email

@cpuguy83
Copy link
Member

How about acs-engine deploy --with-builtin-model --builtin-model-name v0 where, for now, the new flags are mutually exclusive with --api-model and the version flag can be used to deploy a specific version but the default is the latest version.

Alternatively --api-model could default to the value of builtin (thereby also making it a reserved word). This would still include the model name flag as well.

@serbrech
Copy link
Member

serbrech commented Jul 5, 2018

Note that we have the --set flag implemented by @jcorioland that might solve your --override idea

merged PR : #2787
--set flag docs

@cpuguy83
Copy link
Member

Another thought here:

$ acs-engine create [basic opts to configure model] > model.json
<edit and/or commit model>
$ acs-engine deploy --api-model model.json

As well as a simple

$ acs-engine create [basic opts to configure model] --deploy

These could also be flipped to deploy by default and have a flag to output the model that can be consumed by the deploy subcommand.

@stale
Copy link

stale bot commented Mar 9, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution. Note that acs-engine is deprecated--see https://github.com/Azure/aks-engine instead.

@stale stale bot added the stale label Mar 9, 2019
@stale stale bot closed this as completed Mar 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants