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

Earmark feature updated #4363

Closed
wants to merge 50 commits into from
Closed

Earmark feature updated #4363

wants to merge 50 commits into from

Conversation

Harishacharya-redhat
Copy link

Description

test cases: CEPH-83577841 - [Feature Validation] Earmark Feature Validation for NFS-Ganesha

logs : http://magna002.ceph.redhat.com/ceph/ceph-qe-logs/hacharya/earmark/

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

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Harishacharya-redhat

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

Harishacharya-redhat and others added 26 commits January 8, 2025 14:39
@Harishacharya-redhat
adding earmark feature tests
9b0af94 
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Switch test parallel module to use process pool.
3c57d46 
This commits switches to use ProcessPoolExecutor to allow parallel
test modules to spins threads inside their own process.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Remove gevent from requirements
7e5a24b 
This commit helps us to decouple from gevent module and removes
the package installation.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Remove gvent sleep
7f44c88 
As the next steps, removing methods used from gevent. Sleep in
between threads was used and this commit uses time sleep now
instead of gevent's method.

This helps to move out of gevent.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
RHCEPHQE-16740 - Move to concurrent.futures
7f70f7b 
In this PR, parallel is refactored to use concurrent.futures
instead of gvent. This allows us to use threads or processes.

Along with the introduction of timeouts at the parallel phase. It
also allows to avoid the dependency on gvent.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Implemented review comments
ab66819 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Refactoring log format and Enhancing parallel executions
c095e2c 
Signed-off-by: Amarnath K <amk@li-132ff74c-24e7-11b2-a85c-eca3000aa1e4.ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Fix ValueError - formatting field modname
d535ac5 
The right parameter is module. This commit modifies the log
format.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Implemented review comments
29f6bfd 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Implemented review comments
c11b15c 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Implemented review comments
9239b2f 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@tintumathew10 @Harishacharya-redhat
Updating service account psi-ceph-jenkins user to ceph-qe-sa user
88074e1 
Signed-off-by: tintumathew10 <tmathew@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Switch mergify to support main branch
8c46056 
The default branch is main hence modifying mergify to work with
main instead of master branch.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@viduship @Harishacharya-redhat
Changing the test suite for transition tests to 3 way ms in reef as well
668f4a4 
Signed-off-by: Vidushi Mishra <vimishra@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@TejasC88 @Harishacharya-redhat
CI Integration for 2 tests, CRLF Injection using Curl and higher than…
30a6c2e 
… default shard DBR

Signed-off-by: Tejas Chandramouli <tchandra@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
TFA fix for CG quiesce tests
40b7eb2 
Signed-off-by: Suma R <sumar@Sumas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Adding Validation for subvolume info for cloned Volumes
e0efac0 
Signed-off-by: Amarnath K <amk@li-132ff74c-24e7-11b2-a85c-eca3000aa1e4.ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
test backported to pacific
790eaad 
Signed-off-by: Sunil Angadi <Sunil.Angadi@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@tintumathew10 @Harishacharya-redhat
Replacing master branch with main branch in cephci
a3be92d 
Signed-off-by: tintumathew10 <tmathew@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Fix for unmap device with encryption
f0e065b 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
rebase
f5f1e50 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
fix validating mon addrs
34541c2 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
fix tox issue
c4b6d23 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
fix tox issue
45e56ec 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
fix tox issue - new line character
896425e 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Manimaran-MM @Harishacharya-redhat
fix test suite
d940c59 
Signed-off-by: Manimaran M <manim@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
Chaitanya Dommeti and others added 17 commits January 8, 2025 14:39
@Harishacharya-redhat
Addressed review comment
197f01d 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Adding cluster configs to test_parallel
6658c4d 
Signed-off-by: Amarnath K <amk@li-132ff74c-24e7-11b2-a85c-eca3000aa1e4.ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@s-vipin @Harishacharya-redhat
Improvements to monitor election strategy
3e24cb5 
Signed-off-by: Vipin M S <vips@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
adding support for testing rgw ssl deployment with cephadm generate_cert
5bfd47f 
Signed-off-by: Hemanth Sai Maheswarla <hemanthsaimaheswarla@Hemanths-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@yuva-gayam @Harishacharya-redhat
Updated the conf file path in all test suites
b4e8932 
Signed-off-by: YuvaTejaSreeGayam <yuva.teja.sree.gayam@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@yuva-gayam @Harishacharya-redhat
adding conf for suites in rgw squid
18abe55 
Signed-off-by: YuvaTejaSreeGayam <yuva.teja.sree.gayam@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@yuva-gayam @Harishacharya-redhat
adding conf path for suites in rgw squid
de52c3d 
Signed-off-by: YuvaTejaSreeGayam <yuva.teja.sree.gayam@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@yuva-gayam @Harishacharya-redhat
Changes to be committed:
048d515 
modified:   suites/squid/rgw/tier-2_ssl_rgw_ecpool_test.yaml

