Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-19
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: 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

Earlier   Later