| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-26 | |||
| 12:03:05 | openstackgerrit | Merged openstack/nova master: Adapter raise_exc=False by default https://review.openstack.org/577437 | |
| 12:10:17 | openstackgerrit | Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023 | |
| 12:12:47 | jaypipes | frickler: no worries mate :) | |
| 12:13:33 | jaypipes | efried: morning. hey, so I'm fully back online now. batteries at 100%. | |
| 12:13:52 | jaypipes | efried: I missed the scheduler meeting yesterday as I was on a plane. | |
| 12:14:00 | jaypipes | efried: anything pressing to discuss? | |
| 12:32:42 | jangutter | PSA: when in a meeting with jaypipes, be extra extra careful. He will coerce you into saying the phrase 'a bag of dicts', and it will take a long time to explain python to HR. | |
| 12:33:03 | gibi | :) | |
| 12:33:05 | jaypipes | jangutter: *cough* that's "random bag of dicts", sir. | |
| 12:33:41 | jaypipes | jangutter: also see: why to buy a good pair of headphones while working in an office :P | |
| 12:34:15 | jangutter | jaypipes: hence your subtle technique of getting me to say it. | |
| 12:35:48 | jaypipes | jangutter: you've figured me out. | |
| 12:37:36 | openstackgerrit | Merged openstack/nova master: fix PowerVM get_bootdisk_path docstring https://review.openstack.org/576977 | |
| 12:37:43 | openstackgerrit | Merged openstack/nova master: api-ref: Fix parameters about trusted certificate IDs https://review.openstack.org/576046 | |
| 12:37:50 | openstackgerrit | Merged openstack/nova master: Update links in README https://review.openstack.org/574644 | |
| 12:41:38 | frickler | jaypipes: well actually I am a bit worried about this bug having been able to pass testing. makes me wonder whether execute() should get mocked/otherwise trapped globally in unit tests | |
| 13:00:38 | efried | jaypipes: Good morning, welcome back. | |
| 13:01:49 | openstack | Launchpad bug 1777591 in OpenStack Compute (nova) "‘limit’ in allocation_candidates where sometimes make fore_hosts invalid" [High,In progress] - Assigned to xulei (605423512-j) | |
| 13:01:49 | efried | jaypipes: We mainly talked about https://bugs.launchpad.net/nova/+bug/1777591 -- see the bug for what we concluded, and speak up if you have a dissenting stance. | |
| 13:06:39 | jaypipes | efried: ahhh, force_hosts... | |
| 13:06:58 | efried | everyone's favorite, apparently. | |
| 13:21:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-fixed-ips REST API https://review.openstack.org/568516 | |
| 13:33:27 | mriedem | heh PUT /os-floating-ips-bulk/delete | |
| 13:33:29 | mriedem | makes total sense | |
| 13:39:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ips-bulk https://review.openstack.org/578106 | |
| 13:41:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ips-bulk https://review.openstack.org/578106 | |
| 13:41:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ip-dns https://review.openstack.org/578107 | |
| 13:46:16 | openstackgerrit | xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693 | |
| 13:48:51 | openstackgerrit | xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693 | |
| 13:48:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update API reference for os-floating-ip-pools https://review.openstack.org/578108 | |
| 13:51:42 | mriedem | if anybody is looking for something to do, the os-floating-ip-dns and os-floating-ips-bulk APIs are purely nova-network and can be removed following the standard 410 pattern we've established for this kind of work: https://etherpad.openstack.org/p/nova-network-removal-rocky | |
| 13:58:40 | mriedem | huh https://docs.openstack.org/nova/latest/cli/nova-manage.html#nova-api-database "Sync the api cells database up to the most recent version. This is the standard way to create the db as well." that is both confusing and wrong in the same shot | |
| 13:58:55 | mriedem | - there is no 'api cells database' | |
| 13:58:59 | mriedem | - it doesn't create a database | |
| 14:13:12 | gibi | mriedem: did I understand correnctly that you had strange problem with 'field_name in an_ovo_object' calls? | |
| 14:15:11 | gibi | mriedem: based on your statement '17:18 < mriedem> >>> obj.obj_attr_is_set('foo')' | |
| 14:16:09 | gibi | mriedem: I'm troubleshooting my random unit test failures in https://review.openstack.org/#/c/567268/21 | |
| 14:19:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix CLI docs for nova-manage api_db commands https://review.openstack.org/578116 | |
| 14:19:42 | mriedem | gibi: yeah something i was playing with yesterday, but in a venv | |
| 14:20:18 | mriedem | reminds me i need to check back on that | |
| 14:20:22 | gibi | mriedem: did you find any specific issue with oslo.versioned object? | |
| 14:20:34 | mriedem | nope, i didn't figure it out | |
| 14:20:44 | gibi | mriedem: OK thanks. then I dig forward | |
| 14:20:54 | mriedem | i had to go a different direction anyway on https://review.openstack.org/#/c/515423/ for the fix i was investigating | |
| 14:21:14 | gibi | mriedem: I see | |
| 14:24:57 | stephenfin | Anyone fancy looking at the two of these? Closes out kashyap's "bump libvirt" series https://review.openstack.org/#/q/topic:bump_min_libvirt_and_qemu+status:open | |
| 14:25:23 | kashyap | stephenfin: Oh, reminds me to finish off this too: https://review.openstack.org/#/c/567258/ | |
| 14:25:33 | kashyap | (Was buried in something else) | |
| 14:26:10 | kashyap | stephenfin: Zuul is -1 on the VHOSTUSER_MQ: https://review.openstack.org/#/c/565472/ | |
| 14:26:33 | stephenfin | kashyap: zuul lies | |
| 14:26:39 | kashyap | :-) | |
| 14:26:48 | stephenfin | But I'll recheck to be safe | |
| 14:26:51 | kashyap | Oh yeah | |
| 14:27:04 | kashyap | It's the IBM zKVM that seems to fail | |
| 14:27:22 | stephenfin | mriedem, gibi: surely not the same thing, but I had issues with 'field_name in an_ovs_object' vs. 'an_obv_obj.obj_attr_is_set(field_name)' https://review.openstack.org/#/c/564439/9/nova/objects/numa.py | |
| 14:27:49 | stephenfin | the former worked while the latter complained repeatedly about undefined attributes or something | |
| 14:28:43 | kashyap | stephenfin: Do you want to address Takashi's nit? | |
| 14:28:49 | gibi | stephenfin: interesting, thanks | |
| 14:29:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472 | |
| 14:29:26 | stephenfin | kashyap: Sure, done | |
| 14:29:35 | kashyap | Thx! | |
| 14:29:36 | gibi | stephenfin: I'm suspecting something wrong with 'in' in my case | |
| 14:29:51 | stephenfin | gibi: Ah, other way round. Of course :) | |
| 14:29:59 | mriedem | stephenfin: that's because obj_attr_is_set looks for obj_fields | |
| 14:30:21 | mriedem | http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/base.py#n681 | |
| 14:31:01 | mriedem | the 'in' implementation is a bit cleaner http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/base.py#n321 | |
| 14:31:05 | mriedem | but it hides that AttributeError | |
| 14:31:22 | gibi | mriedem: thanks, this could be something that helps my investigation | |
| 14:31:43 | gibi | I thought 'in' and obj_attr_is_set is equivalent :/ | |
| 14:31:52 | stephenfin | mriedem: Oh, good to know. I should probably call that out in the commit message | |
| 14:31:55 | stephenfin | gibi: As did I | |
| 14:32:23 | mriedem | if the field is defined on the object, they should be | |
| 14:33:38 | stephenfin | mriedem: I think it's because I didn't update every unit test to set 'network_info' to something | |
| 14:33:40 | gibi | in my case the field is defined in the fields attribute of the ovo class so for my case it should behave in the same way | |
| 14:33:46 | stephenfin | and didn't call the magic "set defaults" option | |
| 14:40:42 | mriedem | stephenfin: i guess geneve must be newish? https://docs.openstack.org/neutron/pike/admin/intro-overlay-protocols.html | |
| 14:40:52 | mriedem | not listed there for tunneled ^ | |
| 14:41:14 | stephenfin | mriedem: Yeah. I found it referenced in other docs but haven't had a chance to update those | |
| 14:41:25 | stephenfin | Seems to be some kind of uber protocol | |
| 14:42:03 | mriedem | https://bugs.launchpad.net/neutron/+bug/1778741 | |
| 14:42:05 | openstack | Launchpad bug 1778741 in neutron "Overlay (tunnel) protocols in Neutron - missing geneve" [Undecided,New] | |
| 14:42:32 | kashyap | dnf search reno | |
| 14:42:38 | kashyap | Oops | |
| 14:42:53 | stephenfin | https://github.com/openstack/neutron/blob/master/doc/source/feature_classification/provider_network_support_matrix.ini#L65 | |
| 14:43:27 | mriedem | stephenfin: yeah i saw that but it doesn't mention tunneling | |
| 14:44:05 | stephenfin | mriedem: How about...this one? https://github.com/openstack/neutron/blob/master/doc/source/contributor/internals/openvswitch_agent.rst#geneve-tunnels | |
| 14:44:11 | stephenfin | #DocStruggles | |
| 14:48:52 | mriedem | ack | |
| 14:48:54 | mriedem | linked into the bug | |
| 14:49:05 | mriedem | stephenfin: a few things in https://review.openstack.org/#/c/564445/ | |
| 14:49:13 | stephenfin | ack | |
| 14:56:19 | openstackgerrit | Merged openstack/nova master: Ironic update_provider_tree: restore traits override https://review.openstack.org/576877 | |
| 14:56:25 | openstackgerrit | Merged openstack/nova stable/queens: ironic: stop lying to the RT when ironic is down https://review.openstack.org/575628 | |
| 15:03:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip ServerShowV247Test.test_update_rebuild_list_server in nova-cells-v1 job https://review.openstack.org/578125 | |
| 15:03:51 | openstack | bug 1778743 in OpenStack Compute (nova) "When POSTing to /allocations with multiple consumers it is possible violate inventory capacity constraints" [Medium,Triaged] https://launchpad.net/bugs/1778743 | |
| 15:03:51 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add test demonstrating bug 1778743 https://review.openstack.org/578126 | |
| 15:04:32 | stephenfin | mriedem: Right back at ya (just a small question) https://review.openstack.org/#/c/546256/4/nova/api/openstack/compute/servers.py | |
| 15:06:25 | mriedem | oh boy that's an old one | |
| 15:07:56 | gibi | stephenfin: if there is a field in an ovo that has a default value but obj_set_defaults is not called then that field is considered invalid as per http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/base.py#n679 ? | |
| 15:08:27 | mriedem | stephenfin: replied | |