Signed-off-by: YuvaTejaSreeGayam <yuva.teja.sree.gayam@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@yuva-gayam @Harishacharya-redhat
Changes modified: suites/squid/rgw/tier-2_ssl_rgw_ecpool_test.yaml
a659a98 
Signed-off-by: YuvaTejaSreeGayam <yuva.teja.sree.gayam@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@hkadam134 @Harishacharya-redhat
TFA fix - increase timeout for selinux
9376135 
Signed-off-by: hkadam134 <hyelloji@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Addressed review comments
5da3930 
Signed-off-by: Chaitanya Dommeti <chaitanyadommeti@Chaitanyas-MacBook-Pro.local>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Fix codeowners file
123bc0c 
Earlier cephci-top-reviewers had open permissions to merge code
to any place. The teams mentioned for folders had no impact. This
caused improper delegation of ownership.

This change removes the top-reviewers group and sets the complete
privileges to ceph-qe-ci and ceph-qe-admin groups. They have full
access to merge.

The delegation to each DFG is now done via specfic users with their
managers having permissions to merge in their respective components.

Going forward, anyone should be able to raise a PR to request for
merge permissions with admin and ci teams having approval privileges.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Fixing method defnition
4b8bfde 
Signed-off-by: Amarnath K <amk@li-132ff74c-24e7-11b2-a85c-eca3000aa1e4.ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@harshkumarRH @Harishacharya-redhat
RHCEPHQE-16887 | CEPH-83604474: Test to verify MAX_AVAIL updation upo…
9f88459 
…n OSD removal

Signed-off-by: Harsh Kumar <hakumar@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@harshkumarRH @Harishacharya-redhat
[RADOS] TFA: Exclude CBT qfsck and allocmap cmds for Pacific
d8dc949 
Signed-off-by: Harsh Kumar <hakumar@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@harshkumarRH @Harishacharya-redhat
[RADOS] Reduction execution time of tier-3_rados_test-4-node-ecpools …
8dd932d 
…test suite

Signed-off-by: Harsh Kumar <hakumar@redhat.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@psathyan @Harishacharya-redhat
Fix mergify configuration
3ad94d5 
With the fix of code owners, we removed the top level reviewers
group. This enabled us to align with the purpose of codeowners
file.

After the fix merged, the mergify configuration was left unchanged
resulting in a broken condition.

This PR fixes the mergify conditions and moves the merge mechanism
to queue.

Signed-off-by: Pragadeeswaran Sathyanarayanan <pragadeeswaran.sathyanarayanan@ibm.com>
Signed-off-by: Harishacharya-redhat <hacharya@redhat.com>
@Harishacharya-redhat
Copy link
Author

Harishacharya-redhat commented Jan 8, 2025 •
edited
Loading

git rebase HEAD~2 --signoff
git push --force-with-lease origin earmark_feature_updated

When i gave this command its showing 50 commits

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

@msainiRedhat msainiRedhat Awaiting requested review from msainiRedhat

@vdas-redhat vdas-redhat Awaiting requested review from vdas-redhat vdas-redhat is a code owner

@pranavprakash20 pranavprakash20 Awaiting requested review from pranavprakash20 pranavprakash20 is a code owner

@neha-gangadhar neha-gangadhar Awaiting requested review from neha-gangadhar neha-gangadhar is a code owner

@AmarnatReddy AmarnatReddy Awaiting requested review from AmarnatReddy AmarnatReddy is a code owner

@Manimaran-MM Manimaran-MM Awaiting requested review from Manimaran-MM Manimaran-MM is a code owner

@pdhiran pdhiran Awaiting requested review from pdhiran pdhiran is a code owner

@Manohar-Murthy Manohar-Murthy Awaiting requested review from Manohar-Murthy Manohar-Murthy is a code owner

@mkasturi18 mkasturi18 Awaiting requested review from mkasturi18 mkasturi18 is a code owner

@ckulal ckulal Awaiting requested review from ckulal ckulal is a code owner

@TejasC88 TejasC88 Awaiting requested review from TejasC88 TejasC88 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.