| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-05 | |||
| 20:45:05 | melwitt | but that's not subject to spec freeze | |
| 20:45:15 | melwitt | I don't think, anyway | |
| 20:57:28 | mriedem | nova list --limit -1 works like a charm http://paste.openstack.org/show/722757/ | |
| 21:01:59 | melwitt | mriedem: kinda wonder why that's not the default. I guess so you have to opt into paging | |
| 21:07:27 | mriedem | frickler: are you happy with this now? https://review.openstack.org/#/c/312626/ | |
| 21:07:33 | mriedem | i see you've done quite a bit of review on it already | |
| 21:08:04 | mriedem | melwitt: idk, for a smaller cloud with <1000 instances it's not going to be a problem, | |
| 21:08:15 | mriedem | for a public cloud the admin probably wants to opt into getting back a million records | |
| 21:08:23 | melwitt | yeah, good point | |
| 21:12:19 | mriedem | anyway, that patch made around ~400 LOC change for something that wasn't even broken | |
| 21:12:35 | mriedem | no release note on the behavior change either | |
| 21:13:21 | melwitt | those are the best | |
| 21:24:05 | melwitt | mriedem: I proposed adding the nova-lvm job to pike (as experimental) and it passed (phew) https://review.openstack.org/572130 | |
| 21:24:36 | melwitt | the ocata one however is busted, but on a cinder problem AFAICT, not the nova regression/fix https://review.openstack.org/572132 | |
| 21:29:47 | mriedem | ok | |
| 21:31:31 | mriedem | mgagne: a couple of questions for clarification in your spec https://review.openstack.org/#/c/312626/ | |
| 21:31:34 | melwitt | gah why can I never find the novaclient cli command list reference | |
| 21:31:44 | mriedem | melwitt: i don't think there is one | |
| 21:31:51 | mriedem | you're welcome | |
| 21:31:55 | melwitt | oh, to make people use osc. I see how it is | |
| 21:31:59 | mriedem | https://docs.openstack.org/python-novaclient/pike/cli/index.html | |
| 21:32:08 | mriedem | no, i think just because it never existed | |
| 21:32:21 | melwitt | it existed, unless I dreamed it | |
| 21:32:36 | melwitt | I shall google | |
| 21:32:39 | melwitt | some more | |
| 21:33:12 | mriedem | https://docs.openstack.org/mitaka/cli-reference/nova.html ? | |
| 21:33:26 | melwitt | yeah, that's what I'm thinking of. maybe we didn't import that when things got decentralized | |
| 21:33:28 | mriedem | so it existed in the central docs repo | |
| 21:33:34 | mriedem | yeah must not have | |
| 21:33:54 | melwitt | probably to discourage use of the novaclient CLI | |
| 21:34:12 | mriedem | no, probably just because no one imported the docs | |
| 21:34:15 | mriedem | it was on the projects to do that | |
| 21:34:21 | mriedem | https://docs.openstack.org/ocata/cli-reference/nova.html | |
| 21:34:27 | mriedem | https://docs.openstack.org/pike/cli-reference/nova.html | |
| 21:34:55 | melwitt | okay, I thought maybe someone decided it wasn't a great idea because we want to encourage osc use. I could see that point of view | |
| 21:35:28 | melwitt | but, I do know that boot from volume created from image is not possible in osc in one command, still something that needs to be fixed. and there are probably other things people still need novaclient CLI for | |
| 21:36:14 | mriedem | https://bugs.launchpad.net/python-novaclient/+bug/1775281 | |
| 21:36:15 | openstack | Launchpad bug 1775281 in python-novaclient "nova command line reference is gone since Pike" [Undecided,New] | |
| 21:36:27 | mriedem | there is a bunch of stuff that is in the nova CLI that's not in OSC | |
| 21:36:37 | melwitt | yeah | |
| 21:36:41 | mriedem | so purposefully hiding the nova cli docs would be a dick move | |
| 21:36:50 | mriedem | which i don't think was the case, it was just laziness | |
| 21:37:36 | melwitt | yeah, I agree that it would be best to host them in our tree | |
| 21:37:46 | mriedem | https://github.com/openstack/openstack-manuals/blob/before-migration/doc/cli-reference/source/nova.rst | |
| 21:39:01 | melwitt | thanks for finding that | |
| 21:39:27 | melwitt | and opening the bug | |
| 21:43:52 | openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | |
| 21:50:53 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Import nova CLI reference from openstack-manuals https://review.openstack.org/572567 | |
| 21:50:54 | mriedem | melwitt: there you go | |
| 21:54:12 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add a note in the nova CLI reference about using OSC https://review.openstack.org/572569 | |
| 21:54:44 | melwitt | \o/ | |
| 22:36:29 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Spike a direct interface to placement https://review.openstack.org/572576 | |
| 22:50:35 | mriedem | simondodsley_: some comments in your spec https://review.openstack.org/#/c/340168/ | |
| 22:50:42 | mriedem | mostly just needs to be cleaned up and add some more detail | |
| 22:50:48 | mriedem | otherwise this probably didn't even require a spec | |
| 22:57:31 | melwitt | mriedem: trying to understand this part about the policy check moving ... where did it used to be? https://review.openstack.org/#/c/565578/10/nova/tests/functional/api/openstack/placement/gabbits/resource-classes.yaml | |
| 22:58:11 | melwitt | with the move, it fails with 406 but doesn't that mean it passes the policy check when it shouldn't? (406 normally comes from further down the API, right?) | |
| 22:59:32 | mriedem | melwitt: https://review.openstack.org/#/c/524425/16/nova/api/openstack/placement/handler.py@217 | |
| 22:59:45 | mriedem | melwitt: the policy check used to happen before we got to the handler code, | |
| 22:59:52 | mriedem | but now the policy check happens in each handler | |
| 23:00:04 | mriedem | and that happens after the content type checks | |
| 23:00:13 | melwitt | I see, thank you | |
| 23:00:55 | mriedem | https://review.openstack.org/#/c/565578/10/nova/api/openstack/placement/handlers/resource_class.py@116 | |
| 23:01:10 | openstackgerrit | Merged openstack/nova stable/ocata: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571432 | |
| 23:01:16 | openstackgerrit | Merged openstack/nova master: Fix some wrong urls in doc https://review.openstack.org/571147 | |
| 23:01:45 | melwitt | ++ thanks | |
| 23:02:43 | mriedem | once you get past that, the rest of these are all pretty mechanical | |
| 23:02:57 | mriedem | https://review.openstack.org/#/c/524425/ was the majority of the meat of the series | |
| 23:04:58 | melwitt | sweet | |
| 23:06:03 | cdent | mriedem: you might be interested in some of my comments in the commit message on https://review.openstack.org/572576 which is the way it is in part because of easy way to manage context and policy | |
| 23:17:53 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: WIP: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583 | |
| 23:21:30 | openstackgerrit | Merged openstack/nova master: [placement] Do not import oslo_service for log_options https://review.openstack.org/572231 | |
| 23:23:23 | melwitt | mriedem: why does https://review.openstack.org/570621 need to use scope types whereas the others don't? | |
| 23:25:10 | mriedem | melwitt: it was the first place that it was obvious in the series; there was some discussion about it in https://review.openstack.org/#/c/524425/16 | |
| 23:25:22 | mriedem | so after that i did it for the rest and then filled the gap at the end | |
| 23:25:43 | mriedem | cdent: ack | |
| 23:25:45 | melwitt | I see, okay | |
| 23:25:50 | mriedem | going for a bike ride | |
| 23:39:44 | gmann | mgagne: glad to know that :) thanks | |
| 23:51:14 | openstackgerrit | Merged openstack/python-novaclient master: Import nova CLI reference from openstack-manuals https://review.openstack.org/572567 | |
| 23:55:03 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 23:55:04 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 23:55:05 | openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 23:58:16 | openstackgerrit | Merged openstack/python-novaclient master: Add a note in the nova CLI reference about using OSC https://review.openstack.org/572569 | |
| #openstack-nova - 2018-06-06 | |||
| 01:10:03 | mriedem | Kevin_Zheng: fyi https://review.openstack.org/#/c/569386/ | |
| 01:12:55 | openstackgerrit | Chris Dent proposed openstack/nova master: Ensure resource class cache when listing usages https://review.openstack.org/572652 | |
| 01:23:02 | Kevin_Zheng | mriedem ack | |
| 01:24:26 | Kevin_Zheng | mriedem: BTW, the max bulk boot BP will need further discussion? | |
| 01:25:22 | mriedem | Kevin_Zheng: https://review.openstack.org/#/c/510235/ ? yeah, it's a very rough work in progress | |
| 01:25:45 | mriedem | ideally i'd get some operator feedback before revising it | |
| 01:26:17 | mriedem | not really pushing to get that in for rocky | |
| 01:26:42 | Kevin_Zheng | OK, maybe we will take it over togother with the dettach root volume for next cycle | |
| 01:26:53 | mriedem | i think it's arguably also a bug fix / security hardening opportunity | |
| 01:39:34 | openstackgerrit | Merged openstack/nova-specs master: Rename spec to match blueprint https://review.openstack.org/568870 | |
| 01:40:00 | mriedem | lyarwood: i put some notes in my spec for you https://review.openstack.org/#/c/552078/ - i tried a readonly to readonly swap volume and didn't hit any failures | |
| 02:06:01 | pvc | h | |
| 02:06:02 | pvc | guys | |
| 02:07:30 | gmann | alex_xu: what you think of this - https://review.openstack.org/#/c/571335/1 | |
| 02:08:28 | alex_xu | gmann: I will check it soon | |
| 02:08:44 | gmann | thanks | |