| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-16 | |||
| 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 | |
| 05:25:37 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_XXXpin_cpuset() https://review.openstack.org/527631 | |
| 05:25:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: trivial: omit condition evaluations https://review.openstack.org/545248 | |
| 05:25:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add NumaTopology support for libvirt/qemu driver https://review.openstack.org/530451 | |
| 06:13:54 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561 | |
| 07:50:51 | openstackgerrit | melissaml proposed openstack/nova master: fix typos in documentation https://review.openstack.org/542866 | |
| 08:07:33 | openstackgerrit | melissaml proposed openstack/nova-specs master: fix typos in documentation https://review.openstack.org/545528 | |
| 09:52:12 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/545549 | |
| 10:16:15 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/545567 | |
| 10:26:00 | mdurrant | This is driving me bonkers - trying to start nova-compute, getting a DuplicateOptError regarding 'host'. I disable neutron in my nova.conf, no error. Nowhere in my neutron configuration do I have a host option. Any ideas? | |
| 10:27:00 | mdurrant | I looked at the source and can see two 'host' options being registered by netconf.py | |
| 10:30:05 | mdurrant | Yea - neutron/conf/common.py defines 'host' and nova/conf/netconf.py defines "host". | |
| 15:44:19 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/545567 | |
| 15:49:04 | openstackgerrit | Merged openstack/nova master: Remove single quotes from posargs on stestr run commands https://review.openstack.org/545476 | |
| 15:49:19 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/545549 | |
| #openstack-nova - 2018-02-18 | |||
| 06:12:47 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561 | |
| 06:53:01 | openstackgerrit | Ameed Ashour proposed openstack/nova stable/ocata: Detach volumes when VM creation fails https://review.openstack.org/545087 | |