Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-08
15:39:52 mriedem melwitt: can we use an IN clause to delete all server group members using the list of instance uuids at once? https://review.openstack.org/#/c/457858/16
15:52:46 sfinucan melwitt: Could you take another look at https://review.openstack.org/#/c/379524/ today, please? :)
15:54:28 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
15:56:29 openstackgerrit Merged openstack/python-novaclient master: Better handle key=value pair not being key=value https://review.openstack.org/469988
16:04:27 cfriesen re: my comment above, I just hit another one...neutron wants os-xenapi>=0.2.0. Shouldn't devstack handle the pip upgrade itself?
16:08:20 sdague johnthetubaguy: https://dogpilecache.readthedocs.io/en/latest/api.html#memory-backends if we wanted to go down that path
16:08:52 sdague cfriesen: RECLONE=True ?
16:09:10 sdague because if not you can hit requirements files out of sync with upper-constraints
16:12:29 cfriesen sdague: thanks, will give that a try. I thought that was the default, for some reason.
16:13:20 sdague no, because then it hard resets people's code
16:13:50 cfriesen wonder if I should try PIP_UPGRADE=True
16:17:13 sdague I don't think that does anything any more with constraints in play
16:19:34 mriedem kashyap: is there any reason that nova "needs" to set the 'serial' field in the disk config for the libvirt domain xml?
16:19:41 mriedem reading https://libvirt.org/formatdomain.html#elementsDisks
16:19:48 mriedem it says,
16:19:50 mriedem "If present, this specify serial number of virtual hard drive. For example, it may look like <serial>WD-WMAP9A966149</serial>. Not supported for scsi-block devices, that is those using disk type 'block' using device 'lun' on bus 'scsi'. Since 0.7.1"
16:20:04 mriedem however, we make no distinction of when we set that in the disk config, block type or not
16:20:17 mriedem and it's always the cinder volume id i believe
16:20:27 mriedem does qemu actually care about it?
16:21:31 johnthetubaguy sdague: yeah, good call, spied your comment on that
16:22:49 mriedem except avoid pickle
16:27:35 cfriesen sdague: with RECLONE=True I still hit it: ContextualVersionConflict: (pyroute2 0.4.13 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('pyroute2>=0.4.15'), set(['neutron']))
16:28:31 sdague cfriesen: I don't know
16:28:51 sdague definitely going to need more than a single error log to know how you got there
16:29:10 sdague go through the log and see why there is a disconnect there
16:29:37 cfriesen sdague: http://paste.openstack.org/show/612007/
16:30:24 sdague cfriesen: and the value in /opt/stack/requirements/upper-constraints.txt
16:30:54 cfriesen sdague: pyroute2===0.4.15
16:31:47 sdague hmmm... idk
16:32:00 sdague I thought that was always supposed to override
16:35:03 cfriesen sdague: it choked on pyroute2 again even with PIP_UPGRADE=True
16:36:36 cfriesen I'm tempted to just blow everything away and start from scratch
16:37:37 sdague yeh, I'm rerunning locally to try to figure out if I see anything
16:38:02 sdague there could be a pip regression
16:45:03 openstackgerrit Merged openstack/nova master: Default to 0 when merging values in limit check https://review.openstack.org/471973
16:49:25 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update https://review.openstack.org/466296
16:55:55 dansmith mriedem: looks like that service version set is good now
16:56:28 mriedem dansmith: did you see artom's questions in the top change?
16:56:33 dansmith nope
16:57:10 dansmith oh, yeah, I was just fixing the device metadata thing here
16:57:13 mriedem the min version is checked in 2 places in conductor
16:57:15 dansmith I can do the others now
16:57:20 dansmith artom: ^
16:57:26 mriedem one is safely top-level super conductor,
16:57:37 mriedem the other is rpc, so it's not possibly (i think)
16:57:55 dansmith yeah, the rpc one he points to needs to stay single-cell, but I'll look to see if maybe we should do it higher
16:57:56 cfriesen sdague: I have no idea what's going on. I blew away ~/devstack and cloned it again, and blew away /opt/stack, and I'm still hitting version conflicts.
16:58:10 dansmith mriedem: we may also be able to drop lots of these
16:59:16 mriedem i'd like to drop paging from the hypervisors api
16:59:31 dansmith heh
17:01:52 clarkb cfriesen: sdague fwiw a trivial attempt at reproducing using a venv with 0.4.13 installed then an install of 0.4.15 using constraints works ok
17:02:05 mriedem btw i'm out tomorrow
17:03:04 clarkb so whatever it is seems more complicated than just pyroute2 and constraints
17:03:15 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
17:03:59 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor instance-exists docstring and unit test https://review.openstack.org/462738
17:05:21 clarkb cfriesen: maybe try with pip -v and/or --log ?
17:05:50 cfriesen clarkb: I can upgrade this manually no problem....but I expect devstack to be doing the upgrading and I don't understand why it isn't
17:06:09 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
17:06:35 clarkb cfriesen: so if you run the command yourself it works fine?
17:06:49 cfriesen clarkb: I'm hitting this with multiple packages. If I manually upgrade one package it continues on until it hits another one that needs upgrading.
17:06:58 mriedem dansmith: are you ok with this now after the cells v1 rpc version bump? https://review.openstack.org/#/c/461532/
17:07:03 cfriesen clarkb: something like this works fine: sudo pip install --upgrade ovsdbapp==0.4.0
17:07:52 dansmith mriedem: with that approach yeah, I need to look at the rest of the patch though
17:07:53 sdague cfriesen: I for installed pyroute2==0.4.13 then ran ./clean.sh && ./stack.sh and everthing works fine
17:07:57 clarkb perhaps something with the -H then and pip caching things? But also getting output from pip -v and/or --log would likely be helpful
17:08:27 sdague this is a 16.04 box
17:08:31 cfriesen sdague: I hadn't tried clean.sh, will give that a shot
17:09:20 cfriesen sdague: same here
17:11:19 openstackgerrit Merged openstack/nova master: code comments incorrectness https://review.openstack.org/449416
17:12:42 cfriesen mriedem: if you've got suggestions on how to properly mock the api_version in https://review.openstack.org/#/c/435141/ I'd be happy to make changes. I couldn't figure it out short of modifying each testcase using the "list" command.
17:16:56 cfriesen sdague: ./clean.sh wasn't enough. just hit ContextualVersionConflict with neutron-lib
17:18:07 cfriesen sdague: clarkb: http://paste.openstack.org/show/612034/
17:32:29 artom dansmith, huh, didn't notice the ping - so basically the other places where get_min_service_version is called will be fixed in different patches?
17:32:51 dansmith artom: yeah, I was just fixing that one place in device metadata because of mriedem's bug he opened
17:32:53 dansmith but I
17:32:58 dansmith will take a look at the rest in a few
17:33:18 artom dansmith, makes sense :)
17:33:26 artom Keep commits topical
17:36:04 ihrachys seems like port detach issues still happens even with https://review.openstack.org/#/c/441204/ in
17:36:04 mriedem i like my creams just like i like my commits
17:36:10 ihrachys mriedem, ^
17:36:20 ihrachys example: http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/logs/screen-n-cpu.txt.gz?level=TRACE#_Jun_08_16_58_01_395989
17:37:17 mriedem hmm, that's not fun
17:38:11 mriedem ihrachys: it's always this same test right http://logs.openstack.org/63/471763/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/dccf187/console.html#_2017-06-08_17_13_16_844927 ?
17:38:14 mriedem the hotpluc nic one?
17:38:19 mriedem and usually on the linuxbridge job?
17:38:25 artom mriedem, the word you're looking for is "ointments" - it's up there with "moist" as the most shudder-inducing word in the English language
17:38:32 mriedem yeah it's like 99% LB job
17:38:35 ihrachys I think yes
17:38:43 mriedem i wonder what's different about LB in thiscase
17:38:55 mriedem maybe the vif isn't unplugging how we'd expect before we detach from the guest?
17:39:13 mriedem artom: i think "topical cream"
17:39:16 mriedem not "topical ointment"
17:39:21 ihrachys afaik for linuxbridge case, neutron now manages the whole plugging cycle ?
17:39:28 ihrachys since late switch to tap device type
17:39:38 mriedem ihrachys: i don't know anything about that
17:39:47 ihrachys I mean https://review.openstack.org/#/c/447150/
17:39:55 artom mriedem, because of you my search history is weird now
17:40:08 mriedem artom: search on your phone, not your work system
17:40:25 mriedem nothing weird about grown man pain reliefe

Earlier   Later