| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-07 | |||
| 16:19:22 | mriedem | i don't see why --version is different from --local_cell | |
| 16:19:25 | mriedem | since they are both options | |
| 16:19:36 | mriedem | plus i feel like this is a distraction right now | |
| 16:21:56 | mriedem | sdague: how is https://review.openstack.org/#/c/490644/ different from https://review.openstack.org/#/c/478485/ ? | |
| 16:22:05 | mriedem | note i can't get to docs-draft.openstack.org/44/490644/3/check/gate-nova-docs-ubuntu-xenial/bd5e01d//doc/build/html/ right now | |
| 16:22:45 | openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | |
| 16:23:43 | sdague | mriedem: the TOC approach | |
| 16:24:10 | sdague | mriedem: see commit message | |
| 16:39:05 | openstackgerrit | Merged openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | |
| 16:44:01 | mriedem | sdague: do we need to worry about redirects for anything in https://review.openstack.org/#/c/490644/ ? | |
| 16:44:41 | mriedem | sdague: or is that handled by https://review.openstack.org/#/c/490644/3/doc/source/index.rst@186 ? | |
| 16:45:34 | mnaser | jaypipes do you think proposing a fix to replace the existing ram weighter by percentage would be an acceptable change | |
| 16:45:43 | mnaser | or introduce a new weighter instead? | |
| 16:45:59 | mnaser | because i genuinely think a ram weighter based on free memory only is not correct | |
| 16:47:08 | mriedem | sdague: nvm, looks like nothing actually moves | |
| 16:47:13 | mriedem | so no redirects needed | |
| 16:49:03 | jaypipes | mnaser: a new weigher would be better. | |
| 16:50:21 | openstackgerrit | Ed Leafe proposed openstack/nova master: Fix migrate single instance when it was created concurrently https://review.openstack.org/491439 | |
| 16:51:26 | edleafe | bauzas: ^^ just fixed the pep8 issue | |
| 16:55:22 | sdague | mriedem: right, I did not move any source files | |
| 16:55:43 | sdague | I was just trying to build landing pages, and tame the TOC on the left sidebar which is a confusing mess right now | |
| 16:56:08 | mnaser | ehh | |
| 16:58:37 | dansmith | jaypipes: mnaser agreed | |
| 16:58:46 | dansmith | behavioral change may surprise people | |
| 16:59:10 | mnaser | yep, that's what i thought, i'm working on something now | |
| 16:59:15 | jaypipes | cool | |
| 16:59:48 | stvnoyes | mriedem mdbooth - I updated review https://review.openstack.org/#/c/389798/ - it's a small change. please take a look when you have a minute. Thanks. | |
| 17:01:16 | jaypipes | dansmith: correct. | |
| 17:16:46 | stephenfin | mriedem: Further replies left on https://review.openstack.org/#/c/453808/ | |
| 17:17:04 | stephenfin | I know it's a distraction and would understand if it were ignored but it totally blocks the (undoubtedly fabulous) cliff integration for another cycle which would be rather depressing | |
| 17:18:26 | openstackgerrit | Merged openstack/nova master: Add Contributor Guide section page https://review.openstack.org/490644 | |
| 17:18:30 | mriedem | stephenfin: why does --verbose block that but --local_cell does not? | |
| 17:18:47 | mriedem | why doesn't cliff allow us to have options, in other words | |
| 17:58:08 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher https://review.openstack.org/491561 | |
| 17:58:12 | mnaser | dansmith jaypipes i havent contributed to nova in a while so i welcome all critique whenever you have time ^ | |
| 17:58:17 | openstackgerrit | Merged openstack/python-novaclient master: tools: Remove dead script https://review.openstack.org/480138 | |
| 17:58:59 | jaypipes | mnaser: cheers | |
| 18:26:06 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle ironicclient failures in Ironic driver https://review.openstack.org/487925 | |
| 18:37:40 | jaypipes | dansmith: what's the difference between rebuild and evacuate? | |
| 18:38:03 | dansmith | jaypipes: rebuild is in place on the same compute node | |
| 18:38:04 | dansmith | jaypipes: evacuate means "rebuild me somewhere else" and has to be run on a down compute node | |
| 18:38:12 | dansmith | jaypipes: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | |
| 18:38:33 | jaypipes | dansmith: thx. sorry, just needed a reminder after looking at the code... :( | |
| 18:38:45 | dansmith | np | |
| 18:44:45 | mnaser | dansmith jaypipes and evacuate when using ceph means start me somewhere else (not rebuilding) | |
| 18:45:07 | jaypipes | mnaser: when using ceph *for the root disk*. :) | |
| 18:45:08 | dansmith | mnaser: it's still a rebuild under the covers in nova, | |
| 18:45:14 | dansmith | it just doesn't erase the disk | |
| 18:45:17 | dansmith | which is covered in the above link | |
| 18:45:56 | mnaser | yes, that's correct. when reading that link, volume-backed made me think cinder but indeed | |
| 18:46:35 | dansmith | well, it says shared storage in one place too, but yeah | |
| 18:54:38 | mriedem | +2 on bauzas' IndexError fix https://review.openstack.org/#/c/491439/ | |
| 18:58:18 | sdague | mriedem: also this doc page when you get a chance - https://review.openstack.org/#/c/490994/ | |
| 18:59:03 | sdague | mriedem: +A | |
| 18:59:47 | mriedem | yup the tab is open | |
| 19:25:36 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/491296 | |
| 19:26:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc https://review.openstack.org/491581 | |
| 19:26:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | |
| 19:26:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | |
| 19:46:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | |
| 19:49:09 | openstackgerrit | Merged openstack/python-novaclient master: Fix reservation_id not supported by Nova API https://review.openstack.org/468444 | |
| 19:58:35 | mriedem | stephenfin: why didn't we migrate the hypervisor-specific config references? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/config-reference/source/compute/hypervisor-hyper-v.rst | |
| 20:00:41 | sdague | mriedem: I think the answer to many things is going to be there are so many documents, and the code reviews are slow enough, that it's taking a while to get them in | |
| 20:01:20 | mriedem | i didn't know if it was intentional | |
| 20:01:47 | sdague | it's also about finding the right places for things to go | |
| 20:02:32 | sdague | yeh, I don't know, but I do know in building some of these sub pages that giving this thing any narative cohesion starting with 100 pages of mostly random notes people made at different depths, is challenging | |
| 20:03:10 | openstackgerrit | Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | |
| 20:06:34 | openstackgerrit | Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | |
| 20:10:38 | mriedem | sdague: ok questions in https://review.openstack.org/#/c/490994/ | |
| 20:21:21 | sdague | looking | |
| 20:23:13 | sdague | mriedem: hopefully answers | |
| 20:23:57 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Base _ContextAuthPlugin on BaseIdentityPlugin https://review.openstack.org/491203 | |
| 20:25:25 | mriedem | ok reply to the reply | |
| 20:25:33 | sdague | mriedem / sfinucan / asettle - https://etherpad.openstack.org/p/ideal-nova-docs-landing-page as I stare at things about how to get to the next slice here, this is kind of what I was thinking about for an ideal front page flow | |
| 20:25:40 | sdague | and how things would chunk up the rest of the way | |
| 20:25:49 | mriedem | if the cells v2 layout thing stays in "Architecture Evolution Plans" as an ops guide then we probably need to rename "Architecture Evolution Plans" | |
| 20:26:37 | sdague | mriedem: right, so that was next on the chopping block | |
| 20:26:47 | sdague | you want to look at that etherpad quick? | |
| 20:26:58 | mriedem | just did | |
| 20:28:26 | sdague | so I think we could get there in 3 more patches. One that was the For Operators and one For Users and one smoothing out the index | |
| 20:29:47 | sdague | and if we get a general ack that's how we want the world to look, I'll help write those bits | |
| 20:32:14 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher https://review.openstack.org/491561 | |
| 20:44:02 | mriedem | i've already found some gems in the docs about stuff that's way old, like the nova-api service description only mentions xml requests | |
| 20:44:23 | mriedem | and all the ec2 api stuff needs to go away | |
| 20:47:00 | dansmith | mriedem: we're dropping CD support, we might as well go back to XML anyway right? | |
| 20:47:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | |
| 20:47:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc https://review.openstack.org/491581 | |
| 20:47:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | |
| 20:47:39 | mriedem | did we mark json as experimental and never unmark it? | |
| 20:56:18 | cdent | dansmith: naw, backward to SOAP and forward to GRPC | |
| 20:57:52 | sdague | mriedem dropped already... eh? | |
| 21:02:51 | openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for local delete allocations https://review.openstack.org/470578 | |
| 21:31:56 | openstackgerrit | Merged openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520 | |
| 21:41:05 | openstackgerrit | Ed Leafe proposed openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491 | |
| 21:47:35 | openstackgerrit | Merged openstack/nova master: Detach device from live domain even if not found on persistent https://review.openstack.org/488545 | |
| 21:48:03 | openstackgerrit | Merged openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730 | |
| 21:58:50 | cdent | jaypipes, dansmith: i’m somewhat lost on the current scheduler/placement situation. What’s currently on the urgent to do list? (I’m trying to make my plan for tomorrow morning) | |
| 21:59:18 | dansmith | cdent: waiting for jaypipes to refresh the patches, AFAIK | |
| 21:59:43 | jaypipes | dansmith, cdent: yeah, I'm still on it :( I'm lost in unit test hell. :( | |
| 22:00:04 | cdent | okay, I’ll go looking for those in the morning | |