| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-06 | |||
| 20:45:32 | itlinux | nova compute | |
| 20:46:16 | itlinux | so when my nova-compute node reboots.. which is my hypervisor in this case i want to have vms restart | |
| 20:47:24 | mriedem | then i think you're looking for the resume_guests_state_on_host_boot config option | |
| 20:48:31 | itlinux | ok thanks | |
| 20:49:13 | itlinux | I have set this resume_guests_state_on_host_boot to true but did not work :) | |
| 20:49:50 | mriedem | "did not work" would need some expanding | |
| 20:50:07 | mriedem | which compute driver are you using? | |
| 20:50:20 | itlinux | well I rebooted the hypervisor and the vm virsh list still down | |
| 20:50:24 | dansmith | itlinux: if your init system starts libvirt shutdown before nova is down, then nova will record that the guest powered off and won't start it back up | |
| 20:50:39 | dansmith | and there's not much we can do about that | |
| 20:50:42 | itlinux | I am on centos | |
| 20:50:44 | itlinux | 7 | |
| 20:51:09 | mriedem | yeah the code that checks that option says if the nova db thinks the instance should be running, but the hypervisor says it isn't, and the option is true, nova reboots the guest | |
| 20:51:12 | itlinux | so does this https://ask.openstack.org/en/question/66880/start-vms-after-reboot-compute-node/ show the right info? | |
| 20:51:44 | mriedem | if libvirt stopped and stopped the guests with it, nova would detect that and mark them as stopped in the nova db too, so on startup we won't try to reboot those guests | |
| 20:52:17 | itlinux | ok so assuming the doc I posted is correct.. | |
| 20:53:27 | mriedem | that doc is similar to what i remember, depends on how libvirt is configured to handle the guests on restart | |
| 20:54:01 | itlinux | this is standard ooo implementation which option should I check? mriedem: | |
| 20:54:01 | itlinux | thanks | |
| 20:54:21 | dansmith | that's from 2015 and mentions chkconfig, so it's likely that it's not up-to-date with what happens on systemd in parallel | |
| 20:54:58 | dansmith | and also, ask.o.o != "docs" | |
| 20:55:09 | dansmith | some people answer questions with pretty wrong suggestions | |
| 20:56:02 | itlinux | ok | |
| 21:00:00 | dansmith | mriedem: archiving a lot more data now :) http://logs.openstack.org/71/550171/4/check/nova-next/4986796/job-output.txt.gz#_2018-03-06_20_44_12_527465 | |
| 21:00:19 | dansmith | dunno what that failure is though, given it _just_ ran | |
| 21:00:24 | mriedem | dansmith: easy fix for something i forgot about during the ptg https://review.openstack.org/#/c/548572/ - needs to go to queens | |
| 21:00:56 | dansmith | mriedem: purge purged a bunch of stuff | |
| 21:01:16 | mriedem | i don't know what that failure is either | |
| 21:02:20 | mriedem | hmm, something about instance_mappings | |
| 21:02:25 | mriedem | which is relatively new... | |
| 21:02:28 | mriedem | tssurya added | |
| 21:02:59 | dansmith | ah it must be hitting api db for something | |
| 21:03:31 | tssurya | dansmith, mriedem : removing request specs and instance mappings | |
| 21:03:35 | tssurya | during archival ? | |
| 21:03:36 | mriedem | yeah http://logs.openstack.org/71/550171/4/check/nova-next/4986796/logs/etc/nova/nova_cell1.conf.txt.gz | |
| 21:03:43 | dansmith | ah | |
| 21:03:46 | mriedem | nova_cell1.conf doesn't have [api_database] configured | |
| 21:03:55 | dansmith | right | |
| 21:04:05 | dansmith | well, if were going to do the --all-cells thing I guess it doesn't matter | |
| 21:04:12 | dansmith | s/were/we're/ | |
| 21:04:43 | mriedem | so what would happen if we also passed --config-file /etc/nova/nova.conf which has [api_database] but also [database] pointed at cell0? i've always wondered how oslo.config handles specifying 2 config files that have conflicting options set | |
| 21:05:13 | mriedem | we could at least avoid the ugly traceback if we checked if CONF.api_database.connection is set | |
| 21:05:21 | mriedem | before trying to delete instance mappings and request specs | |
| 21:05:29 | dansmith | I'd have to look to see which order they need to be in, | |
| 21:05:42 | dansmith | but that's what I'd do for now if we weren't going to do the --all-cells thing | |
| 21:06:09 | dansmith | mriedem: yeah, because I think it actually did archive things and failed to print status, which is why the subsequent run found nothing | |
| 21:06:22 | tssurya | dansmith : doesn't Kevin have a patch for --all-cells ? | |
| 21:06:33 | mriedem | yeah i'll open a bug quick | |
| 21:06:38 | dansmith | tssurya: yep, that's what I'm saying | |
| 21:06:41 | mriedem | tssurya: he does but it needs work | |
| 21:07:23 | tssurya | dansmith, mriedem : oh okay, let me know if I can help | |
| 21:07:39 | tssurya | or if my extra stuff needs an extra check | |
| 21:07:57 | dansmith | tssurya: yeah I think mriedem is filing a bug against it right now | |
| 21:08:07 | dansmith | tssurya: just to catch that exception if we can't do the api bit and report it cleanly | |
| 21:08:47 | tssurya | dansmith : okay, nice | |
| 21:09:35 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Propose standardized provider descriptor file https://review.openstack.org/550244 | |
| 21:09:37 | mriedem | tssurya: congratulations. you have advanced to the level of nova developer that not only fixes bugs, but introduces regressions from your bug fixes. | |
| 21:09:42 | mriedem | this is a joyous occasion | |
| 21:09:55 | jaypipes | amen | |
| 21:10:06 | mriedem | we should have colored belts | |
| 21:10:13 | tssurya | mriedem : :D | |
| 21:11:09 | tssurya | jaypipes : I will start working on the CLI as planned this week | |
| 21:11:16 | tssurya | I mean as promised8 | |
| 21:11:50 | tssurya | I was wondering if it needed a bp? | |
| 21:11:52 | jaypipes | w00t :) | |
| 21:12:41 | tssurya | mriedem : let me know if you have filed the bug; I will fix it considering I created it :( | |
| 21:13:08 | mriedem | tssurya: https://bugs.launchpad.net/nova/+bug/1753833 | |
| 21:13:09 | openstack | Launchpad bug 1753833 in OpenStack Compute (nova) "archive_deleted_rows --until-complete stops if api database is not configured" [Undecided,New] | |
| 21:13:22 | tssurya | thanks | |
| 21:14:02 | mriedem | tssurya: the CLI to help set allocation ratios for all providers in an aggregate? | |
| 21:14:11 | mriedem | that could probably be done under the same bp jaypipes has for the same issue | |
| 21:14:22 | tssurya | mriedem : yes that one | |
| 21:14:34 | mriedem | https://blueprints.launchpad.net/nova/+spec/placement-aggregate-allocation-ratios | |
| 21:14:39 | mriedem | just do it under that one ^ | |
| 21:15:06 | tssurya | mriedem : thank you , will do | |
| 21:16:11 | mriedem | toabctl: sorry about forgetting about your patch, it's approved now | |
| 21:17:50 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Allow 'network' in RequestContext service_catalog https://review.openstack.org/550259 | |
| 21:24:37 | mriedem | odyssey4me: approved https://review.openstack.org/#/c/549737/ - can you propose a backport to stable/queens? | |
| 21:25:06 | itlinux | looks like the only option needed is the one in nova resume_guests_state_on_host_boot | |
| 21:25:10 | itlinux | mriedem: | |
| 21:25:29 | mriedem | itlinux: that's the only one we have for that thing you're trying to do | |
| 21:26:23 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction https://review.openstack.org/550265 | |
| 21:26:52 | itlinux | yes saying that's working mriedem: | |
| 21:26:54 | itlinux | thanks | |
| 21:27:09 | mriedem | ah, cool | |
| 21:33:01 | cdent | It's not possible to depends-on python library changes is it? I've been assuming I can't test microversion-parse changes exercised by the above DNM in the gate, but maybe I'm forgetting something? | |
| 21:36:16 | mriedem | hongbin: just need a simple test for this https://review.openstack.org/#/c/535532/3 | |
| 21:36:36 | mriedem | cdent: not sure about that in the zuulv3 world | |
| 21:36:44 | mriedem | used to be you couldn't unless you were using a src job | |
| 21:36:45 | hongbin | mriedem: ack, will get back to it today or tomorrow | |
| 21:38:00 | cdent | mriedem: I considered doing some kind of crazy weird thing inside requirements.txt and then decide testing it locally is easy enough to prove things out well enough to lead to an (eventual) release of a new microversion_parse | |
| 21:46:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add more functional test for placement.aggregates https://review.openstack.org/514109 | |
| 21:51:34 | mriedem | jroll: can i pull the trigger on this? https://review.openstack.org/#/c/544772/ | |
| 22:00:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | |
| 22:00:32 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Change a validation in creating a server group https://review.openstack.org/546484 | |
| 22:00:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | |
| 22:01:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | |
| 22:01:36 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 22:02:35 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 22:03:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform servergroup.addmember notification https://review.openstack.org/541101 | |