| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-21 | |||
| 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_create method https://review.openstack.org/459111 | |
| 00:55:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: cinder: add attachment_update method https://review.openstack.org/459113 | |
| 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 | |
| 08:29:55 | sfinucan | moshele: No? Would there be somewhere better suited? | |
| 08:31:45 | moshele | sfinucan: vif_type describe the version objects of the os-vif. I guess we should have also documentation per plugin ovs and lb | |
| 08:34:03 | moshele | sean-k-mooney: what do you think ^ | |
| 08:34:14 | johnthetubaguy | there is the longer term plan around os-vif type binding | |
| 08:34:37 | johnthetubaguy | I was catch up with kevinbenton about that on friday I think | |
| 08:35:07 | moshele | can you point me to the discussion? | |
| 08:35:37 | openstackgerrit | jichenjc proposed openstack/nova-specs master: Add os-quota-class-sets-query-update https://review.openstack.org/459185 | |
| 08:36:32 | johnthetubaguy | moshele: the wider context is implement the vif binding negotiation thing in the original os-vif spec, maybe thats not too relevant here, except for the versioning | |
| 08:39:10 | moshele | johnthetubaguy: ok I will read the os-vif spec | |
| 08:39:48 | sfinucan | moshele: The main thing I was looking for was a comparison of that particular VIF type to other VIF types | |
| 08:39:53 | sfinucan | I took a stab at documenting a glossary, but that doesn't really solve the problem here | |
| 08:40:02 | sfinucan | So if that's in the form of a new doc, so be it | |
| 08:40:33 | johnthetubaguy | sfinucan: docs are a good point here, I think that is slowing down the discussions with neutron | |
| 08:41:01 | johnthetubaguy | there is something that Nova calls to inspect the host to say what is supported, I guess we have to check the kernel to see if this new feature is supported | |
| 08:41:19 | johnthetubaguy | Nova then (eventually) will send to neutron the os-vif response to say what the host supports | |
| 08:41:35 | johnthetubaguy | neutron then picks from that what it thinks its best from its side of things | |
| 08:42:07 | johnthetubaguy | the port binding info is then returned as a serialized os-vif object, that needs to tell os-vif exactly what needs to get setup | |
| 08:42:19 | johnthetubaguy | or something like that (wibble inspect the instance and the host, etc) | |
| 08:42:36 | johnthetubaguy | its not clear how this all fits into that pattern, for me at least | |
| 08:43:46 | johnthetubaguy | sfinucan: is that the kinda stuff you were thinking, or is that orthogonal? | |
| 08:47:50 | moshele | johnthetubaguy: this is orthogonal because we to doument the plug/unplug of each vif plugin | |
| 08:50:24 | moshele | johnthetubaguy: basiclly doumenting parts like https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L154-L174 as it become to match complex. sfinucan are we on the same page? | |
| 08:53:16 | johnthetubaguy | moshele: ah, understood, that certainly looks out of hand | |
| 08:54:08 | johnthetubaguy | moshele: I was kinda expecting each of those vif objects to do the thing they needed, rather than having massive If statement | |
| 08:57:10 | moshele | johnthetubaguy: the ovs-plugin just say it used with the OVS mechanism driver, but there are several ways to achieve binding (plain ovs/ hybrid with lb/DPDK/windows) each one is an if statement | |
| 08:58:00 | sfinucan | moshele: Yeah, exactly | |
| 08:58:03 | sfinucan | As a start | |
| 08:58:12 | johnthetubaguy | moshele: yeah thats certainly the current state, that seems bad, you could use the object structure to remove the all the ifs I would have thought | |
| 08:58:37 | johnthetubaguy | I guess I was expecting those to all get advertised as separate things in some ways, just sharing a common set of utils | |
| 08:58:38 | sfinucan | johnthetubaguy: There's definitely good reason to go the way you're suggesting but that'd be a different series | |
| 08:58:45 | johnthetubaguy | sfinucan: oh, totally | |
| 08:58:52 | johnthetubaguy | just thinking out loud really | |
| 08:58:57 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Rework descriptions in os-hypervisors https://review.openstack.org/452030 | |