-
Notifications
You must be signed in to change notification settings - Fork 80
OADP-5782: Add hypershift-oadp-plugin E2E tests #1684
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
Conversation
Skipping CI for Draft Pull Request. |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
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.
Pull Request Overview
This PR introduces end-to-end tests for the hypershift-oadp-plugin and extends functionality to support hypershift scenarios throughout the codebase. It also adds new functions, updates the CRDs, manifests, and configuration files to include hypershift plugin references.
- Added YAML sample manifests for hypershift
- Introduced hypershift support in the scheme, backup, credentials, and controller tests
- Updated CRDs, CSVs, and config files for hypershift plugin support
Reviewed Changes
Copilot reviewed 30 out of 31 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
tests/e2e/sample-applications/hostedcontrolplanes/hypershift/hostedcluster-pull-secret.yaml | Added YAML for pull secret with hypershift label |
tests/e2e/sample-applications/hostedcontrolplanes/hypershift/hostedcluster-etcd-enc-key.yaml | Added YAML for etcd encryption key with hypershift label |
tests/e2e/sample-applications/hostedcontrolplanes/hypershift/hostedcluster-agent-capi-role.yaml | Added YAML Role for agent with capi access |
tests/e2e/lib/scheme.go | Registered hypershift API scheme |
tests/e2e/lib/backup.go | Added function CreateCustomBackupForNamespaces |
tests/e2e/lib/apps.go | Logged deployment status for debugging |
tests/e2e/hcp_backup_restore_suite_test.go | Introduced the hypershift backup & restore test suite and helper functions |
tests/e2e/e2e_suite_test.go | Updated client scheme initialization for test runtime |
pkg/credentials/* | Added hypershift related test cases and plugin image logic |
pkg/common/common.go | Added hypershift plugin constants |
internal/controller/* | Updated unit tests to include hypershift plugin in deployment and nodeagent checks |
docs/developer/testing/TESTING.md | Added env variable for hypershift plugin image |
config/* | Updated configuration files with hypershift plugin references |
api/v1alpha1/dataprotectionapplication_types.go | Extended enum and constants to include hypershift plugin |
Files not reviewed (1)
- go.mod: Language not supported
Comments suppressed due to low confidence (1)
tests/e2e/hcp_backup_restore_suite_test.go:25
- The variable 'dpaCR' is used without a visible declaration in this file. Please ensure that 'dpaCR' is defined or passed appropriately.
err := lib.AddHCPPluginToDPA(h, dpaCR.Namespace, dpaCR.Name, false)
Need to add coverage: #1676 (comment) |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
56e0c6a
to
3df12fd
Compare
The PR is ready to be reviewed 🚀 |
tests/e2e/lib/scheme.go
Outdated
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.
whats the why behind this change?
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.
Mostly order and reusability. I think that function could be reused from most of the places and have a dynamic client just calling a function.
@jparrill the typical oadp e2e test expects that OADP will be installed and the dpa is configured. This should also follow that pattern imho if I'm understanding things correctly.
|
We'll start reviewing to merge, but sorry to say we've already branched oadp-1.5 :) |
@jparrill: This pull request references OADP-5782 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
👍 I will backport to oadp-1.5 branch if needed (in the end it's E2E coverage) |
hrm.. I think the images we're pulling here are NOT multi-arch. so I have an arm cluster w/ 3 cont, 3 workers.. I think this is the second time trying the hypershipft e2e on this cluster. Is there cleanup, or additional resources I need? Any gotchas? it's stalling on 2025/04/21 13:02:46 Error getting etcd statefulset, retrying...: statefulsets.apps "etcd" not found -1, we need this to work out of the box on ARM64 as that is the only arch we are allowed to devel on in OADP. |
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.
Needs to work as written for ARM64
Hey @weshayutin looks like the CAPI image is not being using multi, to solve that is just changing a line: https://github.com/openshift/oadp-operator/pull/1684/files#diff-683c1f305231a8b35496874f630b287ea73c40ff73f633c4d27c9851b798c977R28 The question I have is, should I build the plugin in multiarch to make it work in the ARM cluster, or just change the OCP payload to use multiarch? |
either works if it works. ideally the upstream quay plugin for devs will also be multi arch so we don't have to override when on diff arches. |
Signed-off-by: Juan Manuel Parrilla Madrid <[email protected]>
the deployments and b/r are working, but I do see 2 velero pods running and 1 in crash loop
Perhaps just my deployment.. will retry.. First test: test-hc-none
Second test: test-hc-agent
Errors bringing up: test-hc-agent cluster
@jparrill this is pretty cool stuff, impressive and well done. Can you please check out these errors on the second test. It seems like not all the services came up on the test-hc-agent cluster, is that to be expected?
|
/test 4.19-e2e-test-hcp-aws-periodic |
I think this could be a flakiness on the restoration part. I think it's worth to:
Regarding the Multiarch part:
I think with these things we have all the fronts covered. WDYT @weshayutin @kaovilai ? |
Sound good to me |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jparrill, kaovilai, weshayutin The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@jparrill: all tests passed! 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. |
/jira backport oadp-1.5 |
@jparrill: The following backport issues have been created: Queuing cherrypicks to the requested branches to be created after this PR merges: In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@openshift-ci-robot: #1684 failed to apply on top of branch "oadp-1.5":
In response to this:
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. |
Why the changes were made
This PR add the E2E tests for hypershift-oadp-plugin. It includes also:
How to test the changes made
Execute the E2E with this command: