Skip to content

squid: qa: do not iterate list being modified#57063

Merged
batrick merged 2 commits intoceph:squidfrom
batrick:wip-65620-squid
May 22, 2024
Merged

squid: qa: do not iterate list being modified#57063
batrick merged 2 commits intoceph:squidfrom
batrick:wip-65620-squid

Conversation

@batrick
Copy link
Member

@batrick batrick commented Apr 23, 2024

backport tracker: https://tracker.ceph.com/issues/65620


backport of #56638
parent tracker: https://tracker.ceph.com/issues/65022

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

batrick added 2 commits April 23, 2024 14:27
This is done in CephFSTestCase.tearDown.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 98f161b)
Fixes: https://tracker.ceph.com/issues/65022
Fixes: bb71023
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 57022d9)
@batrick batrick added this to the squid milestone Apr 23, 2024
@batrick batrick added the tests label Apr 23, 2024
@github-actions github-actions bot added the cephfs Ceph File System label Apr 23, 2024
@batrick
Copy link
Member Author

batrick commented Apr 24, 2024

jenkins test api

@batrick
Copy link
Member Author

batrick commented Apr 24, 2024

jenkins test windows

@batrick
Copy link
Member Author

batrick commented May 17, 2024

This PR is under test in https://tracker.ceph.com/issues/66101.

@batrick batrick modified the milestones: v19.1.0, v19.1.1 May 17, 2024
Copy link
Member Author

@batrick batrick left a comment

Choose a reason for hiding this comment

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

@batrick batrick requested a review from a team May 22, 2024 18:02
@batrick batrick merged commit 207e116 into ceph:squid May 22, 2024
@batrick batrick deleted the wip-65620-squid branch May 22, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants