Skip to content

qa/rgw: rgw suite runs rgw-related dashboard api tests#63306

Draft
cbodley wants to merge 1 commit intoceph:mainfrom
cbodley:wip-qa-rgw-dashboard
Draft

qa/rgw: rgw suite runs rgw-related dashboard api tests#63306
cbodley wants to merge 1 commit intoceph:mainfrom
cbodley:wip-qa-rgw-dashboard

Conversation

@cbodley
Copy link
Contributor

@cbodley cbodley commented May 15, 2025 •
edited
Loading

allow the rgw suite to catch regressions to the dashboard api tests

copies bits from qa/suites/rados/dashboard into rgw/dashboard, except:

  • omit tasks/e2e.yaml which doesn't exercise rgw
  • don't customize 'mon election default strategy'

this adds a single job to the rgw suite, where the rados suite had 4

Show available Jenkins commands

@cbodley
Copy link
Contributor Author

cbodley commented May 15, 2025

first attempt just symlinked rados/dashboard under rgw/ but showed consistent failures:
https://pulpito.ceph.com/cbodley-2025-05-13_15:47:27-rgw-main-distro-default-smithi/
https://pulpito.ceph.com/cbodley-2025-05-15_15:38:59-rgw-main-distro-default-smithi/

@cbodley cbodley mentioned this pull request May 16, 2025
Open
@epuertat epuertat requested review from a team, afreen23, dnyanee1997 and prgoel-code and removed request for a team May 16, 2025 16:37
@cbodley
Copy link
Contributor Author

cbodley commented May 16, 2025

from https://qa-proxy.ceph.com/teuthology/cbodley-2025-05-15_18:10:45-rgw-main-distro-default-smithi/8286305/teuthology.log

2025-05-15T18:30:58.718 INFO:tasks.cephfs_test_runner:======================================================================
2025-05-15T18:30:58.718 INFO:tasks.cephfs_test_runner:ERROR: setUpClass (tasks.mgr.dashboard.test_rgw.RgwApiCredentialsTest)
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:----------------------------------------------------------------------
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:Traceback (most recent call last):
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_cbodley_ceph_dfb3c940e5d99e287062e91383c08a5c1542ab19/qa/tasks/mgr/dashboard/test_rgw.py", line 28, in setUpClass
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:    super(RgwTestCase, cls).setUpClass()
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_cbodley_ceph_dfb3c940e5d99e287062e91383c08a5c1542ab19/qa/tasks/mgr/dashboard/helper.py", line 218, in setUpClass
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:    cls.update_base_uri()
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_cbodley_ceph_dfb3c940e5d99e287062e91383c08a5c1542ab19/qa/tasks/mgr/dashboard/helper.py", line 266, in update_base_uri
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:    cls._base_uri = cls._get_uri("dashboard").rstrip('/')
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_cbodley_ceph_dfb3c940e5d99e287062e91383c08a5c1542ab19/qa/tasks/mgr/mgr_test_case.py", line 211, in _get_uri
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:    cls.wait_until_true(lambda: _get_or_none() is not None, 30)
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/github.com_cbodley_ceph_dfb3c940e5d99e287062e91383c08a5c1542ab19/qa/tasks/ceph_test_case.py", line 339, in wait_until_true
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:    raise TestTimeoutError("Timed out after {0}s and {1} retries".format(elapsed, retry_count))
2025-05-15T18:30:58.719 INFO:tasks.cephfs_test_runner:tasks.ceph_test_case.TestTimeoutError: Timed out after 30s and 0 retries

guessing that tasks.mgr.dashboard.test_rgw has some dependencies, so trying again with tasks.mgr.test_dashboard added first

@cbodley
qa/rgw: rgw suite runs rgw-related dashboard api tests
e0ae150 
allow the rgw suite to catch regressions to the dashboard api tests

copies bits from qa/suites/rados/dashboard into rgw/dashboard, except:
* omit tasks/e2e.yaml which doesn't exercise rgw
* don't customize 'mon election default strategy'

this adds a single job to the rgw suite, where the rados suite had 4

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley cbodley force-pushed the wip-qa-rgw-dashboard branch from d6e90ec to e0ae150 Compare May 19, 2025 20:53
@cbodley
Copy link
Contributor Author

cbodley commented May 20, 2025

testing seems to be blocked on https://tracker.ceph.com/issues/70669

ERROR: test_list_enabled_module (tasks.mgr.dashboard.test_mgr_module.MgrModuleTest)

looking back through https://pulpito.ceph.com/?suite=rados, that same test is failing in the rados suite on main, tentacle, and squid

@cbodley
Copy link
Contributor Author

cbodley commented Jul 3, 2025

testing seems to be blocked on https://tracker.ceph.com/issues/70669

ERROR: test_list_enabled_module (tasks.mgr.dashboard.test_mgr_module.MgrModuleTest)

looking back through https://pulpito.ceph.com/?suite=rados, that same test is failing in the rados suite on main, tentacle, and squid

fyi @epuertat and others, this is still failing on main

ex https://qa-proxy.ceph.com/teuthology/skanta-2025-07-02_03:56:39-rados-wip-bharath9-testing-2025-07-01-2106-distro-default-smithi/8366747/teuthology.log

@github-actions
Copy link

github-actions bot commented Sep 1, 2025

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@github-actions github-actions bot added the stale label Sep 1, 2025
@github-actions
Copy link

github-actions bot commented Oct 1, 2025

This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution!

@github-actions github-actions bot closed this Oct 1, 2025
@adamemerson adamemerson reopened this Oct 3, 2025
@adamemerson adamemerson added pinned Use this label if you want to exempt a PR from being stalled and removed stale labels Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dnyanee1997 dnyanee1997 Awaiting requested review from dnyanee1997 dnyanee1997 was automatically assigned from ceph/dashboard

@prgoel-code prgoel-code Awaiting requested review from prgoel-code prgoel-code was automatically assigned from ceph/dashboard

@afreen23 afreen23 Awaiting requested review from afreen23

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pinned Use this label if you want to exempt a PR from being stalled rgw tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cbodley @adamemerson