-
Notifications
You must be signed in to change notification settings - Fork 1
test(ui): add ui tests with local-deploy #1735
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
base: master
Are you sure you want to change the base?
Conversation
|
A single node development cluster (infra-pr-1735) was allocated in production infra for this PR. CI will attempt to deploy 🔌 You can connect to this cluster with: 🛠️ And pull infractl from the deployed dev infra-server with: 🚲 You can then use the dev infra instance e.g.: Further Development☕ If you make changes, you can commit and push and CI will take care of updating the development cluster. 🚀 If you only modify configuration (chart/infra-server/configuration) or templates (chart/infra-server/{static,templates}), you can get a faster update with: LogsLogs for the development infra depending on your @redhat.com authuser: Or: |
9f4d184 to
77b48f0
Compare
77b48f0 to
184603c
Compare
|
|
||
| ## Quick Start - Running E2E Tests Against Local Backend | ||
|
|
||
| ### Prerequisites |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a notice to run these commands from the repository root (not just in the second item in the list).
| 0. **Build** images: | ||
|
|
||
| ```bash | ||
| make image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The top-level README should mention that docker is required for this.
|
|
||
| ```bash | ||
| # From the repository root | ||
| make deploy-local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
helm is required for this command.
This also doesn't work with remote clusters (as the image is not available). A local cluster that has access to the local image registry is required or the image must be pushed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, I also don't see an easy way to overwrite the version (result of make tag). I had to hardcode 0.13.55 as the result of make tag to continue (as I didn't want to deploy a container registry bridge between podman and Kind)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Next issue: No pull secrets for the quay.io image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While running these steps, I asked myself why you didn't chose to run the infra-server locally as a binary, connected to a remote argo-workflows server, like we do with the existing e2e tests?
Originally written by claude(claude-sonnet-4-5@20250929): #1722
Refactored (hacked to pieces because it changed too much) by @davdhacs