Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-11-02
08:35:57 gibi good morning
08:45:56 lyarwood Morning
08:48:42 bauzas fwiw, our sanitary protocols become stricter for schools (and my kids are back in school after 2 weeks), so I'll need to disappear like 30 mins every 3 hours
09:45:19 alex_xu gibi: stephenfin, If I have a feature have to do numa affinity, then we must implement numa in placement first, right? As my understand, the two phase commit is only for resolving the race problem, we still need back to numa in placement when we need affinity, is it right?
09:52:34 gibi alex_xu: I don't think this is a must any more. I think we agreed that we all would prefere having the numa modelled in placement. but it did not happened in the last cycles. so we are not sure it will ever happen. If you have time and willingnes to work on the numa in placement implementation then I think that is welcomed.
09:53:19 gibi alex_xu: but if you propose a numa affinity feature without numa in placement then it is also OK at least to discuss
09:54:28 alex_xu gibi: yea, I see that, I will get that back to team and pm, see if we have resource
09:54:38 alex_xu gibi: thanks, I clear the path now
09:54:51 gibi OK, cool
10:56:59 hemanth_n Hi Nova folks, I am looking for the following review to get merged https://review.opendev.org/#/c/749175/ unless there is something else to take care of.. it is already +2ed couple of time
10:58:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional-py39 tox target https://review.opendev.org/760884
11:34:49 gibi sean-k-mooney: hi! it is just me or the os-vif unit test os_vif.tests.unit.internal.ip.linux.test_impl_pyroute2.TestIpCommand.test_add_exit_code fails consistently (at least locally for me)
11:37:57 openstackgerrit Balazs Gibizer proposed openstack/os-vif master: DNM https://review.opendev.org/760891
11:39:10 gibi sean-k-mooney: never mind, it was a local issue for me
11:51:58 slaweq sean-k-mooney: gibi: hi, can You take a look at https://bugs.launchpad.net/nova/+bug/1902516 when You will have few minutes? I saw it at least couple of times in the ci in last few days, maybe You know already what is the root cause of it :)
11:51:58 openstack Launchpad bug 1902516 in OpenStack Compute (nova) "[CI] Libvirt error "TCG doesn't support requested feature: CPUID.01H:ECX.vmx" cause failure while spawning instance" [Undecided,New]
11:54:28 gibi slaweq: ack, I haven't seen this yet
11:54:38 slaweq gibi: thx
11:58:10 gibi slaweq: thanks for reporting it
11:58:49 slaweq gibi: yw :)
13:46:23 lyarwood gibi: https://review.opendev.org/#/c/758928/ - would you mind taking a look at this when you have a chance? melwitt++
13:50:22 gibi lyarwood: will check
13:51:16 lyarwood many thanks
13:58:02 gmann gibi: it will be good to add gate job too for py3.9 along with tox env. what you say?
13:58:07 gmann this one https://review.opendev.org/#/c/760884/1
13:59:07 gibi gmann: ohh, now I got efried's comment in the review. Sure I can try to add functional jobs to the zuul.yaml
13:59:36 gmann because functional jobs does not come form generic template, it is added in project side explicitly
13:59:43 gmann thanks
13:59:49 efried ^
14:00:48 gibi efried: hey! can I say that welcome back?
14:01:14 efried Afraid not. I just happened to see your email in the ML and thought, "Hey, there's an easy review for me!"
14:01:20 gibi :)
14:01:23 gibi how is life?
14:01:34 efried Life is *good*.
14:02:05 gibi I'm glad to hear that
14:08:06 gibi lyarwood: +A https://review.opendev.org/#/c/758928/
14:18:18 lyarwood thanks
14:26:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional-py39 testing https://review.opendev.org/760884
14:26:55 bauzas gibi: gave a couple of +2s
14:27:00 bauzas for the py39 tests
14:29:59 gibi bauzas: thanks
14:33:16 bauzas let's see how it goes
14:36:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional-py39 testing https://review.opendev.org/760884
14:47:37 gmann gibi: adding base func job for py3.9 - https://review.opendev.org/#/c/760932/
14:47:42 gmann that we can use as depends-on
14:49:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional-py39 testing https://review.opendev.org/760884
14:50:07 gibi gmann: thanks. now the nova patch points to it ^^
14:50:17 gmann gibi: thanks.
15:14:17 bauzas gibi: fwiw, working on the RPC Compute API bump
15:14:29 bauzas hopefully will provide a patch in between tomorrow and wed
15:14:46 bauzas but we could merge it around m-3
15:14:59 bauzas (in case people want to provide a new RPC API version)
15:18:30 dansmith bauzas: you're going to do the bump now and just keep maintaining it until m3?
15:18:54 dansmith I mean, if we don't change anything, then.. that will work fine, but if we do, it's just extra work
15:47:33 bauzas dansmith: yup
15:47:52 bauzas just in case we don't have time close to m-3
15:47:56 dansmith okay
16:01:56 gibi slaweq: this does not seem to be a nova issue https://bugs.launchpad.net/nova/+bug/1902516 . the hypervisor run out of memory.
16:01:56 openstack Launchpad bug 1902516 in OpenStack Compute (nova) "[CI] Libvirt error "TCG doesn't support requested feature: CPUID.01H:ECX.vmx" cause failure while spawning instance" [Undecided,New]
16:02:18 elod lyarwood + stable cores: I'm planning to propose a release patch for Stein (final-release-before-em :)) somewhere during the week. Do you see any patch that would be good to be part of the release? Anything that we should wait to merge before the release?
16:03:41 elod a list for some help: https://etherpad.opendev.org/p/nova-stable-stein-em
16:03:46 gibi slaweq: btw I've now changed the heading of the bug https://bugs.launchpad.net/nova/+bug/1902516 to reflect the real error. the original heading was a red herring
16:03:46 openstack Launchpad bug 1902516 in OpenStack Compute (nova) "[CI] Libvirt error "cannot set up guest memory 'pc.ram': Cannot allocate memory" cause failure while spawning instance" [Undecided,New]
16:04:25 slaweq thx gibi
16:05:13 gibi slaweq: you might consider lowering the concurrency in tempest or using bigger flavor for the job
17:08:58 melwitt dansmith: cherry pick fix, stein edition https://review.opendev.org/760673
17:09:45 stephenfin Anyone care to review the OSC change for "nova evacuate" support? https://review.opendev.org/#/c/643578/
17:10:39 lyarwood stephenfin: did you fix the tests?
17:10:45 stephenfin yarp
17:10:50 lyarwood stephenfin: I pulled that down a while ago but didn't have time to sort them out
17:10:58 lyarwood stephenfin: cool, I'll review tonight in that case
17:11:08 stephenfin (it was just an unnecessary mock assertion)
17:30:04 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP Add os-volume_attachments reference docs https://review.opendev.org/760971
17:53:58 openstackgerrit melanie witt proposed openstack/nova stable/ussuri: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760977
17:56:40 openstackgerrit melanie witt proposed openstack/nova stable/train: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760978
18:11:49 dansmith melwitt: I like this game
18:12:39 melwitt :)
18:15:37 openstackgerrit melanie witt proposed openstack/nova stable/stein: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760984
18:16:55 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760985
18:19:08 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760985
18:22:35 openstackgerrit melanie witt proposed openstack/nova stable/queens: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/760987
19:16:07 openstackgerrit Merged openstack/nova stable/stein: Follow up for cherry-pick check for merge patch https://review.opendev.org/760673
19:41:49 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP Add os-volume_attachments reference docs https://review.opendev.org/760971
19:46:55 gmann dansmith: can you check this- glance policy override update https://review.opendev.org/#/c/760422/
19:49:11 dansmith gmann: I assume that you know that "" means "anyone" in which case, I'm sure that's fine
19:49:37 dansmith the only thing is, it doesn't allow us to write a test that confirms that the policy gets honored (i.e. by trying to create a non-public image and copy it)
19:49:52 dansmith but assuming that is not a big deal, this seems okay to me
19:50:55 gmann dansmith: yeah, if we add such test then we can run that in other jobs. means exclude to run that from this job
19:51:25 dansmith ack
19:55:13 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Follow up for cherry-pick check for merge patch https://review.opendev.org/761000
20:05:21 melwitt dansmith: more game time ^
20:05:49 dansmith how many more episodes of this game are there?
20:06:02 melwitt one more I think. let me double check
20:06:49 melwitt two more, queens and pike. bug made it all the way back there :(
20:07:03 dansmith shesus
20:07:35 melwitt yeah
20:48:56 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
20:48:56 openstack bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
20:48:57 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
20:48:57 openstackgerrit melanie witt proposed openstack/nova stable/queens: Follow up for cherry-pick check for merge patch https://review.opendev.org/761023

Earlier   Later