| opendevreview | Jianjian Huo proposed openstack/swift master: Container-server: add container namespaces GET https://review.opendev.org/c/openstack/swift/+/890470 | 06:01 |
|---|---|---|
| *** ozzzo1 is now known as ozzzo | 07:02 | |
| *** lpiwowar|PTO is now known as lpiwowar | 14:33 | |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: fix HEAD request on multipart-manifest raw format https://review.opendev.org/c/openstack/swift/+/890246 | 16:10 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: slo: Implement efficient slo object metadata handling https://review.opendev.org/c/openstack/swift/+/893578 | 16:36 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: slo: Support GET/HEAD request with partnum query param https://review.opendev.org/c/openstack/swift/+/894570 | 16:37 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber https://review.opendev.org/c/openstack/swift/+/894580 | 16:37 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: fix HEAD request on multipart-manifest raw format https://review.opendev.org/c/openstack/swift/+/890246 | 16:46 |
| opendevreview | Merged openstack/swift stable/2023.2: debug_logger: ensure NOTICE level is registered https://review.opendev.org/c/openstack/swift/+/894599 | 17:28 |
| opendevreview | Jianjian Huo proposed openstack/swift master: Container-server: add container namespaces GET https://review.opendev.org/c/openstack/swift/+/890470 | 19:43 |
| timburke | #startmeeting swift | 21:00 |
| opendevmeet | Meeting started Wed Sep 13 21:00:17 2023 UTC and is due to finish in 60 minutes. The chair is timburke. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:00 |
| opendevmeet | The meeting name has been set to 'swift' | 21:00 |
| timburke | who's here for the swift meeting? | 21:00 |
| timburke | after standing y'all up last week, fair enough ;-) | 21:04 |
| timburke | sorry about that, i was head-down trying to translate some C to python and lost track of time | 21:04 |
| timburke | there were just a few things i wanted to call out this week: | 21:05 |
| timburke | #topic vPTG | 21:05 |
| timburke | if you haven't already, please register at https://ptg2023.openinfra.dev/ -- the PTG will take place Oct 23-27 | 21:05 |
| timburke | i still haven't seen any finalized slots for meeting times, but i might just put together a doodle poll for meeting times based on prior vPTGs for next week | 21:06 |
| timburke | please add topics to the etherpad at https://etherpad.opendev.org/p/swift-ptg-caracal | 21:07 |
| timburke | #topic server side copy and sysmeta | 21:07 |
| timburke | clayg, acoles, and i have been thinking more about whether it makes sense to have the copy middleware copy sysmeta -- clayg has written up https://review.opendev.org/c/openstack/swift/+/894830 to try removing it | 21:09 |
| seongsoocho | o/ | 21:10 |
| timburke | this came up in the context of an slo refactor (https://review.opendev.org/c/openstack/swift/+/893578) which had some ceph-tests failures | 21:11 |
| timburke | which in turn were traced back to s3api writing down some empty sysmeta: https://github.com/openstack/swift/blob/master/swift/common/middleware/s3api/controllers/multi_upload.py#L228-L236 | 21:11 |
| timburke | it's one of those sorts of problems where once you start poking at it a little, it keeps getting stranger the more you investigate :-/ | 21:13 |
| timburke | among the take-aways: s3api shouldn't be intentionally writing empty metadata values like that; s3api and slo need to continue tolerating the empty values (as there are almost certainly objects on disk that have them); and copy's current sysmeta behavior is likely just a result of how it used to be a part of the proxy-server app | 21:16 |
| timburke | if anyone can think of a reason that we *should* continue copying sysmeta as part of server-side copy, please comment on https://review.opendev.org/c/openstack/swift/+/894830 | 21:17 |
| timburke | and last | 21:18 |
| timburke | #topic diskfile and file extensions | 21:18 |
| timburke | as part of making fallocate_reserve work (at least to some degree) with chunked puts, i found myself wanting to have more context about whether this was part of a PUT, POST, or DELETE request down in diskfile | 21:19 |
| timburke | at first, i thought i'd be able to just look at self._extension, but we currently update the extension *after* we've `open`ed the diskfile: https://github.com/openstack/swift/blob/2.32.0/swift/obj/diskfile.py#L3022-L3024 | 21:21 |
| timburke | so i took a stab at plumbing it in as a keyword arg at https://review.opendev.org/c/openstack/swift/+/894820 | 21:21 |
| timburke | and it's reminded us of just how ill-defined our diskfile API is -- as clayg put it, "essentially the only guidance we have is duck typing - whatever the object server asks it to do is what a working df interface has to support. Everything else that it does is just internal details." | 21:24 |
| timburke | so, that might be another conversation to wade into if anyone's interested | 21:25 |
| timburke | that's all i've got | 21:25 |
| timburke | #topic open discussion | 21:26 |
| seongsoocho | I have one topic on Open discussion. | 21:26 |
| seongsoocho | https://bugs.launchpad.net/swift/+bug/2034054 | 21:26 |
| seongsoocho | My mentee and I are proposing to add hostname to the information that only comes out as ip in swift recon, add hostname to the response header in the recon middleware, and show it in the cli. | 21:26 |
| seongsoocho | The reason I'm suggesting this is because I check the cluster status with recon, and sometimes it's hard to tell which host is which just by looking at the IP, so I thought it would be nice to add the hostname as well. | 21:27 |
| timburke | oh yes, i saw that! https://review.opendev.org/c/openstack/swift/+/893583 | 21:27 |
| seongsoocho | Initially we were going to show it only if the human readable option is put in by the user, but I think it's okay to show it as a default option. | 21:27 |
| seongsoocho | We're still working on the patch, but wanted to get your thoughts on adding hostname. | 21:28 |
| timburke | seems reasonable to me -- i could go either way on whether it should be gated on the human-readable option | 21:30 |
| timburke | don't forget to consider what happens when the lookup fails -- pretty sure getfqdn will return the ip address again | 21:30 |
| seongsoocho | okay. I will. probably be able to upload a patch in the next week or so. | 21:32 |
| seongsoocho | I will ask for a review after | 21:32 |
| timburke | oh, i see now that's covered in tests -- though i'm not sure i like the "10.1.1.2(10.1.1.2:6210)" output | 21:32 |
| timburke | can do! thanks for working with the mentees, seongsoocho! always nice to have more contributors around :-) | 21:33 |
| seongsoocho | :-) | 21:33 |
| timburke | all right, i'll let you get on with your morning | 21:35 |
| timburke | thanks for coming, and thank you for working on swift! | 21:35 |
| timburke | #endmeeting | 21:35 |
| opendevmeet | Meeting ended Wed Sep 13 21:35:57 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2023/swift.2023-09-13-21.00.html | 21:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2023/swift.2023-09-13-21.00.txt | 21:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2023/swift.2023-09-13-21.00.log.html | 21:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!