Skip to content

qa/task/rgw: instruct Valgrind to don't exit on first error.#38966

Merged
cbodley merged 1 commit intoceph:masterfrom
rzarzynski:wip-qa-rgw-no-valgrind-early-exit
Jan 20, 2021
Merged

qa/task/rgw: instruct Valgrind to don't exit on first error.#38966
cbodley merged 1 commit intoceph:masterfrom
rzarzynski:wip-qa-rgw-no-valgrind-early-exit

Conversation

@rzarzynski
Copy link
Contributor

This requires ceph/teuthology#1600.

Signed-off-by: Radoslaw Zarzynski rzarzyns@redhat.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

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 api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
@rzarzynski rzarzynski added the rgw label Jan 19, 2021
@github-actions github-actions bot added the tests label Jan 19, 2021
@jdurgin jdurgin requested a review from cbodley January 19, 2021 15:29
@cbodley
Copy link
Contributor

cbodley commented Jan 19, 2021

thanks @rzarzynski!

@jdurgin any idea what this will look like with cephadm?

Copy link
Contributor

@cbodley cbodley left a comment

Choose a reason for hiding this comment

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

we'll have to coordinate merging of this with the teuthology pr, but i'll pull it into my valgrind branch for testing

@jdurgin
Copy link
Member

jdurgin commented Jan 19, 2021

@cbodley cephadm isn't running valgrind yet, we should keep this in mind when we add that

@cbodley
Copy link
Contributor

cbodley commented Jan 19, 2021

jenkins test docs

@cbodley cbodley merged commit ae8db35 into ceph:master Jan 20, 2021
@cbodley
Copy link
Contributor

cbodley commented Jan 20, 2021

thank you gentlemen

@mattbenjamin
Copy link
Contributor

thank you gentlemen

+1 !!

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.

4 participants