| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-29 | |||
| 13:07:17 | bauzas | well, I know I'm French, but when I use the "merge" verb, it has a very clear meaning to me :) | |
| 13:07:29 | bauzas | anyway | |
| 13:07:41 | edleafe | bauzas: for background, the discussion moved from gerrit to IRC at around 14:20 here: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-06-27.log.html | |
| 13:07:51 | bauzas | I loved those off days, but I hate how it hitted me | |
| 13:08:18 | edleafe | And to clarify, *in Jay's design*, the claim is now done in the manager | |
| 13:10:50 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 13:12:55 | bauzas | edleafe: just for my brain sanity, could we please use the word "proposed" instead of "done" if we speak of changes up ? :) | |
| 13:13:24 | bauzas | emotional lift, y'know :p | |
| 13:13:56 | gcb | mriedem: Can you revisit https://review.openstack.org/#/c/466787. it blocks oslo.messaging stable/newton release | |
| 13:21:09 | bauzas | edleafe: so, FWIW, after reading all of this, including https://review.openstack.org/#/c/478291/ I honestly think it's totally understandable to return HostStates instead of freeform dicts | |
| 13:21:30 | bauzas | the interface was very poor within the scheduler code, so I don't think it hurts | |
| 13:21:41 | mriedem | gcb: ok, i got really confused by the various backports for those changes, and they weren't consistent about the change id and some were abandoned, i.e. https://review.openstack.org/#/c/474456/ | |
| 13:22:01 | bauzas | the only drawback I would have said would be about custom drivers, but mriedem already got the word | |
| 13:22:11 | bauzas | which is that we no longer support custom drivers | |
| 13:22:32 | mriedem | bauzas: i already reported a docs bug abou that | |
| 13:22:39 | mriedem | since the docs tell you how to write your own scheduler driver | |
| 13:22:47 | bauzas | that sounds reasonable | |
| 13:22:56 | bauzas | we could still opt-in custom drivers | |
| 13:23:02 | openstack | Launchpad bug 1701105 in OpenStack Compute (nova) "Customizing the OpenStack Compute (nova) Scheduler in Operations Guide" [Medium,Confirmed] | |
| 13:23:02 | mriedem | bauzas: https://bugs.launchpad.net/nova/+bug/1701105 | |
| 13:23:03 | bauzas | after all, it's just adding a choice to an opt | |
| 13:23:26 | mriedem | if someone wants to hack a custom driver they are going to hack the config options, sure | |
| 13:23:32 | bauzas | but I strongly disapprove this | |
| 13:23:38 | mriedem | we just shouldn't advertise this as a built in capability that we support | |
| 13:23:42 | bauzas | given it's merely not interoperable | |
| 13:24:04 | bauzas | honestly, we did cut the possibility and nobody complainted | |
| 13:24:08 | bauzas | complained | |
| 13:24:18 | bauzas | if someone comes with a good usecase about that, we could discuss | |
| 13:24:32 | bauzas | but for the moment, no need to open a debate like this | |
| 13:25:06 | bauzas | and most of the operators wanting to run a specific scheduler can still use FilterScheduler and custom filters that call out to a 3rd party scheduler | |
| 13:26:42 | openstackgerrit | Merged openstack/python-novaclient master: Cleanup duplicated methods https://review.openstack.org/476493 | |
| 13:26:49 | mriedem | gcb: +W | |
| 13:27:08 | mriedem | bauzas: or they aren't upgraded to ocata yet | |
| 13:27:10 | mriedem | but anyway | |
| 13:27:40 | bauzas | mriedem: well, actually, that leaves me wonder one thing | |
| 13:27:59 | bauzas | mriedem: in the past, you could just disable all filters but just run your custom one | |
| 13:28:10 | bauzas | and then litterally do all nasty things you had in mind | |
| 13:28:20 | bauzas | like not counting against CPU or TAM | |
| 13:28:21 | bauzas | RAM | |
| 13:28:51 | bauzas | now that we check that by placement, I do wonder if someone could come up and say they wouldn't like to count like this | |
| 13:29:46 | bauzas | well, actually, we did enforced the CPU and RAM verifications in the compute claims | |
| 13:29:56 | bauzas | so, maybe it's just me overthinking | |
| 13:29:58 | bauzas | anyway | |
| 13:30:47 | openstackgerrit | Merged openstack/python-novaclient master: Deprecate binary argument in nova service enable/disable/force-down CLIs https://review.openstack.org/477212 | |
| 13:36:13 | MR2000 | Hi, Anybody there? I have a question about adding new subcommand to nova. | |
| 13:42:56 | mriedem | alex_xu: i'm +2 on the tagged attach api change now https://review.openstack.org/#/c/376021/ - you were +2 before | |
| 13:43:47 | mriedem | MR2000: it's ok to just ask | |
| 13:44:03 | artom | mriedem, is that our new coverage standard, "exxxxxxxtreme"? ;) | |
| 13:44:24 | artom | Aka - how tired are you right now? | |
| 13:44:59 | mriedem | artom: only 2xtreme | |
| 13:45:16 | mriedem | not 7x | |
| 13:45:45 | artom | Our Xs are like SLAs's 9s | |
| 13:51:16 | sfinucan | artom: mriedem likes his tests like he likes his pancake flipping | |
| 13:53:59 | mriedem | dansmith: why did we make instance.keypairs a list? | |
| 13:54:16 | mriedem | am i missing something or can you only associate a single keypair with an instance when it's created | |
| 13:54:43 | dansmith | mriedem: yeah, I think we just went future-looking | |
| 13:55:03 | dansmith | I vaguely remember a conversation | |
| 13:55:14 | mriedem | hmmm, ok because that's going to be codified in the versioned notifications https://review.openstack.org/#/c/419730/ | |
| 13:55:30 | dansmith | is that a problem? | |
| 13:55:31 | mriedem | the future conan? | |
| 13:55:33 | mriedem | no | |
| 13:55:41 | mriedem | i was just comparing instance.key_name to keypairs | |
| 13:56:02 | mriedem | sometimes i forget why we do what we do | |
| 13:56:05 | dansmith | heh | |
| 13:57:06 | mriedem | sfinucan: i don't get the pancake flipping thing | |
| 13:57:10 | mriedem | you should only flip a pancake once | |
| 13:58:21 | sfinucan | mriedem: Pancake flipping was the most mundane thing I could think of at the time | |
| 13:58:43 | sfinucan | Besides, only flipping once doesn't let you demonstrate your pancake flipping prowess to you non-flipping buddies | |
| 14:00:22 | bauzas | FWIW, creps are larger than pancakes hence harder to flip :p | |
| 14:00:29 | mriedem | you are going to have to work on some new analogies | |
| 14:00:45 | mriedem | i like my tests like i like my mt dew | |
| 14:00:50 | mriedem | exxxxxxxxxxxxxxxxxxxxtreme! | |
| 14:00:52 | mriedem | there | |
| 14:00:54 | mriedem | that's an analogy | |
| 14:02:58 | openstackgerrit | Alex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)' https://review.openstack.org/478900 | |
| 14:02:58 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore https://review.openstack.org/478901 | |
| 14:02:59 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore https://review.openstack.org/478902 | |
| 14:02:59 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903 | |
| 14:03:00 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore https://review.openstack.org/478499 | |
| 14:03:00 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463 | |
| 14:03:01 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes instead of stevedore https://review.openstack.org/478905 | |
| 14:03:01 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore https://review.openstack.org/478498 | |
| 14:03:02 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904 | |
| 14:03:02 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for volumes instead of stevedore https://review.openstack.org/478502 | |
| 14:04:02 | openstackgerrit | Merged openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037 | |
| 14:06:11 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | |
| 14:23:05 | sean-k-mooney | sfinucan: o/ can you take a look at https://review.openstack.org/#/c/467173/8 our requirement conflict witht he upstrem ones | |
| 14:23:48 | sean-k-mooney | sfinucan: or rather we are missing some blacklisted package versions | |
| 14:23:52 | sfinucan | sean-k-mooney: Done and done | |
| 14:24:02 | sean-k-mooney | sfinucan: cool thanks | |
| 14:25:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: create functional test base class https://review.openstack.org/407055 | |
| 14:25:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Validate huge pages https://review.openstack.org/399653 | |
| 14:29:06 | openstackgerrit | Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627 | |
| 14:30:51 | openstackgerrit | Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/467173 | |
| 14:31:21 | edleafe | mriedem: ^^ | |
| 14:32:46 | mriedem | oooo just in time | |
| 14:35:23 | mriedem | wow that was a nasty diff | |
| 14:35:51 | edleafe | mriedem: yeah, the section I changed was moved to another file | |
| 14:35:56 | edleafe | mriedem: always fun! | |
| 14:39:26 | mriedem | edleafe: dead code in the rebase | |