Skip to content
This repository has been archived by the owner on May 1, 2020. It is now read-only.

[in synopsysctl] Clarify which inputs are absolutely required for deploying Polaris #1067

Open
yashbhutwala opened this issue Nov 7, 2019 · 2 comments
Assignees
Milestone

Comments

@yashbhutwala
Copy link
Contributor

What would you like to be added:
Make it clear to the "deployer of Polaris" which fields are required to deploy. This includes conditional requirements, for example if postgres container is used, only postgres-password is required. if postgres container is not used, postgres-host, postgres-port, postgres-username, and postgres-password are required.

Why is this needed:
Better user experience.

@mphammer
Copy link
Member

One way we could start to do this is by adding more examples to the --help that show the flags that go together.

@yashbhutwala
Copy link
Contributor Author

Good idea! 😄 Added an example in #1107. While better than nothing, this is still just a workaround, need to find a better way to build our own template with cobra similar to kubectl to group options together and mark group requirements 😄

@msenmurugan msenmurugan modified the milestones: 2019.12.0, 2020.1.0 Dec 9, 2019
@msenmurugan msenmurugan modified the milestones: 2020.1.0, 2020.4.0 Feb 3, 2020
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

3 participants