| *** mhen_ is now known as mhen | 02:16 | |
| tkajinam | croelandt, which comment is that ? sorry if I overlooked it somehow but I see no comment posted in that change | 03:02 |
|---|---|---|
| opendevreview | Merged openstack/glance master: Fix ZIP file extraction in image_decompression plugin https://review.opendev.org/c/openstack/glance/+/968432 | 03:26 |
| opendevreview | Merged openstack/glance master: Add functional tests for image decompression plugin https://review.opendev.org/c/openstack/glance/+/969282 | 03:26 |
| croelandt | tkajinam: oops, left it as a draft :) It should be ok now | 09:42 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Migrate py313 job to Debian Trixie https://review.opendev.org/c/openstack/glance/+/970054 | 10:22 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Migrate upper functional job to Python 3.13 https://review.opendev.org/c/openstack/glance/+/970059 | 10:23 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove py312 tips jobs https://review.opendev.org/c/openstack/glance/+/970060 | 10:23 |
| tkajinam | croelandt, thanks now I can see it. | 10:23 |
| tkajinam | croelandt, ^^^ updated the commit message to explain why that is needed | 10:23 |
| croelandt | tkajinam: thanks | 12:25 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: Cache PropertyRules instances in Gateway https://review.opendev.org/c/openstack/glance/+/968766 | 13:42 |
| croelandt | dansmith: we're trying to merge all relevant specs before tomorrow EOD | 15:36 |
| croelandt | so if you want to take another look at the updated version of https://review.opendev.org/c/openstack/glance-specs/+/962570 , you still have a bit of time | 15:37 |
| dansmith | croelandt: ack, I'm replying on it right now | 15:37 |
| croelandt | (though I'm not sure how bad it would be for us to merge these on Friday) | 15:37 |
| dansmith | like literally stopped typing to answer :D | 15:37 |
| dansmith | but I must say I'm still a bit puzzled about the real goal and plan here | 15:37 |
| abhishekk | about spec freeze or NFS spec? | 15:38 |
| dansmith | the nfs spec | 15:38 |
| abhishekk | timeout is the main concern | 15:38 |
| abhishekk | If NFS share is unavailable then the call hangs forever | 15:40 |
| abhishekk | Also not related to this topic, but I have one more spec up which needs attention (may be we can target it next cycle) but there is huge demand for this work | 15:42 |
| abhishekk | https://review.opendev.org/c/openstack/glance-specs/+/967046 | 15:42 |
| abhishekk | croelandt: dansmith just FYI ^^ | 15:42 |
| croelandt | abhishekk: is Florian the person who requested this enhancement? | 15:45 |
| abhishekk | likey he can be one among many | 15:45 |
| croelandt | abhishekk: regarding parallel image import, please remove the empty line change from index.rst :) | 15:45 |
| abhishekk | :D | 15:46 |
| dansmith | abhishekk: I just published my comments | 15:46 |
| abhishekk | dansmith: ack, will have a look | 15:47 |
| opendevreview | Merged openstack/glance-specs master: Add RBD Erasure-Coded Pools Support specification https://review.opendev.org/c/openstack/glance-specs/+/863110 | 15:47 |
| dansmith | abhishekk: I totally get the desire to have an IO threadpool for things _like_ NFS, but ext4 on iSCSI can be pretty similar and so I just feel like trying to make this very NFS specific (with NFS-specific error messages) is just sort of weird | 15:47 |
| abhishekk | dansmith: ack, let me go through the recent comments. | 15:49 |
| abhishekk | I was thinking just from NFS prospective so didn'd considered others but yeah I think we can consider others at the same time as well | 15:50 |
| opendevreview | Mathieu Tortuyaux proposed openstack/glance master: glance/utils: override urllib 'User-agent' header https://review.opendev.org/c/openstack/glance/+/972507 | 15:50 |
| opendevreview | Mathieu Tortuyaux proposed openstack/glance master: glance/utils: override urllib 'User-agent' header https://review.opendev.org/c/openstack/glance/+/972507 | 15:51 |
| dansmith | abhishekk: it's more just that it's too specific/targeted IMHO.. as you noted, NFS behaves differently even on its own with soft/hard, but also ext4 on iSCSI, CIFS, etc, etc | 15:53 |
| abhishekk | dansmith: croelandt, I think we can revisit autodetect NFS related work next cycle, the concerns raised by Dan sounds valid and needs to be addressed properly. I will not be able to respin it before the spec freeze deadline :( | 15:54 |
| abhishekk | dansmith: thank you for the thorough inputs, (and sorry for the time loss it caused for you) | 15:55 |
| dansmith | I'm not sure what major problem in my latest round makes you think we can't respin it.. aside from the error message goal, I'm mostly just looking for clarification on how the local ones will really behave | 15:56 |
| abhishekk | Nothing major but the time left for the spec freeze (I am also not sure whether we can accept it post tomorrow or not ) | 15:59 |
| dansmith | this parallel import one looks pretty heavy too... | 16:00 |
| dansmith | there's a lot of stuff proposed.. what's the priority ordering? | 16:00 |
| abhishekk | yes, this is definitely not for this cycle | 16:00 |
| abhishekk | We need this before I | 16:01 |
| abhishekk | OR in I | 16:01 |
| dansmith | parallel import is not for this cycle? | 16:01 |
| abhishekk | Not if we have other priorities, I have just proposed it so that we have attention on this topic | 16:02 |
| dansmith | okay well, then I will punt on that one in the short term | 16:02 |
| abhishekk | sure | 16:03 |
| dansmith | abhishekk: croelandt do you wanna jump on a gmeet to more quickly iterate on the NFS, error messages, etc? | 16:03 |
| abhishekk | may be in an hour? | 16:04 |
| dansmith | yep, in an hour is good | 16:04 |
| abhishekk | ack, will ping you, thank you | 16:04 |
| croelandt | abhishekk: ok just added a message about the parallel image spec | 16:13 |
| croelandt | the implementation through "batches" is interesting | 16:13 |
| croelandt | ok in 45 minutes for a gmeet? | 16:13 |
| abhishekk | ack | 16:14 |
| dansmith | ah crap, I have another meeting 45, sorry | 16:17 |
| dansmith | could we do 1h45m? | 16:17 |
| dansmith | or now? | 16:17 |
| dansmith | croelandt: abhishekk ^ | 16:19 |
| abhishekk | I need to head for dinner but if croelandt is available now I can join | 16:20 |
| abhishekk | looks like croelandt is not around now, I am ok for 1 hr 45 mins then | 16:23 |
| * abhishekk going for dinner | 16:24 | |
| croelandt | damn, was gone for a bit | 16:50 |
| croelandt | 70 minutes from now should be good | 16:50 |
| dansmith | croelandt: abhishek_: whenever you're ready: https://meet.google.com/ybg-yses-fcx | 17:56 |
| opendevreview | Merged openstack/glance master: Cache PropertyRules instances in Gateway https://review.opendev.org/c/openstack/glance/+/968766 | 18:09 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!