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.

Sign up for GitHub

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

Jump to bottom

devices/simple/scan: Fix string in log statement #34412

Merged
merged 1 commit into from Apr 7, 2020
Merged

devices/simple/scan: Fix string in log statement #34412

merged 1 commit into from Apr 7, 2020

Conversation

jan--f
Copy link
Contributor

@jan--f jan--f commented Apr 6, 2020

A multiline string enclosed by () gained a comma, making it into a
tuple, which has no format() method.

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

Signed-off-by: Jan Fajerski jfajerski@suse.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 crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

devices/simple/scan: Fix string in log statement
292ae03 
A multiline string enclosed by () gained a comma, making it into a
tuple, which has no format() method.

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

Signed-off-by: Jan Fajerski <jfajerski@suse.com>
@jan--f jan--f requested a review from a team as a code owner April 6, 2020 08:54
@jan--f
Copy link
Contributor Author

jan--f commented Apr 6, 2020

jenkins test ceph-volume tox

@jan--f
Copy link
Contributor Author

jan--f commented Apr 6, 2020

retest this please

1 similar comment
@yuriw
Copy link
Contributor

yuriw commented Apr 7, 2020

retest this please

@tchaikov
Copy link
Contributor

tchaikov commented Apr 7, 2020

jenkins, retest this please.

@jan--f
Copy link
Contributor Author

jan--f commented Apr 7, 2020

jenkins test ceph-volume tox

1 similar comment
@tchaikov
Copy link
Contributor

tchaikov commented Apr 7, 2020

jenkins test ceph-volume tox

@jan--f jan--f merged commit 2822c53 into ceph:master Apr 7, 2020
This was referenced Apr 7, 2020
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guits guits guits approved these changes

@rishabh-d-dave rishabh-d-dave rishabh-d-dave approved these changes

@yuriw yuriw Awaiting requested review from yuriw

Assignees
No one assigned
Labels
bug-fix ceph-volume
Projects
None yet
Milestone
No milestone
5 participants
@jan--f @yuriw @tchaikov @guits @rishabh-d-dave