| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-20 | |||
| 19:27:08 | efried | gryf: I think johnthetubaguy ought to have another look at ^ and see if his concerns have been addressed. Other cores will be reluctant to approve the patch with another core's -1 on it. | |
| 19:27:50 | melwitt | dansmith: could you take a look at the proposed fix? ^ it seems to make sense but I'm not that familiar with the particulars of resource reporting considering overcommit | |
| 19:28:22 | dansmith | melwitt: oh yeah I looked at this last week but never said anything | |
| 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 | dansmith | but clearly I'm not familiar enough with that to review it without a deep dive, so.. :) | |
| 19:34:22 | gryf | dansmith, no rush | |
| 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 | openstack | Launchpad bug 1788014 in OpenStack Compute (nova) "when live migration fails due to a internal error rollback is not handeled correctly." [Undecided,New] | |
| 09:57:59 | mriedem | sean-k-mooney: question in https://bugs.launchpad.net/nova/+bug/1788014 | |
| 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 | mriedem | if not that's like another bug | |
| 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:53 | mriedem | correct | |
| 10:43:02 | mriedem | that's what i'm saying in the review | |
| 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:44:00 | mriedem | melwitt: i've marked https://bugs.launchpad.net/nova/+bug/1788115 for rc potential | |
| 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 | |