| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-16 | |||
| 21:58:54 | jroll | but even before 487925 we would return an empty list | |
| 21:59:57 | jroll | ah jeez https://github.com/openstack/nova/commit/cce06a1e9855d9eed3f7c653200853f23466d791 | |
| 22:01:19 | jroll | hm, 5pm friday | |
| 22:01:35 | fried_rice | jroll: Need anything from me? | |
| 22:01:52 | jroll | fried_rice: whiskey may be needed | |
| 22:01:54 | jroll | :) | |
| 22:01:56 | fried_rice | Can https://review.openstack.org/545464 be abandoned? | |
| 22:02:06 | jroll | yes, believe so | |
| 22:02:27 | fried_rice | jroll: The fact that you're a time zone ahead of me indicates I have no way of getting you a bottle in time to save you. | |
| 22:02:27 | TheJulia | jroll: I will buy you whiskey in Dublin | |
| 22:02:34 | fried_rice | Yeah, that ^ | |
| 22:02:36 | jroll | heh | |
| 22:02:57 | TheJulia | And next time I'm through your part of the country, I'll make a point of bringing really good whiskey on my RV | |
| 22:03:15 | jroll | :o <3 | |
| 22:16:19 | mrjk | About nova.cfg, something is not clear. Let's say I've conductor, api and any other service. Most of settings are in the DEFAULT section, but is it possible to override some default parameters for a specific service ? Let's say I want to change the debug mode for only one service, and not the others ... | |
| 22:17:01 | mriedem | debug is global | |
| 22:17:19 | mriedem | as long as you have all of your controller services running on the same host, they are going to share config from the [DEFAULT] section | |
| 22:17:21 | mrjk | How could I find out this info by myself ? | |
| 22:17:50 | mriedem | you could split configs and create an /etc/nova/nova-api.conf which has config specific to your API service | |
| 22:17:59 | mriedem | and remove debug from the base /etc/nova/nova.conf | |
| 22:18:05 | mriedem | then run the service with both config files | |
| 22:18:19 | mrjk | Ok, this would be the way to go. I was unsure, I believed there was a kind defaulting/override values | |
| 22:18:21 | mriedem | nova-api --config-file /etc/nova/nova.conf --config-file /etc/nova/nova-api.conf | |
| 22:26:28 | openstackgerrit | Matthew Treinish proposed openstack/nova master: Remove single quotes from posargs on stestr run commands https://review.openstack.org/545476 | |
| 22:26:43 | mtreinish | melwitt: ^^^ but lets test this and make sure I'm not just seeing things... | |
| 22:27:16 | melwitt | k, I'll try it | |
| 22:42:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix error handling in compute API for multiattach errors https://review.openstack.org/545478 | |
| 22:42:54 | mriedem | well i wish i would have found this before we cut RC2 ^ because that's an annoying UX problem | |
| 22:46:28 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: ironic: stop lying to the RT when ironic is down https://review.openstack.org/545479 | |
| 22:46:54 | jroll | TheJulia: fried_rice: ^ that fixes it, but will crash at startup if ironic is down | |
| 22:47:26 | fried_rice | jroll: Maybe we *should* crash at startup if ironic is down. | |
| 22:48:17 | jroll | fried_rice: yeah, I kind of agree, kind of don't, regardless crashing when ironic is down was a huge pain in CI in the past that I don't want to live again | |
| 22:48:37 | jroll | I also feel like I want to be able to start my computes whenever and have them do stuff when ironic comes back | |
| 22:49:19 | jroll | though I don't believe in upgrading nova and ironic at the same time (or even the same maintenance window), other people do and this makes their life easier | |
| 22:49:37 | mriedem | nova-compute doesn't start if we can't connect to libvirt | |
| 22:49:52 | mriedem | i think the same for powervm? | |
| 22:49:58 | mriedem | not sure about hyperv/xen/vmware | |
| 22:50:17 | mriedem | nova-compute shouldn't come up, | |
| 22:50:27 | mriedem | because then the service will say it's up, and be around for scheduling, | |
| 22:50:38 | mriedem | and will just not work if the scheduler picks it and the hypervisor is gone | |
| 22:50:44 | jroll | mriedem: yeah, but libvirt isn't some external service, that just means you've configured your hypervisor wrong | |
| 22:51:05 | mriedem | vcenter is an external service | |
| 22:51:07 | jroll | and at least in ironic's case, there won't be any resources to schedule to, until it can connect to ironic | |
| 22:51:19 | jroll | or I guess there will, sigh | |
| 22:51:23 | fried_rice | In powervm, it looks like we'll hold up init_host for a while if we can't talk to the hypervisor, and then we'll ultimately blow up. | |
| 22:51:50 | mriedem | i thought someone's dog was going to get walked? | |
| 22:51:54 | mriedem | i can hear him whining from here | |
| 22:51:56 | jroll | good point | |
| 22:52:00 | fried_rice | But I've got a nice TODO there to make it work like I73a34eb6e0ca32d03e54d12a5e066b2ed4f19a61 which will actually disable the compute service (but not crash it) in that case. | |
| 22:52:00 | jroll | bbiab | |
| 22:53:16 | mriedem | melwitt: did you figure this out? https://review.openstack.org/#/c/340614/18/nova/compute/api.py@2029 | |
| 22:53:21 | mriedem | you had >1 attachment right> | |
| 22:53:22 | mriedem | ? | |
| 22:53:35 | mriedem | and that's why the volume status wasn't changing to 'available'? | |
| 22:55:39 | melwitt | mriedem: I had multiple attachments because I was having trouble getting the code path to hit in devstack. so I tried the scenario multiple times with the same volume by reset-state on it | |
| 22:55:45 | melwitt | and didn't notice it was building up attachments | |
| 22:56:14 | melwitt | so once I started from a clean slate, new volume and did the scenario, it worked as expected. the volume actually had no attachments in the fresh volume case. it was 'reserved' with no attachments | |
| 22:56:42 | melwitt | then the attachment_delete change it from 'reserved' -> 'available', then the volume_api.delete deleted the volume properly | |
| 22:57:34 | melwitt | so I think all is well | |
| 22:58:23 | mriedem | that was tied to the thing on L2063 too? | |
| 22:58:30 | mriedem | if detach fails, you definitely can't delete | |
| 23:00:32 | mriedem | i just updated the comments after you realized it was a test env issue | |
| 23:00:38 | mriedem | and yes volume attachments can build up if not managed | |
| 23:00:47 | mriedem | that's why i was using force detach earlier, | |
| 23:00:54 | mriedem | because you can do reset-state on the volume, but that doesn't remove the old attachments | |
| 23:01:11 | mriedem | really need a CLI for force detach in cinder | |
| 23:03:45 | melwitt | yeah | |
| 23:04:24 | melwitt | yeah, I know that if the detach fails you definitely can't delete. I was just trying to work out whether we should add a new try-except there to try the delete even if the detach fails | |
| 23:04:35 | melwitt | but I think that was legacy from when detach could fail with "nothing to detach" | |
| 23:04:43 | melwitt | so I took it out | |
| 23:09:39 | TheJulia | jroll: fwiw, we put some restarts in for issues with having to restart nova-compute due to restarts. We should just be able to make it a default thing... I think.... we'll likely want to verify that it works across multinode grenade jobs for ironic since they are different | |
| 23:10:27 | jroll | TheJulia: so what you're saying is allowing a crash at startup shouldn't be an issue for CI? | |
| 23:10:49 | TheJulia | afaik it should not be | |
| 23:11:10 | TheJulia | we have a default restart if it is not multinode if I'm remembering correctly | |
| 23:11:26 | TheJulia | which likely is why... | |
| 23:11:27 | TheJulia | ugh | |
| 23:11:54 | jroll | hrm | |
| 23:12:05 | jroll | agree, this sounds like a monday thing | |
| 23:14:50 | TheJulia | jroll: yes ++ | |
| 23:14:52 | mriedem | melwitt: ok +2 on https://review.openstack.org/#/c/340614/ now | |
| 23:15:12 | mriedem | melwitt: now we need to rope in superdan to peruse the series | |
| 23:15:17 | mriedem | at 3:15pm on a friday | |
| 23:15:36 | mriedem | melwitt: did you take a look at https://review.openstack.org/#/c/545123/ and the one after it? | |
| 23:37:43 | melwitt | \o/ hallelujah | |
| 23:38:13 | melwitt | mriedem: not yet, it's next on my list | |
| 23:41:34 | melwitt | good, the patches are small. yess | |
| 23:43:06 | openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | |
| 23:43:06 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | |
| 23:43:07 | openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 23:43:07 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 23:43:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 23:43:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324 | |
| 23:43:09 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | |
| 23:43:10 | fried_rice | Because you know I'm all about rebase, 'bout rebase... | |
| 23:54:52 | fried_rice | edleafe: Making sure I'm not seeing things - do we not have GET /resource_providers?with_traits=... ? | |
| #openstack-nova - 2018-02-17 | |||
| 00:19:20 | openstackgerrit | Eric Fried proposed openstack/nova master: Only pull associated *sharing* providers https://review.openstack.org/545494 | |
| 00:19:32 | fried_rice | leakypipes: There's for that thing about only pulling down sharing providers ^ | |
| 00:19:38 | fried_rice | edleafe: ^ | |
| 00:19:49 | fried_rice | And cdent, you purplin so and so | |
| 00:20:09 | fried_rice | We may want to backport this... | |