| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-10 | |||
| 20:33:13 | gmann | jamesbenson: cirros is one we use in upstream too and password is configurable in tempest config. But tempest is not ready to run for all combination of micrversion. | |
| 20:33:38 | gmann | I will say try with min_microversion=None and max_microversion='2.87' and then we can see if they still fail then its tempest issue | |
| 20:34:29 | jamesbenson | ok, sounds good. These are our setups for tempest.conf | |
| 20:34:31 | jamesbenson | https://gitlab.com/utsa-ics/osias/-/blob/master/test_setup.sh | |
| 20:36:11 | gmann | yeah, that seems correct image_ssh_password but there might be some test issue, if you face issue ping me on #openstack-qa or file bug in tempest i can fix that | |
| 20:36:37 | jamesbenson | you are a life saver! Thank you! | |
| 20:37:13 | gmann | np! | |
| 20:39:53 | jamesbenson | I do have one re-occuring bug, it always happens in ussuri up in the tearDownClass (tempest.api.compute.servers.test_multiple_create.MultipleCreateTestJSON). Details: {'type': 'SubnetInUse', 'message': 'Unable to complete operation on subnet 5be11612-2a84-4723-b683-3cf06e3567d4: One or more ports have an IP allocation from this subnet.', 'detail': ''} | |
| 20:40:22 | jamesbenson | It can't destroy the subnet it creates.... | |
| 20:41:54 | gmann | jamesbenson: can you please log bug in tempest for that, I will check on Monday | |
| 20:42:08 | gmann | here https://bugs.launchpad.net/tempest | |
| 20:42:15 | jamesbenson | sure thing | |
| 20:42:33 | gmann | thanks | |
| 20:44:14 | jamesbenson | Any special logs you need or just the output from Refstack on that? | |
| 20:44:49 | gmann | you can add tempest log also | |
| 20:50:23 | jamesbenson | https://bugs.launchpad.net/tempest/+bug/1954535 | |
| 23:05:31 | jamesbenson | @gmann, 'None' had issues so I changed it to 2.1 for the min. I'm rerunning it now, I'll report back on Monday. Thanks again and have a great weekend! | |
| 23:07:11 | gmann | jamesbenson: sure, you too. | |
| #openstack-nova - 2021-12-11 | |||
| 21:11:22 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/c/openstack/nova/+/710847 | |
| 21:36:09 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/c/openstack/nova/+/710847 | |
| 22:00:16 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| #openstack-nova - 2021-12-12 | |||
| 03:13:39 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/c/openstack/nova/+/710847 | |
| 03:40:56 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 14:28:33 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/c/openstack/nova/+/710847 | |
| 16:08:14 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 16:47:06 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/760354 | |
| 16:48:58 | opendevreview | Stanislav Dmitriev proposed openstack/nova master: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/760354 | |
| #openstack-nova - 2021-12-13 | |||
| 01:56:52 | sdmitriev | https://review.opendev.org/c/openstack/nova/+/760354/5 | |
| 01:56:54 | sdmitriev | https://review.opendev.org/c/openstack/nova/+/710848/6 | |
| 01:57:06 | sdmitriev | https://review.opendev.org/c/openstack/nova/+/710847/8 | |
| 01:57:23 | sdmitriev | Hello there, just rebased these commits above, would really appreciate if someone could review them | |
| 01:57:33 | sdmitriev | I'm really interested in fixing that bug since deployments are pretty much affected | |
| 08:47:49 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova-specs master: Implements: blueprint soft-delete-instance-actions https://review.opendev.org/c/openstack/nova-specs/+/821387 | |
| 09:42:23 | gibi | bauzas, sean-k-mooney: this is my last week before vacation and I will not be back until M2 so I'd like to get over with the placement spec https://review.opendev.org/q/topic:any-traits-support this week. | |
| 09:42:42 | bauzas | gibi: ack good point | |
| 09:42:46 | gibi | it was already approved a long time ago so I don't think it is controversioal | |
| 09:42:52 | bauzas | and reminder that we have our spec review day tomorrow | |
| 09:43:03 | gibi | yepp, tomorrow is good for me | |
| 09:43:04 | gibi | :) | |
| 09:50:36 | gibi | as a trade I'm willing to look at many reviews this week as I don't want to start any new on my side just closing down open things | |
| 09:50:41 | gibi | so hit me with reviews | |
| 11:03:45 | gibi | sean-k-mooney: I'm +2 on the recent version of the instance action softdelete spec https://review.opendev.org/c/openstack/nova-specs/+/821387 | |
| 11:11:25 | gibi | bauzas: and easy stable/xena backport needing a second core (lyarwood is already on PTO) https://review.opendev.org/q/topic:%2522bug/1945310%2522+branch:stable/xena | |
| 11:17:48 | bauzas | gibi: done | |
| 11:18:14 | bauzas | gibi: I'm just reviewing pslestang's spec already :) | |
| 11:50:07 | sean-k-mooney | gibi: just reading back | |
| 11:50:33 | sean-k-mooney | ill look at the placement one shortly | |
| 11:51:27 | sean-k-mooney | gibi: i was pretty happy with the instance action spec so ill look at that again this morning since its been revised | |
| 11:52:57 | sean-k-mooney | gibi: by the way i dong thave +2 rights on placemt/placemsnt spec directory since its in tree but ill review it anyway | |
| 11:56:12 | sean-k-mooney | gibi: oh this is the spect that like yuou do required=in:TRAIT1,TRAIT2 | |
| 11:57:55 | opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | |
| 12:07:30 | gibi | sean-k-mooney: yes, that is in :TRAIT1,TRAIT2 | |
| 12:07:58 | sean-k-mooney | gibi: i have one open qustion/suggestion for that spec | |
| 12:09:52 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/placement/+/649992/5/doc/source/specs/yoga/approved/2005346-any-traits-in-allocation_candidates-query.rst#68 | |
| 12:10:59 | sean-k-mooney | oh i see the last spec is for mixing which is what i was asking us to support | |
| 12:11:50 | sean-k-mooney | the proposal for mixing will work for placment but now for nova/glance | |
| 12:12:47 | sean-k-mooney | i tink just using ; is simpler | |
| 12:24:56 | gibi | sean-k-mooney: it is almost directly following how member_of is modelled in the placement API | |
| 12:25:18 | gibi | so repeat means AND. OR expressed with in: prefix | |
| 12:26:02 | gibi | sure ';' could be used instead of repeat of the required query param but then we would create inconsistency with member_of | |
| 12:32:00 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova-specs master: Implements: blueprint soft-delete-instance-actions https://review.opendev.org/c/openstack/nova-specs/+/821387 | |
| 12:33:07 | sean-k-mooney | gibi: it would yes but the ofllow on proposal of using multiple &required=...&required=... | |
| 12:33:17 | sean-k-mooney | cant be modled in flavors or images | |
| 12:33:30 | sean-k-mooney | since we cannot support multi opts and merge them | |
| 12:33:48 | sean-k-mooney | so we will need to have some other way to mix them in the flavor/image | |
| 12:34:20 | sean-k-mooney | which is why i think either requiring all in parmater to be at the end or using ; makes sense | |
| 12:35:40 | gibi | the current falvor syntax is trait:HW_CPU_X86_AVX2=required , so it can extended to trait:inFOO,BAR=required | |
| 12:35:49 | opendevreview | Merged openstack/nova stable/xena: Reproduce bug 1945310 https://review.opendev.org/c/openstack/nova/+/811405 | |
| 12:36:32 | gibi | also trait:inFOO,BAR=required and trait:BAZ=required can be mixed | |
| 12:36:35 | sean-k-mooney | i guess that could work | |
| 12:36:38 | pslestang | gibi: thanks for reviewing the spec, I pushed an other patch to fix all the syntax error you underligned | |
| 12:36:44 | gibi | the flavor syntax is ugly | |
| 12:36:47 | gibi | but that is a different story | |
| 12:36:48 | gibi | :D | |
| 12:36:50 | sean-k-mooney | gibi: this might be nicer trait:FOO,BAR=required_in | |
| 12:37:03 | gibi | sean-k-mooney: yeah, that could be another option | |
| 12:37:24 | gibi | the point is that we have a way to modell repetition as the key contains the trait name | |
| 12:37:33 | gibi | pslestang: I will check soon | |
| 12:37:58 | sean-k-mooney | gibi: ok the image/flavor represnetiaotn is out of scope of the placment spec anyway | |
| 12:38:10 | sean-k-mooney | so i think you have convinced me that this can work | |
| 12:38:41 | gibi | yes, but you had a point, if the flavor have had required=FOO syntax then we would be in deep trouble | |
| 12:38:57 | gibi | so it was a good excersize to see how flavor will fit | |
| 12:38:57 | sean-k-mooney | ya i forgot we reversted it | |
| 12:39:19 | sean-k-mooney | and didn trait:*=required | |
| 12:40:33 | sean-k-mooney | ill update my review with this conversation and change to a +1 | |
| 12:41:54 | gibi | sean-k-mooney: thanks | |
| 12:41:58 | gibi | pslestang: +2 | |
| 12:43:01 | sean-k-mooney | pslestang: ill review that sortly after i grab a coffee | |
| 12:43:31 | sean-k-mooney | i unexpectly have meeting for the next 4 hours but i should have time to review your spec before i have to join them | |
| 12:44:27 | sean-k-mooney | its almost as if there is pending holidays that people will be taking on mass and everything is beeing cramed into this week :) | |
| 12:44:29 | gibi | sean-k-mooney: I hope that 4 hour meeting is some light gathering before the vacation time to chill out and slow down :) | |
| 12:44:44 | pslestang | gibi: thanks | |
| 12:44:49 | plibeau3 | lyarwood: I have apply a part of proposition. I have put some question on patchset3. -> https://review.opendev.org/c/openstack/nova/+/820531 | |
| 12:44:55 | pslestang | sean-k-mooney: sure, don't worry | |
| 12:45:06 | gibi | pslestang: thank you for the nicely written spec! | |
| 12:45:20 | gibi | plibeau3: lyarwood is on PTO already | |
| 12:45:31 | gibi | plibeau3: so be prepared for slow response :) | |
| 12:46:00 | plibeau3 | gibi: oki :) | |
| 12:47:50 | opendevreview | Merged openstack/nova stable/xena: Query ports with admin client to get resource_request https://review.opendev.org/c/openstack/nova/+/811407 | |
| 12:54:58 | sean-k-mooney | bauzas: unless you want to to specifically look at the instance-action sepc im also not +2 on it and was going to +w | |