| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-05 | |||
| 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 | |
| 11:48:23 | openstackgerrit | Pranab proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | |
| 11:50:47 | sean-k-mooney | stephenfin: yes agreed. also i would like to tackover rodolfo patches to add support for using the python binding so that we can stop using the commandline clients | |
| 11:51:58 | stephenfin | sean-k-mooney: +1 from me. I saw an email on openstack-dev about that earlier this week | |
| 11:52:14 | stephenfin | I'll review...probably not this week, but definitely next week | |
| 11:54:13 | sean-k-mooney | ill leave a comment regarding the backport and +w the patch for master. | |
| 11:55:22 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | |
| 12:25:18 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Don't wait for direct vnic plugged events during a reboot https://review.openstack.org/550046 | |
| 12:25:38 | lyarwood | stephenfin: ^ re the downstream ping | |
| 12:34:49 | sahid | sean-k-mooney, stephenfin - i don't think it make sense to have a special condition. I think that old fix was merged without real issue but in prevention | |
| 12:35:35 | sean-k-mooney | the old bug happens if a vm tap device does not exist during ovs startup | |
| 12:36:04 | sean-k-mooney | this happens if libvirt has not finished spawning the vms before ovs starts | |
| 12:36:42 | sean-k-mooney | so there is a race between libvirt/qemu starting the instance and ovs starting for ovs <2.6 | |
| 12:36:45 | sahid | did you noticed that for newton only hybrid vif was supported ? and for hybrid vif we have a special condition to check whether the hybrid vif already exists and so in that case we do not execute that command again | |
| 12:38:00 | sean-k-mooney | sahid: in the non hybrid case the vif is plugged by libvirt not os-vif | |
| 12:39:10 | sahid | can you point me the code where that issue could happen? | |
| 12:39:58 | sean-k-mooney | sahid: its a race condtion between ovs and vm tap devices being created its not an issue in the code itself | |
| 12:40:38 | sahid | sean-k-mooney: yes but we do not create any tap device when restarting nova-compue service | |
| 12:40:41 | sahid | that is what i mean | |
| 12:42:20 | sean-k-mooney | correct the "bridge|WARN|could not open network device tap2cf7dbad-9d (No such device)" only happens if the ovs-db has an entry for an interface that does not currently exist in the root namespace when ovs-vswitchd is starting | |
| 12:42:48 | sean-k-mooney | the fact the interface is prefixed with tap mean that hybrid plug is not in use meaning libvirt is pluging the interface not os-vif | |
| 12:43:33 | sean-k-mooney | so this will happen if ovs start before libvirt starts the vm after a reboot | |
| 12:43:40 | sean-k-mooney | but only on ovs <2.6 | |
| 12:44:26 | sahid | sean-k-mooney: after a reboot of what? | |
| 12:44:40 | sean-k-mooney | sahid: the host | |
| 12:46:11 | sahid | how possible that happen? | |
| 12:46:42 | sahid | the tap device would have been loaded by the kernel before ovs and libvirt | |
| 12:46:53 | sahid | then i don't understnad why libvirt is in relation here? | |
| 12:47:04 | sean-k-mooney | basicaly when ovs trys to add all ports in the ovsdb to the ovs kernel datapath it skips any it does not find, e.g. not running vms. the ovs revalidtor tread does not redo this check untill another port is added to ovs before 2.6 | |
| 12:47:17 | sean-k-mooney | sahid: tap devices are not persited across reboots | |