Skip to content
This repository has been archived by the owner on Jun 12, 2024. It is now read-only.

add draft check command #910

Open
michelleN opened this issue Dec 12, 2018 · 4 comments
Open

add draft check command #910

michelleN opened this issue Dec 12, 2018 · 4 comments

Comments

@michelleN
Copy link
Contributor

like the linkerd check command, ensuring all systems go

@bjornmagnusson
Copy link
Contributor

God yes! I´m so impressed by how it works in Linkerd.
I would be down to help out with this.
What are your thoughts about this for draft?
check that k8s api / draft config setup / helm is available on working?
or completely different? :)

@michelleN
Copy link
Contributor Author

yes exactly!! Generally for features, we aim to write a Draft Enhancement Proposal/DEP before starting so everyone is on the same page about the feature. If you could take a stab at writing a DEP first, that would be awesome. If we don't want it to be top-level, we can also make a draft check plugin first. Either approach is fine in my opinion but we'll need buy-in from the other core maintainers to make this a top level command first.
cc/ @fibonacci1729 @bacongobbler @radu-matei

Thank you!

@bacongobbler
Copy link
Contributor

bacongobbler commented Jan 22, 2019

I'd love to have an "all systems nominal" check command baked right into the tool. A DEP would be great so we can figure out what parts of the system we need to check and how that'd look from a UX perspective. That way we can also explain what purpose draft check intends to solve (and more importantly what it does not intend to solve).

Let us know if you need any help!

@bjornmagnusson
Copy link
Contributor

Sure thing, will do that. Thanks for the pointer. Makes sense

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