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

WIP: chore: add ci script for e2e testing #268

Closed
wants to merge 1 commit into from

Conversation

salasberryfin
Copy link
Contributor

@salasberryfin salasberryfin commented Nov 17, 2023

What this PR does / why we need it:

Based on the configuration used in upstream projects, e.g. CAPA, and setting up E2E tests running periodically in Prow as a PoC (which hopefully solves the current stability of nightly tests and can be promoted to full-time solution), this PR adds a script used to trigger the full E2E test suite.

In the future, if required, this script can include the logic to interact with boskos and run the janitors.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

Signed-off-by: Carlos Salas <carlos.salas@suse.com>
@richardcase
Copy link
Contributor

@salasberryfin - what else do we need from this to move it from wip? Guessing we need to update the GHA workflow to call the script?

@salasberryfin
Copy link
Contributor Author

I was thinking about re-using these scripts as a base for adding the AWS environment to run Rancher Manager but only did some tests with eksctl last week. I'll try to find some time to look at it this week and we can use a single PR with all the changes, if that's the decision we go for. Does that sound good? If so, we can maybe close this and work in a separate PR.

@richardcase
Copy link
Contributor

Yeah happy to close this and pick it up as part of #382

auto-merge was automatically disabled February 21, 2024 10:38

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants