Skip to content

Conversation

@davdhacs
Copy link
Contributor

@davdhacs davdhacs commented Dec 8, 2025

  • local-deploy (no secrets, no auth)
  • ui e2e tests (cypress)
  • run ^ in github actions (ci)

Originally written by claude(claude-sonnet-4-5@20250929): #1722
Refactored (hacked to pieces because it changed too much) by @davdhacs

@davdhacs davdhacs requested review from a team and rhacs-bot as code owners December 8, 2025 02:08
@davdhacs davdhacs marked this pull request as draft December 8, 2025 02:09
@rhacs-bot
Copy link
Contributor

A single node development cluster (infra-pr-1735) was allocated in production infra for this PR.

CI will attempt to deploy quay.io/rhacs-eng/infra-server: to it.

🔌 You can connect to this cluster with:

gcloud container clusters get-credentials infra-pr-1735 --zone us-central1-a --project acs-team-temp-dev

🛠️ And pull infractl from the deployed dev infra-server with:

nohup kubectl -n infra port-forward svc/infra-server-service 8443:8443 &
make pull-infractl-from-dev-server

🚲 You can then use the dev infra instance e.g.:

bin/infractl -k -e localhost:8443 whoami

⚠️ Any clusters that you start using your dev infra instance should have a lifespan shorter then the development cluster instance. Otherwise they will not be destroyed when the dev infra instance ceases to exist when the development cluster is deleted. ⚠️

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:

make helm-deploy

Logs

Logs for the development infra depending on your @redhat.com authuser:

Or:

kubectl -n infra logs -l app=infra-server --tail=1 -f

@davdhacs davdhacs force-pushed the rox-29520-ui-testing branch 4 times, most recently from 9f4d184 to 77b48f0 Compare December 8, 2025 04:44
@davdhacs davdhacs requested a review from pedrottimark December 8, 2025 05:02
@davdhacs davdhacs marked this pull request as ready for review December 8, 2025 05:02
@davdhacs davdhacs force-pushed the rox-29520-ui-testing branch from 77b48f0 to 184603c Compare December 8, 2025 05:04
@davdhacs davdhacs requested a review from tommartensen December 9, 2025 02:04

## Quick Start - Running E2E Tests Against Local Backend

### Prerequisites
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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.

Copy link
Contributor

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)

Copy link
Contributor

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.

Copy link
Contributor

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?

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