| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-01 | |||
| 13:47:32 | yonglihe | thanks, i gonna run, have a good day. | |
| 13:49:55 | gmann | sean-k-mooney: lyarwood gibi basically if new return error code is new to that API (not from existing one) then we need microversion bump. and these are general return error code for all APis 400, 403, 404 and 415 | |
| 13:51:00 | gmann | sean-k-mooney: lyarwood gibi which APIs is this? I think most of server APIs have 409 exist return code | |
| 13:55:11 | gibi | os-services/{id}/force_doewn PUT | |
| 13:59:27 | gmann | yeah 409 is not there https://github.com/openstack/nova/blob/50fdbc752a9ca9c31488140ef2997ed59d861a41/nova/api/openstack/compute/services.py#L358 | |
| 14:02:28 | gibi | yepp | |
| 14:02:43 | gibi | I accept 400 as a compromise for backportability | |
| 14:03:17 | gmann | +1 | |
| 14:17:16 | sean-k-mooney | gmann: yep that is basically what we discusssd on irc this morning 400 for backport reasons. 409 is more correct but would require version bump so we cant use it | |
| 14:17:20 | sean-k-mooney | at least not yet | |
| 14:17:48 | sean-k-mooney | we can make it use 409 in a followup patch if we wish too | |
| 14:23:23 | gmann | but we would not be able to backport 409 if do in follow up due to microvesion bump. | |
| 14:27:27 | sean-k-mooney | correct | |
| 14:27:44 | sean-k-mooney | so we were suggeing using 400 and backporting that | |
| 14:27:57 | sean-k-mooney | and leavign a todo to make it a 409 at some other point in the future | |
| 14:28:15 | sean-k-mooney | we coudl bundel it with some of the ohter todos we have | |
| 14:28:28 | sean-k-mooney | we have one for consitent return codes for unimplemnted feature for example | |
| 14:28:34 | gmann | yeah. make sense | |
| 15:19:02 | artom | Is Zuul... OK? | |
| 15:19:36 | artom | https://downforeveryoneorjustme.com/zuul.opendev.org ah :( | |
| 15:20:36 | melwitt | see #opendev for details, they've been debugging issues since yesterday | |
| 15:51:47 | bauzas | folks, gentle reminder, nova meeting in 9 mins on #openstack-meeting-3 | |
| 16:00:52 | bauzas | meeting started on #openstack-meeting-3 | |
| 17:01:35 | gibi | stephenfin: enjoy o/ | |
| 18:18:26 | sean-k-mooney | artom: by the way you proably want https://zuul.openstack.org/ | |
| 18:19:01 | sean-k-mooney | https://zuul.opendev.org/tenants has all then tenants | |
| 18:19:30 | sean-k-mooney | but zuul.openstack.org will show you the openstack ci dashboard | |
| 18:34:12 | artom | sean-k-mooney, I was just copy-pasting from the logs url | |
| 18:41:27 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/784129 | |
| 18:41:30 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/784130 | |
| 18:41:30 | openstack | bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,In progress] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood) | |
| 20:27:57 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Neutron fixture: don't clobber profile and vif_details if empty https://review.opendev.org/c/openstack/nova/+/783385 | |
| 20:27:58 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 20:27:58 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | |
| 20:28:16 | sean-k-mooney | :) | |
| 20:28:22 | sean-k-mooney | no more WIP | |
| 20:28:45 | artom | Yeah, rewrite the ugly [0] logic, and added a unit test | |
| 20:28:48 | artom | And a release note | |
| 20:28:51 | artom | And the bug reference | |
| 20:28:54 | artom | And my axe! | |
| 20:29:40 | sean-k-mooney | ah yes your using next() | |
| 20:29:53 | sean-k-mooney | you technially have extra () | |
| 20:29:59 | sean-k-mooney | that you dont need | |
| 20:30:15 | sean-k-mooney | equest = next((r for r in instance.pci_requests.requests | |
| 20:30:17 | sean-k-mooney | if r.requester_id == port['id'])) | |
| 20:30:37 | artom | No, that's needed | |
| 20:30:40 | artom | Wait, is it? | |
| 20:30:44 | artom | Maybe it isn't... | |
| 20:30:54 | artom | What you pass to next() has to be an interator | |
| 20:30:55 | sean-k-mooney | you are creating a touple | |
| 20:31:04 | artom | *iterator, not a list/tuple | |
| 20:31:07 | sean-k-mooney | but you can pass the generator expression driectly to next | |
| 20:31:38 | sean-k-mooney | iterator or generator | |
| 20:31:58 | sean-k-mooney | "r for r in instance.pci_requests.requests" is a generator expression | |
| 20:32:08 | artom | Oh yeah, works without the extra ( ) | |
| 20:32:11 | artom | TIL | |
| 20:32:36 | sean-k-mooney | with the extra () its doing a touple comprehention and then it taking the first element | |
| 20:32:36 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 20:32:53 | sean-k-mooney | without it its iterating the generator untile the first value is yeilded | |
| 20:33:20 | sean-k-mooney | so its faster too since it will early exit | |
| 20:35:03 | sean-k-mooney | artom: thats a python 3 thing by the way | |
| 20:35:20 | artom | Oh | |
| 20:35:21 | artom | Hrmm | |
| 20:35:34 | artom | Well, no, it's fine, Train is Python3 | |
| 20:35:36 | sean-k-mooney | its easy to add them back when you get to train | |
| 20:35:44 | sean-k-mooney | its both | |
| 20:35:52 | artom | Ah, it's OSP16 that's py3-only | |
| 20:35:58 | artom | Because RHEL8 | |
| 20:36:03 | sean-k-mooney | ussuri was the first one that is pyton 3 only | |
| 20:36:07 | sean-k-mooney | yes | |
| 20:36:21 | sean-k-mooney | which is proably the only time rhel has done someting first :) | |
| 20:37:16 | artom | *snerk* | |
| 20:40:19 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 20:40:21 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 20:40:22 | openstackgerrit | sean mooney proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | |
| #openstack-nova - 2021-04-02 | |||
| 03:00:08 | openstackgerrit | melanie witt proposed openstack/nova master: Remove unused DeleteFromSelect db api method https://review.opendev.org/c/openstack/nova/+/784528 | |
| 03:07:49 | yonglihe | alex_xu, we reproposal smart nic spec to this release, needs your bandwidth for that, and whole refactored codes plus fucntional test: https://review.opendev.org/c/openstack/nova-specs/+/783632 | |
| 05:39:36 | alex_xu | yonglihe: yea, that is in my list | |
| 05:39:52 | yonglihe | thanks, | |
| 06:08:19 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Trival change: correct some nits https://review.opendev.org/c/openstack/nova-specs/+/780443 | |
| 06:10:56 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Trival change: correct some nits https://review.opendev.org/c/openstack/nova-specs/+/780443 | |
| 21:55:54 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 21:55:56 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 22:00:22 | openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | |
| 22:00:23 | openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | |
| 22:02:41 | openstackgerrit | sean mooney proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | |
| #openstack-nova - 2021-04-03 | |||
| 11:23:21 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Update contributor guide for Xena https://review.opendev.org/c/openstack/nova/+/783534 | |
| 13:27:14 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/stein: Correctly disable greendns https://review.opendev.org/c/openstack/nova/+/784581 | |
| #openstack-nova - 2021-04-06 | |||
| 01:34:09 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/784870 | |
| 01:37:20 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/714439 | |
| 01:41:33 | openstackgerrit | Kevin Zhao proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/c/openstack/nova/+/714439 | |
| 03:28:14 | openstackgerrit | Nobuhiro MIKI proposed openstack/nova master: libvirt: update the backing file path during hard_reboot https://review.opendev.org/c/openstack/nova/+/784875 | |
| 07:03:30 | gibi | good morning | |
| 07:20:08 | bauzas | good morning folks | |
| 07:20:30 | bauzas | my kids are at home as the school is closed, so today I'll be a bit off this morning | |
| 07:21:23 | gibi | bauzas: ack | |
| 08:10:34 | gibi | yonglihe: brinzhang_: hi! do you happen to know if the cyborg team has any topics for the PTG that would like to discuss with the nova team? If there is any then we can try to find a timeslot in the schedule to meet | |
| 08:13:19 | brinzhang_ | gibi: as I know, we have not any topic need to be talk at this PTG, for example, the cybrog vGPU support we have an AGGREMENT but less of implementation at Wallaby, just need to re-propose | |
| 08:17:34 | brinzhang_ | gibi: I will check with yonglihe and xinran, is there something need to be discuss with smartNIC, later ping you again^ | |
| 08:23:34 | brinzhang_ | gibi: I have checked with xinranwang, we dont have any topics with smarNIC for the PTG, thanks | |