| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-19 | |||
| 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 | |
| 22:12:07 | jaypipes | mriedem: err, same. http://www.fox13news.com/weather/skytower-radar/manatee-sarasota | |
| 22:12:27 | mriedem | is fox 13 your action news leader in the tri-state area? | |
| 22:12:49 | mriedem | http://www.kttc.com/category/162320/kttc-interactive-radar | |
| 22:12:52 | jaypipes | mriedem: there is no tri-state area, but sure :) | |
| 22:13:12 | mriedem | where i grew up in south dakota, there was not only a tri-state area, but it was also called kelo land | |
| 22:13:20 | mriedem | http://www.keloland.com/ | |
| 22:13:27 | mriedem | you won't find it on any map | |
| 22:13:35 | jaypipes | heh | |
| 22:14:21 | mriedem | my favorite weather guy, randy brock, went into real estate and i still want to find him to make a joke about his forecast for a great deal on selling me a house | |
| 22:16:42 | cdent | jaypipes: gibi’s func test flapping is the result of differing sort order on the sharing_providers dict | |
| 22:16:59 | cdent | I noted that on the bug: https://bugs.launchpad.net/nova/+bug/1705231 | |
| 22:17:00 | openstack | Launchpad bug 1705231 in OpenStack Compute (nova) "Placement returns no allocation candidate for request that needs both compute resources and custom shared resources" [High,Triaged] | |
| 22:18:17 | cdent | messes up the order of the joins which for reasons unknown to me messes up the query | |
| 22:28:50 | openstackgerrit | melanie witt proposed openstack/nova-specs master: Add a new section: "Upgrade impact" to the template https://review.openstack.org/456756 | |
| 22:30:31 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/485358 | |
| 22:42:59 | mriedem | melwitt: so the counting quotas changes are all approved, | |
| 22:43:08 | mriedem | once that last one is done i think we can close that bp unless there is something i'm missing, | |
| 22:43:17 | mriedem | the last 2 changes in the series are the quotas in api db bp | |
| 22:44:18 | melwitt | mriedem: yeah. the rest will be adding test coverage and the like | |
| 22:44:44 | mriedem | speaking of | |
| 22:44:45 | mriedem | http://logs.openstack.org/52/484152/6/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/2bee19d/testr_results.html.gz | |
| 22:44:52 | mriedem | so what i'm seeing in this test is, | |
| 22:44:56 | mriedem | we resize down, | |
| 22:44:59 | mriedem | usage changes, as expected | |
| 22:45:12 | mriedem | b/c no more reservations and the instance vpus/memory_mb is changed in the compute | |