Skip to content

quincy: qa: add POOL_APP_NOT_ENABLED to ignorelist for cephfs tests#53863

Merged
yuriw merged 5 commits intoceph:quincyfrom
vshankar:tr-62538
Oct 9, 2023
Merged

quincy: qa: add POOL_APP_NOT_ENABLED to ignorelist for cephfs tests#53863
yuriw merged 5 commits intoceph:quincyfrom
vshankar:tr-62538

Conversation

@vshankar
Copy link
Contributor

@vshankar vshankar commented Oct 6, 2023

https://tracker.ceph.com/issues/62538

This backport contains commits from 3 PRs:

in that order.

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

vshankar and others added 4 commits October 6, 2023 22:11
@vshankar
qa: add POOL_APP_NOT_ENABLED to ignorelist for cephfs tests
698e378 
Fixes: http://tracker.ceph.com/issues/62508
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 97b76a4)
@batrick @vshankar
Revert "Merge PR ceph#53077 into main"
c019ffd 
This reverts commit b8bf0c6, reversing
changes made to fe07f64.

Silencing this health warning is unnecessary if we stop creating the rbd pool
in CephFS testing.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit af4558f)
@batrick @vshankar
qa: do not create rbd pool for CephFS QA
da092a1 
Fixes: https://tracker.ceph.com/issues/62482
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 636d2a4)
@vshankar
Revert "Revert "Merge PR ceph#53077 into main""
5a7d5bb 
This reverts commit af4558f.

Fixes: http://tracker.ceph.com/issues/62482
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 7d5795d)
@vshankar vshankar requested a review from a team October 6, 2023 16:45
@vshankar
Copy link
Contributor Author

vshankar commented Oct 6, 2023

@yuriw please include this change with

@github-actions github-actions bot added this to the quincy milestone Oct 6, 2023
@vshankar vshankar changed the title quincy": qa: add POOL_APP_NOT_ENABLED to ignorelist for cephfs tests quincy: qa: add POOL_APP_NOT_ENABLED to ignorelist for cephfs tests Oct 6, 2023
@vshankar
qa: use exisitng ignorelist override list for fs:mirror[-ha]
438cb17 
The custom ignorelist yaml would also require `POOL_APP_NOT_ENABLED`
to be ignored. The custom yaml is unnecessary anyway.

Fixes: http://tracker.ceph.com/issues/62482
Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit a728d17)
vshankar added a commit to vshankar/ceph that referenced this pull request Oct 9, 2023
@vshankar
Merge PR ceph#53863 into wip-vshankar-testing-20231009.053123
64ec891 
* refs/pull/53863/head:
	qa: use exisitng ignorelist override list for fs:mirror[-ha]
	Revert "Revert "Merge PR ceph#53077 into main""
	qa: do not create rbd pool for CephFS QA
	Revert "Merge PR ceph#53077 into main"
	qa: add `POOL_APP_NOT_ENABLED` to ignorelist for cephfs tests
vshankar added a commit to vshankar/ceph that referenced this pull request Oct 9, 2023
@vshankar
Merge PR ceph#53863 into wip-vshankar-testing-20231009.111138
bb47e9a 
* refs/pull/53863/head:
	qa: use exisitng ignorelist override list for fs:mirror[-ha]
	Revert "Revert "Merge PR ceph#53077 into main""
	qa: do not create rbd pool for CephFS QA
	Revert "Merge PR ceph#53077 into main"
	qa: add `POOL_APP_NOT_ENABLED` to ignorelist for cephfs tests
vshankar
vshankar commented Oct 9, 2023
View reviewed changes
Copy link
Contributor Author

@vshankar vshankar left a comment

Choose a reason for hiding this comment

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

#53285

@vshankar
Copy link
Contributor Author

vshankar commented Oct 9, 2023

Reviewed-by: Venky Shankar vshankar@redhat.com

yuriw
yuriw approved these changes Oct 9, 2023
View reviewed changes
Copy link
Contributor

@yuriw yuriw left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Venky Shankar vshankar@redhat.com

@yuriw yuriw merged commit b00ee4e into ceph:quincy Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuriw yuriw yuriw approved these changes

Assignees

No one assigned

Labels

cephfs Ceph File System needs-qa quincy-batch-1 wip-yuri-testing

Projects

None yet

Milestone

quincy

Development

Successfully merging this pull request may close these issues.

3 participants

@vshankar @yuriw @batrick