Skip to content

Onboard Argus (plans and instance) #158

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

Merged
merged 6 commits into from
Mar 19, 2024
Merged

Onboard Argus (plans and instance) #158

merged 6 commits into from
Mar 19, 2024

Conversation

DiogoFerrao
Copy link
Contributor

No description provided.

DiogoFerrao and others added 6 commits March 11, 2024 17:35
* Onboard Argus instance: create command init

* create command: complete functionality and start testing

* create command: added testing for utils

* update utils error message

* Add custom Argus error and fix create command testing

* fix error formatting in utils

* fix error formatting
* Onboard Argus Instance: update command

* update command: support no plan info provided

* update command: testing

* fix linting

* update command: handle empty instance name

* update documentation

* update command: handle empty name update and corner cases

* fix linting

* update command: improve error message

* address comments in PR

* Update internal/cmd/argus/instance/update/update.go

Co-authored-by: João Palet <[email protected]>

* fix linting

---------

Co-authored-by: João Palet <[email protected]>
* Onboard Argus Instance: delete command

* delete command: add docs
* Onboard Argus Instance: list command

* Onboard Argus Instance: describe command

* describe command: dont display name if empty

* describe command: update output

* describe command: remove additional check

* describe command: add grafana url

* add docs and address comments
@joaopalet joaopalet merged commit 3b90267 into main Mar 19, 2024
@DiogoFerrao DiogoFerrao deleted the feature-onboard-argus branch April 2, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants