| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-30 | |||
| 19:14:57 | melwitt | I see. ironically when I implemented counting quotas I had to set in_use = 0 for keypairs to keep the same behavior. I was thinking we might have to microversion to change that or otherwise signal it | |
| 19:17:01 | melwitt | so yeah, there are probably many improvements that could happen with the APIs. one thing to consider is that the keystone team is looking to add quota limits to keystone. which would make things separated where you'd have to query keystone for limits and nova for usage | |
| 19:22:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: Use oslo_db Session in resource_provider.py https://review.openstack.org/523192 | |
| 19:23:27 | jaypipes | efried: are you currently working on https://review.openstack.org/#/c/517119/? | |
| 19:24:10 | openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Handle concurrent registrations of the VC extension https://review.openstack.org/484667 | |
| 19:25:30 | cdent | mriedem: ^ thanks | |
| 19:27:01 | cfriesen | melwitt: do you know what the timeline is on the keystone quotas? If it's going to be a while I might have to propose an API change. | |
| 19:27:23 | mriedem | cdent: thanks - i was coding that in the comment out of memory, does it actualy pass the test? | |
| 19:27:31 | cdent | yes | |
| 19:27:35 | mriedem | cool | |
| 19:27:46 | efried | jaypipes No | |
| 19:28:03 | cdent | and I went to look at the excutils code just to make sure it made sense (because it was new to me) and your example matched the docstring’s example | |
| 19:28:10 | jaypipes | efried: k, I'm going to pull and work on that then | |
| 19:28:17 | efried | jaypipes Cool beans. | |
| 19:28:17 | melwitt | cfriesen: you'll want to ask lbragstad about it as he's in the know about keystone timelines | |
| 19:28:45 | cdent | efried, jaypipes : is there a new stable review-entry point for n-r-p? | |
| 19:29:07 | jaypipes | cdent: unfortunately not. | |
| 19:29:14 | efried | cdent I thought it was here: https://review.openstack.org/#/c/377215/ | |
| 19:29:47 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 19:29:48 | efried | cdent The bottom three patches | |
| 19:29:48 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 19:29:48 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 19:29:54 | lbragstad | cfriesen: we're in the process of firming up the specification for unified limits https://review.openstack.org/#/c/455709/ (wxy_ is actively responding to comments and making updates) | |
| 19:29:59 | efried | Oh, just updated | |
| 19:30:14 | jaypipes | cdent: I just addressed that microversion.py rebase flub, nothing more. | |
| 19:30:22 | cdent | roger | |
| 19:30:29 | lbragstad | cfriesen: wxy_ also has an early implementation proposed for review | |
| 19:30:43 | lbragstad | cfriesen: https://review.openstack.org/#/q/status:open+project:openstack/keystone+branch:master+topic:bp/unified-limits | |
| 19:31:02 | cdent | I think my best plan is perhaps to be done with openstack for today | |
| 19:31:30 | cdent | have fun everyone | |
| 19:32:31 | lbragstad | cfriesen: specification freeze for keystone is next week - so we're trying to get the details ironed out this week and next (all feedback welcome!) | |
| 19:33:36 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | |
| 19:33:36 | openstackgerrit | Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | |
| 19:33:37 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | |
| 19:33:37 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | |
| 19:33:38 | openstackgerrit | Jay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | |
| 19:33:38 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | |
| 19:33:53 | jaypipes | efried: rebased and fixed conflicts up to ^ | |
| 19:34:44 | jaypipes | efried: I stopped there because I don't know which of alex and tetsuro's add func tests patches are duplicated and which are real. | |
| 19:35:00 | efried | jaypipes ack | |
| 19:56:55 | melwitt | mriedem: turns out your test just always passes locally for me (on top of my celldatabases patch) even without the periodic task fix. so something must be afoot if you're getting a different result locally than me | |
| 19:58:47 | mriedem | melwitt: at some point i rebased out your change since it wasn't fixing stuff, that's when i started writing that workaround in start_service | |
| 19:59:13 | melwitt | oh, okay | |
| 20:01:19 | melwitt | if you still had my change when you fixed the wrong assert you mentioned earlier and it didn't pass, then that's really unexpected | |
| 20:04:30 | mriedem | unrelated, but if you use gerrit-dash-creator for your nova review dashboard, it's been fixed for zuul https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/nova.dash | |
| 20:04:42 | mriedem | i was wondering why my dashboard was showing me changes with a -2 as ready for final +2 | |
| 20:05:09 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 20:05:37 | melwitt | but I need to re-create my dashboard right? my old dashboard doesn't work anymore | |
| 20:05:56 | mriedem | correct | |
| 20:06:06 | mriedem | you have to git clone gerrit-dash-creator | |
| 20:06:15 | melwitt | okay, thanks | |
| 20:06:15 | mriedem | and then: tox -e venv -- gerrit-dash-creator dashboards/nova.dash | |
| 20:06:29 | mriedem | i put the url in a shortener and then put that in my gerrit settings | |
| 20:07:33 | mriedem | efried: seen this https://review.openstack.org/#/c/521764/ ? | |
| 20:07:36 | melwitt | sweet. good tips | |
| 20:18:19 | openstackgerrit | Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022 | |
| 20:28:05 | mriedem | dansmith: melwitt: cellsv2 thingamajig https://review.openstack.org/#/c/521837/5 | |
| 20:28:10 | mriedem | ux stuff | |
| 20:32:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update description of Rebuild in server_concepts.rst https://review.openstack.org/522086 | |
| 20:41:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715 | |
| 20:49:55 | openstackgerrit | Merged openstack/os-traits master: Updated from global requirements https://review.openstack.org/520477 | |
| 20:50:55 | efried | mriedem Nope, hadn't seen it. Thanks for the pointer. | |
| 20:52:53 | melwitt | mriedem: yeah, seems like it would be uncool of us to remove 'nova-manage host list' before we have another way to list hosts in a cell, even though it's deprecated | |
| 20:54:38 | melwitt | I was thinking discover_hosts had a --dry-run mode that just shows the hosts. but looks like not | |
| 20:54:45 | efried | jaypipes Care to eyeball https://review.openstack.org/#/c/524263/ ? Looks like we may want to backport it (mriedem?) | |
| 20:54:52 | mriedem | discover_hosts is idempotent so that's the fallback | |
| 20:54:56 | melwitt | although I guess a dry run would only show unmapped hosts | |
| 20:55:00 | melwitt | or does it show all? | |
| 20:55:09 | mriedem | efried: yes i think we probably want to backport that | |
| 20:55:15 | mriedem | i marked the bug as affecting stable | |
| 20:55:31 | efried | mriedem Should we tag the bug with placement, or is that only for stuff on the server side of placement? | |
| 20:55:42 | melwitt | I wouldn't be surprised if ppl would like to be able to see what it's going to map before it maps it | |
| 20:55:49 | mriedem | efried: not sure if that matters | |
| 20:55:55 | efried | ight | |
| 21:02:11 | rwellum | Hi folks. I'm seeing an issue with Pike nova, I assume it's my setup - but wondering if I could get a few pointers what to debug? This is running on kolla containers I should add. https://www.irccloud.com/pastebin/WH8k1XV3/ - looks like a database sync issue? | |
| 21:02:54 | mriedem | rwellum: yes, | |
| 21:03:04 | mriedem | it's looking in the cell0 database for the compute_nodes table, | |
| 21:03:11 | mriedem | so you must not have run 'nova-manage db sync' yet | |
| 21:03:20 | mriedem | after creating the nova_cell0 database | |
| 21:06:45 | rwellum | And that should be run on presumably one of the nova containers | |
| 21:08:03 | mriedem | shrug, i've never used kolla | |
| 21:08:15 | mriedem | i think they are in #openstack-kolla | |
| 21:09:09 | rwellum | Yeah I'm there :) Just not sure who should execute the sync. | |
| 21:09:18 | rwellum | But thanks this is really helpful | |
| 21:09:32 | openstackgerrit | Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/520478 | |
| 21:16:10 | jaypipes | efried: I'm kind of annoyed right now... I can't figure out for the life of me why https://review.openstack.org/#/c/384807/77//COMMIT_MSG changed back from 1.14 to 1.13... | |
| 21:18:03 | efried | jaypipes Probably you had PS74 locally. edleafe published PS75 with that being the only change. | |
| 21:18:31 | jaypipes | efried: but I pulled a fresh branch before I did anything. | |
| 21:18:42 | jaypipes | efried: whatevs, I've fixed it.. | |
| 21:18:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 21:18:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 21:18:46 | efried | jaypipes ack | |
| 21:19:33 | efried | jaypipes +1. Ship it! | |
| 21:19:40 | rwellum | mriedem: that indeed fixed the database sync error! However I still am getting 'No valid host was found. There are not enough hosts available' | |
| 21:19:58 | mriedem | rwellum: have you created cell1 and discovered hosts? | |
| 21:20:07 | mriedem | rwellum: i thought kolla automated all of this? | |
| 21:20:54 | dansmith | I thought kolla was just containers of stuff | |
| 21:23:42 | mriedem | i thought there was an ansible part to kolla for actually deploying openstack in those containers | |
| 21:23:52 | rwellum | Yeah kolla is just the images and some config built into the image. I'm orchestrating with kolla-kubernetes. Works with ocata btw. | |
| 21:23:55 | mriedem | clearly this isn't the place to ask about wtf kolla is or does :) | |