Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-06
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
22:26:40 mriedem then i stepped in the ptg pile of poo in -tc
22:26:45 mriedem and am trying to also fix up a patch locally..
22:27:10 cdent yeah, the change touches lots of stuff, but it is almost all mechanical
22:34:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Save admin password to sysmeta in libvirt driver https://review.openstack.org/543032
22:40:42 mriedem and gerrit is down
22:41:37 mriedem that's my cue to exit
22:42:02 cdent meh
22:46:16 openstackgerrit Chris Dent proposed openstack/nova master: Move resource provider objects into placement hierarchy https://review.openstack.org/540049
22:46:17 openstackgerrit Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
22:46:17 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
22:46:18 openstackgerrit Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
22:48:10 openstackgerrit Russell Tweed proposed openstack/nova master: Fix invalid UUIDs in remaining tests https://review.openstack.org/550238
22:53:10 openstackgerrit Merged openstack/nova master: Fix the notification devref location in exception https://review.openstack.org/549816
22:58:27 openstackgerrit Hongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot https://review.openstack.org/511074
23:17:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update contributor guide for Rocky https://review.openstack.org/550288
23:38:20 openstackgerrit Hongbin Lu proposed openstack/nova master: Handle IpAddressAlreadyAllocated exception https://review.openstack.org/535532
23:40:06 openstackgerrit Merged openstack/nova master: Remove MigrationPreCheckClientException https://review.openstack.org/542704
#openstack-nova - 2018-03-07
00:55:37 openstackgerrit Merged openstack/nova master: Add more functional test for placement.aggregates https://review.openstack.org/514109
01:16:34 Kevin_Zheng gibi, sure, I will be glad to help
01:17:55 Spaz-Work Morning Novaers
01:55:33 openstackgerrit Merged openstack/nova master: Save admin password to sysmeta in libvirt driver https://review.openstack.org/543032
02:11:59 Spaz-Work jianghuaw_: You back home safe and in work?
03:31:21 Spaz-Work Whew i'm bad at tests
04:47:06 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI: XCP2.1+ Swallow VDI_NOT_IN_MAP Exception https://review.openstack.org/538415

Earlier   Later