Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-25
16:15:14 melwitt yeah, I think he's not saying he's reporting it for ocata. "First, why the latest version of nova not supports migrating instance to same host event through some drivers support that function..Second, if there is any problem when user migrates instance without migration destination host but there is just one compute node."
16:15:57 melwitt eh, I dunno, I think I'm just confused by the wording of these comments
16:18:33 mriedem i left a comment to try and answer the questions from comment 2
16:20:11 melwitt thanks. I think I see now, he's saying why go to error status when it "fails" because migrating to self doesn't really do anything
16:21:07 melwitt ok that comment makes it clear. good
16:21:13 mriedem correct, but the rejection in the API is not the same as the bug in the compute that sets the instance to error status
16:21:31 mriedem i've got fixes posted above for the latter, not the former
16:21:54 melwitt gotcha. I've got it starred for review
16:29:30 mriedem melwitt: if https://blueprints.launchpad.net/openstack/?searchtext=conf-max-attach-volumes is ready for review again you want to queue it up in runways?
16:29:46 mriedem jangutter's can probably go in the open slot as well
16:30:10 melwitt mriedem: sure. I don't know why I didn't think of doing that
16:34:51 jangutter melwitt, mriedem: thanks!
16:35:41 mriedem pretty easy fix here for someone https://review.openstack.org/#/c/624959/
16:35:46 mriedem needs to be backported to queens as well
16:39:49 melwitt jangutter: the bp dependencies show that vrouter-hw-offloads also depends on the generic-os-vif-offloads too. should I add that patch link in there too for the runway review?
16:40:37 sean-k-mooney melwitt: i think the os-vif stuff was all in the 1.14 release we did this week
16:40:41 jangutter melwitt: the dependency's unblocked on that one (generic-os-vif-offloads has follow-on work)
16:41:09 sean-k-mooney jangutter: more stuff in os-vif "e.g. that i should review" or in nova
16:41:15 melwitt oh, I see, the dependency piece is unblocked but bp still needs more work to be complete
16:41:16 sean-k-mooney which i should proably also review
16:41:17 melwitt ok thank you
16:41:48 jangutter melwitt: the dominoes are all ready to fall... for the vrouter bits though.
16:42:05 melwitt coolness
16:45:51 jangutter sean-k-mooney: for the generic-os-vif-offloads I think there's just one more bit in nova (for Stein) to populate the data. The next os-vif change should really only be appropriate for Train.
16:48:48 sean-k-mooney ok
16:49:40 sean-k-mooney if there is stuff i need to review for stien let me know. the non client lib frezze is in 3-4 weeks so if another os-vif release is need we need to merge the code before then
16:50:14 sean-k-mooney form the non-client-lib freeze untill rc1 os-vif will basically be frozen
16:50:41 sean-k-mooney form rc1/rc2 on it will be reopen for train
16:51:09 melwitt feb 28 is non-client lib freeze
16:51:38 jangutter sean-k-mooney: thanks, the only open one I have for Stein is the serialization cleanup patch, and that kinda-sorta depends on a style choice.
16:52:13 sean-k-mooney ok thats what i had understood
16:52:38 jangutter sean-k-mooney: I _think_ the only affected party to that would be kuryr and not Nova.
17:06:58 melwitt leakypipes, mriedem: am I correct in my understanding that because this spec about "how to use initial allocation ratios" got stuck https://review.openstack.org/544683, we don't and will not have support for being able to set allocation ratios per aggregate?
17:08:41 mriedem you can do it via placement externally if you want, and could have since forever - and is what we said people could do in dublin
17:08:44 mriedem but no one worked on that,
17:08:48 mriedem or added anything to osc-placement for it
17:09:06 mriedem the nova spec is about mirroring aggregate allocation ratios to placement
17:09:34 mriedem which is a nice to have but less necessary since you can do that directly with placement and we have rbac support in placement now too (which was a previous reason this should go through nova - for rbac)
17:10:06 mriedem so the only sticking point on the nova spec is deciding if nova should do that mirroring / proxy orchestration work
17:10:25 mriedem clearly that isn't going to happen in stein though
17:10:28 mriedem bbiab
17:12:41 efried_mtg sean-k-mooney: not sure if you knew about this, but when resubmitting a series, git-review normally re-topics the whole series to match the top patch. You can make it not do that - i.e. leave the topic unchanged (or empty if it's the first time you're uploading it) by using the -T flag.
17:12:50 melwitt mriedem_afk: thanks, I'll make some notes about it on the blueprint tracking etherpad
17:13:31 sean-k-mooney efried_mtg: ya it has bitten me in the past
17:13:56 melwitt I've had a hard time following what was going on there. didn't realize the only thing left was mirroring, and I agree I'm not sure whether we should
17:14:32 sean-k-mooney efried_mtg: generally its does what i want as if i am working on a patch serise its all for the same bug/blueprint but it can be annoying
17:15:31 sean-k-mooney efried_mtg: i just like the fact i can "git checkout -b whatever" and then "git revew -f" and have the topic set correctly to "whatever"
17:16:42 leakypipes melwitt: I'll abandon that spec.
17:18:14 sean-k-mooney leakypipes: you spec for mirroring nova aggrages as placement aggrates was complete in rocky rigt? seperate form the allcoation ratio stuff
17:18:16 melwitt leakypipes: k. I was only asking because I'm trying to round up things we identified as priority items and that one I had trouble parsing what the second spec was about
17:18:32 melwitt *priority items at the ptg
17:19:41 melwitt sean-k-mooney: mirroring aggregates was a different thing yeah, and is already in nova
17:19:47 leakypipes sean-k-mooney: yes.
17:20:27 leakypipes melwitt: np
17:20:56 sean-k-mooney cool so in thery you coudl write a scipt/tool to find all the RPs with a specific resouce in an aggrate and set the allocation ratio in the inventory relitivly simply if you wanted too
17:26:51 melwitt yeah, we talked about adding a command to osc-placement CLI to help with that. we should do that. maybe I'll try to do it if I can get my other stuff done
17:39:59 openstackgerrit Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178
17:40:30 openstackgerrit Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178
18:17:05 artom We can just disable the Keystone middleware and run without authentication, right?
18:17:26 sean-k-mooney artom: you can disable auth yes
18:17:30 artom (Asking for a performance thing downstream, suspecting Keystone is the culprit, want to propose testing without it)
18:21:09 sean-k-mooney i guessed. let me see if i can find the config
18:21:33 artom sean-k-mooney, too late, email went out. I'll blame you if people pick me up on it :D
18:22:37 cdent artom, sean-k-mooney: I'm not positive but I think you have to choose a different auth strategy (noauth2) and still set x-auth-token headers so that policy checks still have something to chew on. If that can be avoided too, I'd love to hear about it.
18:23:01 sean-k-mooney artom: i think you set this https://docs.openstack.org/nova/latest/configuration/config.html#keystone_authtoken.auth_type
18:23:12 artom cdent, right, but that avoids the authentication calls to the Keystone API, right?
18:23:19 cdent yes
18:23:30 artom OK, because that's what I care about
18:23:59 cdent I do that noauth2 stuff all the time with placement and it seems to work fine with a 'x-auth-token: amdin' header
18:24:00 sean-k-mooney cdent: ya our functional test use noauth2 i think when they spin up apis
18:24:18 cdent except I spell admin correctly...
18:24:20 cdent sigh
18:24:39 sean-k-mooney just alias it it will be fine
18:24:52 cdent :)
18:24:58 artom Am Din. You be?
18:25:09 cdent Me be me
18:25:10 sean-k-mooney i have always beeen temeted to make "suod" and alais of "sudo"
18:25:31 artom And "temeted" an alias of "tempted", and "alais" of "alias"
18:25:33 artom ;)
18:25:46 artom cdent, but will the FCC let you be?
18:26:22 sean-k-mooney hehe the problem is i mispell things inconsitently
18:26:35 cdent artom++
18:27:11 artom cdent, .... and now I'm listening to it
18:27:25 cdent yeah, I was headed that way too
19:11:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Isolate cell-targeting code in MigrationTask https://review.openstack.org/621310
19:11:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Extract compute API _create_image to compute.utils https://review.openstack.org/621311
19:11:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
19:11:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035
19:11:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
19:11:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
19:11:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
19:11:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313
19:11:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123
19:11:55 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add TargetDBSetupTask https://review.openstack.org/627892
19:11:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581
19:11:56 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
19:11:57 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
19:43:00 melwitt is there no way to link to an archived ML post from openstack-dev@?
19:43:20 melwitt from before the switch to the new list
19:44:09 melwitt ok, I found a way. just couldn't get to the root page for openstack-dev anymore

Earlier   Later