| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-02 | |||
| 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 | |
| 20:50:51 | openstackgerrit | melanie witt proposed openstack/nova stable/pike: Follow up for cherry-pick check for merge patch https://review.opendev.org/761024 | |
| 21:00:56 | dansmith | that smug bastard lyarwood is going DOWN | |
| 21:01:31 | lyarwood | oh my | |
| 21:03:08 | melwitt | lol | |