| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-05 | |||
| 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 | |
| 11:41:08 | sean-k-mooney | well both bugs are resolved with a vm hard reboot so i dont think this new bug is more serious | |
| 11:41:26 | sean-k-mooney | they both resolt in the same user observable behavior | |
| 11:41:32 | stephenfin | sean-k-mooney: So Newton was released 6 October 2016 (per Wikipedia). I wonder if it's fair to say anyone using...Ocata and upwards would be using OVS 2.6+ ? | |
| 11:41:51 | stephenfin | We could include a release note to that effect (do we do release notes for os-vif?) | |
| 11:42:06 | sean-k-mooney | i belive neutron has a minium version for ovs as we have for libvirt | |
| 11:42:37 | sean-k-mooney | well what im considering is should we confige this change to only when plugging a dpdkvhostuser port | |
| 11:43:19 | sean-k-mooney | s/confige/confine/ | |
| 11:43:23 | stephenfin | Hmm, that could be possible | |
| 11:43:47 | stephenfin | But if we do, I'd like to remove that check immediately afterwards and not backport that | |
| 11:44:13 | stephenfin | That could be discussed separately though | |
| 11:44:17 | sean-k-mooney | stephenfin: well no i would like it in the backport as the old bug was specifically for kernel ovs | |
| 11:45:02 | stephenfin | You mean you would like the check to be included in the backport? | |
| 11:45:03 | sean-k-mooney | when https://github.com/openstack/nova/commit/33cc64fb817 was done this fuction did not support vhost user | |
| 11:45:09 | stephenfin | If so, that's what I mean :) | |
| 11:45:24 | stephenfin | Add the check, backport that patch, then remove that check in a follow-up patch | |
| 11:45:50 | sean-k-mooney | ya so in the backport i think we should have the check so that we only change the behavior for vhost user ports | |
| 11:45:53 | stephenfin | ...given that we don't need it anymore (since OVS 2.6.0), and using '--if-exists' seems far saner to me | |
| 11:46:05 | stephenfin | OK, if that's possible then I've no objections | |
| 11:46:09 | stephenfin | sahid: Thoughts? ^ | |
| 11:46:56 | sean-k-mooney | stephenfin: did you mean '--may-exists' seams saner? | |
| 11:47:03 | sean-k-mooney | for master | |
| 11:47:54 | stephenfin | I did 🙈 | |
| 11:48:22 | stephenfin | i.e. don't special case dpdkvhostuser on master | |