Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-16
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 jroll bbiab
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: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: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
23:43:06 openstackgerrit Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
23:43:07 openstackgerrit Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
23:43:07 openstackgerrit Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246
23:43:08 openstackgerrit Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324
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: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...
00:20:36 openstackgerrit Merged openstack/nova master: Trivial: Update help of enabled_filters https://review.openstack.org/545431
00:28:01 openstackgerrit Eric Fried proposed openstack/nova master: Clarify update_provider_tree docstring https://review.openstack.org/545495
00:28:30 fried_rice leakypipes, edleafe, cdent: And wording about externally-owned providers in the docstring ^
02:07:26 dgedia Hi, I am unable to create instances larger than m1.tiny? nova-conductor log shows "No Valid Host Found"? Any help would be really appreciated.
02:19:48 dgedia Hi, I am unable to create instances larger than m1.tiny? nova-conductor log shows "No Valid Host Found"? Any help would be really appreciated.
03:08:54 openstackgerrit Matthew Treinish proposed openstack/nova master: Remove single quotes from posargs on stestr run commands https://review.openstack.org/545476
03:10:03 mtreinish melwitt: ^^^ I figured out the functional failures we were using --group-regex instead of --group_regex and that was being treated as an unknown arg (and therefore a regex)
03:10:41 mtreinish it does mean that flag has been doing nothing for a while, which makes you wonder if it's even needed anymore
05:00:58 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add NumaTopology support for libvirt/qemu driver https://review.openstack.org/530451
05:00:58 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_XXXpin_cpuset() https://review.openstack.org/527631
05:00:59 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: trivial: omit condition evaluations https://review.openstack.org/545248

Earlier   Later