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

Test Automation for encryption dashboard summary #11043

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Test Automation for encryption dashboard summary #11043

merged 1 commit into from
Jan 15, 2025

Conversation

paraggit
Copy link
Contributor

@paraggit paraggit commented Dec 12, 2024 •
edited
Loading

Test Automation for encryption dashboard summary.

Following encryption summary details validated against storagecluster and noobaa spec.

Object Storage Encryption Summary details

  1. Navigate to object details page
  2. Open encryption details.
  3. verify encryption data with the nooba and storagecluster spec.
    -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Screenshot 2024-12-13 at 7 57 54 PM

Block And File Encryption summary details

  1. Navigate to file and block details page
  2. Open encryption details.
  3. verify encryption data with the nooba and storagecluster spec.
    -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Screenshot 2024-12-13 at 7 57 34 PM

@paraggit paraggit requested review from a team as code owners December 12, 2024 18:54
@pull-request-size pull-request-size bot added the size/L PR that changes 100-499 lines label Dec 12, 2024
@paraggit paraggit changed the title [WIP] Test Automation for encryption dashboard summary Test Automation for encryption dashboard summary Dec 13, 2024
@paraggit paraggit force-pushed the encryption_configuration_dashboard branch from a51d176 to b4bee92 Compare December 15, 2024 03:49
ocs-ci
ocs-ci reviewed Dec 15, 2024
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci
ocs-ci reviewed Dec 15, 2024
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci
ocs-ci reviewed Dec 16, 2024
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job UNSTABLE (some or all tests failed).

@paraggit paraggit force-pushed the encryption_configuration_dashboard branch 2 times, most recently from c3d01cb to 230c70a Compare January 1, 2025 11:24
ebenahar
ebenahar reviewed Jan 1, 2025
View reviewed changes
ocs_ci/ocs/ui/page_objects/block_and_file.py Outdated Show resolved Hide resolved
ocs_ci/ocs/ui/page_objects/object_details_tab.py Outdated Show resolved Hide resolved
ocs_ci/ocs/ui/page_objects/object_details_tab.py Outdated Show resolved Hide resolved
ocs_ci/ocs/ui/page_objects/object_details_tab.py Outdated Show resolved Hide resolved
ocs-ci
ocs-ci reviewed Jan 1, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

@paraggit paraggit requested review from a team as code owners January 6, 2025 07:45
@pull-request-size pull-request-size bot added size/XXL and removed size/L PR that changes 100-499 lines labels Jan 6, 2025
@paraggit paraggit force-pushed the encryption_configuration_dashboard branch from 1dcab09 to c52b666 Compare January 6, 2025 07:52
@pull-request-size pull-request-size bot added size/L PR that changes 100-499 lines and removed size/XXL labels Jan 6, 2025
@paraggit paraggit removed request for a team January 6, 2025 07:53
@pull-request-size pull-request-size bot added the size/L PR that changes 100-499 lines label Jan 7, 2025
ocs-ci
ocs-ci reviewed Jan 7, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job UNSTABLE (some or all tests failed).

@paraggit paraggit force-pushed the encryption_configuration_dashboard branch 2 times, most recently from 2372db1 to 42a2c74 Compare January 8, 2025 10:41
ocs-ci
ocs-ci reviewed Jan 8, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci
ocs-ci reviewed Jan 8, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

ocs-ci
ocs-ci reviewed Jan 8, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job UNSTABLE (some or all tests failed).

@paraggit paraggit force-pushed the encryption_configuration_dashboard branch from f7f4211 to 55064f3 Compare January 14, 2025 04:10
ocs-ci
ocs-ci reviewed Jan 14, 2025
View reviewed changes
Copy link

@ocs-ci ocs-ci left a 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: tier1
PR Test Path: tests/functional/encryption/test_encryption_configuration_dashboard.py
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job PASSED.

@paraggit paraggit added the Verified Mark when PR was verified and log provided label Jan 14, 2025
@DanielOsypenko
Copy link
Contributor

/lgtm

are there known platforms that do not support encryption? If so, we may want to add skip_if_
ROSA? DR? External?

@openshift-ci openshift-ci bot added the lgtm label Jan 14, 2025
@DanielOsypenko DanielOsypenko removed their assignment Jan 14, 2025
@paraggit
Copy link
Contributor Author

/lgtm

are there known platforms that do not support encryption? If so, we may want to add skip_if_ ROSA? DR? External?
It should support both external and internal modes. I will confirm with the ROSA and DR setups and make changes accordingly.

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Jan 15, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: DanielOsypenko, ebenahar, jilju, paraggit, shyRozen

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ebenahar ebenahar merged commit 6f44593 into red-hat-storage:master Jan 15, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ocs-ci ocs-ci ocs-ci left review comments

@DanielOsypenko DanielOsypenko DanielOsypenko approved these changes

@jilju jilju jilju approved these changes

@shyRozen shyRozen shyRozen approved these changes

@ebenahar ebenahar ebenahar approved these changes

Assignees

@ebenahar ebenahar

@shyRozen shyRozen

@jilju jilju

@DanielOsypenko DanielOsypenko

Labels
lgtm size/L PR that changes 100-499 lines Verified Mark when PR was verified and log provided
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@paraggit @DanielOsypenko @ebenahar @shyRozen @jilju @ocs-ci @openshift-merge-robot