| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-24 | |||
| 09:56:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add MigrationTypeField https://review.opendev.org/706013 | |
| 09:58:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_MULTIATTACH https://review.opendev.org/710238 | |
| 10:22:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Rework how we check for neutron extensions https://review.opendev.org/705792 | |
| 10:39:15 | zigo | When building Nova for buster-backports, I get 26 failures of this kind: | |
| 10:39:15 | zigo | http://paste.openstack.org/show/792658/ | |
| 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 | zigo | My tooling does a "git archive" to generate the tarball, I've been doing this since the begining of OpenStack. | |
| 10:42:23 | lyarwood | yup, smells like something is off with the version of mock it's pulling in | |
| 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 | |