| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 14:34:57 | mriedem | stephenfin: https://review.openstack.org/#/c/489650/ fixes a bunch of links, | |
| 14:35:05 | mriedem | but some are still broken https://review.openstack.org/#/c/489650/ | |
| 14:35:14 | stephenfin | mriedem: Yeah, looks like we got bogged down with those patches. So just that one change left? | |
| 14:35:26 | mriedem | oops https://review.openstack.org/#/c/489641/ | |
| 14:35:32 | stephenfin | Oh, two so | |
| 14:35:37 | stephenfin | I assume the docs patches still open aren't subject to the feature freeze? | |
| 14:35:38 | mriedem | well, those are the 2 we know of | |
| 14:35:44 | mriedem | no they aren't | |
| 14:35:53 | stephenfin | ...but should now include additions to this htaccess file | |
| 14:35:56 | mriedem | sdague and i are concerned about the massive one that changes the indexes | |
| 14:36:03 | tikitavi | hi, I’m from ec2 team, we need some help | |
| 14:36:17 | stephenfin | mriedem: index.rst ? | |
| 14:36:24 | mriedem | this one https://review.openstack.org/#/c/478485/ | |
| 14:36:30 | tikitavi | why extensions stopped working? in particular, extended_volumes.py stopped adding info about attached volumes to server list and server show? | |
| 14:36:50 | mriedem | stephenfin: i also noted in https://review.openstack.org/#/c/477497/ that it's missing a bunch of nova-specific CLI guides that are now gone from the admin guide | |
| 14:36:58 | mriedem | but are still in the stable/ocata branch for openstack-manuals | |
| 14:37:15 | sdague_ | stephenfin: I'm more concerned about breaking up things like the policy doc, so that the only way to find information about nova policy is to already understand it, and know which subdoc to be in | |
| 14:37:19 | stephenfin | mriedem: Yeah, I'm working with asettle on that this week | |
| 14:37:46 | sdague_ | for stuff like that long pages that you can search in browser in are often more useful | |
| 14:37:52 | mriedem | stephenfin: also, do we know if these got in from the user guide? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-launch-instances.rst https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-delete-an-instance.rst ? | |
| 14:38:33 | mriedem | stephenfin: maybe we push https://review.openstack.org/#/c/478485/ to the end and actually just work on getting the migration done for pike | |
| 14:38:35 | stephenfin | sdague_: tbh, I'm not sure what's changed in that doc so I need to look at that. If you haven't left that comment on Gerrit yet, can you do so and I'll tackle it once I get there? :) | |
| 14:38:43 | mriedem | and worry about completely re-arranging everything after that's done | |
| 14:38:45 | sdague_ | stephenfin: I did | |
| 14:38:54 | stephenfin | sdague_: Ta | |
| 14:39:27 | stephenfin | mriedem: Yeah, a mass move and later refactor was my initial plan | |
| 14:39:28 | mriedem | it also seems if we're going to re-arrange the home page for nova to be a certain form, that template should follow suit in the other projects, | |
| 14:39:34 | mriedem | and today it's not for cinder/glance/neutron either | |
| 14:39:36 | mriedem | they are all different | |
| 14:40:25 | mordred | efried: ok - I left a pile of additional comments | |
| 14:40:30 | mordred | efried: don't hate me too much | |
| 14:40:52 | efried | mordred Thanks! I will report back on hate-level once I've read 'em. | |
| 14:41:40 | stephenfin | OK, I'll work with asettle on that | |
| 14:41:59 | mriedem | stephenfin: i can also summarize concerns in the ML quick | |
| 14:42:03 | stephenfin | mriedem: As for user-guide etc., Chason Chan (no idea what his IRC nick is) has patches up to move those | |
| 14:42:12 | asettle | We will! | |
| 14:42:15 | asettle | (what are we working on?) | |
| 14:42:32 | mriedem | stephenfin: he has install guide and admin guide, not the user guide | |
| 14:42:33 | stephenfin | that will also include the non-autogenerated cli stuff | |
| 14:42:46 | mriedem | i'll post a quick brain dump to the ML | |
| 14:42:48 | stephenfin | I'll do that one so | |
| 14:42:51 | stephenfin | mriedem: Sounds good | |
| 14:43:40 | stephenfin | asettle: The same thing we're always working on, asettle. Trying to take over the world | |
| 14:43:54 | asettle | Of course, Pinky | |
| 14:43:56 | stephenfin | Also, index.rst hijinks | |
| 14:44:07 | asettle | That's less cool. | |
| 14:51:38 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 14:55:33 | mriedem | stephenfin: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120482.html | |
| 14:56:01 | stephenfin | mriedem: Correct - *temporarily back | |
| 14:56:28 | stephenfin | but let's skip over that | |
| 14:56:38 | mriedem | my god | |
| 14:57:05 | mriedem | how dare you enjoy being young and single | |
| 15:01:08 | stephenfin | mriedem: Replied there. tl;dr: I agree with all point and will handle them this week | |
| 15:04:11 | ildikov | stephenfin: do the full experience with a tent, etc? | |
| 15:04:45 | stephenfin | ildikov: Yup, the full deal | |
| 15:05:15 | openstackgerrit | Gergely Csatari proposed openstack/nova master: Empty adminPass in Change Administrative Password https://review.openstack.org/490055 | |
| 15:05:27 | ildikov | stephenfin: sounds cool! | |
| 15:05:32 | stephenfin | Not looking forward to the 8am starts because the sun is up and you either get out of the tent and die from a hangover or slowly melt in the tent | |
| 15:05:54 | ildikov | stephenfin: yeah, weather is a bit crazy over here | |
| 15:06:19 | ildikov | stephenfin: the forecast says some storms for next week, but it's a whole week so they have no idea yet anyway :) | |
| 15:06:40 | ildikov | stephenfin: but hopefully will cool down to manageable by next week regardless | |
| 15:08:06 | mordred | efried: ok - I've been nerd-sniped ... | |
| 15:08:16 | stephenfin | ildikov: I hope so. No one wants rain but sub 30 temperatures wouldn't go amiss, heh | |
| 15:08:17 | efried | mordred ? | |
| 15:08:32 | stephenfin | I'm sure I'll survive in any case though :) | |
| 15:08:58 | ildikov | stephenfin: yeah, I didn't want to scare you with it, just wanted to be sure you're prepared :) | |
| 15:09:01 | mordred | efried: I just wrote most of the auth followup patch as a result of reading through to make sure I understood where the issues were ;) | |
| 15:09:08 | efried | mordred Figuring out how to get the auth into the glance client ... | |
| 15:09:15 | mordred | yup. done. patch coming | |
| 15:09:19 | sdague_ | mriedem: you misread my email about what's not being used in multinode | |
| 15:09:21 | stephenfin | ildikov: Ha, cheers :) | |
| 15:09:24 | efried | mordred Cool beans. | |
| 15:09:25 | mordred | (this is the nerd-sniping that just hapepned) | |
| 15:09:46 | mordred | efried: there is a test case that I didn't fix because it's a bit yuck and I don't know enough about the nova test cases | |
| 15:09:51 | mriedem | sdague_: i realized after i sent that | |
| 15:09:58 | ildikov | stephenfin: :) | |
| 15:10:00 | mordred | (basically, there's a test case that doesn't have any auth context info already prepared) | |
| 15:10:08 | mriedem | "it" with multiple nouns in the same sentence has been hurting me lately | |
| 15:10:31 | sdague_ | heh, no problem | |
| 15:10:35 | stephenfin | ildikov: I've learnt from my ill fated "Sure, you'll only need a light cotton sheet to sleep with" experience at Zurich last year. Turns out it gets colddd at night | |
| 15:10:51 | sdague_ | mriedem: I also responded, email should hit soon | |
| 15:11:05 | mriedem | i will prepare to respond to your response | |
| 15:12:18 | mriedem | wow, has anyone realized that a single trove change runs 26 jobs? | |
| 15:15:36 | ildikov | stephenfin: yeah, better to prepare as if you're lucky it gets down to 15 degrees Celsius at night the second half of next week | |
| 15:16:31 | ildikov | stephenfin: which I like way better than 27 even if I have to sleep in a tent, but we all have different tastes :) | |
| 15:22:38 | openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Use auth from context for glance api servers https://review.openstack.org/490057 | |
| 15:22:41 | mordred | efried: ^^ | |
| 15:22:50 | efried | mordred Ack | |
| 15:23:01 | mordred | efried: I didn't fix nova/tests/unit/api/openstack/compute/test_images.py because of the auth context thing | |
| 15:23:13 | efried | mordred You gonna? | |
| 15:27:53 | mordred | efried: not this moment - I hit my nerd-snipe limit for the day - feel free to use/takeover/ignore that patch - or if I get stuck on other things later today and need a different mental task I may try poking again | |
| 15:28:23 | efried | mordred Rgr. My comments may be for myself, then :) | |
| 15:29:01 | mordred | efried: woot! | |
| 15:38:48 | mriedem | cdent: coming back to your question about the plan for fixing https://bugs.launchpad.net/nova/+bug/1707252 i don't know of one | |
| 15:38:48 | openstack | Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed] | |
| 15:39:38 | mriedem | but it's not part of https://review.openstack.org/#/c/488510/ | |
| 15:39:47 | cdent | mriedem: yeah | |
| 15:40:28 | cdent | mriedem: the ideas I’ve heard batted around include: | |
| 15:40:40 | mriedem | the issue is here https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L196 | |
| 15:41:02 | cdent | adding migration uuids and adding the old allocation using the migration uuid, setting the consumer_uuid to the new allocation | |