| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-19 | |||
| 19:09:59 | mriedem | ok | |
| 19:10:11 | TheJulia | 470307 likely just needs to be abandoned at this point | |
| 19:10:19 | TheJulia | since it is just creating confusion | |
| 19:10:52 | mriedem | sdague: can you abandon? https://review.openstack.org/#/c/470307/ | |
| 19:11:39 | sdague | mriedem: done | |
| 19:12:07 | TheJulia | thanks sdague | |
| 19:14:39 | openstackgerrit | melanie witt proposed openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | |
| 19:14:43 | openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 19:14:47 | openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 19:25:18 | mriedem | melwitt: doc build failed on the man page https://review.openstack.org/#/c/446243/43/doc/source/cli/nova-manage.rst | |
| 19:25:21 | mriedem | you need a blank line in there | |
| 19:25:25 | mriedem | otherwise +2 | |
| 19:25:37 | melwitt | oh, dangit | |
| 19:27:16 | melwitt | forgot about the tox docs target | |
| 19:27:55 | openstackgerrit | Merged openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | |
| 19:28:07 | openstackgerrit | Merged openstack/python-novaclient master: Add resize down test which also verifies quota changes https://review.openstack.org/484151 | |
| 19:28:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 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: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 | |