Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-05
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
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?

Earlier   Later