Posted	Nick	Remark
#openstack-nova - 2018-01-16
19:16:26	sean-k-mooney	til well i shoudl be able to help in anycase. there is a lot you can do with unitest to fake things out + there is the IBM PowerVM CI so if you remind me/ i get time i might take a go at it but not before the PTG/end of Q1
19:19:07	melwitt	cfriesen: so it was fixed by having https://review.openstack.org/#/c/227278 and removing VIR_MIGRATE_TUNNELLED from the block_migration_flag in nova.conf
19:19:29	melwitt	and I think config drive has to be vfat also
19:20:49	cfriesen	melwitt: okay, was wondering about that last part. we're still (in Pike) hitting issues with iso9660 config drives and boot-from-volume on a compute node with local storage.
19:21:45	cfriesen	melwitt: specifically with block-live-migrating such instances
19:22:03	cfriesen	I'm trying to see if it's something we broke or an upstream issue.
19:22:07	melwitt	cfriesen: yeah, I can't remember the context now but if you dig into the libvirt driver code there are a lot of conditions that have to be met before live block migration will work, and if you have a config drive, that's local storage so has to be a live BLOCK migration
19:22:43	melwitt	you can't have the tunneling but I don't remember why
19:23:48	cfriesen	melwitt: that part I know...tunnelling prevents the "block migrate some disks but not others" behaviour in libvirt
19:23:56	melwitt	oh, okay. cool
19:24:12	melwitt	now I know then :)
19:29:10	melwitt	cfriesen: I know that in the past iso9660 config drives couldn't be live migrated, I don't know when/if that was fixed. based on the absence of 'vfat' in the pike code I'm guessing it was fixed
19:30:12	melwitt	I just updated the BZ to make a lot of the comments public that didn't need to be private
19:30:32	melwitt	in case that might help ppl that happen upon it
19:36:37	melwitt	looks like iso9660 migration was enabled by https://review.openstack.org/#/c/234659
20:09:20	openstackgerrit	Merged openstack/nova master: Rename block_device_info_get_root https://review.openstack.org/529028
20:21:50	sean-k-mooney	jaypipes: looks like the gate is happy with it. https://review.openstack.org/#/c/534307/2 once that is merged im happy to go with a 1.8.1 release.
20:24:07	jaypipes	sean-k-mooney: question... why don't we just straight revert the pyroute patch from rodolfo>?
20:46:03	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
20:46:36	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
20:47:07	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
20:47:51	sean-k-mooney2	jaypipes rodolfo patch also introduces the infra for functional testing and deduplicates the logic from linux bridge plugin and ovs
20:48:52	sean-k-mooney2	So i would like to keep that and just delay moving to pyroute2 until a new release is out
20:49:52	jaypipes	sean-k-mooney2: ok...
21:07:19	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384
21:42:50	matrohon	sean-k-mooney: hi
21:45:33	cfriesen	melwitt: mdbooth: it turns out that if you have an instance with an iso9660 config drive and a ceph-backed attached cinder volume, then block-live-migration fails in libvirt with "qemuMigrationPrecreateStorage:463 : Operation not supported: pre-creation of storage targets for incremental storage migration is not supported". With iscsi-backed cinder volumes it works fine.
21:46:51	cfriesen	At least in our code....still trying to track down whether we broke something or if it's upstream.
22:53:55	openstackgerrit	Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222
22:59:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629
22:59:58	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
#openstack-nova - 2018-01-17
00:20:19	openstackgerrit	Merged openstack/nova master: archive_deleted_instances is not atomic for insert/delete https://review.openstack.org/526065
00:20:32	openstackgerrit	Merged openstack/nova master: Enable py36 unit tests in tox https://review.openstack.org/534223
01:03:14	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
01:03:15	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216
01:03:15	openstackgerrit	Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094
01:30:26	openstackgerrit	Merged openstack/nova master: Fix missing rps in allocation candidates https://review.openstack.org/522409
03:19:40	openstackgerrit	nafeiyang proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240
03:57:46	openstackgerrit	Merged openstack/nova master: func tests for nested providers in alloc candidate https://review.openstack.org/531443
05:01:14	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/533046
05:15:21	openstackgerrit	Merged openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953
05:15:57	openstackgerrit	Merged openstack/nova master: Fix accumulated nits https://review.openstack.org/534368
06:03:20	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector from ports/portgroups https://review.openstack.org/517322
06:38:22	ameeda	Morning :)
06:42:57	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
06:59:42	openstackgerrit	Merged openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263
08:44:22	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682
08:49:24	openstackgerrit	Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728
08:53:41	openstackgerrit	Eli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping https://review.openstack.org/522084
09:12:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Cleanup tox.ini https://review.openstack.org/534382
09:12:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383
09:29:40	ameeda	gmann: please don't forget me about this gerrit "https://review.openstack.org/#/c/526900/"
09:42:10	kashyap	stephenfin: Heya, when you get a moment, mind taking a gander at this (one of the use cases is right in your wheel house)? https://review.openstack.org/#/c/534384/ -- "[WIP] libvirt: Allow to specify granular CPU feature flags"
09:42:29	kashyap	stephenfin: It's still WIP, I need to fix the 11 or so broken tests
09:42:58	kashyap	But thought I'd put it out there, as we should it'll alleviate the performance impact that the recent "Meltdown" CVE fixes cause.
09:43:11	kashyap	The commit message tells it all (I mean, all) :-)
09:47:06	stephenfin	kashyap: Sounds good
09:48:18	bauzas	stephenfin: just in case, you could also be interested in https://review.openstack.org/#/c/528832/
09:48:20	kashyap	stephenfin: Just to note --- next two days I'm a bit sporadic in my availability, Friday night, I'lll be back home
09:48:31	bauzas	stephenfin: ;)
09:56:42	BKaminski	There any etiquette on adding someone as a reviewer to a commit if you know they know that code well?
09:58:03	kashyap	BKaminski: Sure, it's fine.
09:58:28	BKaminski	Alrighty, thanks :)
09:59:38	kashyap	BKaminski: E.g. sometimes I run this command: git shortlog -nse nova/$File-of-Interest
09:59:46	kashyap	To get a sense of who might know a certain area
10:00:03	BKaminski	Ah that does maek sense
10:00:03	kashyap	Then check if they're still active, and add them to Cc.
10:00:28	kashyap	(I do it for mailing lists based projects mostly, as it requires explicit Cc)
10:00:47	kashyap	But obviously, the trick applies for any Git-based project
10:00:58	BKaminski	That makes sense.. i'll give it a go just to get it on a couple of peoples radars that I know worked on this code, since XenAPI code is kind of hard to catch a reviewer on I think :D
10:01:42	kashyap	Yup
10:04:00	BKaminski	Last commit into .Xenapi was Aug 2017 .. whew...
10:05:43	BKaminski	OH nevermind.. December.. i apparently had a temporary brain to vision disease there hehe
10:14:00	openstackgerrit	Chen Hanxiao proposed openstack/nova master: _cleanup_resize: remove rbd swap disk if resize to flavor without swap https://review.openstack.org/534711
10:15:37	gmann	ameeda: sure, i was away today. ll check soon
10:16:20	ameeda	gmann: thank you !
10:16:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: Fix openstackdocstheme options for api-ref https://review.openstack.org/534712
10:16:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Remove crud from 'conf.py' https://review.openstack.org/534713
10:28:16	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958
10:37:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544
10:38:44	openstackgerrit	sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008
10:53:25	openstackgerrit	sahid proposed openstack/nova master: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489
11:08:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544
11:08:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Remove duplicate 'vnc' config opt descriptions https://review.openstack.org/534724
11:57:11	openstackgerrit	sahid proposed openstack/nova master: libvirt: rework _get_vpcu_total to return online pCPUs https://review.openstack.org/509730
11:57:11	openstackgerrit	sahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus https://review.openstack.org/510897
12:08:32	Rambo	Good mornig,everyone, Can you help me to review the spec ?The link is https://review.openstack.org/#/c/532410/
12:08:47	Rambo	Another is :https://review.openstack.org/#/c/532407/
12:08:47	Rambo	Thank you very much
12:27:09	sean-k-mooney	stephenfin: hi o/ if jaypipes want any changes ill add the link to the pull request in the iptools patch. its currently in the commit message. is that ok. alternitivly i can do a quick respin now but we will have to wait a while for the gate to come back.
12:28:29	jaypipes	sean-k-mooney: I'm cool with what you've done
12:28:46	jaypipes	sean-k-mooney: mostly so I can just blame you if anything messes up :P
12:29:42	sean-k-mooney	jaypipes: :) well if anything does come up ill fix it but it think its good based on my testing and the ci
12:32:52	jaypipes	sean-k-mooney: now if only you could get rid of this chest congestion and cough for me...
12:33:29	sean-k-mooney	jaypipes: may i suggest irish coffee + a good spicy curry
12:33:44	sean-k-mooney	also drugs lots of drugs and rest
12:33:53	jaypipes	sean-k-mooney: lol. my boss, who's british, suggested whiskey as well.
12:34:35	jaypipes	sean-k-mooney: also, I have the curry handled ;) https://twitter.com/jaypipes/status/951974933835399169
12:35:52	sean-k-mooney	jaypipes: haha ya i saw that tweet. i dont know what people did for coldes before spicy food was commonplace in europe/america but its pretty effectiv
12:37:50	sean-k-mooney	on a slightly different tangent. for https://review.openstack.org/#/c/465160/ we were waiting for the intel nfv ci to run. that shoudl be up before the ptg however the ci-worker vm i was doing the os-vif work in was created to allow me to replicate how the nfv ci would work.

Earlier   Later