| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-13 | |||
| 09:36:49 | stephenfin | bauzas: https://docs.openstack.org/nova/latest/i18n.html | |
| 09:36:57 | stephenfin | Log messages should not be translated. Any use of _LI(), _LW(), _LE(), _LC() are vestigial and will be removed over time. | |
| 09:37:11 | stephenfin | gcb: It's good - we figured it out :) | |
| 09:37:26 | bauzas | stephenfin: cool thanks | |
| 09:37:30 | gcb | stephenfin: yes, look at https://docs.openstack.org/oslo.i18n/latest/user/usage.html | |
| 09:37:42 | openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | |
| 09:37:54 | gcb | Starting with the Pike series, OpenStack no longer supports log translation. It is not necessary to add translation instructions to new code, and the instructions can be removed from old code. Refer to the email thread understanding log domain change on the openstack-dev mailing list for more details. | |
| 09:38:07 | gcb | stephenfin^^ | |
| 09:38:23 | stephenfin | gcb: Excellent. Thank you :) | |
| 09:40:39 | cdent | stephenfin: I'm not wise in the ways of policy, does this change (or the linked bug) make sense: https://review.openstack.org/#/c/481443/ | |
| 09:42:04 | stephenfin | cdent: I was purposefully leaving that until if/when johnthetubaguy was back. Apparently we were removing all default rules, so I'd imagine that will have some bearing on that | |
| 09:43:03 | cdent | stephenfin: ah okay, so the burden of setting the default is on the deployment? | |
| 09:44:02 | stephenfin | cdent: Wait, no - that was _discoverable_ policies (because they no longer make sense) | |
| 09:44:40 | stephenfin | Looks like claudiub added a load of those default rules. Maybe he can review https://review.openstack.org/#/c/481443/ and see if it makes sense? | |
| 09:56:36 | openstackgerrit | Rikimaru Honjo proposed openstack/nova master: compute: Do not create bdms before successfully reserving a volume https://review.openstack.org/290793 | |
| 10:07:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Always set MTU when plugging devices https://review.openstack.org/408781 | |
| 10:12:10 | pallavi_ | Hi , I have done ubuntu manual installation for ocata release. I have launched instance. But accessing to the instance is not happening.?? Can anyone please help me out to figure it out | |
| 10:17:00 | claudiub | stephenfin: replied. that patch LGTM. | |
| 10:18:30 | stephenfin | claudiub: Thanks :) If you're happy, so am I | |
| 10:23:27 | claudiub | stephenfin: if you're happy and you know it, clap your hands! *clap* *clap*. :D | |
| 10:23:49 | cdent | thanks claudiub | |
| 10:28:11 | gibi | bauzas: hi! There is a small test and doc only patch that needs a second +2. Could you check it please? https://review.openstack.org/#/c/480469/ | |
| 10:28:22 | bauzas | gibi: roger this. | |
| 10:28:43 | gibi | bauzas: thanks! | |
| 10:35:51 | sean-k-mooney | jangutter: hi the bridge name is currently set here https://github.com/openstack/neutron/blob/8326a65a5e3c70c66a5297c618d0202c9395cc3f/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L94-L105 | |
| 10:37:03 | sean-k-mooney | jangutter: so in you mech driver if you just overrided that fuction you can pass the bridge name via the vif binding details. | |
| 10:38:40 | sean-k-mooney | jangutter: this then get populated into the os-vif network object here https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/os_vif_util.py#L225-L226 | |
| 10:40:17 | sean-k-mooney | jangutter: and its used in os-vif here: https://github.com/openstack/os-vif/blob/4a8e07f6c502e5e3b99db9ba543a501fbb138596/vif_plug_ovs/ovs.py#L97 and anywayer else we need the bridge name | |
| 10:43:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove plug_ovs_hybrid, unplug_ovs_hybrid https://review.openstack.org/483030 | |
| 10:43:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove useless test https://review.openstack.org/483031 | |
| 10:44:00 | bauzas | stephenfin: gibi: something got messed up with our man pages | |
| 10:44:10 | stephenfin | ? | |
| 10:44:21 | bauzas | eg. <http://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v2 is pointing nowhere | |
| 10:44:32 | stephenfin | Yeah, that's a pbr bug | |
| 10:44:43 | stephenfin | *Oh, that's like that pbr bug | |
| 10:44:57 | stephenfin | *likely | |
| 10:44:58 | stephenfin | dammit | |
| 10:45:25 | stephenfin | bauzas: https://bugs.launchpad.net/pbr/+bug/1702872 | |
| 10:45:26 | openstack | Launchpad bug 1702872 in PBR "pbr + Sphinx 1.6+ ignores the '[sphinx_build] builders' setuptools option" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane) | |
| 10:46:17 | stephenfin | I'll tack a fix onto the end of the https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova series, once pep8 fix patch goes in (it needs a rebase again) | |
| 10:46:23 | bauzas | stephenfin: I just +2d https://review.openstack.org/#/c/478472/7 because it workarounds the issue | |
| 10:46:47 | bauzas | stephenfin: I'm not sure it should be a cli/ section, but meh | |
| 10:47:10 | bauzas | at least it's understandable that nova-manage and nova-status are CLI tools | |
| 10:47:53 | stephenfin | Yeah, sdague said the same thing. However, we've no where else to put those man pages | |
| 10:48:05 | stephenfin | I wanted that section called 'man' instead, but didn't make my case strongly enough :( | |
| 10:55:17 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 11:13:54 | openstackgerrit | Chris Dent proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | |
| 11:54:14 | gibi | cdent: thank you for your detailed answer on my placement question on the ML. It helps a lot | |
| 11:57:00 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | |
| 11:57:31 | jangutter | sean-k-mooney: hmmm, I still get: ERROR os_vif NotImplementedError: Cannot load 'bridge' in the base class | |
| 11:58:28 | sean-k-mooney | jangutter: how are you accessing the bridge? via the network object? | |
| 11:59:33 | jangutter | sean-k-mooney: vif.network.bridge -> I think that might not be populated for vhostuser | |
| 11:59:49 | sean-k-mooney | jangutter: if you do a neutron port show or openstack port show do you see the bridge correctly set in the vif_binding_details | |
| 12:00:37 | sean-k-mooney | jangutter: it should be populated or it means that vlan aware vms dose not work for vhost-user and this is a nova bug we need to fix | |
| 12:01:55 | sean-k-mooney | jangutter: yep you are right.... https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/neutronv2/api.py#L2157-L2179 | |
| 12:02:17 | sean-k-mooney | jangutter: do you want to submit the patch or will i | |
| 12:02:27 | jangutter | sean-k-mooney: binding_vif_details -> ovs_hybrid_plug='False', port_filter='True' | |
| 12:03:40 | jangutter | sean-k-mooney: depends on what will get it merged faster :-) | |
| 12:03:40 | sean-k-mooney | actully no i misread that https://github.com/openstack/nova/blob/d955eb1db753f191acb02eb9758080329da5b866/nova/network/neutronv2/api.py#L2172-L2173 it is saved for vhost-user | |
| 12:04:21 | sean-k-mooney | jangutter: so ya the bug is in the neutron ml2 driver yuou are missing the bridge key in the vif_binding_details | |
| 12:05:21 | jangutter | sean-k-mooney: Aah, and the workaround is to add: elif vif_type == network_model.VIF_TYPE_AGILIO_OVS to that list. | |
| 12:05:42 | sean-k-mooney | jangutter: you really should not have a new vif_type | |
| 12:06:20 | sean-k-mooney | there will not be a new vif object in os-vif and you should not need one for nova or neutron | |
| 12:06:52 | sean-k-mooney | the vif type shoud just be VIF_TYPE_VHOSTUSER | |
| 12:07:14 | jangutter | sean-k-mooney: I am using VIF_TYPE_VHOSTUSER for the actual plugging. | |
| 12:07:28 | sean-k-mooney | jangutter: you should also be using it for port binding | |
| 12:08:11 | sean-k-mooney | we are trying to stop using vif types as a flag for how to plug interfaces and retrun them to being a statemant about what type if interface you want | |
| 12:09:03 | sean-k-mooney | or in otherwords the vif_type is an abstion for the hyperviser interface type and is not related to how that interface is pluged to the network backend at all | |
| 12:10:06 | jangutter | sean-k-mooney: yeah. so one thing that kinda breaks the abstraction is that the "forwarder" is actually a kind of translation layer on top of the hypervisor. | |
| 12:10:29 | jangutter | sean-k-mooney: and introducing another layer just leads to madness. | |
| 12:10:29 | sean-k-mooney | jangutter: that is captured by the vnic_type | |
| 12:11:00 | jangutter | sean-k-mooney: yep. let me see if I can figure out if I'm doing the wrong thing in my ML2 driver. | |
| 12:12:50 | jangutter | sean-k-mooney: (facepalm) network_model.VIF_DETAILS_VHOSTUSER_OVS_PLUG | |
| 12:13:43 | sean-k-mooney | jangutter: ya you might need to set that | |
| 12:14:14 | sean-k-mooney | that is a legacy thing that should go away when we can specify the plugin to use in the vif details | |
| 12:17:19 | foutatoro | hi all, I would like to know how to deploy hypervisors VMware vSphere and Hyper-V on compute nodes ? | |
| 12:18:14 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:18:41 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | |
| 12:19:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance. |
|
| 12:30:54 | cdent | gibi: glad it was of some use, looking forward to seeing more of what you're doing | |
| 12:32:05 | stephenfin | sean-k-mooney, jaypipes: Fancy taking a look at this? Easy +1/2, imo https://review.openstack.org/#/c/483030/ | |
| 12:34:09 | toabctl | is there a way to list hosts which are not attached to a cell (v2) ? | |
| 12:38:48 | sean-k-mooney | stephenfin: sure, looking at it now | |
| 12:42:57 | stephenfin | sean-k-mooney: Cheers. Out of curiosity, is the expectation that all the old, non-os-vif plugging/unplugging code would eventually be replaced by code that harnesses vendor-developed os-vif plugins? | |
| 12:43:04 | stephenfin | e.g. All of these https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L599-L764 would be replaced by these https://github.com/openstack/nova/blob/master/nova/network/os_vif_util.py#L346-L403 | |
| 12:44:09 | openstackgerrit | Merged openstack/nova master: doc: Enable pep8 on doc generation code https://review.openstack.org/480469 | |
| 12:44:33 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/483337 | |
| 12:47:03 | sean-k-mooney | stephenfin: yes that was the original intention was to add an os-vif pluging for all the old types. modify neutron ml2 driver to pass plugin name in vif details and then eventurlly remove os_vif_util when we finally have neutron retrunin serialised os-vif objects from its api | |
| 12:50:54 | stephenfin | sean-k-mooney: OK, so regarding the comment here https://review.openstack.org/#/c/483030/2/nova/virt/libvirt/vif.py ... | |
| 12:52:13 | stephenfin | would we be submitting a bug against os-vif, or would that be a separate plugin maintained in its own repo by the floodlight team? | |
| 12:52:15 | sean-k-mooney | stephenfin: yep i was suggesting opening a bug against https://github.com/openstack/networking-bigswitch | |
| 12:52:31 | stephenfin | ahhh | |
| 12:54:12 | sean-k-mooney | stephenfin: IVS is a bigswitch networks competitor to ovs . it is the only thing that uses the IVS vif type https://github.com/openstack/networking-bigswitch/blob/master/networking_bigswitch/plugins/ml2/drivers/mech_bigswitch/driver.py#L422-L431 | |
| 12:54:57 | stephenfin | sean-k-mooney: Yeah, so they'd "own" the repo. It would be merged into 'openstack/os-vif'? | |
| 12:56:01 | sean-k-mooney | stephenfin: no its an non reference backend so im proposing that we wright an os-vif plugin for them and proposes it to there repo then deprecate the nova code and remove it once it is merged | |
| 12:56:32 | sean-k-mooney | stephenfin: alternitively we can create our only os-vif-ivs repo and that we own for this backend | |
| 12:57:18 | sean-k-mooney | it depend on whetere you want to tie the packaging of the plugin to the packaging of the ml2 driver which arguably you might want to keep seprate | |
| 13:00:35 | stephenfin | We'll let them make that decision, I guess | |
| 13:00:57 | stephenfin | sean-k-mooney: ...though IMO they're separate products so they should be. We'll see | |
| 13:01:00 | stephenfin | I'll file that bug now | |