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
hammer: build/ops: improve ceph.in error message #10905
Conversation
@tchaikov Do you know of any tests that rely on the old behavior? |
@smithfarm AFAIK, no. it's just a better error message. the old one is sort of misleading. |
…essage Reviewed-by: Nathan Cutler <ncutler@suse.com>
…essage Reviewed-by: Nathan Cutler <ncutler@suse.com>
Fixes: ceph#11101 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit e19e8f1)
b66c417
to
a3ba985
Compare
@liewegas This backport passed a rados suite at http://tracker.ceph.com/issues/17151#note-7 with failures that I believe have been addressed (except for http://tracker.ceph.com/issues/15139 which is caused by the build system no longer providing dumpling-era packages). I have rebuilt the integration branch to include the two fixes and scheduled a new run at http://tracker.ceph.com/issues/17151#note-14 Do you think it's OK to merge provided the second run succeeds? |
…essage Reviewed-by: Nathan Cutler <ncutler@suse.com>
…essage Reviewed-by: Nathan Cutler <ncutler@suse.com>
👍 |
Note this also passed an upgrade suite at http://tracker.ceph.com/issues/17151#note-18 |
Fixes: #11101
Signed-off-by: Kefu Chai kchai@redhat.com
(cherry picked from commit e19e8f1)