add new binary pluralctl #531
Conversation
zreigz
requested review from
michaeljguarino,
floreks and
maciaszczykm
as code owners
cmd/pluralctl/main.go
Outdated
| cd.Command(plural.Plural, plural.HelmConfiguration), | ||
| up.Command(plural.Plural), | ||
| pr.Command(), | ||
| stack.Command(plural.Plural), |
There was a problem hiding this comment.
Are these stacks from console or plural? We'd only care about the console stack commands here, and I do think they deserve a separate subcommand.
There was a problem hiding this comment.
those are plural stacks. We have console stacks in cd subcommand
There was a problem hiding this comment.
Yeah in that case transfer out the ones from cd into the top-level here if possible.
There was a problem hiding this comment.
the idea is the cli should sort of mimic the sidebar tab structure in the console ui.
| pr.Command(), | ||
| stack.Command(plural.Plural), | ||
| cmdinit.Command(plural.Plural), | ||
| mgmt.Command(plural.Plural), |
There was a problem hiding this comment.
We should probably also keep:
profilecommandsclonecommand (to help clone + decrypt repos)versioncommand (always useful to have)
There was a problem hiding this comment.
some of the basic commands are here: https://github.com/pluralsh/plural-cli/pull/531/files#diff-784c7665274506f73ee507fdb0b07802971b16fa0ce946f3553c1d935d567bf8R31
I will add rest of the commands
There was a problem hiding this comment.
We do still want a top-level login command, not sure if it's here still from the pr (maybe cmdinit brings it in?)
There was a problem hiding this comment.
yes, the plural init can handle login.
|
@michaeljguarino |
We should still keep the I think you might have gotten confused when I said to transfer commands from there, I just meant make the |
|
CD commands are in |
* add stable sort to helm dependencies * add new binary pluralctl * make CD commands on top level * move CD commands to CD group, remove plural login --------- Co-authored-by: michaeljguarino <[email protected]>
Summary
A new distribution of the CLI for fleet-management.
Labels
Test Plan
Checklist