| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-03 | |||
| 09:38:38 | lyarwood | tbh you could even sell that as a bugfix in W | |
| 09:38:42 | lyarwood | for me at least | |
| 09:38:48 | lyarwood | if it's replacing broken behaviour | |
| 09:39:10 | lyarwood | stephenfin: ^ would you agree? | |
| 09:39:32 | lyarwood | stephenfin: context is https://review.opendev.org/c/openstack/nova/+/762330 | |
| 09:39:49 | stephenfin | This is about libvirt using the wrong APIs? | |
| 09:39:50 | lyarwood | stephenfin: switching over to a different libvirt API for checking CPUs | |
| 09:39:53 | lyarwood | yeah | |
| 09:40:06 | stephenfin | Yeah, I thought that was a bug tbh. Is someone saying otherwise? | |
| 09:40:15 | lyarwood | it's associated with a blueprint | |
| 09:40:17 | kashyap | lyarwood: Yeah, it could be argued as a bug-fix in good-faith | |
| 09:41:06 | lyarwood | cool, lets drop that reference and proceed as a bugfix | |
| 09:41:13 | kashyap | lyarwood: It is associated with a spec even: https://opendev.org/openstack/nova-specs/commit/70811da221035044e27 | |
| 09:41:35 | kashyap | See the work items for the recommended split of patches | |
| 09:42:04 | openstackgerrit | Wenping Song proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | |
| 09:42:05 | openstackgerrit | Wenping Song proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | |
| 09:42:12 | lyarwood | MIN_{LIBVIRT,QEMU}_VERSION is beyond the required versions now so I'd say this is even more of a bugfix | |
| 09:42:33 | lyarwood | with some pre-written documentation we can use from the spec ;) | |
| 09:42:35 | kashyap | lyarwood: Yeap, yeap | |
| 09:44:13 | gibi | lyarwood, stephenfin: sent a mail about placement-stable-maint to ML http://lists.openstack.org/pipermail/openstack-discuss/2021-February/020186.html | |
| 09:44:23 | lyarwood | gibi: many thanks | |
| 09:44:39 | gibi | I will couple of days and then I will make the change | |
| 09:45:55 | lyarwood | ack | |
| 10:02:06 | elod | lyarwood: kashyap: +2+W'd >>> https://review.opendev.org/c/openstack/nova/+/758763 | |
| 10:02:16 | kashyap | elod: Thanks! | |
| 10:02:28 | elod | kashyap: np :) | |
| 11:36:08 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380 | |
| 13:45:52 | sean-k-mooney | V comes after U ... | |
| 13:46:11 | sean-k-mooney | meaning i need to backport to Victoria then Ussuri | |
| 13:51:24 | openstackgerrit | sean mooney proposed openstack/nova stable/victoria: only wait for plugtime events in pre-live-migration https://review.opendev.org/c/openstack/nova/+/770744 | |
| 13:56:28 | sean-k-mooney | lyarwood: can you take a look at ^ again | |
| 13:56:37 | sean-k-mooney | now that its in the correct order | |
| 14:01:14 | bauzas | sean-k-mooney: fwiw, I'm splitting my -W change for routed networks | |
| 14:01:27 | bauzas | the functests are saying OK so we know it works | |
| 14:01:42 | bauzas | (for all the move operations) | |
| 14:02:33 | sean-k-mooney | ok | |
| 14:03:06 | sean-k-mooney | i assume your going to either move teh func tests or the unit tests to a second patch? | |
| 14:03:47 | sean-k-mooney | for this type of change i trust the func tests more so would prefer those to stay with the code but ill leave that up to you | |
| 14:03:56 | sean-k-mooney | we will be reviewing the serise as a whole anyway | |
| 14:07:08 | bauzas | no, I'll have the functests in the same change for the filter | |
| 14:08:11 | bauzas | but I'll move the renaming of ensure_network_metadata() in a dependent change | |
| 14:10:02 | sean-k-mooney | cool | |
| 14:48:18 | gibi | fyi cores, I'm +2 on the vnc password patch https://review.opendev.org/c/openstack/nova/+/622336/ but it needs a second set of eyes | |
| 14:56:56 | sean-k-mooney | hum that touches a bunch of files but most are small changes it seams | |
| 14:57:13 | sean-k-mooney | im not a core but ill take a look at it | |
| 14:57:31 | gibi | sean-k-mooney: cool, thanks | |
| 14:57:34 | sean-k-mooney | it dose not look that difficult to reiew | |
| 15:31:54 | sean-k-mooney | gibi: -1 because i have a question on allowing console without a password to be created when the nova config has [vnc]/auth_schemes = none,vnc,vencrypt | |
| 15:32:03 | gibi | sean-k-mooney: thanks | |
| 15:32:07 | sean-k-mooney | gibi: but other then that the code looks ok | |
| 15:33:05 | sean-k-mooney | my comments are on the patch but while i woudl prefer not to save the pasword in plain text in novas db it is out of scope to use barbican so i view that as a future imporvment not a requirement for the feature | |
| 15:41:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/773930 | |
| 15:41:07 | openstack | bug 1907522 in OpenStack Compute (nova) victoria "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host |
|
| 15:42:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Fallback to same-cell resize with qos ports https://review.opendev.org/c/openstack/nova/+/773932 | |
| 16:08:17 | melwitt | kashyap: took me awhile (sorry) but here's the qemu bug I opened about the block copy taking a long time to end the job after the last block is done https://bugs.launchpad.net/qemu/+bug/1914282 | |
| 16:08:18 | openstack | Launchpad bug 1914282 in QEMU "block copy job sometimes hangs on the last block for minutes" [Undecided,New] | |
| 16:08:34 | kashyap | melwitt: Hello | |
| 16:08:41 | melwitt | hello | |
| 16:09:19 | kashyap | melwitt: Thanks for filing; no need for sorry :-) | |
| 16:09:27 | kashyap | melwitt: Nice write-up | |
| 16:09:37 | kashyap | melwitt: One small comment; do you want to attach the pastebin content as a text file to the bug? | |
| 16:09:43 | kashyap | As pastebins expire | |
| 16:10:04 | melwitt | kashyap: oh, yeah, I will do that | |
| 16:10:10 | kashyap | melwitt: Also, the bug will show up on the main 'qemu-devel' list; so the Block Layer devs should notice it | |
| 16:10:51 | melwitt | I thought pastebin wasn't expiration by default but it's a good idea to attach it even so | |
| 16:10:55 | kashyap | If we don't hear a response in a week, I'll get Eric Blake to have a gander; he knows this inside out, as someone who's worked on QEMU and libvirt | |
| 16:11:39 | melwitt | ok great | |
| 16:13:40 | kashyap | melwitt: Is this affecting the Gate quite often? | |
| 16:14:10 | melwitt | kashyap: I think no but I don't know for sure bc I haven't figured out a unique fingerprint to logstash for it | |
| 16:25:17 | melwitt | lyarwood: I opened this bug yesterday based on a gate failure I saw and was curious if you've ever seen this before? https://bugs.launchpad.net/nova/+bug/1914296 | |
| 16:25:19 | openstack | Launchpad bug 1914296 in OpenStack Compute (nova) "tests involving encrypted volumes sometimes (rarely) fail with libvirtError: [...] Volume is not in LUKS format" [Undecided,New] | |
| 16:26:00 | kashyap | melwitt: Hmm, so I can't see a unique signature either ... as most of the messages are "usual", but they're on loop | |
| 16:26:28 | kashyap | Unless I'm missing something badly, looks like it's tricky to check for its frequency | |
| 16:26:33 | melwitt | kashyap: yeah exactly. the only place where I think we have a chance is in the job-output.txt but IIRC that text is also quite generic | |
| 16:26:57 | melwitt | I will take another look to see if anything jumps out | |
| 16:27:36 | kashyap | melwitt: Thank you. I hope this won't come back to bite us - as we can't (yet) see how often we're hitting it | |
| 16:28:46 | lyarwood | melwitt: can't say I have, *clicks* | |
| 16:29:32 | lyarwood | melwitt: if it's a block device n-cpu should format it with LUKS before using it | |
| 16:30:03 | melwitt | lyarwood: it seems weird bc the error message says both format=luks and then "Volume not in LUKS format", like how is that possible? | |
| 16:31:12 | lyarwood | melwitt: urgh that chrome tab just died, I'll pull down the log after the call if that's okay | |
| 16:31:32 | melwitt | sure np, just wanted to link you before I forgot | |
| 16:31:42 | melwitt | it's not urgent | |
| 17:13:58 | kashyap | melwitt: I have attached the QEMU <-> libvirt exchange from the dest-host in the bug. | |
| 17:14:22 | melwitt | kashyap: thanks! | |
| 17:15:12 | kashyap | NP. Thx, you too | |
| 17:44:48 | sean-k-mooney | lyarwood: can you reveiw https://review.opendev.org/c/openstack/nova/+/622336/29/nova/api/openstack/compute/remote_consoles.py#183 and confirm if you expect that to have teh same effect as the issue we discussed downstream | |
| 17:45:21 | sean-k-mooney | we do not want the password to end up in the image data or in config_dirver/metadata api | |
| 17:45:49 | sean-k-mooney | but here it may end up in both correct | |
| 17:49:38 | melwitt | sean-k-mooney: hm, I'd have thought that should be stored encrypted, same as we do for admin password | |
| 17:49:57 | sean-k-mooney | melwitt: ya i mentioned that | |
| 17:50:08 | sean-k-mooney | although i dont think it can be | |
| 17:50:16 | sean-k-mooney | the admin password is not in the libvirt xml | |
| 17:50:24 | sean-k-mooney | we use the ssh public key to encypt it yes | |
| 17:50:31 | melwitt | yes | |
| 17:50:43 | sean-k-mooney | in this case we wont have the private key to decyprti it ot stick it in the libvirt xml | |
| 17:50:55 | melwitt | oh :\ | |
| 17:50:57 | sean-k-mooney | so we can use the ssh public key here | |
| 17:51:07 | sean-k-mooney | but we coudl use soemthign else | |
| 17:51:11 | sean-k-mooney | e.g. a keystore | |
| 17:51:31 | sean-k-mooney | or static encyption with a fixed key seeded with the ssk public key or somehtihng | |