| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-26 | |||
| 15:48:31 | gmann | gibi: bauzas on RBAC BP, ack. I will try to join next meeting but might not due to hospital appointment(let's see how long it takes me to drive in snow :)) but dansmith can answer the queries on that if he joining. | |
| 15:48:41 | bauzas | ack | |
| 15:49:57 | gmann | stephenfin: yeah :). I am not sure why people did not brought those py3.6 which we discussed since PTG and re-re-updated the testing runtime - https://review.opendev.org/c/openstack/governance/+/815851/3..6/reference/runtimes/yoga.rst | |
| 15:50:56 | opendevreview | Merged openstack/nova stable/wallaby: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/818519 | |
| 15:52:32 | gibi | stephenfin, gmann: I think it is a good discussion on the ML, it is probably a late discussion but a good one :) | |
| 15:55:01 | gmann | gibi: IMO, it is not late but revised one :) I have kept centos-stream 8 in my updating the testing runtime because it was not clear for TC in PTG if centos-stream 9 is released or not and later in patch review one of TC checked with centos team and confirmed it is fine to move to centos stream9 | |
| 15:55:34 | gmann | or different opinion from centos team may be | |
| 15:56:38 | gibi | I think there is some missunderstanding / miscommunication around the centos 9 readyness | |
| 15:57:02 | gmann | yeah | |
| 15:57:25 | gmann | anyways let's see if we end up reverting then we can revert the changes inclduing https://review.opendev.org/c/openstack/nova/+/819415 | |
| 15:58:00 | gmann | bauzas: I am +2 on lyarwood changing python_requires = >=3.8 just leaving +A in case you are ok - https://review.opendev.org/c/openstack/nova/+/819415 | |
| 15:58:32 | lyarwood | do we want to see what happens with the thread first | |
| 15:59:25 | lyarwood | in case the TC need to revert back to py36 and CS8 | |
| 15:59:26 | bauzas | gmann: lyarwood: yeah we can hold | |
| 15:59:28 | gmann | lyarwood: either is ok but if we revert to py36 then anyways we have to revert 819194 too | |
| 15:59:37 | lyarwood | true | |
| 15:59:48 | lyarwood | FWIW that depends on https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/819431 that I'm less sure about | |
| 15:59:55 | lyarwood | I think the branches need to be listed there for this to work | |
| 16:00:03 | lyarwood | and it obviously has an impact on other projects | |
| 16:00:09 | lyarwood | let me -W the nova change for now | |
| 16:00:16 | lyarwood | and that change | |
| 16:00:22 | gibi | I agree ^^ | |
| 16:00:26 | gibi | lets wait a bit | |
| 16:00:43 | gibi | and try not to land things that breask py3.6 support in the meantime :) | |
| 16:00:48 | gibi | breaks | |
| 16:01:23 | bauzas | ++ | |
| 16:01:38 | gmann | sure, I have -W on tempest one also | |
| 16:02:06 | lyarwood | thanks | |
| 17:28:16 | gmann | lyarwood: gibi bauzas stephenfin this is the proposal on Yoga testing runtime we agreed in TC channel with RDO team members, http://lists.openstack.org/pipermail/openstack-discuss/2021-November/026024.html | |
| 17:29:25 | gmann | I will keep this proposal as open for next week before we conclude it as most of USA and other folks are on holiday today. meanwhile please add your opinion there | |
| 17:42:55 | gibi | gmann: thanks I will check it | |
| 17:50:06 | lyarwood | gmann: ack will do over the weekend, thanks! | |
| 20:59:35 | opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | |
| #openstack-nova - 2021-11-27 | |||
| 00:18:09 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/817303 | |
| 00:18:09 | opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | |
| 02:59:15 | opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | |
| 15:01:44 | opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | |
| #openstack-nova - 2021-11-28 | |||
| 09:23:41 | opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 18:16:13 | opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | |
| #openstack-nova - 2021-11-29 | |||
| 06:54:54 | opendevreview | Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225 | |
| 08:09:19 | qiujunting | sean-k-mooney I have updated this commit:https://review.opendev.org/c/openstack/nova/+/802225 | |
| 13:55:28 | opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | |
| 14:02:48 | opendevreview | Nicolas Parquet proposed openstack/nova master: Add v2.91 microversion, allowing @ and dot (.) characters in keypair name https://review.opendev.org/c/openstack/nova/+/781076 | |
| 14:06:25 | opendevreview | Nicolas Parquet proposed openstack/nova master: Add v2.91 microversion, allowing @ and dot (.) characters in keypair name https://review.opendev.org/c/openstack/nova/+/781076 | |
| 16:32:08 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | |
| 16:32:16 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | |
| 16:44:15 | gibi | bauzas: what is you way to mark placement bugs in storyboard triaged or incomplete? | |
| 16:44:33 | bauzas | good question, I f*** don't know | |
| 16:45:23 | gibi | there is invalid state, so I might use that for incomplete | |
| 16:45:36 | gibi | or I can tag it with incomplete | |
| 16:45:38 | gibi | tag | |
| 16:46:19 | bauzas | cool | |
| 16:46:22 | bauzas | context ? | |
| 16:46:27 | gibi | https://storyboard.openstack.org/#!/story/2009315 | |
| 16:46:37 | gibi | I need more info to try to reporduce | |
| 16:46:50 | gibi | I'm not even sure what generate file the reporter referring to | |
| 16:53:27 | bauzas | gosh, even hyperlinks don't work | |
| 16:53:41 | gibi | melwitt: hi! I think we fixed https://storyboard.openstack.org/#!/story/2009178 isn't it? if yes could you mark the story implemented? | |
| 16:54:22 | gibi | bauzas: where? | |
| 16:54:43 | bauzas | gibi: https://storyboard.openstack.org/#!/story/2009315 redirects me to my main board | |
| 16:54:55 | gibi | bauzas: that is interesting | |
| 16:55:05 | gibi | bauzas: such links tent to work for me | |
| 16:55:40 | gibi | (it worked now in an private window too) | |
| 16:55:46 | bauzas | fun, it works with private mode | |
| 16:56:11 | gibi | hahh | |
| 16:56:20 | bauzas | hmmm, now it works | |
| 16:56:26 | gibi | storyboard knows you don't like itr | |
| 16:56:34 | bauzas | I just clicked on "Stories" before calling the URL again | |
| 17:02:57 | gibi | bauzas: could you hit https://review.opendev.org/c/openstack/placement/+/800611 it is an easy fix | |
| 17:04:11 | bauzas | gibi: ack | |
| 17:05:02 | gibi | thanks | |
| 17:09:43 | melwitt | gibi: yes I will check on it and update it, thanks for heads up | |
| 17:14:04 | gibi | thanks | |
| 17:23:50 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | |
| 17:23:58 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add yoga spec directory https://review.opendev.org/c/openstack/placement/+/819660 | |
| 17:24:03 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | |
| 17:32:00 | gibi | bauzas: I did a couple of triage and invalidation of placement stories, the incomplete ones just tagged but not marked as invalid so they are still in the primary list | |
| 17:32:06 | bauzas | ++ | |
| 17:32:22 | gibi | it seems cdent used rfe and bug tags, so I did that too | |
| 17:32:35 | gibi | and now I leave for today :) | |
| 17:35:22 | dansmith | gmann: lbragstad: I had to add this to our functional fixture to fix the fails on my revert patch: https://termbin.com/3v6a | |
| 17:35:42 | dansmith | but I'm not sure why, as we should still be passing with the legacy rules in place for functional, AFAIK | |
| 17:36:07 | dansmith | I thing that ^ change is appropriate anyway, just so we're mocking keystone like we expect in the real world, but... I'm not sure why it's suddenly necessary | |
| 17:36:30 | sean-k-mooney | hum "echo just testing! | nc termbin.com 9999" that is a userful site to knowabout | |
| 17:36:44 | lbragstad | that change makes sense\ | |
| 17:36:49 | lbragstad | from a role implication standpoint | |
| 17:37:17 | gmann | dansmith: for unit test this is done in base context https://github.com/openstack/nova/blob/master/nova/tests/unit/policies/base.py#L74 | |
| 17:37:28 | gmann | so you mean for functional tests right | |
| 17:38:16 | gmann | dansmith: ohk, got it. this one | |
| 17:38:17 | gmann | https://bddca1d27ef6e7362ec8-bff4e0c7dcef4e0ba8c46a21501c0936.ssl.cf5.rackcdn.com/816206/9/check/nova-tox-functional-py39/f17f814/testr_results.html | |
| 17:38:18 | dansmith | gmann: yeah functional | |
| 17:38:23 | gmann | +1 | |
| 17:38:43 | dansmith | I have to jump on a call in a sec | |
| 17:38:56 | dansmith | with that patch applied, I pass all those functional fails, I just don't know why I need it | |
| 17:40:23 | gmann | dansmith: I do not think we run functional test on real policy instead on fake policy so it should not fail but I will be able to check tomorrow. | |
| 18:47:17 | dansmith | gmann: hmm, okay then I don't know why it would have changed at all then | |
| 21:54:56 | opendevreview | Merged openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| #openstack-nova - 2021-11-30 | |||
| 08:51:35 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: check if doc build is broken on master https://review.opendev.org/c/openstack/placement/+/819766 | |
| 08:59:02 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/741529 | |
| 09:51:11 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add yoga spec directory https://review.opendev.org/c/openstack/placement/+/819660 | |