Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-11
14:54:23 jgwentworth lyarwood: a-ha, thanks. so with that, the bug can be marked Fix Released? someone was asking about it last week I think
14:55:39 lyarwood jgwentworth: yeah I think so
14:56:03 jgwentworth lyarwood: cool, thanks. I'll update it as such
14:59:02 sean-k-mooney finucannot: ill have a basic noop plugin up for review in the next hour or so. just need to add docs and unit tests working but have a scrum meeting in 2 minutes
14:59:21 sean-k-mooney finucannot: ill comment back on your patch when its ready
15:00:08 finucannot sean-k-mooney: Sounds good to me :)
15:06:48 vdrok mriedem: hrm, it seems like images_type is being overridden somewhere http://logs.openstack.org/60/567860/1/check/legacy-grenade-dsvm-neutron-multinode-live-migration/73d8b8a/logs/etc/nova/nova.conf.txt.gz
15:07:47 mriedem vdrok: the live migration job runs first w/o ceph and then with ceph
15:08:08 mriedem https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/ceph.sh#L79
15:08:25 mriedem but the first run before ceph should fail
15:08:41 mriedem https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/run_tests.sh#L27
15:14:38 openstackgerrit Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
15:16:31 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877
15:16:34 mriedem naichuans: hi, i'll probably need some help with fixing the unit tests in ^ because of the way the fake / stubs are setup to trace xenserver session records
15:16:46 mriedem that entire fake/stub structure is a bit weird if you're not used to it
15:21:43 openstackgerrit Surya Seetharaman proposed openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/567878
15:22:11 mriedem vdrok: although something else is up, because in the run before rbd, it uses 'default' which defaults to qcow2, not raw http://logs.openstack.org/60/567860/1/check/nova-live-migration/90babb9/logs/screen-n-cpu.txt.gz#_May_11_14_04_55_065852
15:23:12 mriedem i'll just override the config option defaults in code rather than mess with setting the values in the CI run
15:24:45 openstackgerrit Surya Seetharaman proposed openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/567878
15:26:43 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860
15:29:27 cfriesen_ sean-k-mooney: sriov question for you. according to the docs at https://docs.openstack.org/neutron/pike/admin/config-sriov.html, if we have both PFs and VFs on the host and pass the PF through to the guest, the VF number stored in "sriov_numvfs" is lost. how do we restore that when we detach the PF from the guest and return it to the host?
15:30:35 sean-k-mooney cfriesen_: udev rules is the only way i have found really
15:31:11 sean-k-mooney you could perhaps use a systemd service and a filepath trigger
15:31:37 sean-k-mooney cfriesen_: somthing needs to echo it back into sysfs
15:32:33 cfriesen_ sean-k-mooney: yeah, that's what we found. the docs talk about setting it in "ifup", but that didn't sound right
15:33:12 sean-k-mooney cfriesen_: you can use ifcfg or /etc/network/interfaces files
15:33:17 sean-k-mooney cfriesen_: they only work for nics
15:33:34 sean-k-mooney and the require the nic to be bound back to a driver other then vfio-pci
15:34:06 sean-k-mooney the udev/systemd approch works for GPUs or QAT devices too
15:34:30 cfriesen_ sean-k-mooney: makes sense, thanks
15:41:17 mriedem jgwentworth: superdan: looking at tssurya's online data migration here https://review.openstack.org/#/c/567878/ - i realize that it, and the one before it for populating BDM uuids, aren't multi-cell aware, so anyone running multiple cells has to run this manually per-cell database config, which kind of sucks
15:41:42 mriedem we could probably easily pull the non-cell0 cells from the api db and iterate those doing the online data migration
15:42:29 jgwentworth well, if we do that, we'd also want to make db sync multi-cell too right? because those are usually run side-by-side
15:42:37 mriedem there is a patch for that
15:42:39 jgwentworth or is it already
15:42:41 jgwentworth okay
15:43:05 mriedem https://review.openstack.org/#/c/519275/
15:44:05 jgwentworth nice
15:58:06 sean-k-mooney fyi if people have ATC codes and are planning to go to the summit in in a week they expire today. also im now going to the summit
15:59:10 mriedem jgwentworth: ah crap,
15:59:23 mriedem the in-tree live migration job won't run when we've only made changes to nova/tests/*
15:59:28 mriedem https://review.openstack.org/#/c/567860/
15:59:55 mriedem https://github.com/openstack/nova/blob/master/.zuul.yaml#L22
16:00:05 mriedem er https://github.com/openstack/nova/blob/master/.zuul.yaml#L49
16:00:39 jgwentworth is that not what we wanted? I copied it from the nova base job I thought
16:01:03 mriedem the nova-live-migration setup is in nova/tests/live-migration
16:01:45 mriedem i can post a fix i think for the regex
16:04:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix irrelevant-files in nova-dsvm-multinode-base https://review.openstack.org/567884
16:04:43 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860
16:05:40 mriedem yup that fixed it, https://review.openstack.org/#/c/567860/ is queued up to run nova-live-migration onw
16:05:41 mriedem *now
16:12:25 openstackgerrit Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
16:12:57 jgwentworth mriedem: oh I see
16:28:12 jgwentworth we have a review for increasing the max number of attached volumes to a single instance from 26 https://review.openstack.org/567472
16:29:15 jgwentworth mdbooth or lyarwood: would appreciate your review on that ^
16:30:29 lyarwood jgwentworth: ack yeah will have to be early next week now
16:30:47 jgwentworth np, no rush. just a heads up
16:31:30 mdbooth jgwentworth: Have you tested it, btw? What's the limit on the default storage controller?
16:32:55 jgwentworth mdbooth: uh, I have no idea?
16:34:13 jgwentworth I need to do some research to find out what makes sense there. I didn't know we currently limit to 26 until that patch
16:35:11 jgwentworth and I'm aware that sahid did some work with virtio-scsi to allow 256 devices. but that must be somehow separate from this, maybe those aren't volumes
16:35:57 sean-k-mooney finucannot: still around?
16:36:09 finucannot yup
16:36:25 openstackgerrit Merged openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
16:36:34 sean-k-mooney notice anything wrong with this https://github.com/openstack/os-vif/blob/master/tox.ini#L26
16:37:33 sean-k-mooney ill give you a hint look at the python 3 tests below it
16:37:36 finucannot I'm assuming '.tests' shouldn't be in the regex?
16:37:47 finucannot (or we should drop 'test-path')
16:38:07 sean-k-mooney yep we are not running the unit test for the plugins in python 2
16:38:16 sean-k-mooney for the last 10 months...
16:38:38 finucannot Boo, ralonsoh. Booooo
16:38:45 sean-k-mooney im going to fix this. this is is why i was not able to run the tests for the noop plugin
16:39:31 sean-k-mooney speaking of which ill finish that before i leave today but im guess you will be gone
16:39:44 finucannot I'll be around for another while yet
16:39:57 finucannot But if I am, I can review Tuesday for sure
16:40:02 finucannot (I'm gone Monday)
16:40:31 sean-k-mooney cool no worries i should have the review up in the next 15 mins or so
16:41:22 sean-k-mooney it was driving nuts trying to figure out why my test were not running lol i should start testing with py35 by default
16:52:15 openstackgerrit sean mooney proposed openstack/os-vif master: fix tox py27 job https://review.openstack.org/567895
16:58:36 PaulN Hi, quick question, I'm trying to create a stack via Heat Templates containing a OS::Nova::ServerGroup in which I want to use the soft-anti-affinity policy which was added in nova compute api microversion 2.15. I can create it via the CLI by specifying --os-compute-api-version 2.15, but I can't create it via heat. I can't seem to find any information on how one specifies the nova api microversion from a Heat Template.
17:00:26 PaulN This in on an older OpenStack cluster that is only running Newton.
17:03:17 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899
17:06:43 jgwentworth PaulN: I think you'll need to try the heat channel or tripleo channel. not sure anyone here will know how to specify microversions via heat templates
17:06:44 PaulN Yeah I have the Heat API version set to the value appropriate for Newton, but thats just the Heat Template version, doesn't really seem to have any correlation to what Nova API version is used.
17:06:57 cfriesen_ PaulN: based on https://bugzilla.redhat.com/show_bug.cgi?id=1447798 it seems heat support for nova microversions was only added in Ocata
17:07:06 jgwentworth nvm
17:07:08 PaulN Okay, I can try over in the heat channel.
17:07:51 PaulN Ugh, based on that seems I may be unable to use it via Heat on this cluster.
17:08:18 openstack bugzilla.redhat.com bug 1447798 in openstack-heat "soft-anti-affinity is not accepted by python-openstackclient but is with python-novaclient" [High,Closed: duplicate] - Assigned to zbitter
17:11:38 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899
17:20:00 prometheanfire mriedem_lunch: the new keystonemiddleware breaks nova tests
17:20:03 prometheanfire http://logs.openstack.org/49/567449/1/check/cross-nova-py27/c2d43ef/testr_results.html.gz
17:35:10 jgwentworth cdent ^ you might understand what's going on there. only difference is the string quotes?
17:38:02 cdent jgwentworth: huh, wacky.
17:38:10 jgwentworth yeah
17:41:07 cdent jgwentworth: it's this https://review.openstack.org/#/c/559925/
17:41:24 jgwentworth aaahhh
17:42:00 cdent so the test was relying on apparently not quite correct behavior by the middleware
17:42:20 cdent so it failing now makes sense
17:42:24 jgwentworth good find

Earlier   Later