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.

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

Automation of testlib for 8.1 cephfs fscrypt support #4490

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sumabai
Copy link
Contributor

@sumabai sumabai commented Feb 27, 2025

Description

JIRA: https://issues.redhat.com/browse/RHCEPHQE-17982

Test Plan: https://docs.google.com/document/d/1DLzimNfe9lieobUiiybEyDRnrr4JpYDpSdn5aiK1l0c/edit?tab=t.0

Fix Description : Adding testlib for most fscrypt cli command options
setup
status
encrypt
lock
unlock
purge
metadata - create, destroy,add,remove

Logs : http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-RUBJ4D
There is one script issue in log with get_status method which has been fixed with latest changes, but i could not validate the same as the upstream priv build setup got purged few mins ago.

Known kernel client prod issues in the log:

  • incorrect passphrase in prompts when id is inline, but cmds suceed
  • fscrypt lock mentions some files are open, but none of test files were open, also even after running suggested action cmds, the prompt for open files continue. But test files were locked
  • After purge, the policy and protetcor info is still listed in fscrypt status but it was to be wiped of with purge

Will discuss with dev and raise Kernel BZs.
Please include Automation development guidelines. Source of Test case - New Feature/Regression Test/Close loop of customer BZs

click to expand checklist
  • Create a test case in Polarion reviewed and approved.
  • Create a design/automation approach doc. Optional for tests with similar tests already automated.
  • Review the automation design
  • Implement the test script and perform test runs
  • Submit PR for code review and approve
  • Update Polarion Test with Automation script details and update automation fields
  • If automation is part of Close loop, update BZ flag qe-test_coverage “+” and link Polarion test

self.mgrs = ceph_cluster.get_ceph_objects("mgr")
self.osds = ceph_cluster.get_ceph_objects("osd")
self.mdss = ceph_cluster.get_ceph_objects("mds")
self.clients = ceph_cluster.get_ceph_objects("client")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we access this from the self.fs_util object since these elements are also available in the utils object?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants