Skip to content

qa: ignore cluster warnings generated from forward-scrub task#56699

Merged
vshankar merged 2 commits intoceph:mainfrom
vshankar:wip-48562
May 13, 2024
Merged

qa: ignore cluster warnings generated from forward-scrub task#56699
vshankar merged 2 commits intoceph:mainfrom
vshankar:wip-48562

Conversation

@vshankar
Copy link
Contributor

@vshankar vshankar commented Apr 4, 2024 •
edited by mchangir
Loading

THe forward-srub tasks intentioanlly removes objects from the metadata pool, so this warning is expected. This is porbably a fallout from the additional clog changes.

Fixes: http://tracker.ceph.com/issues/48562

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. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

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
  • jenkins test rook e2e

@vshankar vshankar added the cephfs Ceph File System label Apr 4, 2024
@vshankar vshankar requested a review from a team April 4, 2024 12:17
@mchangir
Copy link
Contributor

mchangir commented Apr 5, 2024 •
edited
Loading

My qa tests revealed that you need to ignore a log of Corrupt dentry as well

2024-04-05T01:32:12.389268+0000 mds.d (mds.0) 1 : cluster [WRN] Corrupt dentry 'file_to_be_damaged' in dir frag 0x10000000000: Invalid tag char 'd' pos 0: Malformed input [buffer:3](/subdir)

@mchangir mchangir mentioned this pull request Apr 5, 2024
Closed
14 tasks
@vshankar
Copy link
Contributor Author

vshankar commented Apr 8, 2024

My qa tests revealed that you need to ignore a log of Corrupt dentry as well

doh. I'll add that (it's coming from CDir due to the test)

vshankar added 2 commits April 10, 2024 09:53
@vshankar
qa: ignore object missing on disk warning
5cf3a64 
THe forward-srub tasks intentioanlly removes objects from the
metadata pool, so this warning is expected. This is porbably
a fallout from the additional clog changes.

Fixes: http://tracker.ceph.com/issues/48562
Signed-off-by: Venky Shankar <vshankar@redhat.com>
@vshankar
qa: ignore Invalid tag char warning
f4f135b 
@mchangir mentioned about this showing up in fs suite. One of
the test mangles a omap value causing this warning to show up
when loading a dentry.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
@vshankar vshankar requested a review from a team April 10, 2024 04:23
@vshankar vshankar changed the title qa: ignore object missing on disk warning qa: ignore cluster warnings generated from forward-scrub task Apr 10, 2024
mchangir
mchangir approved these changes Apr 10, 2024
View reviewed changes
Copy link
Contributor

@mchangir mchangir left a comment

Choose a reason for hiding this comment

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

LGTM

vshankar added a commit to vshankar/ceph that referenced this pull request Apr 12, 2024
@vshankar
Merge PR ceph#56699 into wip-vshankar-testing-20240411.061452
a3082c6 
* refs/pull/56699/head:
	qa: ignore `Invalid tag char` warning
	qa: ignore `object missing on disk` warning

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
vshankar added a commit to vshankar/ceph that referenced this pull request Apr 30, 2024
@vshankar
Merge PR ceph#56699 into wip-vshankar-testing-20240430.111407-debug
df420f0 
* refs/pull/56699/head:
	qa: ignore `Invalid tag char` warning
	qa: ignore `object missing on disk` warning

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
vshankar added a commit to vshankar/ceph that referenced this pull request May 6, 2024
@vshankar
Merge PR ceph#56699 into wip-vshankar-testing-20240506.064357
89f981a 
* refs/pull/56699/head:
	qa: ignore `Invalid tag char` warning
	qa: ignore `object missing on disk` warning

Reviewed-by: Milind Changire <mchangir@redhat.com>
Reviewed-by: Leonid Usov <leonid.usov@ibm.com>
@vshankar
Copy link
Contributor Author

vshankar commented May 13, 2024

https://tracker.ceph.com/projects/cephfs/wiki/Main#2024-05-13

@vshankar
Copy link
Contributor Author

vshankar commented May 13, 2024

jenkins test make check

@vshankar vshankar merged commit bd42acb into ceph:main May 13, 2024
This was referenced May 22, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mchangir mchangir mchangir approved these changes

@joscollin joscollin joscollin approved these changes

+1 more reviewer

@leonid-s-usov leonid-s-usov leonid-s-usov approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cephfs Ceph File System

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vshankar @mchangir @leonid-s-usov @joscollin