Skip to content
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

docs(agents|workflows): Update usage documentation #78

Merged
merged 6 commits into from
Jun 14, 2022

Conversation

PrayagS
Copy link
Contributor

@PrayagS PrayagS commented Jun 1, 2022

This PR adds the following changes in the usage documentation for litmusctl.

  • Refactor create agent to connect agent.
  • Add REGISTRATION field in get agents.
  • Add disconnect agents command under additional commands.
  • Add instructions for create workflow command.
  • Add other workflow commands viz. get, delete, and describe under additional commands.

Signed-off-by: PrayagS prayagsavsani@gmail.com

Signed-off-by: PrayagS <prayagsavsani@gmail.com>
Signed-off-by: PrayagS <prayagsavsani@gmail.com>
Signed-off-by: PrayagS <prayagsavsani@gmail.com>
Signed-off-by: PrayagS <prayagsavsani@gmail.com>
Copy link
Collaborator

@Saranya-jena Saranya-jena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Usage.md Outdated

* To disconnect an agent, issue the following command..
```shell
litmusctl disconnect agent f9799723-29f1-454c-b830-ae8ba7ee4c30 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also add one more command displaying agent_id as a placeholder instead of the actual agent_id and use the current command as an example.
litmusctl disconnect agent <agent_id> --project-id="<project_id>"

```


* To list the created workflows within a project, issue the following command.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also add examples to display the usage of different flags to display workflows and workflow runs

Usage.md Outdated

* To describe a particular chaos workflow, issue the following command.
```shell
litmusctl describe workflow 9433b48c-4ab7-4544-8dab-4a7237619e09 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we also add command without actual workflowID, so that user knows what is - 9433b48c-4ab7-4544-8dab-4a7237619e09

Usage.md Outdated

* To delete a particular chaos workflow, issue the following command.
```shell
litmusctl delete workflow df91c6b2-ad33-45ae-9a2f-00cb87978657 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here


* To disconnect an agent, issue the following command..
```shell
litmusctl disconnect agent f9799723-29f1-454c-b830-ae8ba7ee4c30 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here


* To describe a particular chaos workflow, issue the following command.
```shell
litmusctl describe workflow 9433b48c-4ab7-4544-8dab-4a7237619e09 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here


* To delete a particular chaos workflow, issue the following command.
```shell
litmusctl delete workflow df91c6b2-ad33-45ae-9a2f-00cb87978657 --project-id=""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

@imrajdas imrajdas merged commit 19bf342 into litmuschaos:master Jun 14, 2022
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.

5 participants