Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

quincy: osd/scrub: Reintroduce scrub starts message #47621

Merged
merged 1 commit into from Sep 2, 2022

Conversation

pdvian
Copy link

@pdvian pdvian commented Aug 16, 2022

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


backport of #46438
parent tracker: https://tracker.ceph.com/issues/55798

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

@pdvian pdvian requested a review from a team as a code owner August 16, 2022 04:23
@pdvian pdvian added this to the quincy milestone Aug 16, 2022
@pdvian pdvian added the core label Aug 16, 2022
@pdvian
Copy link
Author

pdvian commented Aug 16, 2022

@ronen-fr Kindly review this quincy backport PR. I have not included formatter<pg_t> changes in this PR as <pg_t> formatter is not part of quincy release yet :

template <>
struct fmt::formatter<pg_t> {
  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }

  template <typename FormatContext>
  auto format(const pg_t& pg, FormatContext& ctx)
  {
    return fmt::format_to(ctx.out(), "{}.{:x}", pg.pool(), pg.m_seed);
  }
};

@pdvian pdvian requested a review from ronen-fr August 16, 2022 04:28
Whenever the scrubbing process starts for a PG, We used to log
scrub/deep-scrub "starts" message in the cluster log. We are
not seeing scrub "starts" messages anymore. Reintroduce scrub
starts message in order to calculate exact time taken to
scrub/deep-scrub the PG.

Fixes: https://tracker.ceph.com/issues/55798

Signed-off-by: Prashant D <pdhange@redhat.com>
(cherry picked from commit 2f6272a)

Conflicts:
	src/osd/osd_types_fmt.h: Add formatter for pg_t. Remove
                                 formatter for spg_t, pg_history_t,
                                 pg_info_t and others as they are
                                 not part of quincy yet.
@pdvian
Copy link
Author

pdvian commented Aug 16, 2022

@ronen-fr Kindly review this quincy backport PR. I have not included formatter<pg_t> changes in this PR as <pg_t> formatter is not part of quincy release yet :

template <>
struct fmt::formatter<pg_t> {
  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }

  template <typename FormatContext>
  auto format(const pg_t& pg, FormatContext& ctx)
  {
    return fmt::format_to(ctx.out(), "{}.{:x}", pg.pool(), pg.m_seed);
  }
};

@ronen-fr I have added fmt::formatter<pg_t> changes to this PR. It needs to be included in quincy as we are using pg_t formatter in set_scrub_begin_time. Kindly review this PR.

@ronen-fr
Copy link
Contributor

@pdvian : LGTM. Not sure if the 'make check' failure (which is unrelated) means this should be rebased.

@ronen-fr
Copy link
Contributor

jenkins retest this please

@pdvian
Copy link
Author

pdvian commented Aug 24, 2022

jenkins retest this please

@ljflores
Copy link
Contributor

Rados suite review: https://pulpito.ceph.com/?branch=wip-yuri-testing-2022-08-23-1120-quincy

Failures, unrelated:
1. https://tracker.ceph.com/issues/57303 -- Pending Quincy backport
2. https://tracker.ceph.com/issues/57269
3. https://tracker.ceph.com/issues/57165 -- Fix under review
4. https://tracker.ceph.com/issues/57270
5. https://tracker.ceph.com/issues/57311
6. https://tracker.ceph.com/issues/49287

Details:
1. rados/cephadm: Failed to fetch package version from https://shaman.ceph.com/api/search/?status=ready&project=ceph&flavor=default&distros=ubuntu%2F22.04%2Fx86_64&sha1=b34ca7d1c2becd6090874ccda56ef4cd8dc64bf7 - Ceph - Orchestrator
2. rook: unable to read URL "https://docs.projectcalico.org/manifests/tigera-operator.yaml" - Ceph - Orchestrator
3. expected valgrind issues and found none - Ceph - RADOS
4. cephadm: RuntimeError: Failed command: apt-get update: E: The repository 'https://download.ceph.com/debian-octopus jammy Release' does not have a Release file. - Ceph - Orchestrator
5. rook: ensure CRDs are installed first - Ceph - Orchestrator
6. podman: setting cgroup config for procHooks process caused: Unit libpod-$hash.scope not found - Ceph - Orchestrator

@ljflores
Copy link
Contributor

jenkins test make check

3 similar comments
@yuriw
Copy link
Contributor

yuriw commented Aug 29, 2022

jenkins test make check

@yuriw
Copy link
Contributor

yuriw commented Aug 30, 2022

jenkins test make check

@yuriw
Copy link
Contributor

yuriw commented Aug 31, 2022

jenkins test make check

@yuriw
Copy link
Contributor

yuriw commented Aug 31, 2022

@pdvian
This PR failed make check:
The following tests FAILED:
31 - run-rbd-unit-tests-1.sh (Timeout)
273 - run-tox-alerts-lint (Failed)

@ilya FYI

https://jenkins.ceph.com/job/ceph-pull-requests/102848/

@yuriw
Copy link
Contributor

yuriw commented Sep 1, 2022

jenkins test make check

1 similar comment
@yuriw
Copy link
Contributor

yuriw commented Sep 1, 2022

jenkins test make check

@yuriw
Copy link
Contributor

yuriw commented Sep 1, 2022

@pdvian can't merge it :(

@rzarzynski
Copy link
Contributor

jenkins test make check

@rzarzynski rzarzynski merged commit 2c54363 into ceph:quincy Sep 2, 2022
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants