Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-18
02:46:16 kevinbenton mriedem: yeah, it's an OR list
02:47:18 mriedem hmm, but the API ref says "OpenStack Networking does not offer an OR mechanism for filters."
02:47:45 kevinbenton mriedem: it wouldn't be the first time the docs were wrong :)
02:47:45 mriedem i figured the client would unpack the list and turn that into fixed_ip_address=1&fixed_ip_address=2&port_id=port_id
02:47:47 mriedem and AND them all
02:50:49 mriedem ok i can post a wip for that too and see what happens,
02:50:56 mriedem i don't know if there are any tests in tempest which would tickle that
02:52:50 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port https://review.openstack.org/465787
02:52:51 mriedem kevinbenton: ^ is the kind of awful i was talking about earlier
02:53:12 kevinbenton mriedem: yeah, repeats turn into the list which becomes the OR in that first query link i sent
02:53:17 kevinbenton mriedem: here is the request parse https://github.com/openstack/neutron/blob/master/neutron/api/api_common.py#L46-L47
02:53:40 mriedem cool
02:54:11 kevinbenton mriedem: yeah, not exactly a trivial change :)
02:54:23 mriedem it would really be split into 2 changes, the first being a refactor
02:54:25 mriedem but yeah
02:54:31 mriedem i'm not going for pretty at 10pm
02:55:01 kevinbenton mriedem: move to hawaii
02:55:10 kevinbenton more time in the day that way
02:55:12 kevinbenton :)
02:55:15 mriedem heh
02:55:27 mriedem you are a proven thought leader
03:00:32 mriedem apparently we handle GET /floatingips results as a list when we filter on fixed IP and port,
03:00:37 mriedem but wouldn't that really be 1:1?
03:00:49 mriedem can you have more than one floating IP associated to the same fixed ip and port?
03:00:56 mriedem i'd think not
03:01:26 kevinbenton no
03:01:34 mriedem ok good, this is just confusing nova code then
03:01:53 kevinbenton well let me confirm
03:04:59 kevinbenton mriedem: there actually may be a way to get there
03:05:06 mriedem ew
03:05:14 kevinbenton mriedem: two different routers
03:05:20 mriedem ok
03:05:21 kevinbenton mriedem: attached to two different ext nets
03:05:29 kevinbenton mriedem: both attached to same internal
03:05:37 kevinbenton mriedem: and both with a floating IP associated to the instance
03:06:09 kevinbenton mriedem: and it would actually work okay if you setup some routing rules on the instance
03:06:18 kevinbenton mriedem: so i think you'll have to preserve that use case :)
03:06:24 mriedem damn
03:06:24 mriedem ok
03:07:19 kevinbenton mriedem: the constraint for floating IP is external_network + IP + port_id https://github.com/openstack/neutron/blob/master/neutron/db/models/l3.py#L101
03:09:42 mriedem i'll bake that into a comment in here
03:16:17 mriedem if we only have one fixed ip per port then this doesn't buy us much
03:16:19 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Bulk query floating IPs in _nw_info_get_ips https://review.openstack.org/465792
03:16:24 mriedem but here is the thing ^
03:17:15 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Bulk query floating IPs in _nw_info_get_ips https://review.openstack.org/465792
03:29:28 openstackgerrit Jay Pipes proposed openstack/nova master: Ensure image metadata when unshelving BFV instance https://review.openstack.org/347063
03:32:12 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Don't warn on expected network-vif-unplugged events https://review.openstack.org/465794
03:54:25 openstackgerrit Tao Li proposed openstack/nova master: Optimize the logged message https://review.openstack.org/465797
03:57:34 openstackgerrit Luo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados https://review.openstack.org/465404
04:00:59 cfriesen mriedem: you've been a busy beaver on get_instance_nw_info()...taking a look at your patch.
04:32:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unused exceptions https://review.openstack.org/464525
04:33:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
04:35:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
05:45:16 openstackgerrit Eli Qiao proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984
05:51:33 openstackgerrit Jay Pipes proposed openstack/nova master: Fix wrong disk size for sparse flat files https://review.openstack.org/396873
05:53:23 openstackgerrit Luo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados https://review.openstack.org/465404
05:55:40 openstackgerrit Tao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver. https://review.openstack.org/414463
06:13:52 openstackgerrit christopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951
06:19:50 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
06:30:06 openstackgerrit Eli Qiao proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984
07:02:52 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
07:40:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
07:41:00 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-quota-sets endpoint instead of stevedore https://review.openstack.org/461630
07:42:32 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641
07:46:40 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818
07:48:12 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore https://review.openstack.org/460813
07:50:57 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628
08:20:04 openstackgerrit Andrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values https://review.openstack.org/448008
08:24:30 openstackgerrit sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741
08:24:30 openstackgerrit sahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller https://review.openstack.org/459740
08:24:31 openstackgerrit sahid proposed openstack/nova master: libvirt: rearange how scsi controller is defined https://review.openstack.org/459739
08:24:31 openstackgerrit sahid proposed openstack/nova master: libvirt: update LibvirtConfigGuestDeviceAddress to provide XML https://review.openstack.org/459737
08:24:32 openstackgerrit sahid proposed openstack/nova master: libvirt: set full description of the controller used by disk https://review.openstack.org/459738
08:25:56 openstackgerrit Tao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver. https://review.openstack.org/414463
08:32:01 lyarwood bauzas, sfinucan, johnthetubaguy ; https://review.openstack.org/#/c/462348/ - A pretty straight forward rootwrap fix for something I broke a few weeks ago if you have time to review this morning, I might be feeling slightly guilty about this ;)
08:34:19 sfinucan lyarwood: Yup, that looks fine by me. +2
08:37:27 lyarwood sfinucan: thanks
08:43:20 openstackgerrit jichenjc proposed openstack/nova master: remove unused policy from policy doc https://review.openstack.org/465505
09:08:29 mlakat Hi Stackers - we have an interesting problem - basically we have a broken connection to an NFS server, which results that os.path.getsize() hangs forever on a mounted file. Do we have any timeout like functionality in OpenStack? Greenthreads seem to be blocked.
09:09:14 mlakat this is the bug btw: https://bugs.launchpad.net/nova/+bug/1691131
09:09:14 openstack Launchpad bug 1691131 in OpenStack Compute (nova) "IO stuck causes nova compute agent outage" [Undecided,In progress] - Assigned to Daniel Gonzalez Nothnagel (dgonzalez)
09:15:58 kashyap artom: Looking at your patch
09:16:23 artom kashyap, thanks :)
09:17:07 artom Also, I tried playing around with updating the volume serial number when we do volume-update, but I don't think that's supported in a live domain
09:17:27 artom Ie, we can only change it in the persistent XML
09:17:31 kashyap artom: Yeah, I saw your note in the LP bug
09:17:48 kashyap artom: I was also wondering, if that line in driver.py should be:
09:17:50 kashyap flags |= copy and (libvirt.VIR_DOMAIN_BLOCK_REBASE_COPY | libvirt.VIR_DOMAIN_BLOCK_REBASE_COPY_DEV) or 0
09:18:14 kashyap But maybe doing it as two lines is consistent with the rest
09:18:23 artom And having the serial number not match the actual volume ID has apparently not been a problem until now, so I just left it
09:19:14 artom kashyap, it's still bitwise or'ing all the flags, right? Might as well stay consistent with what's there already
09:19:39 kashyap artom: Yes, it is bitwise OR
09:19:46 kashyap s/is/is still/
09:21:06 openstackgerrit Yongfeng Du proposed openstack/nova master: Update the file for IPv4-only or IPv6-only network https://review.openstack.org/465891
09:29:46 openstackgerrit Eli Qiao proposed openstack/nova master: Correct force host checking for LM and evacuation. https://review.openstack.org/465895
09:32:17 openstackgerrit Eli Qiao proposed openstack/nova master: Correct force host checking for LM and evacuation https://review.openstack.org/465895
09:33:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix uuid replacement in aggregate notification test https://review.openstack.org/465896
09:41:32 openstackgerrit Jianghua Wang proposed openstack/nova master: Skip policy rules on attach_network for none network allocation https://review.openstack.org/465903
09:43:58 openstackgerrit Sylvain Bauza proposed openstack/nova master: Scheduler returning alternatives per destination https://review.openstack.org/465175
09:43:58 openstackgerrit Sylvain Bauza proposed openstack/nova master: Delete existing allocations when retrying https://review.openstack.org/460433

Earlier   Later