| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-06 | |||
| 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 | |
| 22:03:12 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839 | |
| 22:04:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform live_migration.post.dest notifications https://review.openstack.org/469784 | |
| 22:06:38 | openstackgerrit | Chris Dent proposed openstack/nova master: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | |
| 22:06:48 | cdent | jaypipes: unwipped ^ | |
| 22:09:14 | jaypipes | cdent: w00t. | |
| 22:13:47 | openstackgerrit | Merged openstack/nova master: Pass user context to virt driver when detaching volume https://review.openstack.org/549411 | |
| 22:13:56 | openstackgerrit | Merged openstack/nova master: Check for multiattach before removing connections https://review.openstack.org/548427 | |
| 22:16:27 | cdent | mriedem: https://review.openstack.org/#/c/543469/ (which you just +A'd) was a child of a db change that is currently a WIP, waiting a bit more discussion. I unparented it at some point, but still looking for feedback on the validity of the approach in the old parent: https://review.openstack.org/#/c/543262/ The commit message explains why it would be nice to have | |
| 22:20:05 | mriedem | holy christ | |
| 22:23:26 | cdent | mriedem: are you holy christing that change, or sticky ptg conversations or something else? | |
| 22:26:31 | mriedem | cdent: it's a large change, so holy christ to that | |