| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-02 | |||
| 23:28:51 | melwitt | ok yw o/ | |
| #openstack-nova - 2022-02-03 | |||
| 10:05:35 | opendevreview | yuval proposed openstack/nova master: Lightbits LightOS driver https://review.opendev.org/c/openstack/nova/+/821606 | |
| 10:15:24 | opendevreview | yuval proposed openstack/nova master: Lightbits LightOS driver https://review.opendev.org/c/openstack/nova/+/821606 | |
| 11:02:42 | opendevreview | Balazs Gibizer proposed openstack/placement master: Refactor trait normalization https://review.opendev.org/c/openstack/placement/+/825847 | |
| 11:02:43 | opendevreview | Balazs Gibizer proposed openstack/placement master: DB layer should only depend on trait id not names https://review.opendev.org/c/openstack/placement/+/826490 | |
| 11:02:43 | opendevreview | Balazs Gibizer proposed openstack/placement master: Extend the RP db query to support any-traits https://review.opendev.org/c/openstack/placement/+/825848 | |
| 11:02:44 | opendevreview | Balazs Gibizer proposed openstack/placement master: Extend the RP tree DB query to support any-traits https://review.opendev.org/c/openstack/placement/+/825849 | |
| 11:02:44 | opendevreview | Balazs Gibizer proposed openstack/placement master: Enhance doc of _get_trees_with_traits https://review.opendev.org/c/openstack/placement/+/825780 | |
| 11:03:07 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for listing resource providers https://review.opendev.org/c/openstack/placement/+/826491 | |
| 11:03:08 | opendevreview | Balazs Gibizer proposed openstack/placement master: Remove unused compatibility code https://review.opendev.org/c/openstack/placement/+/826493 | |
| 11:03:08 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for allocation candidates https://review.opendev.org/c/openstack/placement/+/826492 | |
| 11:03:22 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | |
| 11:42:27 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Follow-ups for cells v2, architecture docs https://review.opendev.org/c/openstack/nova/+/827336 | |
| 11:56:54 | artom | I guess we'll need a tracker for the test_tagged_attachment failures :( | |
| 12:27:02 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Testing change to test_tagged_attachment in tempest https://review.opendev.org/c/openstack/nova/+/827549 | |
| 12:46:04 | gibi | artom: yes please :) | |
| 12:46:55 | gibi | artom: and thanks for trying out if switching away from q35 helps | |
| 12:47:13 | artom | gibi, yeah, just trying to gather data points | |
| 12:57:18 | gibi | artom: I will push a tempest change up that will add more logs around that curl command that is used to verify the metadata | |
| 12:57:45 | gibi | as the current code simply drops the exception and returns false so we don't see what fails | |
| 12:58:17 | artom | gibi, yeah, I think the most useful piece of information would be what the metadata API returns | |
| 12:58:21 | artom | Also, https://bugs.launchpad.net/nova/+bug/1959899 is the tracker | |
| 12:58:28 | artom | And I sent an email to the ML | |
| 12:58:55 | gibi | artom: thanks! | |
| 13:10:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: troubleshoot nova-next tagged attach test https://review.opendev.org/c/openstack/nova/+/827661 | |
| 13:25:49 | bauzas | dumb question here, but I can't find any docs about how to test our working Tempest branches with it, I guess we need to just use a venv with pip -e tempest ? | |
| 13:26:04 | bauzas | and then running tempest against our new test classes ? | |
| 13:27:19 | gibi | bauzas: you want to run tempest against a nova stable branch? | |
| 13:27:49 | bauzas | gibi: no I'm working on adding a new tempest test for https://specs.openstack.org/openstack/nova-specs/specs/yoga/approved/boot-vm-with-unaddressed-port.html#id11 | |
| 13:28:03 | bauzas | gibi: so I added a new tempest scenario and I want to test it locally | |
| 13:28:11 | bauzas | against some devstack env | |
| 13:28:31 | bauzas | so I guess I'll just create a build subdir and pip -e the source directly | |
| 13:28:40 | bauzas | with a venv | |
| 13:28:53 | gibi | so you have a devstack with the new nova code and with the new tempest code. Then | |
| 13:28:59 | gibi | tox -evenv -- tempest run --regex <new test case> | |
| 13:28:59 | bauzas | yup | |
| 13:29:19 | gibi | in the tempest source tree | |
| 13:29:20 | bauzas | oh man, I haven't thought about it | |
| 13:29:31 | bauzas | I was about to create the venv by hand | |
| 13:29:36 | gibi | :) | |
| 13:29:47 | bauzas | but yeah, sounds what I thought, just magically done with toix | |
| 13:29:48 | bauzas | tox | |
| 13:29:52 | gibi | yepp | |
| 13:30:03 | bauzas | ack, thanks | |
| 13:30:12 | gibi | if you build you devstack with tempest enabled the tempest is properly preconfigured by devstack so you only need to run it | |
| 13:30:44 | bauzas | gibi: oh good catch | |
| 13:31:01 | gibi | I have a similar blindspot with deployment tools like devstack, kolla, tripleoo, etc | |
| 13:31:04 | bauzas | gibi: I was stupidely about to run tempest on my laptop *against* my devstack vm | |
| 13:31:19 | gibi | bauzas: that would probably work but need a bit more setup | |
| 13:31:31 | bauzas | gibi: yup, I was about to configure etc/tempest | |
| 13:31:33 | gibi | as you need a populated tempest.confg | |
| 13:31:38 | gibi | yepp | |
| 13:31:38 | bauzas | :) | |
| 13:31:51 | bauzas | gosh, I feel stupid | |
| 13:31:58 | bauzas | I know tempest is delivered with devstack | |
| 13:32:09 | bauzas | I even ran it a couple of times | |
| 13:32:15 | bauzas | but I never tweaked it | |
| 13:32:22 | bauzas | I was able to read tempest tests | |
| 13:32:36 | bauzas | but I never contributed to tempest surprinsingly | |
| 13:32:50 | bauzas | gmann: don't look at me like this :D | |
| 13:34:22 | bauzas | now* | |
| 13:34:29 | gibi | :D | |
| 14:14:04 | gibi | dmitriis: went through your series again, looks good. I left some question along the way. | |
| 14:15:03 | gibi | dmitriis: does we have neutron dependencies we need to land first? or are those already landed? | |
| 14:18:07 | yuval | Hey, Is there anyway to tell zuul to install os-brick from master and not pypi? | |
| 14:18:50 | yuval | I added "Depends-On:" in the commit msg but didnt do the trick | |
| 14:28:05 | gibi | yuval: probably need to add | |
| 14:28:06 | gibi | required-projects: | |
| 14:28:10 | gibi | -os-brick | |
| 14:28:14 | gibi | to the job config | |
| 14:28:26 | gibi | required-projects: | |
| 14:28:39 | gibi | - openstack/os-brickj | |
| 14:29:20 | yuval | hmm someone use this option lately so I can see an example? | |
| 14:29:26 | yuval | used | |
| 14:30:06 | gibi | yuval: https://github.com/openstack/neutron/blob/c7f35d3870cb20de997231f2b502973fbcd0c3e7/zuul.d/tempest-singlenode.yaml#L272-L278 I stole the idea from here | |
| 14:31:38 | yuval | Thanks! | |
| 14:38:57 | dmitriis | gibi: RE the depends-on, had a discussion here https://review.opendev.org/c/openstack/nova/+/824833/1/nova/network/neutron.py#669 with sean-k-mooney. So https://review.opendev.org/c/openstack/neutron/+/808961 depends on the Nova change. As such, we don't have Neutron changes that need to be landed first. | |
| 14:39:20 | dmitriis | the VNIC type is already in the neutron lib because of the past Ironic work and we're just reusing it | |
| 14:39:45 | gibi | dmitriis: cool, thanks for the info | |
| 14:41:56 | artom | Huh, so https://review.opendev.org/c/openstack/nova/+/827549 passed | |
| 14:42:06 | artom | Looks like q35 *is* the culprit | |
| 14:42:42 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/805649 | |
| 14:44:26 | opendevreview | yuval proposed openstack/nova master: Lightbits LightOS driver https://review.opendev.org/c/openstack/nova/+/821606 | |
| 14:45:10 | yuval | gibi: I added as you mentioned ^ | |
| 14:46:58 | gibi | yuval: does your patch depends on a new os-brick feature? | |
| 14:47:32 | yuval | yes | |
| 14:48:29 | gibi | yuval: ok. so what you did not with zuul allows you to test the new os-brick feature together with the nova feautre | |
| 14:48:56 | gibi | yuval: but the final solution will be to merge the os-brick change first, then release os-brick, then bump the requirement to use the new os-brick release | |
| 14:58:16 | sean-k-mooney | gibi: yep was talking to yuval in parrallel about that | |
| 14:58:56 | sean-k-mooney | yuval: if you dont feel comforatable creatign the release patch i can submit it if you are willing to take it over and or respond to any question the maintaienr have | |
| 14:59:08 | gibi | ack | |
| 14:59:09 | yuval | yes, thank you both | |
| 14:59:09 | sean-k-mooney | gibi: os-brick is mainly maintained by cidner write | |
| 14:59:44 | sean-k-mooney | i know its kind fo shared owner ship but officaly its a cinder deliverbale in governace so there ptl/release leaision need to approve? | |
| 14:59:53 | yuval | sean-k-mooney: its ok, let me do some checking. there are 2 followups I need to add to my driver before they release | |
| 15:00:06 | gibi | sean-k-mooney: yepp it is under cinder | |
| 15:01:16 | sean-k-mooney | yuval: ok well its proably good to start the process early. looking at the patch delta https://github.com/openstack/os-brick/compare/5.1.0...master it should be a feature bump to 5.2.0 | |
| 15:01:25 | sean-k-mooney | i think it should be uncontoversal | |
| 15:02:34 | sean-k-mooney | just add another release to https://github.com/openstack/releases/blob/master/deliverables/yoga/os-brick.yaml and follow up with the os-brick/cinder folks on #openstack-cinder | |
| 15:03:06 | sean-k-mooney | yuval: unless you ment there are followup to the os-brick part | |