| Emperor | It seems to me that's a bit of a fundamental problem with using 1=WARN, 2=ERROR for a cli tool like this - you naturally want unhandled exceptions to be ERRORs not warnings, I think? | 11:48 |
|---|---|---|
| Emperor | though given the swift-ring-builder executible is a small script that imports swift.cli.ringbuilder.main perhaps putting a try: except round that and arranging for a suitable exit code (and bt etc) for unhandled exceptions is the way to go... | 11:49 |
| opendevreview | Matthew Vernon proposed openstack/swift master: swift-ring-builder: exit ERROR (2) on uncaught exceptions https://review.opendev.org/c/openstack/swift/+/829053 | 14:21 |
| Emperor | ^-- I think this fixes this problem, would be grateful for review please :) | 14:36 |
| opendevreview | Matthew Vernon proposed openstack/swift master: swift-ring-builder: exit ERROR (2) on uncaught exceptions https://review.opendev.org/c/openstack/swift/+/829053 | 16:05 |
| opendevreview | Clay Gerrard proposed openstack/swift master: Delete s3api MPU segments when the manifest expires https://review.opendev.org/c/openstack/swift/+/800701 | 18:18 |
| timburke__ | tag's up! https://github.com/openstack/swift/blob/2.29.0/CHANGELOG | 18:54 |
| *** timburke__ is now known as timburke | 19:02 | |
| opendevreview | Clay Gerrard proposed openstack/swift master: Cleanup cleave dbs empty part dirs as we go https://review.opendev.org/c/openstack/swift/+/829136 | 22:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!