| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-26 | |||
| 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 | |
| 09:51:11 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | |
| 09:51:18 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | |
| 11:07:41 | qiujunting | sean-k-mooney | |
| 11:07:45 | qiujunting | hi | |
| 11:08:42 | qiujunting | can you help me. review this commit about pmem spec:https://review.opendev.org/c/openstack/nova/+/802225 | |
| 13:03:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Log which instance event was timed out https://review.opendev.org/c/openstack/nova/+/819817 | |
| 13:48:34 | opendevreview | Elod Illes proposed openstack/nova stable/wallaby: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816883 | |
| 14:13:42 | bauzas | gibi: thanks for creating the yoga specs directory for placement, I forgot about it | |
| 14:13:57 | gibi | bauzas: I'm still failing to make it right :D | |
| 14:14:05 | gibi | doc generation is hard :/ | |
| 14:14:33 | bauzas | yeah I saw it | |
| 14:16:39 | gibi | the extra pain is that locally I have pip dependency issue with the doc generation tox target | |
| 14:17:28 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add yoga spec directory https://review.opendev.org/c/openstack/placement/+/819660 | |
| 14:17:28 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | |
| 14:17:35 | opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | |
| 14:17:39 | gibi | here is the next stab for it :) | |
| 14:21:48 | gibi | bauzas: btw I did it becuase I reheated two of my old specs for placement :) | |
| 14:21:59 | bauzas | I saw iy | |
| 14:31:59 | elodilles | bauzas: not so much things to add to 'stable branches' section of novameetings, but let me know when i can update that | |
| 14:32:18 | bauzas | elodilles: just do it (c) | |
| 14:32:36 | bauzas | elodilles: and I'll update after you | |
| 14:33:12 | elodilles | bauzas: ack :] | |
| 14:36:18 | elodilles | bauzas: done | |
| 15:19:19 | bauzas | in case people have other opinions about this wontfix... https://bugs.launchpad.net/nova/+bug/1951983 | |
| 15:22:19 | bauzas | also, super weirdo bug https://bugs.launchpad.net/nova/+bug/1951169 | |
| 15:26:59 | bauzas | placement cores, easy peasy for adding specs to yoga https://review.opendev.org/c/openstack/placement/+/819660 | |
| 15:27:35 | bauzas | also, reminder: nova meeting in 33 mins here at #openstack-nova | |
| 15:27:47 | gibi | bauzas: agree about https://bugs.launchpad.net/nova/+bug/1951983 until somebody proposes a better regexp it is simply a wontfix | |