Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-21
21:24:50 coffee_cat and we need to call attachment_update at some point
21:25:05 coffee_cat with the connector
21:25:34 mriedem if bdm.attachment_id is set, we know the api reserved the new volume with the new flow
21:25:47 mriedem so i think https://review.openstack.org/#/c/456971/ gets more complicated than just updating the compute code
21:25:50 mriedem it should also be the api code
21:25:54 mriedem and the rpc api changes
21:26:09 mriedem unless we just want to lookup the new attachment via rest api
21:26:17 mriedem i just don't know if we have that support yet in cinderclient
21:26:53 mriedem i could help work on this if you want
21:27:02 mriedem i just can't do it today
21:28:02 mriedem it would be great if you guys could get the v3.27 APIs documented in https://developer.openstack.org/api-ref/block-storage/v3/
21:28:04 mriedem you == all of cinder
21:28:31 mriedem and by "guys" i mean guys and gals and those that don't identify as guys or gals, of course
21:31:12 coffee_cat mriedem: got it :)
21:31:32 mriedem it looks like from the compute we could list attachments and filter on instance_uuid
21:31:53 coffee_cat and yeah, if the bdm.attachment_id is set then all computes should be on the high enough version, etc.
21:32:23 coffee_cat we should definitely be able to do that if we want to skip the rpc changes, which I'm totally fine with
21:32:28 mriedem we could get attachments via cinderclient for a given volume and instance
21:32:33 mriedem using filters it looks like
21:32:40 jgriffith mriedem correct
21:33:01 mriedem if we got more than one attachment back...idk
21:33:10 mriedem we shouldn't today since we don't support multiattach
21:33:12 mriedem but it gets weird
21:33:21 mriedem that's probably why it's just better to pass the new attachment id over rpc
21:33:22 mriedem to the compute
21:33:22 jgriffith mriedem right, for today that’s our safety net
21:33:26 mriedem rather than hae the compute guess
21:33:30 jgriffith BUT, there’s a get-attachment
21:33:35 coffee_cat mriedem: although for live migrate etc we should be able to have multiple attachments for the same volume
21:33:51 coffee_cat which should not make a mess for swap volume, but still
21:33:58 mriedem i think passing over rpc is safest
21:34:05 mriedem then compute doesn't need to decide
21:34:23 mriedem again, i can help with this later, but i'm getting in trouble for hiding in the office when it's a nice day out
21:34:24 coffee_cat agreed
21:34:38 mriedem so i'm forcefully detaching from this computer, ttyl
21:34:54 coffee_cat mriedem: have a good weekend :)
22:14:18 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: nova.utils.get_service_url(group) https://review.openstack.org/458257
22:15:11 efried mriedem johnthetubaguy mordred If possible I'd like y'all to start looking at ^^ - let me know if the approach is remotely sane, and maybe answer some of the open questions in the comments.
22:15:50 efried Wait, was it johnthetubaguy or jaypipes? Well, jaypipes isn't here, so...
22:17:50 mordred it was jay - I have it open in my browser - I'm heads down on a different patch at the moment, but I'll come back and give you some feedback RSN
22:18:05 mordred leakypipes: ^^
22:22:22 efried mordred Thanks. No hurry, think I'm gonna bounce for the night.
23:16:39 leakypipes mriedem: oh, you didn't appreciate my playground rock-paper-scissors humour? boo.
#openstack-nova - 2017-04-22
01:11:27 chatter29 hey guys
01:11:29 chatter29 allah is doing
01:11:34 chatter29 sun is not doing allah is doing
01:11:36 chatter29 to accept Islam say that i bear witness that there is no deity worthy of worship except Allah and Muhammad peace be upon him is his slave and messenger
#openstack-nova - 2017-04-23
08:50:15 openstackgerrit Abhishek Sharma M proposed openstack/nova master: Catching OverLimit Exception https://review.openstack.org/455118
13:30:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Cinder v3 detach call to _terminate_volume_connections https://review.openstack.org/456896
13:30:55 mriedem ildikov: addressed comments ^ i'm going to add a new patch on top of this that adds attachment_create, and then redo the swap_volume patch on top of that
13:32:46 mriedem and attachment_update i guess
13:37:21 mriedem jgriffith: is there a quota limit on volume attachments?
13:38:33 mriedem doesn't look like it
13:44:30 ildikov mriedem: isn't it still Sunday? I have a longer route to home than I expected so I cannot follow what day it is anymore... :)
13:44:43 mriedem yeah it's sunday morning
13:44:57 mriedem i think there also might be a bug in the cinder v3 attachment create api
13:45:04 mriedem or the attachments view builder rather
13:45:11 mriedem oh wait nvm
13:45:27 ildikov mriedem: i haven't seen any quota limits for attachments
13:45:45 ildikov mriedem: there was so
13:46:34 ildikov *some bug fixing activity with that already
13:46:46 mriedem i was just reading the view builder code incorrectly
13:46:47 mriedem it's ok
13:51:59 ildikov ok, cool
13:52:46 ildikov sorry, I couldn't get to that chain earlier
13:52:52 ildikov thanks for looking into it!
13:53:00 mriedem np
13:53:54 ildikov I will try to entertain myself with the two standalone patches on the plane today and get tests up
14:10:54 openstackgerrit Matt Riedemann proposed openstack/nova master: cinder: add attachment_create method https://review.openstack.org/459111
14:53:33 openstackgerrit Matt Riedemann proposed openstack/nova master: cinder: add attachment_create method https://review.openstack.org/459111
15:10:17 openstackgerrit Matt Riedemann proposed openstack/nova master: cinder: add attachment_update method https://review.openstack.org/459113
15:43:27 mriedem mordred: i'm trying to fill out the project version navigator for nova, and going back to austin, it looks like the openstack route didn't actually return versions :)
15:43:30 mriedem the ec2 route did
15:43:41 mriedem https://github.com/openstack/nova/blob/c364724a0dc7a658058fcb167af66ee7eb5bcd2a/nova/api/openstack/__init__.py
15:44:10 mriedem https://github.com/openstack/nova/blob/c364724a0dc7a658058fcb167af66ee7eb5bcd2a/nova/api/ec2/__init__.py#L266
15:44:16 mriedem that 1.0 in the ec2 route versions must have been for openstak
15:44:19 mriedem *openstack
15:56:16 mriedem or i guess it was 1.0 for ec2?
16:08:05 mriedem ooo snap i found the 2010.1 code on launchpad
20:48:39 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/458834
22:31:34 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Update cinder-new-attach-apis spec for begin_detaching https://review.openstack.org/459134
#openstack-nova - 2017-04-24
00:51:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (2/4) https://review.openstack.org/455568
00:51:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (3/4) https://review.openstack.org/455570
00:52:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers-actions (4/4) https://review.openstack.org/455573
00:52:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
00:55:42 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971
00:55:43 openstackgerrit Matt Riedemann proposed openstack/nova master: cinder: add attachment_update method https://review.openstack.org/459113
00:55:43 openstackgerrit Matt Riedemann proposed openstack/nova master: cinder: add attachment_create method https://review.openstack.org/459111
00:57:22 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971
01:47:02 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030
02:58:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
03:08:00 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Trivial Fix a typo https://review.openstack.org/459149
05:22:01 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova. https://review.openstack.org/459166
06:59:15 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030
07:19:03 bhagyashris Hi all, I need some guidance regarding, configuration: Is it possible to set single resource provider in multinode setup for nfs backend? How should I configure the single resource provider in multinode environment which uses the nfs backend?
07:23:01 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
08:04:20 openstackgerrit jichenjc proposed openstack/nova-specs master: Add os-quota-class-sets-query-update https://review.openstack.org/459185
08:26:24 moshele sfinucan: hi
08:26:32 sfinucan moshele: Heya
08:28:07 moshele sfinucan: regarding https://review.openstack.org/#/c/398277/ so I am not sure that vif_type is the best place to document the ovs plugin

Earlier   Later