Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-14
07:23:34 bauzas stephenfin: gentle reminder for other folks that today is a French holiday in case people look at me
07:26:53 alex_xu bauzas: enjoy
07:33:32 gmann bauzas, nice. take some rest :)
07:46:43 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
08:08:57 openstackgerrit Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
08:16:40 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
08:36:43 gmann stephenfin, done, i only concern about notification one ( gibi can tell more tough ). https://review.openstack.org/#/c/478484/10
08:40:05 gibi gmann, stephenfin: the notification doc has multiple purpose.
08:40:46 gibi gmann, stephenfin: It describe how to add new notification so it helps contribution but it also talks about how now work internally (like versioning) as well as what are the existing notifications
08:40:49 gmann gibi, from user perspective also ? i mainly saw more implementation wise
08:41:03 gmann gibi, ohk
08:41:28 gibi gmann: the bottom of the doc has a list of existing notification with samples as such so it is for notification consumers too
08:41:31 gmann gibi, and version things are helpful for users like horizon etc?
08:41:44 gmann yea
08:42:01 gmann gibi, then we should divide the doc?
08:42:02 gibi gmann: yes. Not horizon today but searchlight and watcher as far as I kknow
08:42:18 gibi gmann: it would be nice to split up the doc but that should not be part of the doc migration
08:43:16 gmann gibi, yea, so IMO current one should go under contribution and then we extract consumer related part under user etc
08:43:21 gmann stephenfin, ^^
08:44:02 gmann because under reference it goes as ref and we also donot maintain those much :) but i may be wrong
08:44:23 gibi gmann: there is no single good place for the current doc so it is fine by me to put it under any section :)
08:50:21 gibi gmann, stephenfin: I summarized my view as a comment in the review
08:50:40 gmann gibi, thanks, that help
09:15:45 jangutter sean-k-mooney: would you ping me if you have a sec?
09:45:37 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
09:49:25 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
09:55:08 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
09:58:53 openstackgerrit yuanyue proposed openstack/nova master: Remove ports on create_port timeout during instance build https://review.openstack.org/483888
10:01:38 openstackgerrit jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013
10:06:59 openstackgerrit jichenjc proposed openstack/nova master: add description for migrate on 'resize' section https://review.openstack.org/481483
10:24:26 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
10:24:26 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
10:24:27 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:27:14 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
10:27:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:28:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774
10:29:03 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
10:33:31 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
11:30:48 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902
11:59:37 openstackgerrit NidhiMittalHada proposed openstack/nova master: Nova treats hostname as pattern https://review.openstack.org/474949
12:02:43 sean-k-mooney jangutter: hi sorry just seeing this now
12:04:05 openstackgerrit Manas Mandlekar proposed openstack/nova master: An unexpected failure during live migration leaves the insatnce to migrating state forever. https://review.openstack.org/483911
12:13:18 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
12:13:53 jangutter sean-k-mooney: no problem, I just got back from lunch.
12:14:34 jangutter sean-k-mooney: so, is os-vif still open for one portbinding?
12:16:04 openstackgerrit Andrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816
12:20:54 sean-k-mooney jangutter: what do you mean by port binding?
12:21:36 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
12:22:05 sean-k-mooney jangutter: do you mean a new VIF_TYPE or an extention to the one of the plugins or A new port Profile. you can still submit a new Algrio portbinding to os-vif untill the end of next week i belive
12:22:38 sean-k-mooney /portbinding/port profile/
12:25:25 sean-k-mooney jangutter: looking at https://releases.openstack.org/pike/schedule.html the non client libary release freeze is next week which is the deadline that you have to meet to have changes to os-vif consumed in pike
12:28:08 jangutter sean-k-mooney: I've just finished up testing with one extra portprofile object added in upstream os-vif
12:28:47 jangutter sean-k-mooney: let me clean up the code a bit and submit it for your perusal...
12:30:22 sean-k-mooney jangutter: cool ill keep an eye out for the patch message in irc and take a lookin when you submit it. adding port profile objects is not a controversal addtion.
12:32:11 openstackgerrit Merged openstack/nova master: Use oslo.polcy DocumentedRuleDefault https://review.openstack.org/472634
12:43:50 openstackgerrit melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700
12:43:52 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
12:43:53 openstackgerrit melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699
12:43:56 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
12:43:56 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
12:44:02 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
12:44:09 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
12:44:09 openstackgerrit melanie witt proposed openstack/nova master: Default reservations=None in Cells v1 and conductor APIs https://review.openstack.org/483917
12:51:54 openstackgerrit Jan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor https://review.openstack.org/483921
13:00:39 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
13:02:38 openstackgerrit Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
13:16:52 mriedem who wants to push this through? https://review.openstack.org/#/c/483133/
13:16:58 mriedem it's at the bottom of the counting quotas series, has a +2
13:17:11 mriedem +5, -18 of fun
13:19:59 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559
13:26:37 jangutter sean-k-mooney, jaypipes: I've done the cleanups and, surprisingly, the end-to-end system still works.
13:27:40 jangutter sean-k-mooney, jaypipes: I haven't seen how to get past introducing the VIF type and adding a check for the bridge name in nova/network/neutronv2/api.py
13:28:20 leakypipes jangutter: I haven't yet properly caffeinated :)
13:29:35 jangutter leakypipes: It's Friday. At this point in the week, people can get their yearly caffeine dose from me by briskly walking past me.
13:31:15 sean-k-mooney jangutter: the way around it is to implemenet the mechanisium for passing the plugin name form neutron via the vif binding details but that needs https://review.openstack.org/#/c/441590/ to be completed and the nova change to pass the host info object to neutron and the neutron changes to consume that and pass back the plugin and os-vif objects. i.e. it wont be done till queens
13:32:04 jangutter sean-k-mooney: aaaah!
13:33:21 sean-k-mooney jangutter: its discribed here https://review.openstack.org/390512 more or less. i had planned on doing a poc of this in pike but have not had time. im still going to try and do a poc before the PTG but we will see how it goes
13:34:16 leakypipes jangutter: :)
13:34:47 leakypipes figleaf: thx for the quick reviews on that series. I'll address those comments shortly.
13:35:19 jangutter sean-k-mooney: "Improve Nova-Neutron port binding through negotiation"... that title is a masterstroke.
13:35:39 leakypipes jangutter: that's one word to describe that...
13:36:40 sean-k-mooney jangutter: well its an attempt to sovle the fact that today unless you are carful it posible to deploy a fully working nova and a fully working neutron that are configure in such a way that they cannont work together
13:39:08 sean-k-mooney jangutter: the intent was to have nova/os-vif validate what binings can be supported by the hypervior and pass that info to neutron where neutron would compare that to what can be supported by the network backend and negociate a common binding by takeing the intersection of the two and selecting the first element of that intersect ordered by the values pass form nova
13:41:11 sean-k-mooney jangutter: anyway for your usecase i think we can introduce a new neutron VIF_TYPE but reuse the existing os-vif vif objects though i hope this will be the last or one of the last vif types we need to add to nova
13:42:28 jangutter sean-k-mooney: yeah, I had really wanted to have almost nothing touch the core, but it looks like there's always just one more bit left.
13:44:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Dump tracked version notifications when swap volume tests fail https://review.openstack.org/483939
13:44:13 sean-k-mooney jangutter: actully could we just use VIF_TYPE vhost-user + Vnic_type virtio-forworder instead of the new VIF_type?
13:44:33 sean-k-mooney for the virtio_forworder case that is
13:44:37 mriedem melwitt: one of your patches failed a versioned notification swap volume functional test, unrelated, so https://review.openstack.org/#/c/483939/ is to help debug that
13:45:46 sean-k-mooney for the sriov case it would be vif_type ovs + vnic type direct as with melonox but no wew are missing passing the plugin for that to work...
13:45:52 jangutter sean-k-mooney: yeah, but how do you trigger the third-party OS-VIF plugin?
13:46:30 sean-k-mooney jangutter: yep alway one thing missing hehe VIF_TYPE is the only way until we have neutron pass the plugin
13:46:40 openstackgerrit Stephen Finucane proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757
13:46:44 sean-k-mooney jangutter: or rather plugin name
13:48:14 jangutter sean-k-mooney: so I've also tested the vhost-user path, the ML2 plugin can tell Nova where it wants the socket and whether it's client or server mode.
13:49:42 melwitt mriedem: cool. looks like I have two legit func test failures on my counting instances patch too. gdi
13:49:54 sean-k-mooney jangutter: cool that will align your impentation with how dpdk alreay work when working with vhost user. do ye have support for server mode by the way? i would suggest making that the default if ye have
13:50:27 jangutter sean-k-mooney: I believe we do: it requires a fairly recent DPDK and QEMU I believe.
13:51:15 sean-k-mooney jangutter: it works with old version of qemu 2.1+ but you dont have reconnect until qemu 2.7 if i rember correctly
13:52:10 mriedem bauzas: you want to send it in https://review.openstack.org/#/c/483133/1 ?

Earlier   Later