Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-05
16:57:25 mriedem gibi: cool, thanks - do you plan on getting 'formal' approval in the nova team meeting this week?
16:57:32 mriedem mel is on vacation this week btw
16:57:49 gibi mriedem: sure, I will bring it up on the meeting
16:57:49 mriedem although we already discussed it in person,
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

Earlier   Later