Skip to content

Conversation

@kush982
Copy link
Collaborator

@kush982 kush982 commented Oct 22, 2025

Pull request description

Remove helm --wait for operator install; rely on kubectl waits to avoid context deadline exceeded under e2e label.
Make operator values OpenShift-compatible (unset runAsUser/runAsGroup) and default to cluster StorageClass; keep templates clean and placeholders intact.

Testing Approach
E2E tests were validated by running workflows directly on the fix/microshift_tests branch rather than using the e2e label. This was necessary because the workflows were using pull_request_target, which reads workflow files from the base branch (main) instead of the PR branch, causing workflow configuration changes to be ignored.
Why Direct Branch Testing Was Required
When using the e2e label with pull_request_target:
❌ Workflow files are read from main branch (not PR branch)
❌ Workflow changes in this PR (like TF_REPO_REF: fix/crowd-db-restore) were ignored
❌ Tests ran with outdated configuration

When running workflows directly on the branch:
✅ Workflow files are read from PR branch (fix/microshift_tests)
✅ All workflow changes included (CloudNativePG fixes, Terraform branch override)
✅ Tests ran with correct configuration

Successful Test Runs
All E2E tests passed successfully on the PR branch:
MicroShift Tests (All Products):
Run #18885444887 - ✅ All products passed (Jira, Confluence, Bitbucket, Bamboo, Crowd)
Run #18884943041 - ✅ All products passed
E2E Terraform Tests:
Run #18885444899 - ✅ Infrastructure deployment and E2E tests passed
Run #18882051203 - ✅ Full E2E suite passed (1h runtime)

Checklist

  • I have added unit tests
  • I have applied the change to all applicable products
  • The E2E test has passed (use e2e label)

@kush982 kush982 added the e2e label Oct 22, 2025
@kush982 kush982 added e2e and removed e2e labels Oct 22, 2025
@kush982 kush982 changed the title Fix/microshift tests Fix MicroShift/e2e install timeout and simplify deploy Oct 22, 2025
@kush982 kush982 added e2e and removed e2e labels Oct 22, 2025
@kush982 kush982 added e2e and removed e2e labels Oct 28, 2025
@kush982 kush982 added e2e and removed e2e labels Oct 28, 2025
@kush982 kush982 marked this pull request as ready for review October 28, 2025 19:48
@bianchi2 bianchi2 merged commit eb67ba1 into main Oct 29, 2025
7 checks passed
@bianchi2 bianchi2 deleted the fix/microshift_tests branch October 29, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants