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

chore(sdk): Adding support for helper based chaoslib #546

Merged
merged 4 commits into from
Aug 3, 2022

Conversation

ispeakc0de
Copy link
Member

@ispeakc0de ispeakc0de commented Aug 3, 2022

Signed-off-by: Shubham Chaudhary shubham.chaudhary@harness.io

What this PR does / why we need it:

  • Adds support for experiments with helper pod in chaoslib in litmus SDK with this, the user can create a chaos template with helper pod model using the same litmus SDK

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Checklist:

  • Fixes #
  • PR messages has document related information
  • Labelled this PR & related issue with breaking-changes tag
  • PR messages has breaking changes related information
  • Labelled this PR & related issue with requires-upgrade tag
  • PR messages has upgrade related information
  • Commit has unit tests
  • Commit has integration tests
  • E2E run Required for the changes

Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io>
Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io>
Signed-off-by: ispeakc0de <ashubham314@gmail.com>
ksatchit
ksatchit previously approved these changes Aug 3, 2022
uditgaurav
uditgaurav previously approved these changes Aug 3, 2022
Co-authored-by: Udit Gaurav <35391335+uditgaurav@users.noreply.github.com>
@ispeakc0de ispeakc0de dismissed stale reviews from uditgaurav and ksatchit via a303506 August 3, 2022 08:33
…-helper

Signed-off-by: Shubham Chaudhary <shubham.chaudhary@harness.io>
@ksatchit ksatchit merged commit f402bf8 into litmuschaos:master Aug 3, 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.

3 participants