Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-04-01
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
08:23:47 gibi brinzhang_: I looked through the current nova PTG etherpad and got the same feeling we have previous agreements with cyborg around features
08:24:03 gibi brinzhang_: thanks for checking
08:24:34 brinzhang_ gibi: yes, thanks too
08:24:46 gibi then I will not dedicate a slot for cyborg in the nova PTG, if anything comes up then we can take them in the normal nova sessions
08:27:11 brinzhang_ gibi: yeah, if there is anything we will take them here ^

Earlier   Later