| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-24 | |||
| 10:39:32 | zigo | Does anyone have an idea of what's going on? | |
| 10:40:40 | lyarwood | zigo: what's buster-backports? | |
| 10:40:51 | lyarwood | Debian? | |
| 10:40:58 | zigo | lyarwood: OpenStack Stable backported to Debian stable. | |
| 10:40:59 | zigo | Yeah. | |
| 10:41:09 | lyarwood | zigo: which branch of OpenStack? | |
| 10:41:15 | zigo | lyarwood: Ussuri RC1. | |
| 10:41:38 | lyarwood | zigo: do you have a source tree somewhere or are you just using the tarball directly? | |
| 10:41:40 | zigo | It did build fine in Debian Experimental though (and I already uploaded there...) | |
| 10:41:59 | zigo | lyarwood: I'm using the git tag, which is kind of close to the tarball. | |
| 10:42:23 | lyarwood | yup, smells like something is off with the version of mock it's pulling in | |
| 10:42:23 | zigo | My tooling does a "git archive" to generate the tarball, I've been doing this since the begining of OpenStack. | |
| 10:42:35 | zigo | It's pock 3.0.5. | |
| 10:42:37 | zigo | mock | |
| 10:43:46 | zigo | So I'm guessing it's a problem with another dependency ... | |
| 10:45:01 | lyarwood | yeah looks like we are using 3.0.5 at the moment | |
| 10:45:06 | lyarwood | upstream that is | |
| 10:55:49 | zigo | Would it be possible that it's because of a newer oslotest package? | |
| 10:56:02 | zigo | In Experimental, I built with oslotest 3.8.0, not 4.1.0 | |
| 11:02:56 | zigo | Oh, I'm lagging behind, it should be olsotest 4.2.0 maybe? | |
| 11:04:14 | zigo | Yeah, the issue is with oslotest 4.1.0 ! \o/ | |
| 11:04:23 | zigo | I'll try upgrading to 4.2.0. | |
| 11:11:12 | artom | bauzas, gibi, yeah, I managed to *actually* triage maybe 3 or 4 - the other dozen or so that I looked at I just made the o_O face | |
| 11:17:34 | lyarwood | Does anyone use vscode for upstream work? If so are there any guides on getting it configured correctly for ^openstack\/.* work? | |
| 11:17:55 | lyarwood | Just installed the flatpak and thought I'd try and give it a go this afternoon | |
| 11:18:43 | artom | lyarwood, actually, I think jparker uses it | |
| 11:19:12 | artom | He's done some stuff for whitebox | |
| 11:19:30 | artom | lyarwood, but also, shuuuun | |
| 11:19:35 | artom | Shuuuuuun the heretic | |
| 11:19:58 | lyarwood | always good to get out and try new things once in a while or so I'm told ;) | |
| 11:20:18 | lyarwood | had a moment this morning with my .vimrc and thought I'd give it a go | |
| 11:20:30 | lyarwood | I'm sure it isn't going to last long | |
| 11:21:37 | nightmare_unreal | lyarwood: i use vscode I haven't done much config. just installed few vscode plugins ,like pylint | |
| 11:22:16 | zigo | You guys should blacklist oslotest 4.1.0 ! :) | |
| 11:22:39 | zigo | Should I propose such a patch ? | |
| 11:23:16 | lyarwood | zigo: please, if it's broken | |
| 11:23:27 | lyarwood | zigo: against openstack/requirements | |
| 11:23:55 | lyarwood | actually maybe not | |
| 11:24:51 | zigo | lyarwood: What do you suggest then? | |
| 11:25:55 | lyarwood | suggest it in openstack/nova and we can work out if that's correct | |
| 11:26:12 | lyarwood | I don't see it listed in the global repo so I'm not sure if we should add it there | |
| 11:26:55 | gibi | artom: thanks a lot for triaging! | |
| 11:35:49 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Make Q35 machine type the default for x86 https://review.opendev.org/631154 | |
| 13:18:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444 | |
| 13:18:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509 | |
| 13:30:48 | openstackgerrit | François Palin proposed openstack/nova stable/stein: Add retry to cinder API calls related to volume detach https://review.opendev.org/722783 | |
| 13:36:14 | gibi | stephenfin: if you have time, could you look at this tox related patch https://review.opendev.org/#/c/722520/ ? | |
| 13:38:22 | stephenfin | gibi: Yeah, that's fine | |
| 13:38:47 | gibi | stephenfin: thanks for checking | |
| 14:02:32 | openstackgerrit | Thomas Goirand proposed openstack/nova master: Blacklist oslotest 4.1.0 https://review.opendev.org/722792 | |
| 14:02:50 | zigo | lyarwood: There you go! ^ | |
| 14:03:16 | zigo | (took ages to clone nova with my poor DSL line...) | |
| 14:04:39 | stephenfin | zigo: https://github.com/openstack/oslotest/commit/cdee551ae6145e4646809bf65e71817a63328d9a | |
| 14:04:52 | stephenfin | I guess that's what's hitting us, since it's the only difference between 4.1.0 and 4.2.0 | |
| 14:05:24 | stephenfin | but we have mock in test-requirements.txt so that shouldn't be it either... | |
| 14:05:53 | zigo | And in my build env, I did get mock 3.0.5 through the package Build-Depends:. | |
| 14:06:50 | stephenfin | zigo: can you paste the failures somewhere? | |
| 14:07:14 | zigo | stephenfin: paste.openstack.org/show/792658/ | |
| 14:07:20 | zigo | 26 failures of the same kind. | |
| 14:08:38 | openstackgerrit | Merged openstack/nova-specs master: Updated Nova-Cyborg interaction spec. https://review.opendev.org/684151 | |
| 14:08:52 | stephenfin | ah, weird, looks like it's falling back to stdlib mock. I've seen that issue when trying to migrate nova to unittest.mock | |
| 14:09:38 | openstackgerrit | Takashi Natsume proposed openstack/python-novaclient master: Switch to using TOX_CONSTRAINTS_FILE https://review.opendev.org/722795 | |
| 14:34:57 | openstackgerrit | Dan Smith proposed openstack/nova-specs master: Update SPEC: Using PUT instead of PATCH API to update delete_on_termination https://review.opendev.org/722194 | |
| 14:45:37 | gmann | dansmith: how about mention the other accepted field also on PUT which were 400 previously ?- https://review.opendev.org/#/c/722194/4/specs/ussuri/approved/destroy-instance-with-datavolume.rst@113 | |
| 14:58:13 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Switch to TOX_CONSTRAINTS_FILE https://review.opendev.org/722814 | |
| 15:07:40 | openstackgerrit | Merged openstack/nova stable/stein: Add config option for neutron client retries https://review.opendev.org/722077 | |
| 15:09:17 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Add config option for neutron client retries https://review.opendev.org/722819 | |
| 15:21:34 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: Add config option for neutron client retries https://review.opendev.org/722822 | |
| 15:43:05 | dansmith | gmann: we could although at some point the spec just becomes duplicative of the documentation and/or the code/schema | |
| 15:46:01 | openstackgerrit | Merged openstack/nova master: Update master for stable/ussuri https://review.opendev.org/722521 | |
| 16:02:20 | gibi | bauzas: o/ have a nice weekend | |
| 16:02:23 | bauzas | mgoddard: I started to review https://review.opendev.org/#/c/695012/5, haven't done yet | |
| 16:02:34 | bauzas | but only have nits for the moment | |
| 16:03:43 | bauzas | gibi: thanks, I have a virtual drink with friends in a couple of mins ;) | |
| 16:03:58 | gibi | \o/ | |
| 16:13:34 | gmann | dansmith: good point. anyways api-ref has those changes documented. good to go from me. | |
| 16:14:09 | dansmith | cool | |
| 16:18:27 | spatel | sean-k-mooney: how are you, hope you doing well | |
| 16:18:45 | spatel | I have question related cpu mode="host-passthrough" vs cpu mode="host-model" | |
| 16:19:24 | spatel | As you know host-passthrough good for performance but do we know how much performance its going to improve | |
| 16:19:49 | spatel | as i know host-passthrough is just exposing more flags to VM but does that improve performance? | |
| 16:26:05 | mgoddard | bauzas: ok, good start | |
| 16:36:53 | openstackgerrit | Merged openstack/nova-specs master: Update SPEC: Using PUT instead of PATCH API to update delete_on_termination https://review.opendev.org/722194 | |
| 16:47:47 | kashyap | artom: Saw your comments; thank you. I'll respond on Monday. But a quick note here after reading your ocmments: | |
| 16:48:40 | kashyap | artom: No, we shouldn't mark 'pc' as deprecated -- the story is "complex" here -- but it remains a valid use for many workloads, 'legacy' _and_ non-legacy | |
| 16:49:35 | kashyap | artom: So, not changing the default in Nova itself is a valid option, for the reason libvirt did it. When you get a minute, take time read this commit message from DanPB: | |
| 16:49:38 | kashyap | https://libvirt.org/git/?p=libvirt.git;a=commit;h=26cfb1a3cd | |
| 16:49:43 | kashyap | I need to head out, but thanks, again! | |
| 16:50:05 | kashyap | (I'll expand more on the spec.) | |
| 16:54:26 | openstackgerrit | Merged openstack/nova stable/stein: Make RBD imagebackend flatten method idempotent https://review.opendev.org/708615 | |
| 17:03:36 | artom | kashyap, akc, thanks! | |
| 17:06:51 | openstackgerrit | Merged openstack/nova stable/stein: Reject boot request for unsupported images https://review.opendev.org/708578 | |
| 17:12:22 | kashyap | artom: Actually, I responded now. Couldn't resist. Some of your comments just "invited me out to respond" :D | |
| 17:12:46 | artom | kashyap, yeah, I have that troll'ish effect :) | |
| 18:17:03 | openstackgerrit | Merged openstack/python-novaclient master: Switch to using TOX_CONSTRAINTS_FILE https://review.opendev.org/722795 | |
| 18:32:37 | masterpe | Hi the cache mode of a vm of e existing vm is it stored in a database? | |
| 18:55:59 | openstackgerrit | Merged openstack/python-novaclient master: Bump default tox env from py37 to py38 https://review.opendev.org/722885 | |
| 20:43:25 | masterpe | I'm trying to change the cache mode of running instances from writeback to writethrough. I have changed the nova.conf to use writethrough. This works find for new instances. But for existing instances after live migration the mode on the new compute node it is still writeback? How can I fix this? | |
| 20:52:42 | melwitt | masterpe: I think you will need to stop/start or hard reboot or migrate the instances to make them generate new xml and take the new cache mode | |
| 20:53:48 | melwitt | oh but you're saying after you changed the conf and live migrated, it kept the old cache mode? maybe cold migration only will regenerate xml. sorry I don't know for sure | |