| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-29 | |||
| 12:36:37 | gmann | gibi: lyarwood checking comments on grenade fixes, | |
| 12:39:16 | gibi | gmann: in short we are questioning whether we need to turn on trunk testing on stable branches | |
| 12:41:15 | gmann | gibi: lyarwood ok, I will say we should as Tempest master is used to test the stable/ussuri -> master and if we do not enable then we skip the trunk test for no reason. it was just a miss in enabling the extension in stable branches. | |
| 12:41:39 | lyarwood | gm | |
| 12:41:41 | lyarwood | ops sorry | |
| 12:42:06 | lyarwood | gmann: Yeah I appreciate that but if wasn't tested until now it seems a little odd to add it in for non-master branches | |
| 12:42:29 | lyarwood | gmann: the missing coverage was mostly around live migration right? | |
| 12:42:37 | gmann | and we can see it is passing in nova grenade jobs in stable so we do not need anythings extra than grenade fix | |
| 12:43:04 | gmann | lyarwood: yeah, live migration trunk tests. those are only tests for trunk currently in tempest | |
| 12:43:06 | lyarwood | gmann: okay well if it's passing and the neutron folks are okay with helping with any fallout that might appear in the coming days then I guess we can go ahead | |
| 12:43:44 | gmann | lyarwood: yeah, if neutron team is not comfortable or confident then we can just keep it like that. | |
| 12:53:46 | gmann | lyarwood: whether we need it for Xena or not, I remember master testing patch passed only when I added xena fix as depends-on here https://review.opendev.org/c/openstack/nova/+/811118 | |
| 12:54:08 | gmann | I think i added due to zuul job inventory but let me debug that what exactly happing here | |
| 12:54:14 | lyarwood | okay if that's the case lets just merge both | |
| 13:08:52 | gmann | lyarwood: zuul inventory seems to take job definition from master only https://zuul.opendev.org/t/openstack/build/01f10e2cb162450e912551026dde8f85/log/zuul-info/inventory.yaml#294-304 | |
| 13:09:08 | gmann | let me remove the xena fix as depends-on and then we can get more clarity | |
| 13:09:45 | opendevreview | Ghanshyam proposed openstack/nova master: DNM testing grenade neutron-trunk fix https://review.opendev.org/c/openstack/nova/+/811118 | |
| 13:16:32 | lyarwood | gmann: kk | |
| 13:55:54 | bauzas | sorry folks, I had issues with my computer | |
| 13:56:08 | bauzas | gmann: thanks, unfortunately gtk | |
| 13:57:37 | bauzas | looks like it's limbo | |
| 13:58:31 | gmann | nova grenade master fix is in gate https://review.opendev.org/c/openstack/grenade/+/811117 | |
| 13:58:57 | gmann | for enabling testing on stable or not is different things and we can continue discussion. | |
| 14:01:40 | dansmith | bauzas: gibi looks like the nova-yoga-ptg etherpad has been emptied | |
| 14:01:51 | bauzas | ORLY ? | |
| 14:02:21 | gibi | shit, I see | |
| 14:02:52 | bauzas | dansmith: this was really done 3 mins before | |
| 14:02:56 | gibi | this is the last known state https://etherpad.opendev.org/p/nova-yoga-ptg/timeslider#8272 | |
| 14:03:14 | gibi | last known good state | |
| 14:03:16 | bauzas | yup | |
| 14:03:18 | dansmith | bauzas: okay I just loaded it and saw it empty | |
| 14:03:24 | bauzas | dansmith: me too | |
| 14:03:36 | bauzas | it's just, something happened between 8272 and 8273 | |
| 14:03:43 | dansmith | certainly you can restore a rev right? | |
| 14:03:58 | gibi | the content yes, the coloring I think no | |
| 14:04:11 | bauzas | I wonder | |
| 14:04:18 | bauzas | lemme ask infra | |
| 14:04:20 | gibi | sure | |
| 14:04:21 | dansmith | hmm, I thought there was some way | |
| 14:04:31 | gibi | what I did before is that I exported the old state and copied that back | |
| 14:05:37 | bauzas | fungi is looking at restoring | |
| 14:05:41 | gibi | ack | |
| 14:05:43 | bauzas | folks, don't touch now | |
| 14:05:49 | dansmith | sweet | |
| 14:06:01 | bauzas | dansmith: thanks for having identified this | |
| 14:06:39 | lyarwood | ah ffs I think that was me | |
| 14:06:59 | lyarwood | I was editing and my dock crashed | |
| 14:07:12 | lyarwood | and then I couldn't reconnect to the pad | |
| 14:08:13 | lyarwood | yeah my items were the last on there | |
| 14:08:28 | lyarwood | https://etherpad.opendev.org/p/nova-yoga-ptg/timeslider#8272 | |
| 14:09:07 | lyarwood | not entirely sure how my laptop dock (and thus network) crashing caused this tbh | |
| 14:09:58 | gibi | an interesting untested edge case in the etherpad code :) | |
| 14:13:08 | fungi | bauzas: gibi: dansmith: i've rolled it back to revision 8272 | |
| 14:13:19 | fungi | lyarwood: ^ | |
| 14:13:20 | dansmith | looks good, thanks fungi ! | |
| 14:13:23 | bauzas | fungi: with all my love | |
| 14:13:36 | fungi | cool, just making sure it's looking like you needed | |
| 14:13:50 | lyarwood | thanks for that and apologies all | |
| 14:13:56 | fungi | we do also back up the db behind it daily, worst case | |
| 14:14:04 | bauzas | fungi: it is, all the colors | |
| 14:14:07 | gibi | fungi: awesome thanks | |
| 14:14:10 | fungi | no problem, glad we could recover it | |
| 14:14:28 | bauzas | and yeah, interesting edge case | |
| 14:14:47 | bauzas | a docking issue with a network problem swallows an etherpad | |
| 14:16:07 | fungi | those aren't so bad. in the past there have also been bugs which clients somehow tickled to make the pads completely unusable | |
| 14:16:46 | fungi | and the most we can do in those cases is dump an earlier text copy with the api and stick that into a new pad, but it loses all the attribution and history | |
| 14:17:40 | fungi | and formatting | |
| 14:18:55 | bauzas | fungi: thanks, gtk | |
| 14:19:23 | bauzas | gmann: i know you're busy with all those devstack and grenade stuff | |
| 14:19:50 | bauzas | gmann: but tell me when you think it would be a good opportunity for the x-p PTG session between oslo and nova re: policy | |
| 14:22:14 | gmann | bauzas: give me 10 min, currently internal meeting | |
| 14:22:19 | bauzas | gmann: nah, no rush | |
| 14:34:38 | lyarwood | https://review.opendev.org/c/openstack/nova/+/811118/ is passing with just the master grenade fix that's in the gate FWIW | |
| 14:35:40 | gibi | \o/ | |
| 14:52:19 | bauzas | lyarwood: https://media.giphy.com/media/HTjcWZwMtHpyhuCGKZ/giphy-downsized-large.gif?cid=ecf05e47cwkexzrwaxwlkbicd2ppy22bb1l52sg7yoa8sk9x&rid=giphy-downsized-large.gif&ct=g | |
| 14:52:49 | bauzas | but that's lasting 0.1sec and then I say "meh" | |
| 14:53:58 | fungi | if you used a matrix client with the matrix-oftc bridge to join this channel, you and other matrix users could share inline media while irc users would just see a url to it in-channel | |
| 15:04:32 | bauzas | fungi: what I said, "meh" :p | |
| 15:05:24 | fungi | heh | |
| 15:05:30 | gmann | lyarwood: perfect then I will say it was late night thing which thought that Xena fix is needed :) | |
| 15:07:17 | gmann | bauzas: for PTG oslo sessions, is it possible on Tuesday Oct 19th between 13-15 UTC | |
| 15:14:18 | opendevreview | melanie witt proposed openstack/nova stable/train: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791807 | |
| 15:14:19 | opendevreview | melanie witt proposed openstack/nova stable/train: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/806629 | |
| 15:19:44 | gmann | bauzas: and as per current topic I need only 30 min unless there is more things to discuss from anyone else | |
| 15:20:34 | bauzas | gmann: sorry, I'm on a meeting but I guess we could run a meeting after 1400UTC as we have a cyborg x-p session first | |
| 15:22:10 | gmann | bauzas: on tuesday right? | |
| 15:22:35 | bauzas | gmann: on the 19th of October, yes | |
| 15:22:56 | gmann | bauzas: perfect, sounds good. thanks | |
| 15:23:10 | bauzas | https://etherpad.opendev.org/p/nova-yoga-ptg L47 tells me the 14-15:00UTC slot is already taken | |
| 15:24:12 | gmann | bauzas: that is 'Cyborg-Nova: Tuesday (19th Oct) 13:00 UTC - 14: 00 UTC:' | |
| 15:24:24 | bauzas | my bad, yeah | |
| 15:24:44 | bauzas | again, wfm for a oslo x-p session on Oct-19 14:00UTC | |
| 15:24:53 | gmann | +1, thanks again | |
| 15:24:58 | lyarwood | gmann: ^_^ no issues thanks for working on it so late | |
| 15:25:04 | bauzas | gmann: all good | |
| 15:25:36 | gmann | lyarwood: np!, and for stable backport we can wait for neutron team opinion so I agree on 'no hurry for those' . | |
| 15:25:45 | lyarwood | awesome | |
| 16:12:53 | bauzas | artom: honestly, I'm torn with https://review.opendev.org/c/openstack/nova/+/808474 | |
| 16:13:00 | bauzas | that's an behavioural change | |
| 16:13:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Enable min pps tempest testing in nova-next https://review.opendev.org/c/openstack/nova/+/811748 | |