| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-05 | |||
| 16:57:54 | mriedem | so likely not really needed | |
| 16:58:12 | gibi | it doesn't hurt | |
| 16:59:02 | mriedem | i've approved it | |
| 16:59:43 | gibi | mriedem: thanks | |
| 17:00:08 | gibi | mriedem: if you are interested in a bit heavier read then I also reworked the minimum bandwidth support spec during the weekend based on the PTG discussion https://review.openstack.org/#/c/502306/ | |
| 17:00:35 | mriedem | omg | |
| 17:00:41 | mriedem | i don't think i can handle that kind of weight right now | |
| 17:01:37 | gibi | mriedem: no problem. I agree that this is not good read for the first day after PTG | |
| 17:02:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test nova-api eventlet deprecation https://review.openstack.org/549770 | |
| 17:03:01 | mriedem | gibi: thanks for taking that on though, i'm glad you're owning it | |
| 17:03:31 | gibi | mriedem: yeah, my employer is pushing me to take it seriously | |
| 17:04:10 | mriedem | my employer also cares about it, but i haven't been physically kicked yet | |
| 17:04:36 | mriedem | it will be easier to kick Kevin_Zheng | |
| 17:04:40 | gibi | mriedem: mlavalle seems interested too and If I know correctly you two have the same employer | |
| 17:04:45 | mriedem | yup | |
| 17:05:44 | mlavalle | gibi: yes, I am interested. Will push it from the Neutron side | |
| 17:05:47 | stephenfin | mriedem: What needs to happen to move this forward now? https://review.openstack.org/#/c/483994 | |
| 17:06:55 | gibi | mlavalle: thanks! I think when you read the nova spec you will find couple of thing that needs to be discussed on the neutron side | |
| 17:06:59 | mriedem | stephenfin: um | |
| 17:07:16 | stephenfin | I'm currently deploying DevStack to see if I can reproduce that issue from Tushar Patil. However, the deprecation doesn't really depend on that | |
| 17:07:24 | mriedem | stephenfin: would be nice to know if tpatil can confirm novnc 1.0.0 works for him | |
| 17:07:29 | mlavalle | gibi: ack. will look for them | |
| 17:08:05 | gibi | mlavalle: I have Bence Romsics (rubasov) who can help with the neutron discussion / implementation work | |
| 17:08:21 | mriedem | stephenfin: ovh was saying they have the novnc fix deployed in their cloud though and it works | |
| 17:09:00 | stephenfin | Right. I can't verify as I can't read Japanese :) | |
| 17:09:03 | mlavalle | gibi: fantastic! | |
| 17:09:07 | mriedem | stephenfin: so i think the thing i was looking for in the deprecation was we need wording in the config option help text that you need novnc >= 1.0.0 | |
| 17:09:15 | mriedem | stephenfin: maybe takashin can help | |
| 17:09:21 | mriedem | he was also pushing a related spec for NTT related to this | |
| 17:09:28 | stephenfin | That's not a bad idea | |
| 17:09:34 | mriedem | it's a *great* idea | |
| 17:09:46 | gibi | mlavalle: I have to run now, talk to you later | |
| 17:26:08 | Spazmotic | whew, third shifters shouldn't be up t his late | |
| 17:59:30 | jgriffith | mriedem: hey.. there's a little bit of a snag in the plan | |
| 17:59:52 | jgriffith | mriedem: it seems the attached_host field wasn't being translated into the viewbuilder from cinder :( | |
| 18:00:15 | jgriffith | Not sure if it was dropped somewhere or if we just missed it altogether | |
| 18:00:29 | jgriffith | mriedem: easy enough to add, but then we get into the whole versioning debate | |
| 18:00:43 | mriedem | jgriffith: we don't want to rely on the attached_host field, | |
| 18:00:53 | jgriffith | mriedem: oh | |
| 18:01:01 | jgriffith | well then I suppose that's good news :) | |
| 18:01:02 | mriedem | because we discussed at the ptg that we need to control that via policy rule, and default that to admin-only | |
| 18:01:37 | jgriffith | mriedem: ok... so on the disconnect, I have a list of attachment-ids | |
| 18:01:50 | jgriffith | and of course can get the list of attachments from said id's | |
| 18:02:06 | mriedem | jgriffith: so i think the flow is: get volume details, from the volume['attachments'], get the list of server uuids attach to that volume, get the list of instances on that host (InstanceList.get_uuids_by_host - something like that), and then cross-reference those 2 lists | |
| 18:02:11 | jgriffith | so are you suggesting then iterate through bdms? | |
| 18:02:19 | mriedem | no i don't think we need bdms | |
| 18:02:33 | jgriffith | mriedem: ahh | |
| 18:02:41 | mriedem | i think we just need to see if there are >1 servers attached to the same volume, and if >1 of those servers are on *this* host | |
| 18:02:42 | jgriffith | I think I see what you're getting at | |
| 18:02:46 | mriedem | if so, don't call disconnect | |
| 18:03:16 | jgriffith | Ok, I think that will work; let me try and put it all together, thanks! | |
| 19:03:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-cells-v1 run with neutron https://review.openstack.org/549789 | |
| 19:43:59 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Localdisk https://review.openstack.org/549300 | |
| 19:58:31 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 20:06:42 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Localdisk https://review.openstack.org/549300 | |
| 20:19:57 | openstackgerrit | John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427 | |
| 20:21:40 | jgriffith | mriedem: ^^ I should have the unit tests finished up here when I get back to my desk after a bit, but just want to make sure you and others ( stvnoyes ) are good with the approach | |
| 20:21:48 | jgriffith | bb in a couple hours :( | |
| 20:31:07 | mriedem | ok | |
| 21:02:25 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 21:16:49 | mriedem | appears that CI is borked, | |
| 21:16:56 | mriedem | seeing RETRY_LIMIT on a most jobs on most patches | |
| 22:06:34 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: DiskAdapter parent class https://review.openstack.org/549053 | |
| 22:38:54 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | |
| 23:02:04 | openstackgerrit | John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427 | |
| 23:34:06 | openstackgerrit | John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427 | |
| 23:50:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make the nova-next job voting and gating https://review.openstack.org/549893 | |
| #openstack-nova - 2018-03-06 | |||
| 01:02:52 | Spazmotic | Well at least I got SOME sleep anyway.. Morning Novaers | |
| 01:12:47 | openstackgerrit | John Griffith proposed openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427 | |
| 01:31:49 | Kevin_Zheng | gibi Thanks alot for the update | |
| 01:32:22 | Kevin_Zheng | mriedem I got kicked everyday T_T | |
| 01:43:14 | Kevin_Zheng | gibi what about fault and traceback in notification? | |
| 03:13:57 | Spazmotic | I want to do more XenAPI devvvvv | |
| 06:19:40 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | |
| 06:51:09 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 07:50:19 | toabctl | melwitt, could you have a look at https://review.openstack.org/#/c/548572/ please? I think this is really something that we want backported to Queens... | |
| 07:57:28 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158 | |
| 08:15:31 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158 | |
| 08:24:00 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 08:50:29 | openstackgerrit | sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188 | |
| 08:55:29 | gibi | Kevin_Zheng: regarding fault and traceback in notification I still have a todo to compare the content of the legacy and versioned notifications and if they are different then do some digging why we did it that way | |
| 08:55:57 | gibi | Kevin_Zheng: and if they are different then I will file a bug against the versioned one | |
| 08:56:14 | Kevin_Zheng | gibi Thanks, our product seems to be also interested in those | |
| 08:57:25 | gibi | Kevin_Zheng: give me couple of days. I will try to progress with this later this week | |
| 08:57:37 | Kevin_Zheng | gibi cool, thanks alot | |
| 08:58:14 | gibi | Kevin_Zheng: meanwhile any review on https://review.openstack.org/#/c/502306/ is highly appreciated. | |
| 08:59:06 | Kevin_Zheng | gibi, sure, will do | |
| 09:07:22 | bauzas | good chilly morning | |
| 09:15:17 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: fix TypeError while get console log https://review.openstack.org/549182 | |
| 09:25:33 | openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | |
| 10:10:03 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | |
| 10:17:23 | Spazmotic | Morning UK nova team | |
| 10:17:40 | Spazmotic | Welcome home and all that jazz :) | |
| 10:18:02 | Spazmotic | UK/EU/Europe/Not America rather | |
| 11:25:21 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077 | |
| 11:27:50 | openstackgerrit | Pranab proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | |
| 11:38:34 | sean-k-m1 | stephenfin: https://review.openstack.org/#/c/546588/1 looking at this i think we are ok to merge this in master however back porting to ocata may be an issue if the bug with adding interfaces to the datapath is only fixed in ovs 2.6 | |
| 11:39:29 | stephenfin | sean-k-mooney: That's a good point. I wonder how old 2.6.0 is? | |
| 11:39:56 | stephenfin | sean-k-mooney: Although, to be fair, I think that bugfix is broken, given that it introduces another, potentially more serious bug | |
| 11:40:08 | sean-k-mooney | stephenfin: september 2016 | |