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

Refactor E2E test scripts: improve output and remove old step #101

Merged
merged 2 commits into from
Dec 13, 2024

Conversation

jgwest
Copy link
Member

@jgwest jgwest commented Dec 6, 2024

What does this PR do / why we need it:

  • We no longer need to use the 'jgwest-redhat/rollouts-demo' image, now that we've moved to nginx-unprivileged, so this step can be removed.
  • Add additional steps to upstream test output

Have you updated the necessary documentation?

  • Documentation update is required by this PR, and has been updated.

How to test changes / Special notes to the reviewer:
N/A

@jgwest jgwest force-pushed the tweak-e2e-test-script branch from 633c89f to 9471bc6 Compare December 7, 2024 02:37
@jgwest jgwest force-pushed the tweak-e2e-test-script branch from c098f5b to e4f3bb6 Compare December 9, 2024 06:46
@Rizwana777
Copy link
Collaborator

LGTM

@jgwest jgwest merged commit 8fcc10b into argoproj-labs:main Dec 13, 2024
6 checks passed
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