| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-19 | |||
| 19:28:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | |
| 19:28:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 19:28:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | |
| 19:28:32 | jlvillal | mriedem, As an FYI, I have asked for reviews on https://review.openstack.org/#/c/480332/ , but I know infra is pretty busy | |
| 19:28:49 | mriedem | ok | |
| 19:29:08 | jaypipes | mriedem, dansmith, bauzas, edleafe: rebased the placement-claims stuff on top of mriedem's Handle None returned from get_allocation... patch and addressed all of your review comments. | |
| 19:30:00 | openstackgerrit | Merged openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826 | |
| 19:32:48 | openstackgerrit | Merged openstack/nova master: conf: remove *_topic config opts https://review.openstack.org/433924 | |
| 19:33:09 | melwitt | tox -edocs fails locally with "dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting" just fyi | |
| 19:33:41 | openstackgerrit | melanie witt proposed openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | |
| 19:33:46 | openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 19:33:50 | openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 19:35:19 | melwitt | this is an interesting func test failure, timed out holding a lock during update_available_resource http://logs.openstack.org/01/484501/1/gate/gate-nova-tox-functional-ubuntu-xenial/080e1fb/testr_results.html.gz | |
| 19:35:39 | melwitt | or waiting for lock | |
| 19:39:25 | mriedem | melwitt: that's a known one | |
| 19:39:33 | mriedem | don't have a fingerprint for it since we can't really fingerprint it very well | |
| 19:39:50 | mriedem | we have this though http://status.openstack.org/elastic-recheck/#1704588 | |
| 19:39:54 | mriedem | which is caused by the same thing | |
| 19:39:59 | mriedem | rebuild fails b/c of novalidhost | |
| 19:40:21 | mriedem | not really sure why rebuild should care if it's on the same host though... | |
| 19:40:31 | mriedem | we shouldn't do a claim for rebuild if it's on the same host | |
| 19:41:26 | mriedem | http://logs.openstack.org/82/482382/4/gate/gate-nova-tox-functional-py35-ubuntu-xenial/fdfd05a/console.html#_2017-07-15_12_30_11_220266 | |
| 19:42:11 | melwitt | it seems weird that it's deadlocked trying to update resources | |
| 19:43:44 | mriedem | melwitt: i've wondered if we should remove this v | |
| 19:43:45 | mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L64 | |
| 19:43:53 | mriedem | and start whittling that down into specific tests that require locking | |
| 19:44:07 | mriedem | surely 90% of our functional tests shouldn't require a global lock | |
| 19:46:07 | melwitt | hm, yeah. I guess I haven't really known what REQUIRES_LOCKING does. looking at test.py now | |
| 19:46:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Remove global lock from _IntegratedTestBase https://review.openstack.org/485335 | |
| 19:46:58 | mriedem | let's see what explodes ^ | |
| 19:46:59 | melwitt | global external locking, okay | |
| 19:47:58 | melwitt | unrelated but I'm having a weird thing with my devstack patch where it keeps failing saying "depends on change that failed to merge" and I tried all the things it depends on and none of them had merge conflicts locally | |
| 19:48:16 | melwitt | it starts running in zuul and then it gets kicked out | |
| 19:48:17 | mriedem | using depends-on? | |
| 19:48:19 | melwitt | yeah | |
| 19:48:29 | mriedem | if the dependencies get rebased, it will kick it out of zuul with that message | |
| 19:48:41 | melwitt | ohh. okay, that's what it is then | |
| 19:49:02 | melwitt | thank you | |
| 19:49:26 | melwitt | it was when I kept updating the nova-manage thing. duh. | |
| 19:49:35 | mriedem | http://i3.kym-cdn.com/photos/images/newsfeed/000/620/434/f9e.gif | |
| 19:49:48 | melwitt | lol for real | |
| 19:50:50 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Adjust test_resize_down_revert to account for counting quotas https://review.openstack.org/484152 | |
| 19:57:33 | openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288 | |
| 19:59:06 | cfriesen | is anyone aware of sources of practical info on setting up and configuring/operating cells? caveats, gotchas, etc. | |
| 20:01:45 | mriedem | cfriesen: yes | |
| 20:02:27 | cfriesen | mriedem: care to share? :) Also, according to http://www.dorm.org/blog/exploring-openstack-nova-cells/ there's no way to configure neutron so that port state changes go to the right cell....is that still true or was that fixed? | |
| 20:02:40 | mriedem | cfriesen: are you talking about cells v1 or v2? | |
| 20:03:04 | mriedem | anything mdorman is talking about wrt cells is probably v1 | |
| 20:03:08 | mriedem | which is what godaddy is running | |
| 20:03:11 | cfriesen | mriedem: this would be in the context of Pike, so I'm assuming v2 | |
| 20:03:28 | mriedem | cells v1 is a dumpster fire you want to avoid | |
| 20:04:17 | melwitt | yeah, no one is running cells v2 multi cell yet. all of the blogs and stuff you'll find from CERN, GoDaddy, Nectar, that's all cells v1 | |
| 20:04:20 | mriedem | https://docs.openstack.org/nova/latest/cells.html | |
| 20:04:40 | mriedem | https://docs.openstack.org/ocata/install-guide-ubuntu/nova.html | |
| 20:04:54 | mriedem | the install guide in ocata covers the things needed to deploy with cells v2 stuff | |
| 20:04:59 | mriedem | the devref is more detail | |
| 20:05:03 | cfriesen | oh, excellent. I had found a few of the linked things but that's a much nicer overview | |
| 20:05:22 | mriedem | https://docs.openstack.org/ocata/config-reference/compute/cells.html is all cells v1 | |
| 20:05:29 | mriedem | i had opened a docs bug at some point to make that clear | |
| 20:05:39 | mriedem | but with the docs transition that's probably lost in the ether | |
| 20:05:43 | mriedem | stephenfin should know | |
| 20:06:02 | cfriesen | cool, that'll give me somewhere to start at least | |
| 20:08:26 | mriedem | https://bugs.launchpad.net/openstack-manuals/+bug/1673616 but that was for the ops guide | |
| 20:08:28 | openstack | Launchpad bug 1673616 in openstack-manuals "Scaling in Operations Guide - cells section needs to be updated" [Medium,Confirmed] - Assigned to Zhenyu Zheng (zhengzhenyu) | |
| 20:08:29 | mriedem | which is dead now i think | |
| 20:15:32 | mriedem | cfriesen: we have some docs todos for cellsv2 multi-cell in pike https://etherpad.openstack.org/p/nova-pike-cells-v2-todos | |
| 20:15:35 | openstackgerrit | Merged openstack/nova master: Avoid false positives of Jinja2 in Bandit scan https://review.openstack.org/479437 | |
| 20:16:22 | openstackgerrit | Merged openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | |
| 20:17:33 | cfriesen | mriedem: excellent, thanks | |
| 20:22:28 | openstackgerrit | Merged openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | |
| 20:23:11 | mriedem | jaypipes: is there anything else in here that we care about before releasing os-vif 1.7? https://review.openstack.org/#/q/project:openstack/os-vif+status:open+branch:master | |
| 20:32:33 | jaypipes | mriedem: I'd like to get https://review.openstack.org/#/c/474914/ in | |
| 20:32:51 | jaypipes | mriedem: I might just go fix that up for ralonso since he's out | |
| 20:32:55 | jaypipes | mriedem: gimme five mins | |
| 20:33:42 | mriedem | jaypipes: that depends on a nova and neutron change | |
| 20:33:46 | mriedem | which aren't closed to merged | |
| 20:33:49 | mriedem | so what's the point? | |
| 20:33:51 | jaypipes | ohhh... | |
| 20:33:56 | mriedem | *close to | |
| 20:33:57 | jaypipes | mriedem: sorry, I did not realize that. | |
| 20:33:59 | mriedem | ok | |
| 20:34:03 | jaypipes | mriedem: never mind then :) | |
| 20:50:35 | openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 21:00:26 | openstackgerrit | Merged openstack/python-novaclient master: doc: Create directory structure for docs migration https://review.openstack.org/480135 | |
| 21:00:32 | openstackgerrit | Merged openstack/python-novaclient master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/477757 | |
| 21:03:01 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/485358 | |
| 21:15:18 | openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.52 - Support tag when boot https://review.openstack.org/484617 | |
| 21:36:39 | openstackgerrit | Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | |
| 21:39:09 | mriedem | jaypipes: sdague: another easy quota one https://review.openstack.org/#/c/446243/ | |
| 21:43:31 | openstackgerrit | Merged openstack/nova master: Modify some comments for virt driver https://review.openstack.org/482022 | |
| 21:46:16 | openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 22:00:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix and optimize external_events for multiple cells https://review.openstack.org/445142 | |
| 22:03:25 | openstackgerrit | Merged openstack/nova master: Fix incorrect docstrings in neutron network API https://review.openstack.org/480167 | |
| 22:09:03 | jaypipes | mriedem: done | |
| 22:09:11 | mriedem | yay | |
| 22:09:48 | jaypipes | mriedem: for the record, I'm still trying to track down the issue that gibi's func test with shared resources that are custom has exposed. there's some sort of race condition I'm trying to uncover. | |
| 22:10:59 | mriedem | happy hunting | |
| 22:11:08 | mriedem | i'm watching sheets of rain pound my domicile | |
| 22:11:31 | mriedem | wrong word there i guess | |
| 22:11:42 | mriedem | *abode | |