| opendevreview | Brian Rosmaita proposed openstack/glance master: DNM: testing devstack lib/tempest import wait https://review.opendev.org/c/openstack/glance/+/841805 | 03:50 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/glance master: DNM: testing devstack lib/tempest import wait https://review.opendev.org/c/openstack/glance/+/841805 | 03:54 |
| opendevreview | Pranali Deore proposed openstack/glance-tempest-plugin master: Refactor rbac metadef tests structure https://review.opendev.org/c/openstack/glance-tempest-plugin/+/842117 | 05:00 |
| *** EugenMayer1 is now known as EugenMayer | 06:20 | |
| opendevreview | Merged openstack/glance master: add doc about KVM support e1000e vif model https://review.opendev.org/c/openstack/glance/+/805532 | 06:46 |
| opendevreview | Merged openstack/glance-tempest-plugin master: Implement API protection testing for metadef tags https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802795 | 06:49 |
| opendevreview | Merged openstack/glance master: doc: add aarch64 to list of architecture image properties https://review.opendev.org/c/openstack/glance/+/791453 | 06:50 |
| *** whoami-rajat__ is now known as whoami-rajat | 10:36 | |
| opendevreview | Mridula Joshi proposed openstack/glance-specs master: Update proposal for duplication image download https://review.opendev.org/c/openstack/glance-specs/+/734683 | 10:40 |
| opendevreview | Mridula Joshi proposed openstack/glance-specs master: Update proposal for duplication image download https://review.opendev.org/c/openstack/glance-specs/+/734683 | 10:42 |
| opendevreview | Pranali Deore proposed openstack/glance master: Release notes for Zed Milestone 1 https://review.opendev.org/c/openstack/glance/+/842330 | 10:52 |
| opendevreview | Mridula Joshi proposed openstack/glance-specs master: Update proposal for duplication image download https://review.opendev.org/c/openstack/glance-specs/+/734683 | 11:05 |
| opendevreview | Pranali Deore proposed openstack/glance master: Refresh Glance example configs for Zed milestone 1 https://review.opendev.org/c/openstack/glance/+/842336 | 11:41 |
| *** sfinucan is now known as stephenfin | 12:04 | |
| opendevreview | Merged openstack/glance master: Migrate CentOS Stream 8 job to CentOS Stream 9 https://review.opendev.org/c/openstack/glance/+/841034 | 12:45 |
| pdeore | jokke_, as per your comment on https://review.opendev.org/c/openstack/glance/+/842330, do you mean Bug 1971521 need to go to only critical section & not in fixes ? | 13:31 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Use show-policy to render default policy in documentation https://review.opendev.org/c/openstack/glance/+/813884 | 13:32 |
| opendevreview | Merged openstack/glance master: Delay string interpolations at logging calls https://review.opendev.org/c/openstack/glance/+/802649 | 14:01 |
| opendevreview | Dan Smith proposed openstack/glance master: DNM: Test performance data https://review.opendev.org/c/openstack/glance/+/842367 | 14:18 |
| opendevreview | Pranali Deore proposed openstack/glance master: Release notes for Zed Milestone 1 https://review.opendev.org/c/openstack/glance/+/842330 | 14:31 |
| opendevreview | Pranali Deore proposed openstack/glance master: Release notes for Zed Milestone 1 https://review.opendev.org/c/openstack/glance/+/842330 | 14:35 |
| pdeore | jokke_, I have updated https://review.opendev.org/c/openstack/glance/+/842330, kindly please have a look. | 14:43 |
| pdeore | abhishekk, dansmith, croelandt, rosmaita, kindly please have a look at ^ when you have time... | 14:45 |
| abhishekk | ack | 14:45 |
| rosmaita | dansmith: if we want to backport https://review.opendev.org/c/openstack/devstack/+/841804 , do i need to file a bug for it? | 14:59 |
| dansmith | rosmaita: not sure the devstack policy on that tbh, I defer to gmann or martin | 15:00 |
| rosmaita | ok, thanks | 15:00 |
| abhishekk | we definitely need to backport it | 15:01 |
| gmann | dansmith: rosmaita we can backport till the branch tests are effected/fixed for this change. no strict policy on devstack backport. bug file is ok to track the things but not mandatory | 15:01 |
| rosmaita | gmann: ty | 15:02 |
| rosmaita | abhishekk: how far back does it need to go? | 15:03 |
| abhishekk | till wallaby | 15:03 |
| abhishekk | whoami-rajat, right, we have added it in wallaby | 15:04 |
| rosmaita | ok, i'll propose a backport to yoga as soon as it merges to master | 15:04 |
| abhishekk | cool, thank you | 15:04 |
| whoami-rajat | ack | 15:05 |
| abhishekk | our stable gate is also stuck due to this | 15:05 |
| whoami-rajat | guess i will need to revert my changes for making the cinder job non-import ... | 15:05 |
| abhishekk | yeah | 15:05 |
| abhishekk | once it is merged in master | 15:05 |
| rosmaita | well, we could backport the job changes short-term, because we control those | 15:05 |
| rosmaita | after the devstack patch makes it to wallaby, we can revert them | 15:06 |
| abhishekk | rosmaita, we are stuck in deadlock situation :D | 15:06 |
| whoami-rajat | rosmaita, there's a cyclic dependency on yoga gate so devstack patch seems to be the only solution | 15:07 |
| abhishekk | on cinder backport tempest-plugin job is failing and on tempest-plugin backport cinder job is failing :D | 15:07 |
| rosmaita | omg | 15:07 |
| rosmaita | glad i did that devstack patch, then | 15:08 |
| rosmaita | also gave me the opportunity to update my bash knowledge | 15:08 |
| rosmaita | has no idea that bash introduced named references in 2018 | 15:09 |
| abhishekk | ++ | 15:10 |
| abhishekk | can someone look at reno patch? | 15:14 |
| abhishekk | so that pdeore can post the release patch for glance | 15:14 |
| rosmaita | looking | 15:15 |
| abhishekk | thank you | 15:16 |
| dansmith | beat ya! | 15:17 |
| rosmaita | ummm ... the markup isn't correct | 15:17 |
| dansmith | dammit | 15:17 |
| rosmaita | https://6bc097249a57e4eccfb8-2c6cda48ae2944413654c9e504ee9baf.ssl.cf5.rackcdn.com/842330/3/check/build-openstack-releasenotes/d6391b0/docs/unreleased.html | 15:17 |
| abhishekk | why it is all coming in bold | 15:18 |
| dansmith | ah | 15:18 |
| rosmaita | but also, i'm not sure we should merge the API response change if we're not going to backport it | 15:18 |
| dansmith | because it still has the * in front | 15:18 |
| abhishekk | ack | 15:19 |
| dansmith | the response code change is already merged on master no? | 15:19 |
| rosmaita | yeah, but it can be reverted | 15:20 |
| dansmith | what does backporting have to do with the reno? | 15:20 |
| rosmaita | well, we should not release if we're not going to backport it IMO | 15:20 |
| dansmith | oh I didn't know we were considering reverting | 15:20 |
| rosmaita | dansmith: https://review.opendev.org/c/openstack/glance/+/841278/1#message-67e3c27df0bd7db4d290f44314fbe0ac1e05e68d | 15:20 |
| dansmith | hrm | 15:21 |
| dansmith | brian's argument makes sense to me | 15:22 |
| rosmaita | i can't hear that often enough! | 15:22 |
| dansmith | yoga was just released, almost zero people have deployed it, | 15:22 |
| dansmith | blast radius is tiny among those that have, | 15:22 |
| abhishekk | can we tag release early next week (Monday)? | 15:22 |
| dansmith | we should fix and backport ASAP and likely nobody would ever notice the change | 15:22 |
| rosmaita | that was the original plan | 15:23 |
| abhishekk | that what was we were thinking | 15:23 |
| abhishekk | If we can tag the release on Monday then we can take this topic for discussion in tomorrows meetig | 15:23 |
| rosmaita | i think i already put it on the agenda | 15:24 |
| rosmaita | i lost track of M-1 though | 15:24 |
| abhishekk | yes, you had | 15:24 |
| rosmaita | i have just become a complete PITA since i stopped being cinder PTL | 15:26 |
| rosmaita | just a few letters difference, i guess | 15:26 |
| abhishekk | :d | 15:26 |
| dansmith | since? :) | 15:26 |
| abhishekk | we can also tag release immediately after our meeting | 15:26 |
| abhishekk | I have restored the patch,looks like jokke_ is not around so lets discuss this in tomorrows meeting | 15:28 |
| rosmaita | I'm seeing this in setupUpClass on some tempest tests: Details: b'409 Conflict\n\nImage status transition from saving to saving is not allowed\n\n ' | 15:28 |
| rosmaita | i thought we had fixed that? or maybe i am thinking of something in cinder | 15:29 |
| abhishekk | rosmaita, can you report this in LP? | 15:29 |
| rosmaita | ok | 15:29 |
| abhishekk | I will try to reproduce it in local env | 15:30 |
| rosmaita | here's an example: https://zuul.opendev.org/t/openstack/build/704674d047224763b7c0a1b90f756def | 15:33 |
| abhishekk | dansmith, I have quick question related to nova | 15:33 |
| abhishekk | rosmaita, ack | 15:33 |
| abhishekk | how does request knows which pipeline to execute? | 15:34 |
| abhishekk | https://github.com/openstack/nova/blob/master/etc/nova/api-paste.ini#L33 | 15:34 |
| abhishekk | and if i define new pipeline then how and where I can change it? | 15:34 |
| dansmith | uh | 15:35 |
| rosmaita | abhishekk: i will file a bug, don't worry about it right now ... it's an intermittent problem | 15:36 |
| abhishekk | in glance we have paste_deploy section where we can set which pipleline to use | 15:36 |
| dansmith | I think L28 defines the url prefix to pipeline for /v2/, which takes you to the definition on L31? | 15:36 |
| abhishekk | yeah but from line no 31 how it knows it has to execute keystone pipeline | 15:37 |
| * dansmith looks shifty-eyed | 15:37 | |
| dansmith | um | 15:37 |
| dansmith | I really don't know, paste has always been pretty mysterious | 15:37 |
| abhishekk | I think line 32 might tell something | 15:37 |
| abhishekk | CONF.api.auth_strategy | 15:38 |
| dansmith | yeah, that | 15:38 |
| * dansmith was just grepping | 15:38 | |
| abhishekk | cool, thank you | 15:39 |
| abhishekk | so glance have paste_deploy.flavor and nova has auth_strategy | 15:39 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Add flavor and image options to select emulated virtual tpm https://review.opendev.org/c/openstack/glance/+/633256 | 16:09 |
| opendevreview | Merged openstack/glance master: Refresh Glance example configs for Zed milestone 1 https://review.opendev.org/c/openstack/glance/+/842336 | 16:33 |
| opendevreview | Merged openstack/glance master: Use show-policy to render default policy in documentation https://review.opendev.org/c/openstack/glance/+/813884 | 17:22 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Add CPU Mode Metadata Def https://review.opendev.org/c/openstack/glance/+/392088 | 18:05 |
| opendevreview | Merged openstack/glance master: Release notes for Zed Milestone 1 https://review.opendev.org/c/openstack/glance/+/842330 | 19:11 |
| rosmaita | dansmith: are you a stable core for devstack? | 21:23 |
| rosmaita | i mean, devstack-stable-maint ... i am not questioning your mental status | 21:23 |
| dansmith | rosmaita: I dunno | 21:48 |
| dansmith | link me and we'll find out | 21:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!