| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-03 | |||
| 13:37:05 | sahid | stephenfin: can you look at this serie https://review.openstack.org/#/c/561912/9, jaypipes already +2ed most of the patches | |
| 13:37:14 | stephenfin | sahid: Can do | |
| 13:37:54 | gibi | stephenfin: is the new link at https://review.openstack.org/#/c/540888/3..4/doc/source/admin/manage-the-cloud.rst@25 more general than the original was? | |
| 13:38:42 | stephenfin | gibi: Possibly. I couldn't find an equivalent for the former | |
| 13:38:54 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: trivial: Fix file permissions https://review.openstack.org/566064 | |
| 13:39:02 | gibi | stephenfin: ahh so the original was a dead link already | |
| 13:39:09 | gibi | stephenfin: then it is OK for me | |
| 13:39:13 | stephenfin | Yup. A couple of them were | |
| 13:39:29 | gibi | stephenfin: then this fix is better than I thought :) | |
| 13:39:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove [scheduler]/host_manager config option https://review.openstack.org/565806 | |
| 13:52:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886 | |
| 13:52:17 | gibi | stephenfin: why do we need this ref? https://review.openstack.org/#/c/540888/3..4/nova/api/openstack/compute/rest_api_version_history.rst@761 | |
| 13:53:58 | stephenfin | gibi: for this https://review.openstack.org/#/c/540888/3..4/doc/source/admin/manage-volumes.rst@37 | |
| 13:54:05 | mriedem | johnthetubaguy: want to re-approve this? https://review.openstack.org/#/c/524027/ had to rebase | |
| 13:54:16 | gibi | stephenfin: cool, thanks | |
| 13:54:44 | johnthetubaguy | mriedem: sorry missed that one, done | |
| 13:55:58 | melwitt | nova meeting in 5 minutes | |
| 13:56:37 | mriedem | johnthetubaguy: thanks | |
| 13:57:09 | mriedem | johnthetubaguy: it's also your mandatory weekly bugging for https://review.openstack.org/#/c/552078 | |
| 13:59:22 | johnthetubaguy | mriedem: so I managed to loose track of what we are doing with specs re freeze, although this one should get approved eiher way | |
| 13:59:43 | mriedem | johnthetubaguy: spec freeze is r-2 | |
| 13:59:45 | mriedem | june 7 i think | |
| 14:00:51 | johnthetubaguy | mriedem: ah, I had in my head r-1 for some reason | |
| 14:01:00 | mriedem | it usually is the 1st milestone, | |
| 14:01:08 | mriedem | but because of runways we decided to move out the spec freeze | |
| 14:01:14 | johnthetubaguy | yeah, makes good sense | |
| 14:01:34 | johnthetubaguy | I am just out of the loop (again), too many stupid hospital trips, but thankfully that is all done now! | |
| 14:01:56 | Shilpa | stephenfin: hi, did you gone through http://paste.openstack.org/show/720231/, is it okay to discuss now? | |
| 14:02:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | |
| 14:02:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489 | |
| 14:02:51 | mriedem | johnthetubaguy: hopefully everything is ok, didn't know that was happening | |
| 14:02:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | |
| 14:03:46 | mriedem | stephenfin: rebased and fixed up https://review.openstack.org/#/c/565367/ - should be happy now | |
| 14:03:52 | johnthetubaguy | mriedem: no worries, not really mentioned it, wasn't too bit a deal, just took time I didn't have! | |
| 14:04:03 | kashyap | Because when I just wanted to ping on something, Matt was already on it & addressed what needs addressing. | |
| 14:04:11 | kashyap | (Not the first time it happened, though...) | |
| 14:04:13 | mriedem | johnthetubaguy: babies are just so gd needy! | |
| 14:05:01 | mriedem | kashyap: the right stuff? https://www.youtube.com/watch?v=tbIEwIwYz-c | |
| 14:05:24 | efried | without clicking, /me assumes nkotb video. | |
| 14:05:50 | kashyap | Haha | |
| 14:05:57 | johnthetubaguy | mriedem: its true | |
| 14:06:23 | kashyap | mriedem: Didn't know of this vintage song; thanks for the education. | |
| 14:08:18 | kashyap | efried: You know your guess was right. | |
| 14:09:09 | kashyap | As it's closer to me | |
| 14:11:25 | mriedem | tssurya: question in https://bugs.launchpad.net/nova/+bug/1767303 for you | |
| 14:11:25 | openstack | Launchpad bug 1767303 in OpenStack Compute (nova) "Scheduler connects to all cells DBs to gather compute nodes info" [Undecided,New] - Assigned to Surya Seetharaman (tssurya) | |
| 14:12:33 | mriedem | tssurya: maybe you just want to propose a backport of https://review.openstack.org/#/c/539617/ for queens? | |
| 14:14:17 | bauzas | stephenfin: do we really need RsT for a docstring ? https://review.openstack.org/#/c/564214/2/nova/conf/devices.py | |
| 14:15:04 | stephenfin | bauzas: It's not a docstring. That's help text that's parsed by oslo_config.sphinext to generate this | |
| 14:15:15 | stephenfin | https://docs.openstack.org/nova/queens/configuration/config.html | |
| 14:15:16 | bauzas | shit | |
| 14:15:27 | bauzas | because yes, it's a docstring | |
| 14:15:34 | bauzas | it's just we use the docstring elsewhere | |
| 14:15:52 | stephenfin | I think our definition of docstring differs :) | |
| 14:16:05 | bauzas | IMHO, help text is for nova.conf | |
| 14:16:10 | bauzas | but meh | |
| 14:16:18 | stephenfin | Um, that is nova.conf? | |
| 14:16:23 | bauzas | a *python* docstring | |
| 14:16:30 | bauzas | 3 brackets | |
| 14:16:38 | bauzas | that's what I call "docstring" | |
| 14:16:47 | edleafe | bauzas: that's just a string | |
| 14:16:50 | stephenfin | That's a mult-line string | |
| 14:16:52 | stephenfin | Yeah | |
| 14:17:07 | edleafe | A docstring is the comment that follows the method definition | |
| 14:17:12 | stephenfin | https://www.python.org/dev/peps/pep-0257/#id15 | |
| 14:18:27 | bauzas | well, if you should name something because of its usage, I disagree | |
| 14:18:30 | bauzas | but meh | |
| 14:18:56 | bauzas | :p | |
| 14:19:05 | edleafe | bauzas: for example: | |
| 14:19:06 | edleafe | def some_method(): | |
| 14:19:06 | edleafe | "this is a docstring" | |
| 14:19:06 | edleafe | return True | |
| 14:19:19 | edleafe | Doesn't need triple quotes | |
| 14:19:25 | edleafe | unless it's multi-line | |
| 14:19:52 | bauzas | I should write my own language | |
| 14:20:17 | stephenfin | Yeah, theoretically you could even avoid using triple quotes then by escaping the new lines but no one would be so evil | |
| 14:20:50 | stephenfin | bauzas: http://logs.openstack.org/14/564214/2/check/build-openstack-sphinx-docs/a23348b/html/configuration/config.html#devices.enabled_vgpu_types | |
| 14:21:10 | bauzas | grmblblb :p | |
| 14:21:31 | stephenfin | Eventually I'll have you all writing perfect rST | |
| 14:21:43 | stephenfin | At which point I can happily retire to Spain or something | |
| 14:21:47 | mriedem | *perfect* rST? | |
| 14:22:01 | stephenfin | (TM) | |
| 14:22:59 | bauzas | that's glorious | |
| 14:25:28 | mriedem | stephenfin: speaking of https://review.openstack.org/#/c/565526/2..3/nova/conf/compute.py@a894 | |
| 14:25:34 | mriedem | you should appreciate that | |
| 14:26:50 | stephenfin | I do :) oslo_config.sphinxext should soon start reporting those warnings too (though it won't raise an error, to avoid breaking all the stuff) | |
| 14:52:16 | mriedem | efried: https://bugs.launchpad.net/nova/+bug/1767139 | |
| 14:52:16 | openstack | Launchpad bug 1767139 in OpenStack Compute (nova) "TypeError in _get_inventory_and_update_provider_generation" [Undecided,New] | |
| 14:52:21 | mriedem | remember https://review.openstack.org/#/c/562739/ ? | |
| 14:52:37 | mriedem | apparently people are hitting a variant of that in pike starting in 16.1.1 which is when the pike version of that backport was released | |
| 14:52:44 | mriedem | looking at the code, i can't figure out what might be causing that | |
| 14:53:10 | mriedem | they are saying in the bug report that it happens if they start the compute before the control plane is up, but i'm not sure what difference that would make - but placement would have to be running | |
| 14:53:23 | efried | mriedem: Sorry, is it the backport of the original or the backport of the revert that's breaking? | |
| 14:54:18 | mriedem | backport of the original | |
| 14:54:24 | mriedem | there is no backport of the revert | |
| 14:54:29 | mriedem | so this https://review.openstack.org/#/c/524618/ | |
| 14:55:25 | efried | ^ needs to be reverted is what you're saying? | |
| 14:55:30 | mriedem | in the bug they said this happens if compute starts before controller, so i have to assume that includes placement isn't up yet, so we're hitting a flow where the RT is posting inventory for the first time and the compute node RP can't be created | |
| 14:56:03 | mriedem | i would like to avoid a revert if possible | |
| 14:56:10 | mriedem | i'm trying to inspect the change and flow to see if there is something we're missing | |