| opendevreview | Shreeya Deshpande proposed openstack/swift master: Add labeled metrics to s3api https://review.opendev.org/c/openstack/swift/+/939481 | 01:10 |
|---|---|---|
| opendevreview | Tim Burke proposed openstack/swift master: Add support of Sigv4-streaming https://review.opendev.org/c/openstack/swift/+/836755 | 02:51 |
| opendevreview | Matthew Oliver proposed openstack/swift master: ring: Add a rebalance history in the ring https://review.opendev.org/c/openstack/swift/+/834621 | 03:34 |
| opendevreview | Matthew Oliver proposed openstack/swift master: proxy: Use last-primaries table https://review.opendev.org/c/openstack/swift/+/857758 | 03:34 |
| opendevreview | Matthew Oliver proposed openstack/swift master: Ringv2: Write the serialization format to a gzip header https://review.opendev.org/c/openstack/swift/+/940491 | 03:34 |
| opendevreview | Matthew Oliver proposed openstack/swift master: Ringv2: Write the serialization format to a gzip header https://review.opendev.org/c/openstack/swift/+/940491 | 06:22 |
| opendevreview | Matthew Oliver proposed openstack/swift master: ring: Add a rebalance history in the ring https://review.opendev.org/c/openstack/swift/+/834621 | 06:22 |
| opendevreview | Matthew Oliver proposed openstack/swift master: proxy: Use last-primaries table https://review.opendev.org/c/openstack/swift/+/857758 | 06:22 |
| opendevreview | Alistair Coles proposed openstack/swift master: Add labeled metrics to s3api https://review.opendev.org/c/openstack/swift/+/939481 | 15:59 |
| opendevreview | Alistair Coles proposed openstack/swift master: WIP s3api: add labeled version of error response metric https://review.opendev.org/c/openstack/swift/+/940535 | 15:59 |
| opendevreview | Tim Burke proposed openstack/swift master: Remove xattr dependency on linux https://review.opendev.org/c/openstack/swift/+/938414 | 17:29 |
| rcmgleite | Hey folks, got another question - this time about quarantined objects and db files. 1. After going through all of the auditors implementations, container and account don't seem to do much. Do we really need those daemons running? I was unable to understand their need. | 17:49 |
| rcmgleite | and Second, the object auditor does a lot of checks - which is great - but it then marks objects as quarantined. It's unclear to me if swift deals with quarantined objects automatically or if this is something we should alarm on and take manual action | 17:49 |
| rcmgleite | any pointers here? | 17:50 |
| opendevreview | Shreeya Deshpande proposed openstack/swift master: Add labeled metrics to s3api https://review.opendev.org/c/openstack/swift/+/939481 | 18:25 |
| opendevreview | Shreeya Deshpande proposed openstack/swift master: Add labeled metrics to s3api https://review.opendev.org/c/openstack/swift/+/939481 | 18:44 |
| timburke_ | rcmgleite, the db auditors don't have many checks, it's true. though they probably should have more -- see https://bugs.launchpad.net/swift/+bug/1823785 | 18:46 |
| timburke_ | object quarantines have to get dealt with manually. usually, just see whether the object exists at all the expected spots and delete if it does | 18:47 |
| timburke_ | sometimes, you get things quarantined that can't be deleted; often that's a sign of a failing disk. if you see a bunch of quarantines on a disk, you probably want to replace it | 18:48 |
| timburke_ | if you see a lot of quarantines spread across all the disks for a node, you might want to check it with memtest | 18:49 |
| clayg | timburke_: clarkb and fungi are looking into the WIP setting on 836755: Add support of Sigv4-streaming | https://review.opendev.org/c/openstack/swift/+/836755 over in #opendev | 19:23 |
| fungi | yep, just need to give your core review team perms to override that flag in your project acls | 19:24 |
| timburke_ | cool, thanks! | 19:25 |
| timburke_ | i really should get my bouncer back up; i'm not in any of my usual rooms... | 19:25 |
| clayg | timburke_: have you ever had to make a change to "project-config" https://opendev.org/openstack/project-config.git | 19:26 |
| timburke_ | on occasion, though it's been a while https://github.com/openstack/project-config/commits?author=tipabu | 19:27 |
| clayg | noice, it's been suggested we need a new acl setting https://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/openstack/ironic.config#L14 | 19:29 |
| clayg | but, i'm no joy so far: ssh://claygerrard@review.opendev.org:29418/openstack/project-config.git did not work. Description: claygerrard@review.opendev.org: Permission denied (publickey). | 19:29 |
| clayg | but i'm not really very good with git review 😠| 19:30 |
| fungi | it... should be like pushing changes to swift? | 19:30 |
| fungi | i'll check the gerrit server logs though | 19:30 |
| clayg | skill? https://review.opendev.org/c/openstack/project-config/+/940552 | 19:31 |
| fungi | yeah, looks like claygerrard is not a user in gerrit | 19:31 |
| clayg | fungi: oh sorry; pls don't - yes I just don't know my gerrit username (clay-gerrard it turns out) | 19:31 |
| fungi | hah | 19:31 |
| fungi | no worries | 19:31 |
| clayg | ROFL - i canz oop;enstacikj | 19:31 |
| fungi | anyway, once that merges and the deploy jobs run, refreshing that change in your browser should give you the ability to switch it from wip to active | 19:34 |
| clayg | yes I hope so! thank you fungi 🤩 | 19:35 |
| fungi | any time | 19:36 |
| clayg | timburke_: BOOM! 836755: Add support of Sigv4-streaming | https://review.opendev.org/c/openstack/swift/+/836755 fungi is a hero!!! | 19:46 |
| fungi | i didn't even wear my cape today! | 19:47 |
| clayg | 🦸 | 19:47 |
| fungi | anyway, now you don't have to worry about other changes getting stuck in wip in the future as long as one of your core reviewers is around to un-wip it | 19:48 |
| fungi | (also your core reviewers can now mark anyone's changes wip in your project if you need to) | 19:49 |
| rcmgleite | @timburke_ thank you very much for the context here! | 19:51 |
| opendevreview | Clay Gerrard proposed openstack/swift master: Limit node_iter on EC HEAD https://review.opendev.org/c/openstack/swift/+/939471 | 21:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!