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(Test): Addition of E2E test for Azure Experiments #354

Merged
merged 16 commits into from
Jan 15, 2022
Merged

Chore(Test): Addition of E2E test for Azure Experiments #354

merged 16 commits into from
Jan 15, 2022

Conversation

avaakash
Copy link
Contributor

@avaakash avaakash commented Dec 10, 2021

Proposed changes

This PR adds the E2E test for the Azure Instance Stop Experiment and Azure Disk Loss Experiment
Link to Sample Run: https://github.com/avaakash/litmus-e2e/runs/4758029750
Partially completes Issue #308

How has this been tested:

  • Test environment
    • GKE
    • AWS
    • OpenShift
    • KinD/k3s/k3d cluster
    • Others
  • Tested with production environment ( For ChaosCenter Tests )
  • Have not tested

Types of changes

What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

  • New feature (non-breaking change which adds functionality)
  • Bugfix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices applies)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have read the CONTRIBUTING doc
  • I have signed the commit for DCO to be passed.
  • I have added necessary documentation (if appropriate)
  • I have added Screenshots or Terminal snippets in the PR comment to validate the successful execution of changes

Documentation

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • data-cy=* parameters have been added in codebase ( ChaosCenter Tests )
  • data-cy=* parameters need to be added in codebase ( ChaosCenter Tests )
  • No updates required.

Code Review

  • Does the test handle fatal exceptions, ie. uncaught exceptions from 3rd party library (if any)

Issue

  • Tasks in issue are checked off

Dependency

  • Please add the links to the dependent PR that need to be merged before this (if any).

Special notes for your reviewer:

Following secrets need to be added for the e2e tests

  1. AZURE_CREDENTIALS - Complete auth file content for Azure CLI login
  2. AZURE_AUTH_FILE - Complete auth file content with escaped quotes character
  3. AZURE_RESOURCE_GROUP - resource group name for azure

Akash Shrivastava added 13 commits September 24, 2021 13:50
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Shortened engine name

Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
Signed-off-by: Akash Shrivastava <[email protected]>
@avaakash avaakash changed the title Chore(Test): Addition of E2E test for Azure Instance Stop Experiment Chore(Test): Addition of E2E test for Azure Experiments Jan 6, 2022
Akash Shrivastava added 2 commits January 6, 2022 17:33
.github/workflows/azure-experiment-pipeline.yml Outdated Show resolved Hide resolved
.github/workflows/azure-experiment-pipeline.yml Outdated Show resolved Hide resolved
.github/workflows/nightly-azure-experiment-pipeline.yml Outdated Show resolved Hide resolved
.github/workflows/nightly-azure-experiment-pipeline.yml Outdated Show resolved Hide resolved
.master-plan.yml Outdated Show resolved Hide resolved
Signed-off-by: Akash Shrivastava <[email protected]>
@uditgaurav uditgaurav merged commit 0862a9a into litmuschaos:master Jan 15, 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