| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-02 | |||
| 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 | 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: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: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 | 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: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: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 | 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: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: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 | 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:56 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636 | |
| 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: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: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 | |
| 21:04:06 | lyarwood | QUICK +W ALL THE THINGS TO WIN | |