| opendevreview | Pierre-Samuel Le Stang proposed openstack/glance master: Implement glance-download internal plugin https://review.opendev.org/c/openstack/glance/+/840318 | 07:35 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/glance-specs master: Add new location APIs https://review.opendev.org/c/openstack/glance-specs/+/840882 | 07:53 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: Immediate caching of an image https://review.opendev.org/c/openstack/glance/+/842070 | 08:24 |
| abhishekk | dansmith, ^^^ | 08:24 |
| abhishekk | needs to fix flaky functional test and add releasenote as well | 08:24 |
| * abhishekk will be taking half day off today | 08:26 | |
| *** tosky_ is now known as tosky | 09:42 | |
| opendevreview | Pierre-Samuel Le Stang proposed openstack/glance master: Implement glance-download internal plugin https://review.opendev.org/c/openstack/glance/+/840318 | 10:04 |
| opendevreview | Pierre-Samuel Le Stang proposed openstack/glance master: Implement glance-download internal plugin https://review.opendev.org/c/openstack/glance/+/840318 | 11:53 |
| *** tosky_ is now known as tosky | 11:55 | |
| *** tosky__ is now known as tosky | 13:17 | |
| *** tosky is now known as Guest2169 | 13:29 | |
| *** tosky__ is now known as tosky | 13:29 | |
| *** tosky__ is now known as tosky | 13:58 | |
| croelandt | jokke_: can you take a look at https://review.opendev.org/c/openstack/glance/+/844596 ? I might need to backport patches to bindep.txt for OSP17 :) | 14:10 |
| jokke_ | looking | 14:16 |
| jokke_ | croelandt: I assume those are fixes for stuff that are directly broken on said platforms? | 14:18 |
| croelandt | so the -libpython3-dev is from one of your own comments :) | 14:22 |
| croelandt | the Suse-related fixes are things I tried on an OpenSuse VM | 14:22 |
| croelandt | cf. https://review.opendev.org/c/openstack/glance/+/828239 | 14:22 |
| dansmith | whoami-rajat: around? | 15:45 |
| whoami-rajat | dansmith, hey | 15:47 |
| dansmith | whoami-rajat: on your api locations spec.. I know I brought this up before, but just giving it another read: the "make nova use this instead" work item involves nova being changed to use a separate service user for the location update, which it doesn't do today, right? | 15:48 |
| whoami-rajat | dansmith, yes, same for cinder | 15:49 |
| whoami-rajat | but that's after we implement the feature in glance | 15:49 |
| whoami-rajat | i mean those changes come later in phase 2 ^ | 15:49 |
| dansmith | whoami-rajat: right, so I just want to know if you've at least socialized those changes with nova | 15:50 |
| dansmith | I expect no problems, but it's a change that affects nova's users, deployment documentation, etc | 15:51 |
| dansmith | is cinder waiting to implement any of this until it's fully released in glance? meaning.. not in zed? | 15:52 |
| whoami-rajat | dansmith, I should've done that but haven't, not sure if discussing it right away in the channel is a good thing or attend the weekly meeting that is next tuesday i guess | 15:53 |
| whoami-rajat | dansmith, i think it would be better to give at least one release cycle for cinder to make changes but some might think that both glance and cinder changes go into same cycle | 15:54 |
| whoami-rajat | anyway, my opinion is we do changes after glance changes are released | 15:54 |
| whoami-rajat | i.e. cinder changes in AA | 15:55 |
| whoami-rajat | rosmaita, sorry to drag you in most of the conversations but what do you think? ^ | 15:55 |
| dansmith | whoami-rajat: ack, so I just want to make sure we don't do this thing in glance and then have concerns over adoption after we're (very) committed | 15:55 |
| rosmaita | whoami-rajat: reading the scrollback now | 15:56 |
| whoami-rajat | that totally makes sense, i will discuss with the nova team and cinder team as well (most of the people were on board as i remember last) | 15:56 |
| dansmith | whoami-rajat: I'm asking in -nova now | 15:56 |
| whoami-rajat | rosmaita, just to summarize, we're adding the new location APIs in Zed so my suggestion is do the cinder changes after the glance changes are released i.e. do cinder changes in AA | 15:57 |
| whoami-rajat | ack, joining | 15:57 |
| whoami-rajat | thinking about it a little more, i think these changes are mostly placeholders as of now until we've the service role in place (or whatever mechanism for service to service interaction will be implemented) | 15:58 |
| whoami-rajat | so we will still use the old flow until then ^ | 15:58 |
| rosmaita | yes, the service-interaction-thingy is more undefined that i had thought | 16:00 |
| rosmaita | *than | 16:00 |
| opendevreview | Merged openstack/glance-tempest-plugin master: Refactor rbac tests structure https://review.opendev.org/c/openstack/glance-tempest-plugin/+/839605 | 17:07 |
| *** jokke is now known as Guest2201 | 17:18 | |
| *** Guest2201 is now known as jokke_ | 17:18 | |
| jokke_ | croelandt: sorry ... dropped ... so the additions looks fine if those things are not working atm we might be even able to backport them. Please just file a bug for it so we can track them and put backport exception mail out | 17:20 |
| croelandt | do we need a backport exception? | 17:45 |
| jokke_ | requirement change so we would yea | 18:38 |
| jokke_ | should be fine 'though as it's affecting just the platforms that would not work atm. without it | 18:39 |
| croelandt | ok | 18:56 |
| *** timburke_ is now known as timburke | 21:03 | |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Bindep fixes for SUSE-like systems https://review.opendev.org/c/openstack/glance/+/844596 | 21:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!