| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-23 | |||
| 02:07:30 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | |
| 05:28:24 | openstackgerrit | Merged openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | |
| 05:28:31 | openstackgerrit | Merged openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance https://review.opendev.org/736997 | |
| 05:52:51 | openstackgerrit | Merged openstack/nova master: objects: Update keypairs when saving an instance https://review.opendev.org/683043 | |
| 06:36:41 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 06:57:48 | openstackgerrit | Brin Zhang proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186 | |
| 07:13:53 | gibi | good morning | |
| 07:15:58 | brinzhang_ | gibi: good moning^ | |
| 07:16:42 | brinzhang_ | gibi: I was update the https://review.opendev.org/#/c/716186/16/nova/exception.py patch, but as your comment, after I update, taht will be an error, you can see in http://paste.openstack.org/show/796234/ | |
| 07:17:10 | brinzhang_ | do you know why? the code looks good | |
| 07:22:29 | gibi | brinzhang_: will check in 20 minutes (on a call right now) | |
| 07:23:13 | brinzhang_ | gibi: thanks, I missed a parameter in __init__, will update | |
| 07:30:56 | openstackgerrit | Brin Zhang proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186 | |
| 07:30:57 | brinzhang_ | gibi: you can check all the update in the latest patch now. | |
| 08:10:20 | kashyap | lyarwood: I see the VIR_ERR_DEVICE_MISSING is merged; sorry, yetesrday I had the draft ACK sitting but didn't hit 'submit'. I see it's taken care by three cores :) | |
| 08:29:14 | gibi | brinzhang_: thanks, I'm +2 | |
| 08:29:34 | brinzhang_ | gibi: coolthanks | |
| 08:33:56 | gibi | stephenfin: I put https://review.opendev.org/#/q/topic:bp/add-emulated-virtual-tpm to the empty runway slot for the next two weeks | |
| 08:34:08 | gibi | stephenfin: is that OK for you? | |
| 08:35:10 | gibi | stephenfin: I will be on PTO next week but I hope others will check | |
| 08:35:27 | gibi | and I can pick that up the week after | |
| 08:45:58 | gibi | dansmith: Do I see it correclty that the work in https://blueprints.launchpad.net/nova/+spec/rbd-glance-multistore has been completed? | |
| 09:18:15 | aarents | Hi nova, | |
| 09:18:44 | aarents | gibi, when it will be possible, can you advise on last comments of https://review.opendev.org/#/c/696084/3 as my change conflicts with yours here: https://review.opendev.org/#/c/739246/1 as lyarwood mentioned | |
| 09:22:58 | gibi | aarents: looking... | |
| 09:34:56 | gibi | aarents: go ahead with your patch my patch can be the last in the chain | |
| 09:38:06 | aarents | gibi: ok thanks | |
| 09:39:51 | stephenfin | gibi: Sure. Could I ask you to grab the base patch in the series again before you go? You reviewed it before and it's just a rebase | |
| 09:46:15 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/ussuri: objects: Update keypairs when saving an instance https://review.opendev.org/742631 | |
| 09:47:51 | jkulik | Do you see any chance of implementing some kind of sharding on the nova-compute level? With the vmwareapi driver managing a cluster of hypervisors, there are hundreds of VMs behind every nova-compute node and at times this leads to tasks queuing up a lot. | |
| 09:48:53 | bauzas | gibi: stephenfin: fwiw, I'll enter the Phantom Zone starting end of next week for the next 3 weeks | |
| 09:49:18 | bauzas | gibi: so when you are back, I'll be off (IIUC your vacation time) | |
| 09:49:19 | jkulik | It would be nice, if we could spawn multiple nova-compute processes per cluster and I was thinking of letting them work on VMs based on the UUID so they don't interfere so easily. | |
| 09:50:05 | openstackgerrit | Merged openstack/nova master: compute: Do not allow rescue attempts using volume snapshot images https://review.opendev.org/729261 | |
| 09:50:23 | stephenfin | bauzas: What's the chances of having you work the vTPM stuff with lyarwood? I can handle the 'provider.yaml' stuff (and anything else) since I didn't write it, but not vTPM | |
| 09:50:47 | bauzas | stephenfin: by 'work', you mean 'reviews' ? | |
| 09:50:55 | stephenfin | Correct | |
| 09:50:57 | bauzas | I surely could for next week | |
| 09:51:17 | bauzas | that being said, I'm not an expert, but I guess this is not a drama. | |
| 09:51:25 | stephenfin | \o/ | |
| 09:51:57 | stephenfin | It's not complicated. libvirt does all the work for us. Very much just wiring up | |
| 10:05:22 | gibi | stephenfin: sure, I will re-review the bottom of vtpm today | |
| 10:06:17 | gibi | bauzas: ack. have a nice vacation | |
| 10:08:49 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301 | |
| 10:10:23 | jsuchome | Hi gibi, thanks for the review, I've just updated those whitespaces ^ | |
| 10:13:56 | gibi | jsuchome: cool. I plugged my +2 back | |
| 10:14:01 | gibi | thanks | |
| 10:34:08 | openstackgerrit | Merged openstack/nova stable/stein: libvirt: Mark e1000e VIF as supported https://review.opendev.org/742390 | |
| 10:47:54 | openstackgerrit | Merged openstack/nova stable/pike: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/742181 | |
| 10:48:00 | openstackgerrit | Merged openstack/nova master: test: add some notes and new tests for mixed instance https://review.opendev.org/741395 | |
| 11:17:10 | gibi | stephenfin: there is small test thing is https://review.opendev.org/#/c/739210/5/nova/tests/unit/objects/test_image_meta.py@424 if you have time to respin the I will re-review, but it is also OK to fix it in a FUP | |
| 11:25:00 | lyarwood | stephenfin: FWIW I have the series open as well | |
| 12:07:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Default request group to None https://review.opendev.org/742651 | |
| 12:07:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Test object backporting against correct version https://review.opendev.org/742650 | |
| 12:07:12 | stephenfin | lyarwood, gibi: ^ | |
| 12:07:57 | stephenfin | The latter one is in merge conflict for now, but I'm hoping bauzas can review https://review.opendev.org/#/c/739210/ today (since he understands this stuff) and I can rebase it then :) | |
| 12:11:35 | stephenfin | gibi: Are versioned notifications the best solution for the race you identified in https://review.opendev.org/#/c/741285/ ? | |
| 12:12:05 | gibi | stephenfin: that could be a good one yes. you can also wait for instance actions | |
| 12:12:27 | gibi | I like notifications better for obvious reasons | |
| 12:12:37 | gibi | I know them better | |
| 12:13:00 | stephenfin | ack | |
| 12:28:50 | gibi | stephenfin: I'm +2 on the unsuffixed group defaulting | |
| 12:29:46 | gibi | also I executed a bunch of test runs on the top of your test refactoring patches to tease out any race but I can report that the tests are solid | |
| 12:29:59 | gibi | so good stuff! thanks! | |
| 12:38:07 | bauzas | stephenfin: ack, will look at the vTPM stuff today then | |
| 13:09:21 | openstackgerrit | Merged openstack/nova master: libvirt: Track blockjob status in DEBUG when polling for completion https://review.opendev.org/729595 | |
| 13:17:36 | openstackgerrit | Merged openstack/nova stable/rocky: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full https://review.opendev.org/742258 | |
| 13:29:14 | dansmith | gibi: yeah, I have one doc patch left that I haven't finished, but otherwise it's done IMHO | |
| 13:31:59 | gibi | dansmith: thanks. then I'll wait for the doc patch to land before I mark the bp implemented | |
| 13:34:42 | dansmith | gibi: ack | |
| 13:35:28 | openstackgerrit | Luigi Toscano proposed openstack/nova stable/queens: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full https://review.opendev.org/742668 | |
| 14:42:53 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: testing tempest copy-image test https://review.opendev.org/742547 | |
| 15:38:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Add helpers for suspend, resume and reboot of server https://review.opendev.org/741285 | |
| 15:42:08 | bauzas | stephenfin: fwiw, reviewing hard your https://review.opendev.org/#/c/739210/5 to see whether making vTPM types be an Enum is safe | |
| 15:42:25 | stephenfin | bauzas: Just replying. tl;dr: Look at https://github.com/openstack/nova/blob/da155cb495979726943806630eff1bed146b8605/nova/objects/instance_numa.py#L47-L52 | |
| 15:43:51 | stephenfin | If we changed the field, we would be forced to bump the version number of any object that contained that object. We would also most likely add something like a service check that prevented use of the new TPM model/version until all computes were updated and could handle it | |
| 15:44:38 | bauzas | yup, I'm just saying it's a decision I need to evaluate | |
| 15:44:43 | bauzas | not saying it's bad | |
| 15:44:49 | bauzas | like, | |
| 15:45:04 | bauzas | you decided to enforce those types at the API level, which prevents a bit of issues | |
| 15:45:09 | stephenfin | ...which is similar to what huaquiang did for the introduction of the MIXED CPU policy. See commit 9ddc60539f53e24bb92ef711674f5be4c7c0bcc3 | |
| 15:46:17 | bauzas | stephenfin: all the vTPM workflow is top-down (or northbound if you prefer), right? | |
| 15:46:43 | bauzas | ie. we just leave operators to define which model/version they want to expose | |
| 15:46:55 | bauzas | and then we pass this info down to libvirt eventually, right? | |
| 15:47:35 | bauzas | and we just make traits exposing those vTPM capabilities, right? | |
| 15:48:07 | stephenfin | Within reason. We check to make sure that (a) libvirt is new enough to support the feature, (b) the swtpm executable is available and (c) a key manager service is configured | |
| 15:48:18 | bauzas | so, per say, someone messes up and defines a flavor with a model that none of the computes know, then eventually it could have got a NoValidHost but we enforce the knowledge by an API verification | |
| 15:48:24 | stephenfin | Assuming all those are configured, we report the trait on the libvirt driver | |
| 15:48:33 | bauzas | amirite ? | |
| 15:48:40 | stephenfin | The API would reject that request outright | |
| 15:49:09 | bauzas | so, the main concern would be a new compute with a new model, we would need to wait a release ideally to bump the API | |
| 15:49:13 | bauzas | schema* | |
| 15:49:14 | stephenfin | Either when attempting to set the extra spec on the flavor if using a recent microversion with the flavor validation work | |
| 15:49:15 | bauzas | right? | |
| 15:49:31 | stephenfin | or when creating the server, since we verify all these stuff there now | |
| 15:50:14 | gibi | nova meeting starts in 10 minutes on #openstack-meeting-3 | |
| 15:50:21 | stephenfin | No, we don't currently consider new/modified flavor extra specs or image metadata properties as warranting a new API microversion | |
| 15:50:24 | stephenfin | If that's what you mean | |
| 15:50:35 | stephenfin | We would need to introduce a new service version | |
| 15:50:46 | stephenfin | and wait until all compute nodes have been updated to that service version | |