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.

Sign up for GitHub

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

Jump to bottom

[WIP] [Test Automation] Automate E2E scenario with use of Operator for installation #1607

Closed
wants to merge 55 commits into from
Closed

[WIP] [Test Automation] Automate E2E scenario with use of Operator for installation #1607

wants to merge 55 commits into from

Conversation

Omar-AlJaljuli
Copy link
Contributor

Description

[Test Automation] Automate E2E scenario with use of Operator for installation

Which issue(s) does this PR fix

https://issues.redhat.com/browse/RHIDP-2919

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@Omar-AlJaljuli Omar-AlJaljuli requested a review from a team as a code owner September 12, 2024 21:23
@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@Omar-AlJaljuli
Copy link
Contributor Author

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@subhashkhileri
Copy link
Contributor

@Omar-AlJaljuli Have you tested in CI?

@Omar-AlJaljuli Omar-AlJaljuli changed the title [Test Automation] Automate E2E scenario with use of Operator for installation [WIP] [Test Automation] Automate E2E scenario with use of Operator for installation Sep 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

1 similar comment
@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@Omar-AlJaljuli
Copy link
Contributor Author

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

2 similar comments
@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@gustavolira
Copy link
Contributor

I think the Operator could have it's own script instead of using openshift-ci-tests.sh.
@josephca @subhashkhileri wdyt ?

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@Omar-AlJaljuli
Copy link
Contributor Author

/retest

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

2 similar comments
@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

@github-actions GitHub Actions
Copy link
Contributor

The image is available at: quay.io/janus-idp/backstage-showcase:pr-1607!

Omar-AlJaljuli and others added 24 commits December 4, 2024 13:33
@Omar-AlJaljuli
add OCM cluster-role-binding to operator projects
5632ae6
@Omar-AlJaljuli
make the pod selector appropriate when testing operators with audit l…
c4f0564 
…og tests
@Omar-AlJaljuli
ignore the auth tests for showcase-operator-nightly
275a2cb
@Omar-AlJaljuli
update dyanmic plugins
47ef707
@Omar-AlJaljuli
fix dynamic plugins yaml spacing
5185090
@Omar-AlJaljuli
temporarily disable acr config and test to pass
82101a5
@Omar-AlJaljuli
install podman and opm to properly install newest IIB and change rbac…
830c684 
… git discover frequency
@Omar-AlJaljuli
fix rebase mistakes
ac813f2
@Omar-AlJaljuli
edit to try new install-catalog-source script
675ac63
@Omar-AlJaljuli
return catalog sched frequency to 1
ba6f5dd
@Omar-AlJaljuli
use flexible dynamic plugins configmap and run IIB script in debug mode
4ffc854
@Omar-AlJaljuli
fix rbac namespace name
57d6e4d
@Omar-AlJaljuli
fix rebase changes
3533c84
@Omar-AlJaljuli @rm3l
Update install-catalog script name after merge
b1357cc 
Co-authored-by: Armel Soro <armel@rm3l.org>
@Omar-AlJaljuli
remove redundant dyanmic plugin configmaps to keep it flexible with m…
d4cd371 
…ain dynamic plugins, and use next tag of rhdh-hub-rhel9
@Omar-AlJaljuli
fix rbac configmap
924a0ef
@Omar-AlJaljuli
adding dynamic pluigns root PVC to operator
74093a1
@Omar-AlJaljuli
add PVC file to rhdh-operator
c4e2689
@Omar-AlJaljuli
upgrade Backstage api version for operator deployment
d41781d
@Omar-AlJaljuli
add redis cache deployment and delete rhdh-operator before installation
ced7a67
@Omar-AlJaljuli
add patch to bypass volume node affinity issue
5fc3277
@Omar-AlJaljuli
remove operator deployment patches
71e1dd1
@Omar-AlJaljuli
Merge branch 'RHIDP-2919' of github.com:Omar-AlJaljuli/janus-idp-back…
078c685 
…stage-showcase- into RHIDP-2919
@Omar-AlJaljuli
remove typo
43199b8
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024

The image is available at:

@openshift-merge-robot
Copy link
Collaborator

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Dec 10, 2024

@Omar-AlJaljuli: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-tests 43199b8 link true /test e2e-tests
ci/prow/images 43199b8 link true /test images

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@Omar-AlJaljuli
Copy link
Contributor Author

Duplicate: #2088

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gustavolira gustavolira gustavolira left review comments

@rm3l rm3l rm3l requested changes

@subhashkhileri subhashkhileri subhashkhileri requested changes

@josephca josephca Awaiting requested review from josephca

Assignees

@rm3l rm3l

@subhashkhileri subhashkhileri

Labels
do-not-merge/work-in-progress needs-rebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Omar-AlJaljuli @subhashkhileri @gustavolira @rm3l @openshift-merge-robot