| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-04 | |||
| 14:26:16 | stephenfin | I should probably keep the other one running with an /away message, but eh :) | |
| 14:26:35 | jangutter | sean-k-mooney: So for older kernels we need a fallback mechanism to do the VF-to-repr mapping. | |
| 14:26:35 | gmann_ | stephenfin: nice | |
| 14:27:39 | sean-k-mooney | jangutter: you would not happen to be useing vlan for that would you? | |
| 14:28:24 | openstackgerrit | Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 14:28:27 | sean-k-mooney | jangutter: https://review.openstack.org/#/c/456481/1 this is the nove patch that is out standing for the legacy version correct | |
| 14:28:40 | stephenfin | gmann_: Could you just put an anchor into https://raw.githubusercontent.com/openstack/nova/master/api-ref/source/os-quota-sets.inc ? | |
| 14:28:48 | stephenfin | Assuming one isn't automatically inserted for you | |
| 14:29:26 | jangutter | sean-k-mooney: yep, that's the old stuff I'm refactoring now. | |
| 14:29:32 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 14:29:39 | stephenfin | e.g. above the 'List Default Quotas For Tenant' title, add '.. _list-default-quotas-for-tenant:' | |
| 14:29:56 | stephenfin | again, assuming ':ref:`list-default-quotas-for-tenant`' doesn't work already | |
| 14:29:59 | jangutter | sean-k-mooney: the external os-vif plugin will make that much cleaner. | |
| 14:30:25 | stephenfin | I know Sphinx can automatically insert references for headers, but I'm not sure if it's enabled for that build | |
| 14:30:56 | gmann_ | stephenfin: humm, ok. let me check otherwise ll add anchor | |
| 14:31:26 | stephenfin | gmann_: Do please. The advantage of doing this is that it always works while the link is valid, and the build fails if not | |
| 14:32:09 | jangutter | sean-k-mooney: https://review.openstack.org/#/c/456481/1/nova/network/neutronv2/api.py <--- this bit is obsoleted by https://review.openstack.org/#/c/398265/14/nova/network/neutronv2/api.py | |
| 14:32:40 | jangutter | sean-k-mooney: I think... | |
| 14:33:25 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: fix read the representor phys_port_name https://review.openstack.org/480213 | |
| 14:33:43 | jangutter | sean-k-mooney: and this is the final piece we need: https://review.openstack.org/#/c/398265/14/nova/virt/libvirt/vif.py | |
| 14:34:53 | sean-k-mooney | jangutter: well in terms of setting the ovs-bridge to attach the representor to yes | |
| 14:35:38 | sean-k-mooney | the code you referenced though in the original patch appears to be constructing a pci request | |
| 14:40:06 | jangutter | sean-k-mooney: agh, I forgot. VNIC_VIRTIO_FORWARDER also needs to be added to VNIC_TYPES_SRIOV. | |
| 14:40:31 | jangutter | sean-k-mooney: that triggers the pci_request for it. | |
| 14:40:35 | sean-k-mooney | jangutter: well yes and no | |
| 14:40:49 | bauzas | alex_xu: looks like I'm fortunate, I hitted the race | |
| 14:41:11 | sean-k-mooney | it does not have to be pci back it will be in your case but for intel i was hoping to use that vnic type without sriov vf | |
| 14:41:34 | bauzas | alex_xu: the good thing is that I had a working run, and then a failed one, so I just need to see how to bisect that | |
| 14:42:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938 | |
| 14:42:16 | sean-k-mooney | at lest in some cases. i had not fully mapped out that but yes if we limit our selves to the netronome usecase you will need a pci device allocated | |
| 14:42:36 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 14:44:17 | sean-k-mooney | jangutter: ignore the intel stuff for now that si still in flux and i cant going into detail anyway at this time | |
| 14:44:18 | jangutter | sean-k-mooney: yep, it's not necessarily that all forwarders will have a 1:1 mapping either. | |
| 14:44:51 | jangutter | sean-k-mooney: maybe this means a new os-vnic library.... | |
| 14:46:08 | sean-k-mooney | sean-k-mooney: this is why i want ovs to track this and feed that info from neutron into the placement api so that we can handel these details in the ml2 driver and os-vif dirver generically | |
| 14:47:15 | bauzas | alex_xu: lots of fun, the working test has 309 tests, the failing test 310 :p | |
| 14:47:37 | jangutter | sean-k-mooney: yep, don't worry, you're not the only one thinking in that direction. | |
| 14:50:04 | jangutter | sean-k-mooney: ok, thanks very much. I'll get back on our TC interfaces post-haste and our refactored legacy driver should be done pretty soon. | |
| 14:50:43 | bauzas | alex_xu: I'm surprised, the test which fails wasn't run by the working test | |
| 14:53:33 | bauzas | actually nevermind | |
| 14:58:33 | moshele | sean-k-mooney: hi | |
| 14:59:21 | moshele | sean-k-mooney: is my comment https://review.openstack.org/#/c/398265/ answer your question ? | |
| 15:00:02 | moshele | sean-k-mooney: I have to go let talk tomorrow | |
| 15:21:29 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 16:41:28 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Remove custom autodoc implementation https://review.openstack.org/480262 | |
| 16:44:34 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Reset the traits sync flag in the placement fixtures https://review.openstack.org/480198 | |
| 16:45:48 | bauzas | stephenfin: guessing it's you :p | |
| 16:46:45 | openstack | Launchpad bug 1702304 in OpenStack Compute (nova) "MISC_SHARES_VIA_AGGREGATE not found still show up in the functional test" [High,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 16:46:45 | bauzas | stephenfin: don't know if you remember our convo with alex_xu but I found the root cause for https://bugs.launchpad.net/nova/+bug/1702304 so I provided a new PS https://review.openstack.org/#/c/480198/2 | |
| 17:10:20 | vdrok | hi folks, super-easy patch if anyone around -- https://review.openstack.org/480167 | |
| 18:20:00 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | |
| 18:20:02 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | |
| 21:18:56 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 21:19:50 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | |
| 21:19:51 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 21:19:51 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | |
| 23:45:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227 | |
| 23:49:01 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463 | |
| #openstack-nova - 2017-07-05 | |||
| 00:05:17 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 00:39:43 | Jeffrey4l | hi guys, is there any consumer for nova versioned notification messages? | |
| 01:29:32 | openstackgerrit | Merged openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |
| 01:35:15 | Kevin_Zheng | Jeffrey4l: Searchlight is going to consume those | |
| 01:37:36 | Jeffrey4l | Kevin_Zheng, is searchlight already finished the work? | |
| 01:37:57 | Kevin_Zheng | very close | |
| 01:37:59 | Jeffrey4l | and is there any other project ( ceilometer?) can consume versioned message? | |
| 01:39:48 | Kevin_Zheng | a good idea, bu no one worked on it in ceilometer | |
| 02:27:45 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | |
| 02:34:34 | Nel1x | what consecuences using admin_auth_url => port 5000? and not 35537 in nova => [neutron] | |
| 03:08:23 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: copy image id to image metadata from volume https://review.openstack.org/319905 | |
| 06:23:08 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | |
| 06:35:05 | masber | hi, is this the right place to ask about the integration of nova with sr-iov? | |
| 06:46:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix arguments in calling _delete_nic_metadata https://review.openstack.org/480369 | |
| 07:12:39 | openstack | Launchpad bug 1702304 in OpenStack Compute (nova) "MISC_SHARES_VIA_AGGREGATE not found still show up in the functional test" [Critical,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 07:12:39 | bauzas | alex_xu: stephenfin: follow-up on https://bugs.launchpad.net/nova/+bug/1702304 with https://review.openstack.org/#/c/480198/ needing reviews | |
| 07:12:55 | alex_xu | bauzas: cool, will check soon | |
| 07:13:17 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP https://review.openstack.org/480379 | |
| 07:13:26 | alex_xu | bauzas: ^ another bug fix for placement | |
| 07:13:46 | bauzas | alex_xu: ack, looking | |
| 07:14:22 | gibi | Jeffrey4l: besides searchlight I think watcher also consumes nova versioned notifications | |
| 07:15:27 | gibi | Jeffrey4l: for example https://github.com/openstack/watcher/blob/3a3ce0268bbb900c0ca3fe40379beda574a4d548/watcher/decision_engine/model/notification/nova.py | |
| 07:35:00 | Jeffrey4l | gibi, roger. thanks a lot. | |
| 07:47:43 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392 | |
| 07:54:53 | openstackgerrit | Chason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme https://review.openstack.org/477757 | |
| 08:13:00 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437 | |
| 08:17:00 | stephenfin | bauzas: https://review.openstack.org/#/c/480198 reviewed. Moving onto the next one now | |
| 08:17:24 | stephenfin | wait - the next one's the same :) | |
| 08:17:45 | bauzas | what's the next one ? :p | |
| 08:18:09 | stephenfin | you pinged me last night to review https://review.openstack.org/#/c/480198, then again this morning | |
| 08:18:49 | bauzas | ha-ah | |
| 08:19:52 | stephenfin | Damn, this place is quiet without ze yanks around - I can actually read the scrollback on one page :O | |
| 08:24:35 | gibi | bauzas: hi! there is a doc generation improvement that only needs a second +2. If you have time could you take a look? https://review.openstack.org/#/c/475349/ | |
| 08:24:50 | bauzas | gibi: sure | |
| 08:25:27 | gibi | bauzas: thanks! | |
| 08:27:46 | bauzas | gibi: ack my comment and I +W https://review.openstack.org/#/c/475349/1 | |
| 08:28:05 | bauzas | based on takashi's valid point | |
| 08:35:14 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437 | |
| 08:35:47 | stephenfin | bauzas: I don't think you +W'd that yet, if you plan to? | |
| 08:36:20 | bauzas | stephenfin: because I'd like to see gibi adding a follow-up | |
| 08:36:37 | stephenfin | aha - gotcha | |
| 08:37:37 | stephenfin | bauzas: or gibi could just fix it in the same patch, seeing as we're both online | |