| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-06 | |||
| 11:43:57 | openstackgerrit | Lingyong Xu proposed openstack/nova master: Updated from global requirements https://review.openstack.org/454130 | |
| 12:05:32 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 12:16:54 | openstackgerrit | XueFeng Liu proposed openstack/python-novaclient master: Revise `nova list` description https://review.openstack.org/454148 | |
| 12:19:46 | johnthetubaguy | sfinucan: which ones? | |
| 12:20:23 | johnthetubaguy | sfinucan: I think the problem is getting microversion support into OSC, I lost track of how that is going | |
| 12:20:33 | sfinucan | johnthetubaguy: For example, `nova list` | |
| 12:20:47 | sfinucan | vs. `openstack server list` | |
| 12:21:07 | johnthetubaguy | sfinucan: right, so specifically just python-novaclient then | |
| 12:21:44 | sfinucan | johnthetubaguy: Oops - s/clients/client/ | |
| 12:21:57 | sfinucan | Yeah - python-novaclient | |
| 12:22:02 | johnthetubaguy | ah, cool | |
| 12:22:14 | gabor_antal | johnthetubaguy, sfinucan: would you please have a look at volume_attach notification transformation? https://review.openstack.org/#/c/401992/ | |
| 12:22:21 | johnthetubaguy | I mean I hate nova-manage, but we will always need something to bootstrap | |
| 12:22:39 | johnthetubaguy | gabor_antal: I think the notifications are on my hit list, just snowed under right now | |
| 12:22:44 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/453896 | |
| 12:22:50 | johnthetubaguy | sfinucan: trying to find the reference on this | |
| 12:23:27 | gabor_antal | johnthetubaguy: thanks | |
| 12:23:46 | johnthetubaguy | sfinucan: here we go: https://specs.openstack.org/openstack/openstack-specs/specs/deprecate-cli.html | |
| 12:23:55 | johnthetubaguy | sfinucan: I believe thats still the plan, the issue is microversion | |
| 12:23:56 | johnthetubaguy | s | |
| 12:24:24 | johnthetubaguy | sfinucan: I believe we are still adding features in python-novaclient till we get OSC caught up | |
| 12:24:37 | sfinucan | johnthetubaguy: ^ that's the key bit | |
| 12:24:59 | sfinucan | I wasn't sure whether we should be reviewing/accepting new features to python-novaclient | |
| 12:25:06 | johnthetubaguy | I believe we still are | |
| 12:25:11 | sfinucan | It seems we should | |
| 12:25:13 | sfinucan | indeed | |
| 12:25:43 | johnthetubaguy | I would check with matt when he is on, he might know how the other efforts are going, or maybe sdague thinking about it | |
| 12:27:36 | sfinucan | johnthetubaguy: I'll do just that | |
| 12:28:02 | sfinucan | johnthetubaguy: Also, seeing as you've expressed displeasure with nova-manage, you might want to look at [1] at some point [1] https://review.openstack.org/#/q/status:open+branch:master+topic:bp/move-nova-cmds-to-cliff | |
| 12:28:09 | sfinucan | #shamelessplug | |
| 12:44:36 | openstackgerrit | Bob Ball proposed openstack/nova master: Use physical utilisation for cached images https://review.openstack.org/353862 | |
| 13:21:28 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 13:23:21 | njohnston | Hi, quick question! Did the os-quota-sets API exist in Icehouse? I have an old Icehouse installation I am trying to grab info from, and when I curl to https://${NOVA_URL}/v2/os-quota-sets/${TENANT_ID}/detail I get a 404 Not Found. | |
| 13:24:56 | openstack | Launchpad bug 1516703 in OpenStack Compute (nova) "crypto.py generates certs with SHA-1 digest" [Medium,New] | |
| 13:24:56 | mriedem | peter-hamilton: your team might be able to help with this https://bugs.launchpad.net/nova/+bug/1516703 | |
| 13:32:05 | peter-hamilton | mriedem: taking a look | |
| 13:32:41 | johnthetubaguy | sfinucan: I thought we said no to that spec at the PTG? | |
| 13:33:07 | johnthetubaguy | sfinucan: something tells me we didn't hit that till you had left though, so it wasn't a real discussion | |
| 13:33:18 | sfinucan | johnthetubaguy: It was no to use click and yes to using cliff | |
| 13:33:27 | sfinucan | that's per mriedem's comments on the spec, anyway | |
| 13:33:43 | johnthetubaguy | sfinucan: ah... so they need more different names :) | |
| 13:34:14 | sfinucan | johnthetubaguy: You're telling me. I wrote a sphinx extension for cliff and then referenced click in all the docs for same | |
| 13:34:17 | sfinucan | :) | |
| 13:34:57 | sfinucan | Anyway, long story short: I don't need a spec for the work I'm doing this cycle, so I've retargeted that to Queens. Still using the blueprint to group the work though | |
| 13:35:11 | johnthetubaguy | OK | |
| 13:35:31 | johnthetubaguy | I have filed a correction note in my brian | |
| 13:35:35 | mriedem | njohnston: yes https://github.com/openstack/nova/blob/icehouse-eol/nova/api/openstack/compute/contrib/quotas.py#L240 | |
| 13:35:50 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 13:36:25 | mriedem | njohnston: https://developer.openstack.org/api-ref/compute/#quota-sets-os-quota-sets | |
| 13:36:55 | johnthetubaguy | mriedem: njohnston: I wonder if you have to turn on the extension in the API, if you go back that far, or rather your config might have turned it off | |
| 13:38:11 | mriedem | johnthetubaguy: wanted to ask you about this given you're familiarity with all things caching scheduler https://review.openstack.org/#/c/453916/ | |
| 13:38:32 | mriedem | johnthetubaguy: i assume rax is setting CONF.filter_scheduler.track_instance_changes = False? | |
| 13:38:39 | njohnston | johnthetubaguy mriedem: I assumed that call was how Horizon was displaying it's array of quota pie charts, since the data returned is pretty much perfect for it. So since I see the pie charts in my Horizon I was hopeful it would work. | |
| 13:38:40 | mriedem | so the computes don't send instance updates to the caching scheduler, which doesn't use them | |
| 13:39:08 | mriedem | njohnston: to debug your curl request, you could use nova client with the --debug option | |
| 13:39:40 | openstackgerrit | Dinesh Bhor proposed openstack/nova master: Refactor: Move post method to APIValidationTestCase base class https://review.openstack.org/364687 | |
| 13:39:41 | mriedem | njohnston: nova --debug quota-show tenant_id | |
| 13:39:53 | johnthetubaguy | mriedem: not sure we are running a new enough nova | |
| 13:39:55 | mriedem | * nova --debug quota-show --tenant <tenant_id> --detail | |
| 13:40:07 | mriedem | johnthetubaguy: i think those are pretty old options | |
| 13:40:11 | mriedem | for the instance update stuff | |
| 13:41:06 | mriedem | johnthetubaguy: i just noticed yesterday the config docs saying track_instance_updates is only used for the filter scheduler, but it defaults to true and so nova compute is always sending those, unless you change it when you change the scheduler.driver | |
| 13:41:22 | mriedem | seems we could be smarter and just not look at that option at all if you're not using the filter scheduler | |
| 13:41:37 | mriedem | way back before we had a hard list of scheduler drivers and it was just loaded from classpath, we couldn't really do this | |
| 13:41:40 | mriedem | but i think it's safe now | |
| 13:42:05 | johnthetubaguy | maybe, added a comment on that patch, we don't actually configure the scheduler settings on any node other than the scheduler | |
| 13:42:39 | johnthetubaguy | but I like your approach, it makes more sense overall, we just need to train folks about configuration more | |
| 13:44:39 | johnthetubaguy | mriedem: actually, I think we proabably just turned that off because we didn't use filters that need that info | |
| 13:44:48 | johnthetubaguy | mriedem: but anyways, seems worth a peak | |
| 13:45:06 | mriedem | so i was worried about that, | |
| 13:45:14 | mriedem | because if you don't configure the scheduler things on the compute node, | |
| 13:45:29 | mriedem | that means you get the default, so nova compute is doing all of this extra rpc traffic to send something your scheduler doesn't care about | |
| 13:45:49 | johnthetubaguy | yeah, that does suck | |
| 13:46:33 | johnthetubaguy | we had this debate a few times, we should really look at being more specific on the idea of what configurations should be set where | |
| 13:47:14 | mriedem | i didn't realize until yesterday that this was scheduler config used on the comptues | |
| 13:47:18 | mriedem | but yeah it's pretty nasty | |
| 13:47:28 | mriedem | i guess the silver lining is, | |
| 13:47:32 | mriedem | when caching scheduler is gone, | |
| 13:47:37 | mriedem | so is this config option | |
| 13:47:57 | mriedem | because the only options you'd have then are filter, chance and fake, and the latter two are test-only | |
| 13:48:34 | bauzas | mriedem: remember we have 3 drivers in tree | |
| 13:48:37 | bauzas | not only 2 | |
| 13:48:47 | mriedem | we have 4 drivers in tree | |
| 13:48:54 | mriedem | caching, filter, chance and fake | |
| 13:49:01 | mriedem | the scheduler.driver option uses the choices kwarg | |
| 13:49:11 | mriedem | so unless you fork that, it's a ValueError to specify something else | |
| 13:49:25 | johnthetubaguy | remembers macsz saying something about stevedoor^ | |
| 13:50:25 | mriedem | https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L58 | |
| 13:50:42 | mriedem | https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L79 | |
| 13:50:50 | mriedem | i don't see how ^ is right since we use choices | |
| 13:50:56 | mriedem | "A custom scheduler driver. In this case, you will be responsible for creating and maintaining the entry point in your 'setup.cfg' file" | |
| 13:51:05 | mriedem | choices=("filter_scheduler", "caching_scheduler", "chance_scheduler", "fake_scheduler"), | |
| 13:51:11 | bauzas | mriedem: FWIW, the FakeScheduler is not existing in tree | |
| 13:51:30 | bauzas | afaik, it's only in the tests tree | |
| 13:51:45 | bauzas | you can't deploy it | |
| 13:51:47 | johnthetubaguy | yeah, its meant to be stevedoor I think | |
| 13:52:10 | dansmith | mriedem: yeah that scheduler config is nasty, and it's an upcall :( | |
| 13:52:11 | bauzas | johnthetubaguy: we already load with stevedore :) | |
| 13:52:19 | dansmith | mriedem: that was part of the source of my frustration with this crap last week | |
| 13:52:24 | johnthetubaguy | oh, maybe choices was a step too far then | |