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

qa/run-tox-mgr-dashboard: Do not write to /tmp/test_sanitize_password… #42449

Merged
merged 1 commit into from
Jul 23, 2021

Conversation

kevinzs2048
Copy link
Contributor

@kevinzs2048 kevinzs2048 commented Jul 22, 2021

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

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

@kevinzs2048 kevinzs2048 requested a review from a team as a code owner July 22, 2021 08:48
@kevinzs2048 kevinzs2048 requested review from pereman2, aaSharma14 and nizamial09 and removed request for a team July 22, 2021 08:48
@tchaikov
Copy link
Contributor

@kevinzs2048 could you please drop the first commit from this PR?

@kevinzs2048
Copy link
Contributor Author

@tchaikov Aha my fault, will re-push

….txt file

To allow running multiple instances of the same tests.

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

Signed-off-by: Kevin Zhao <kevin.zhao@linaro.org>
@tchaikov
Copy link
Contributor

@alfonsomthd would you kindly help review this change?

Copy link
Contributor

@alfonsomthd alfonsomthd left a comment

Choose a reason for hiding this comment

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

LGTM

@alfonsomthd alfonsomthd added this to In progress in Dashboard via automation Jul 23, 2021
@tchaikov tchaikov merged commit ade690b into ceph:master Jul 23, 2021
Dashboard automation moved this from In progress to Done Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Dashboard
  
Done
3 participants