| opendevreview | Matthew Oliver proposed openstack/swift master: sharder: update shard storage_policy_index if roots changes https://review.opendev.org/c/openstack/swift/+/800748 | 04:41 |
|---|---|---|
| opendevreview | Matthew Oliver proposed openstack/swift master: DNM: test demonstrating broken shard policy migration https://review.opendev.org/c/openstack/swift/+/805389 | 04:41 |
| opendevreview | Matthew Oliver proposed openstack/swift master: use reconciler stuff to fix shards https://review.opendev.org/c/openstack/swift/+/805450 | 04:41 |
| mattoliver | Squashed the 2 followups into the original. And left the probe and clayg afterwards. In otherword collapsed some of the patch chain. | 04:43 |
| opendevreview | Alistair Coles proposed openstack/swift master: sharder: update shard storage_policy_index if roots changes https://review.opendev.org/c/openstack/swift/+/800748 | 09:24 |
| opendevreview | Alistair Coles proposed openstack/swift master: DNM: test demonstrating broken shard policy migration https://review.opendev.org/c/openstack/swift/+/805389 | 09:24 |
| opendevreview | Alistair Coles proposed openstack/swift master: use reconciler stuff to fix shards https://review.opendev.org/c/openstack/swift/+/805450 | 09:24 |
| opendevreview | Alistair Coles proposed openstack/swift master: DNM: shard policy migration: it's still not great https://review.opendev.org/c/openstack/swift/+/805607 | 09:24 |
| opendevreview | Alistair Coles proposed openstack/swift master: sharder: try to be more consistent when updating policy index https://review.opendev.org/c/openstack/swift/+/805623 | 12:10 |
| opendevreview | Alistair Coles proposed openstack/swift master: DNM: shard policy migration: it's still not great https://review.opendev.org/c/openstack/swift/+/805607 | 12:12 |
| opendevreview | Ade Lee proposed openstack/swift master: WIP/DNM: Add FIPS CI jobs https://review.opendev.org/c/openstack/swift/+/796057 | 17:41 |
| *** timburke__ is now known as timburke | 18:14 | |
| timburke | zaitcev, with the quarantining in https://review.opendev.org/c/openstack/swift/+/743797, how do you feel about https://bugs.launchpad.net/swift/+bug/1300850 and https://bugs.launchpad.net/swift/+bug/1514528? is it enough to call them fixed? | 18:14 |
| zaitcev | timburke: oops my reply to that was in draft for a week | 18:15 |
| timburke | no worries, happens to me, too :-) | 18:19 |
| zaitcev | going to switch to wifi | 18:46 |
| clarkb | Heads up that it seems your docker image builds are currently failing in the gate due to https://opendev.org/openstack/swift/src/branch/master/Dockerfile#L49 not seeing value gpg data. Browsing https://keybase.io/justcontainers/key.asc confirms there isn't a key there. A better option is probably to just vendor the ascii armored key data in your repo. | 21:13 |
| clarkb | https://github.com/just-containers/s6-overlay/issues/352 has additional info | 21:14 |
| opendevreview | Clark Boylan proposed openstack/swift master: Fix docker image builds https://review.opendev.org/c/openstack/swift/+/805711 | 21:22 |
| clarkb | Feel free to take that change or not. I don't mind if you update it either. | 21:23 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org has been restarted for a patch version upgrade, resulting in a brief outage | 21:43 | |
| opendevreview | Clark Boylan proposed openstack/swift master: Fix docker image builds https://review.opendev.org/c/openstack/swift/+/805711 | 21:56 |
| timburke | thanks, clarkb! approved | 22:11 |
| clarkb | timburke: the first ps didn't work for a reason I didn't understand but then I noticed there was a better dir to put that file in anyway | 22:11 |
| clarkb | timburke: you might want to check if that passes the jobs in check and remove the approval if not (the jobs are non voting) | 22:11 |
| timburke | i've been watching https://zuul.openstack.org/status#swift -- they pass 👍 | 22:12 |
| clarkb | ah cool | 22:13 |
| timburke | strange that the first patchset wasn't passing :-/ | 22:15 |
| clarkb | ya I assume something weird is happening with the COPY near the top of the file that I just don't understand | 22:15 |
| opendevreview | Merged openstack/swift master: Add and pipe reconstructor stats through recon https://review.opendev.org/c/openstack/swift/+/541141 | 23:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!