Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-24
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
21:03:58 artom melwitt, yeah, I don't see anything about cache in libvirt/migration.py, I'm pretty sure we don't update that
21:05:09 melwitt artom: ah, thanks for checking that. I didn't know if it was magic inside libvirt that moves the guest (xml) over or if it was something in nova. good to know
21:07:28 artom melwitt, it's all Nova
21:08:21 melwitt thanks
21:34:44 openstackgerrit melanie witt proposed openstack/nova master: zuul: Switch to the Zuulv3 grenade job https://review.opendev.org/704364
22:23:05 masterpe so libvirt generated the xml file and nova commands libvirt to migrate an instance to migrate an instance from hostA to hostB?
22:51:22 openstackgerrit melanie witt proposed openstack/nova stable/rocky: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723050
22:58:13 melwitt masterpe: yes and nova passes new xml to use on the destination https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L8846
23:04:46 melwitt so to generate new xml you have to do server stop + start, or hard reboot, or cold migration, or rebuild
23:05:26 melwitt or resize, shelve + unshelve
23:22:27 openstackgerrit melanie witt proposed openstack/nova stable/queens: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723054
23:27:17 openstackgerrit melanie witt proposed openstack/nova stable/pike: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/723055
#openstack-nova - 2020-04-25
08:14:29 openstackgerrit Merged openstack/python-novaclient master: Add py38 package metadata https://review.opendev.org/722753
08:33:21 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/722644
10:56:41 openstackgerrit Takashi Natsume proposed openstack/python-novaclient master: doc: Update Testing document https://review.opendev.org/723078
15:18:06 openstackgerrit Merged openstack/nova master: Add Python3 victoria unit tests https://review.opendev.org/722523
23:10:38 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: [Community goal] Update contributor documentation https://review.opendev.org/722393
#openstack-nova - 2020-04-26
01:16:12 openstackgerrit xuyuanhao proposed openstack/nova master: loads pyc files failed,sys.modules support loads py and pyc files as modules https://review.opendev.org/723130
06:35:12 openstackgerrit Qiu Fossen proposed openstack/python-novaclient master: Remove Babel requirement https://review.opendev.org/723139
06:40:56 openstackgerrit Qiu Fossen proposed openstack/python-novaclient master: Remove Babel requirement https://review.opendev.org/723139
07:18:48 openstackgerrit Takashi Natsume proposed openstack/python-novaclient master: Use unittest.mock instead of third party mock https://review.opendev.org/723152
07:19:42 openstackgerrit Takashi Natsume proposed openstack/python-novaclient master: Remove future imports https://review.opendev.org/723153
07:44:04 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/ussuri: Imported Translations from Zanata https://review.opendev.org/723160
07:51:30 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/722644
10:12:26 openstackgerrit Takashi Natsume proposed openstack/python-novaclient master: Use unittest.mock instead of third party mock https://review.opendev.org/723152
10:41:38 openstackgerrit Qiu Fossen proposed openstack/python-novaclient master: Remove Babel requirement https://review.opendev.org/723139
11:16:43 openstackgerrit Andreas Jaeger proposed openstack/nova master: Remove translation sections from setup.cfg https://review.opendev.org/723206
11:43:32 openstackgerrit Andreas Jaeger proposed openstack/nova master: Remove translation sections from setup.cfg https://review.opendev.org/723206

Earlier   Later