Conversation
Member
Author
|
Still iterating on tests to reproduce problem in https://tracker.ceph.com/issues/54546 |
ffb1d71 to
4652a70
Compare
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Jun 23, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: add missing or correct .qa links
114eea0 to
17b3f80
Compare
|
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
6eb6692 to
b487a78
Compare
90f3773 to
4eaa20c
Compare
Member
Author
|
jenkins test make check arm64 |
aafe1de to
f97d4c5
Compare
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 9, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 17, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 19, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 20, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 21, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
Member
Author
batrick
added a commit
to batrick/ceph
that referenced
this pull request
Sep 26, 2022
* refs/pull/46821/head: qa: add testing for postgresql databases on CephFS qa: fs:workload add journal flush task qa: add snap-schedule task qa: reorganize scrub/workunit tasks qa: raise error if fwd_scrub discovers damage qa: allow specifying mount options for fuse qa: unify mountpoint config behavior qa: allow check_counter to look at nested keys qa: log mounts file if not mounted qa: add missing or correct .qa links
Member
Author
|
Now shows type of dentry damage (we're looking for "dentry" damage): |
find qa/suites/ -type d -execdir ln -sfT ../.qa/ {}/.qa \;
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
For debugging. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Only fuse would support the "mountpoint" yaml config. Move it to mount.py so it works consistently with the kernel mount. Ditto for the cephfs mount point. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Also, fix the type default to None. A list does not work in the code but the default is not used in QA. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
test_client_recovery was also using mntopts to specify additional options to ceph-fuse. Because the two prior commits unify the behavior of ceph-fuse and the kernel mount so that the "-o" option is available for both, that changes breaks this test. Add a special set of args available only for fuse (there is no equivalent on the kernel). Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
So we can enforce an ordering of tasks and ensure that the fs/subvolume is available before the task runs. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
To exercise snapshot creation and deletion during workloads. Fixes: https://tracker.ceph.com/issues/23724 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Idea here is to force the MDS to flush metadata mutations out of the journal. This may help expose any type of corruption seen with postgres. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Fixes: https://tracker.ceph.com/issues/55470 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Member
Author
vshankar
approved these changes
Oct 3, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR also adds snap schedules for fs:workload to exercise snapshots during the postgres task. That will also affect other workunits.
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windows