Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-20
19:28:35 dansmith melwitt: tbh, I'm not sure why we should change that at the moment
19:28:54 dansmith it's just reporting a different value for something we don't look at during scheduling anyway
19:30:34 melwitt they're saying it's being rejected at the check_can_live_migrate_source stage, not related to scheduling
19:31:02 melwitt the disk value is being rejected
19:32:12 dansmith yeah, but neither value is totally right I don't think
19:33:00 gryf current value is totally wrong
19:33:55 gryf instead of remaining space, entire disk size is provided :/
19:34:09 dansmith okay well, this isn't really related to resource reporting and I'm in the middle of something else at the moment,
19:34:22 gryf dansmith, no rush
19:34:22 dansmith but clearly I'm not familiar enough with that to review it without a deep dive, so.. :)
19:34:32 gryf oh :C
19:35:49 gryf either way, we already have that code applied, since we bump on this
19:36:15 gryf I thought that maybe it is worth to have it fixed on stable
19:37:57 mriedem anyone know if big john is going to be in denver?
19:44:06 openstackgerrit Merged openstack/nova master: Use placement microversion 1.26 in update_from_provider_tree https://review.openstack.org/593628
19:51:35 mriedem melwitt: if you're ok with https://review.openstack.org/#/c/593678/ for stable/rocky, you could +1 and then we can find another stable maint team core to approve to move rc2 along
19:52:03 melwitt ok
20:20:50 sean-k-mooney mriedem: FYI my summary of the live migration testing i did last week and a half is here http://lists.openstack.org/pipermail/openstack-dev/2018-August/133546.html
20:21:16 sean-k-mooney i didnt mean to add [all] but that does not matter now
20:22:00 sean-k-mooney i have opened 2 bugs in nova and 3 in neutron
20:22:16 mriedem sean-k-mooney: cool, thanks!
20:22:32 sean-k-mooney by the way i also noticed the irc bot nolonger comments on new bugs in irc since we went to regeistered mode
20:28:00 sean-k-mooney / away until noon IST
22:21:35 mriedem tonyb: if you're up, we've got another rc2 patch for you on stable/rocky https://review.openstack.org/#/c/593678/
22:23:24 tonyb mriedem: Done.
22:23:27 mriedem tonyb: <3
22:23:35 tonyb :D
23:53:15 openstackgerrit Merged openstack/nova stable/rocky: Use placement microversion 1.26 in update_from_provider_tree https://review.openstack.org/593678
#openstack-nova - 2018-08-21
05:52:13 vishakha mriedem: Hi, https://review.openstack.org/#/c/580271/ is this change valid?? Kindly review. As I am little confused with comments. Thanks
06:31:53 openstackgerrit Chen proposed openstack/nova stable/rocky: Update ssh configuration doc https://review.openstack.org/594041
06:37:06 openstackgerrit Chen proposed openstack/nova stable/rocky: Revisons on notifications doc https://review.openstack.org/594042
07:13:35 openstackgerrit Jiri Suchomel proposed openstack/nova master: Ignore deleted instances when populating with availability zones https://review.openstack.org/594050
07:17:13 gmann alex_xu: i am on vacation for 2 weeks (till 31st Aug ) so will not be able to do API office hour.
07:17:55 gmann melwitt: ^^ i will be able to provide API updates from 31st Aug onward (on vacation for 2 weeks.)
09:43:51 mriedem o/
09:44:35 kashyap Isn't it terribly early for you there?
09:44:45 cdent jet lag?
09:44:52 sean-k-mooney mriedem: o/
09:45:23 mriedem jet lag and work on the brain
09:45:40 lbragstad mriedem: same, my sleep schedule is completely screwed
09:46:29 sean-k-mooney mriedem: well do what jay pipes used to do. if your up early get work done early and be done by 1/2 pm and enjoy the rest of your day
09:48:03 kashyap mriedem: ^ Might want to check it out, when you're not sleeping :P
09:49:25 kashyap (FWIW, the author is not a "journalist" writing junk 'pop science'; he's a serious researcher on that topic.)
09:57:59 mriedem sean-k-mooney: question in https://bugs.launchpad.net/nova/+bug/1788014
09:57:59 openstack Launchpad bug 1788014 in OpenStack Compute (nova) "when live migration fails due to a internal error rollback is not handeled correctly." [Undecided,New]
10:03:32 sean-k-mooney mriedem: hi i think that is the issue yes. i have not had time to pin down the exact cause but i supect it because we are not activating the source binding after deleting the dest
10:04:08 mriedem hmm, i guess i would have expected neutron to automatically activate the source host port bindings when the dest host bindings were deleted
10:04:24 mriedem because when we activate the dest host bindings, neutron automatically de-activates the source host bindings,
10:04:51 mriedem so my thinking was when we delete the dest bindings on rollback, neutron would say, oh i need to activate the only other bindings (source) left
10:05:57 mriedem i could have a wip patch for you to test with if you still have that live migration env available
10:16:17 sean-k-mooney mriedem: i have the devstack vms shut down but i can have it set up quickly again
10:16:31 openstackgerrit Slawek Kaplonski proposed openstack/os-vif master: Avoid os-vif to add ovs ports as trunk by default https://review.openstack.org/594118
10:16:37 sean-k-mooney im goint to work on the first two neutron bugs first
10:17:04 sean-k-mooney ^ that is confusing.... we dont
10:20:00 sapd1 sean-k-mooney: Are you working on SR-IOV attach/detach?
10:20:01 sapd1 :D
10:21:55 sean-k-mooney sapd1: its on my todo list
10:26:37 openstackgerrit Jiri Suchomel proposed openstack/nova master: Set default AZ explicitely for instances without host. Ignore deleted instances when populating with availability zones https://review.openstack.org/594050
10:27:57 openstackgerrit Jiri Suchomel proposed openstack/nova master: Set default AZ explicitely for instances without host. https://review.openstack.org/594050
10:31:53 mriedem tssurya: on ^, we should just filter out instances w/o a host
10:32:04 mriedem default_availability_zone isn't the right config option for instances
10:32:19 mriedem default_schedule_zone is, but it defaults to None so it wouldn't fix the bug
10:36:31 sean-k-mooney mriedem: isint the default availableity zone nova?
10:36:46 mriedem default default_availability_zone is nova
10:36:53 mriedem if the instance is on a host
10:37:07 mriedem default_schedule_zone is the thing we set on instance.availability_zone if the user didn't request an az
10:37:09 mriedem and that defaults to None
10:38:39 sean-k-mooney oh ok, does horozon handel that differently?
10:38:51 sean-k-mooney or does devstack set them both to nova?
10:39:02 mriedem no
10:40:14 mriedem GET /servers/{id} will return '' for the az if the instance doesn't have a host set https://github.com/openstack/nova/blob/722d5b477219f0a2435a9f4ad4d54c61b83219f1/nova/api/openstack/compute/views/servers.py#L170
10:41:32 sean-k-mooney mriedem: when can an instance not have a host set. when its shelved?
10:42:00 mriedem if it fails during scheduling
10:42:04 mriedem NoValidHost
10:42:09 sean-k-mooney ah ok
10:42:14 mriedem and yes if it's shelved offloaded
10:42:41 mriedem i'm not sure that we clear out the instance availability_zone on shelve offload though
10:42:47 sean-k-mooney well it makes sense if its not schduled to a node it should not have az right?
10:42:47 mriedem if not that's like another bug
10:42:53 mriedem correct
10:43:02 mriedem that's what i'm saying in the review
10:44:00 mriedem melwitt: i've marked https://bugs.launchpad.net/nova/+bug/1788115 for rc potential
10:44:00 openstack Launchpad bug 1788115 in OpenStack Compute (nova) "nova-manage db online_data_migrations hangs on instances with no host set" [Medium,In progress] - Assigned to Jiri Suchomel (jsuchome)
10:46:13 mriedem yeah we don't clear out the instance.az on shelve offload
10:46:22 mriedem we will update it on unshelve though
10:46:50 mriedem https://github.com/openstack/nova/blob/722d5b477219f0a2435a9f4ad4d54c61b83219f1/nova/conductor/manager.py#L815
10:47:48 mriedem which reminds me https://review.openstack.org/#/c/559828/
10:48:55 mriedem as noted in ^ we also don't clear the port binding
10:49:28 tssurya mriedem: back from lunch, oh okay yea would be better to all together filter out the ones where host is NOne
10:49:59 mriedem tssurya: i think that still works for your original bug too
10:50:01 mriedem comments inline on why
10:59:21 tssurya mriedem: I agree,
11:44:46 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976
11:47:03 openstackgerrit Jiri Suchomel proposed openstack/nova master: Filter out instances without a host when populating AZ https://review.openstack.org/594050
11:53:26 mriedem sean-k-mooney: see how ^ floats your boat
11:53:27 mriedem oops
11:53:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Re-activate source host port bindings on live migration rollback https://review.openstack.org/594139
11:53:37 mriedem sean-k-mooney: ^
11:53:45 mriedem i want to see what miguel thinks about that too
11:55:48 sean-k-mooney mriedem: ill test that and see. the other thing is im not sure what the state of qemu when this partaclar bug happens due to the fact th monitor closed.

Earlier   Later