| opendevreview | Merged openstack/glance master: Increase per project quota to 50 https://review.opendev.org/c/openstack/glance/+/933550 | 02:20 |
|---|---|---|
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Add distributed download spec https://review.opendev.org/c/openstack/glance-specs/+/933632 | 08:45 |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Add distributed download spec https://review.opendev.org/c/openstack/glance-specs/+/933632 | 08:53 |
| croelandt | gmann: Hey! During PTG, we entertained the idea of *removing* our Windows-related code, after deprecating it following the end of the WinStackers group. Do you think it's ok? | 14:58 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove Windows OS support https://review.opendev.org/c/openstack/glance/+/930168 | 15:47 |
| croelandt | tkajinam: sorry, I missed some more lines in the doc :-( | 15:54 |
| tkajinam | croelandt, no, no. thanks for catching it. I should have caught it by my side | 15:56 |
| tkajinam | let me update the change | 15:57 |
| croelandt | I think we're all good after that | 15:59 |
| tkajinam | yeah | 15:59 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove Windows OS support https://review.opendev.org/c/openstack/glance/+/930168 | 15:59 |
| tkajinam | just fyi nova consider that it already removed windows support when hyper-v support was removed | 15:59 |
| croelandt | but they kept the code? | 15:59 |
| tkajinam | and does not require formal deprecation step to drop remaining logic (to support the other processes like nova-conductor on Windows) | 15:59 |
| tkajinam | glance took quite formal way and officially deprecated windows support so there would be no real blocker for removal, I expect | 16:00 |
| tkajinam | I mean, a few projects already think windows support is dead and started more direct approach to remove it. | 16:00 |
| dansmith | croelandt: I thought you said it was deprecated in 2024.2? the patch says 2024.1 | 16:16 |
| dansmith | 2024.2 would have meant that slurp users would never see the deprecation before the removal.. if it was in 2024.1 then we're safe to remove now I think | 16:17 |
| opendevreview | Rajat Dhasmana proposed openstack/glance master: Revert "Increase per project quota to 50" https://review.opendev.org/c/openstack/glance/+/933667 | 16:17 |
| tkajinam | https://github.com/openstack/glance/commit/a8bed388f26eb383449b4d040c4da84671587fb9 | 16:19 |
| tkajinam | dansmith, we merged the deprecation in 28.0.0 so it was deprecated in 2024.1 | 16:20 |
| tkajinam | I remember I pushed it hard at that time because of SLURP | 16:20 |
| dansmith | tkajinam: ack, I'm referencing the discussion in the glance ptg between croelandt and I | 16:21 |
| dansmith | I thought he said it was in 2024.2 and so I was in favor of leaving it until F because of that timer | 16:21 |
| dansmith | but either he was mistaken or I misunderstood I guess | 16:21 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Remove scrubber https://review.opendev.org/c/openstack/glance/+/930241 | 16:23 |
| croelandt | dansmith: I cannot remember what I said :) | 16:24 |
| croelandt | with tkajinam's info, is it fine to remove this cycle? | 16:24 |
| dansmith | yes | 16:24 |
| croelandt | One thing I completely forgot to discuss during PTG was related to the scrubber | 16:26 |
| croelandt | Do we intend to drop the pending_delete status? This means images would no longer be "soft deleted",right? | 16:26 |
| croelandt | Once it's deleted, it's gone "for rea"l | 16:26 |
| abhishekk | croelandt: yes, we don't need pending_delete status any more | 16:41 |
| croelandt | abhishekk: so I guess users were never recovering images | 16:47 |
| croelandt | that's crazy | 16:47 |
| abhishekk | that is why we discussed this and decided to remove it, right? | 16:47 |
| croelandt | yeah no one seemed to use it | 16:49 |
| abhishekk | yes, also if you delete image by enabling pending delete and deleted 10 images, then you need to run scrubber manually by passing single image id at a time | 16:50 |
| abhishekk | * you mean admin | 16:50 |
| croelandt | yeah | 16:50 |
| croelandt | admins are our users :p | 16:50 |
| abhishekk | :D | 16:50 |
| abhishekk | I think we can reintroduce this by refining in upcoming cycle if you think this is important to have | 16:52 |
| * abhishekk back in 30 mins after dinner | 16:52 | |
| croelandt | eh, if no one is complaining | 16:53 |
| croelandt | But I always find it quite annoying that we don't really know what people are doing with Glance :D | 16:53 |
| gmann | croelandt: we are good to remove window support in this cycle. along with deprecation in 2024.1, this has been untested/unmaintained since we retire os-win in 2023.2. Also, we announced it many times in ML, and I talked to the foundation staff to check with Microsoft if they are interested in maintaining it after the Cloudbase company went away. But no interested party on this. So removing it is right thing to do. | 18:04 |
| gmann | I have commented in gerrit also | 18:04 |
| abhishekk | gmann: thanks for confirmation | 19:06 |
| gmann | abhishekk: croelandt: np!. as you are here, can you look into this quick one https://review.opendev.org/c/openstack/glance-tempest-plugin/+/933476 | 19:09 |
| abhishekk | ack | 19:09 |
| abhishekk | I guess I can ninja approve it | 19:11 |
| abhishekk | Done | 19:11 |
| gmann | thanks | 19:13 |
| abhishekk | np! | 19:13 |
| opendevreview | Merged openstack/glance-tempest-plugin master: Support py3.12 and drop py3.8 https://review.opendev.org/c/openstack/glance-tempest-plugin/+/933476 | 19:34 |
| croelandt | gmann: thanks | 20:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!