Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-16
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
15:46:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate' https://review.openstack.org/508190

Earlier   Later