-
Notifications
You must be signed in to change notification settings - Fork 166
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
[System Test] verify cluster-wide-encryption key at clusterfull, noobaa rebuild and after noobaa db recovery #10082
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: nagendra202 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: nagendra202 <[email protected]>
Signed-off-by: nagendra202 <[email protected]>
Signed-off-by: nagendra202 <[email protected]>
b4b168c
to
6b2c183
Compare
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.
PR validation on existing cluster
Cluster Name: nagreddy-s10-417
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job UNSTABLE (some or all tests failed).
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.
Unknown PR validation on existing cluster
Cluster Name: nagreddy-s10-417
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job state: ABORTED.
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.
PR validation on existing cluster
Cluster Name: nagreddy-s10-417
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job UNSTABLE (some or all tests failed).
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.
PR validation on existing cluster
Cluster Name: nagreddy-s12-417
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job UNSTABLE (some or all tests failed).
tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Outdated
Show resolved
Hide resolved
tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Outdated
Show resolved
Hide resolved
…ion_system.py Co-authored-by: Sagi Hirshfeld <[email protected]>
…ion_system.py Co-authored-by: Sagi Hirshfeld <[email protected]>
ocs_ci/helpers/e2e_helpers.py
Outdated
bucket_name = bucket_factory()[0].name | ||
|
||
# Backup and restore noobaa db using fixture | ||
noobaa_db_backup_and_recovery_locally() |
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.
test will fail as it uses the default function scoped fixture bucket_factory()
so you need to pass the parent function argument bucket_factory which is session scoped
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.
done.
Signed-off-by: nagendra202 <[email protected]>
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.
PR validation
Cluster Name:
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job FAILED (installation failed, tests not executed).
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.
addressed review comments.
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.
PR validation on existing cluster
Cluster Name: nagreddy-s25-bz
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job UNSTABLE (some or all tests failed).
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.
PR validation on existing cluster
Cluster Name: nagreddy-s27-pr
Cluster Configuration:
PR Test Suite: system_test
PR Test Path: tests/cross_functional/system_test/test_cluster_wide_key_rotation_system.py
Additional Test Params:
OCP VERSION: 4.17
OCS VERSION: 4.17
tested against branch: master
Job UNSTABLE (some or all tests failed).
…age#10535) Signed-off-by: Sidhant Agrawal <[email protected]>
…vider cluster only (red-hat-storage#10172) In provider mode multicluster run, set 'cluster_kubeconfig' if the resource kind is available only in provider cluster. Created a primary list of 'Kind' which are available in provider cluster. Set the cluster_kubeconfig value to provider cluster kubeconfig path if the Kind is available in the list. Signed-off-by: Jilju Joy <[email protected]>
…0539) Signed-off-by: prsurve <[email protected]>
…storage#10529) This reverts commit 97b6d5d. Signed-off-by: prsurve <[email protected]>
…0536) Signed-off-by: Uday Kurundwade <[email protected]>
Signed-off-by: oviner <[email protected]>
…10505) Signed-off-by: Daniel Horak <[email protected]>
Signed-off-by: Petr Balogh <[email protected]>
Signed-off-by: vavuthu <[email protected]>
Signed-off-by: Coady LaCroix <[email protected]>
Signed-off-by: Shylesh Kumar Mohan <[email protected]>
Signed-off-by: Aviadp <[email protected]>
…storage#10589) Signed-off-by: Itzhak Kave <[email protected]> Co-authored-by: Itzhak Kave <[email protected]>
Signed-off-by: nagendra202 <[email protected]>
@@ -357,6 +357,43 @@ | |||
PROVIDER_CLIENT_DEPLOYMENT_DIR, "native_storage_client.yaml" | |||
) | |||
|
|||
PROVIDER_CLUSTER_RESOURCE_KINDS = [ |
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.
Hi @jilju can you pls check if that is legit to pull this into master?
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.
This commit is already merged a week ago. Please rebase.
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.
@nagendra202 Please check
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. |
Mistakenly pulled different branches to my PR, hence closing this PR and the same code changes raised from an another branch PR- #10603 |
Automated below test case:
a) cluster nearing full ratios
b) noobaa db replacement
c) noobaa db backup recovery