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.

Sign up for GitHub

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

Jump to bottom

Probes describe command #215

Merged
merged 5 commits into from
May 15, 2024
Merged

Probes describe command #215

merged 5 commits into from
May 15, 2024

Conversation

shivam-Purohit
Copy link
Contributor

Addition of describe probe command

usage : litmusctl describe probe --project-id="" --probe-id=""

output :

getprobeyaml: |

  cmdProbe/inputs:

    command: get

    comparator:

      criteria: ==

      type: int

      value: "200"

  mode: Continuous

  name: shivamcmd

  runProperties:

    interval: 1s

    probeTimeout: 1s

  type: cmdProbe

@shivam-Purohit shivam-Purohit marked this pull request as ready for review April 12, 2024 04:50
SarthakJain26
SarthakJain26 reviewed Apr 26, 2024
View reviewed changes
pkg/cmd/describe/probe.go Outdated Show resolved Hide resolved
pkg/cmd/describe/probe.go Outdated Show resolved Hide resolved
@shivam-Purohit
describe probe command
5a9c55c 
Signed-off-by: Shivam Purohit <shivampurohit900@gmail.com>
@shivam-Purohit
add output flag for exp describe
70c39a8 
Signed-off-by: Shivam Purohit <shivampurohit900@gmail.com>
@shivam-Purohit
describe probe: add option to change output type
9c4725c 
Signed-off-by: Shivam Purohit <shivampurohit900@gmail.com>
@shivam-Purohit
fix:fmt
9dd3f42 
Signed-off-by: Shivam Purohit <shivampurohit900@gmail.com>
@shivam-Purohit
remove comments
0a08245 
Signed-off-by: Shivam Purohit <shivampurohit900@gmail.com>
@shivam-Purohit
Copy link
Contributor Author

@SarthakJain26 addressed both the comments

@SarthakJain26 SarthakJain26 merged commit 5d8e1be into litmuschaos:master May 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SarthakJain26 SarthakJain26 SarthakJain26 approved these changes

@Nageshbansal Nageshbansal Nageshbansal approved these changes

@Jonsy13 Jonsy13 Awaiting requested review from Jonsy13 Jonsy13 is a code owner

@Saranya-jena Saranya-jena Awaiting requested review from Saranya-jena Saranya-jena is a code owner

Assignees

@shivam-Purohit shivam-Purohit

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@shivam-Purohit @SarthakJain26 @Nageshbansal