| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-14 | |||
| 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 ? | |
| 13:52:10 | sean-k-mooney | jangutter: do ye use dpdk to provide the vhos-user backend in the virtio forworder case or is that your own impelention of a vhost-user backend | |
| 13:52:14 | jangutter | sean-k-mooney: yep, something like that. And RH's qemu has got tons of backports, so the version in the field doesn't necessarily mean it isn't capable. | |
| 13:52:24 | mriedem | superdan: easy +W https://review.openstack.org/#/c/483917/ | |
| 13:53:07 | jangutter | sean-k-mooney: from what I understand we use the DPDK vhostuser library, it's just not worth it implementing it yourself. | |
| 13:53:34 | superdan | mriedem: done | |
| 13:53:44 | sean-k-mooney | jangutter: tell me about it. there kernel is worse is a "3.10" kernel with back ports for at leat 4.4 that broke some older versions of dpdk that work with kernel 3.10 just not theres | |
| 13:54:22 | jangutter | sean-k-mooney: the term we use is "frankenkernel". | |
| 13:54:36 | jangutter | sean-k-mooney: sometimes affectionately, sometimes not. | |
| 13:56:26 | sean-k-mooney | jangutter: its a mixed blessing. some of the backports i like. and ya dpdk is ment to be a bunch of reusable libary that you use to build you own apps so it makes sense that you reuse it for the vhost backend | |
| 14:00:39 | sean-k-mooney | jangutter: leakypipes stephenfin mriedem im going to send a email to the mailing list on monday with a list of patchs that i would like to see mearged in of-vif before the client libary freeze | |
| 14:01:19 | leakypipes | sean-k-mooney: that would be awesome, thank you sir! | |