| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-07 | |||
| 10:18:52 | openstackgerrit | tianhui proposed openstack/nova master: Fix bug to doc:nova-status https://review.openstack.org/573149 | |
| 10:19:26 | gibi | tssurya: I added a small update to the bug https://bugs.launchpad.net/nova/+bug/1739325/comments/8 | |
| 10:19:27 | openstack | Launchpad bug 1739325 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset non-nullable fields" [Medium,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 10:19:40 | gibi | tssurya: and I will try to talk to dansmith and mriedem about it | |
| 10:22:53 | tssurya | gibi: thanks for the efforts; | |
| 10:30:37 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: placement: Use INNER JOIN for requied traits https://review.openstack.org/573113 | |
| 10:30:53 | tssurya | gibi: also I totally understand your opinion on https://review.openstack.org/#/c/557369/, I am not against a new API bump | |
| 10:31:27 | tssurya | will think more about this and maybe like mriedem was suggesting, we should have a call sometime soon like early next week maybe | |
| 10:31:32 | tssurya | to discuss the details | |
| 10:52:08 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | |
| 10:52:09 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Use list instead of set for duplicate check https://review.openstack.org/569720 | |
| 10:59:09 | openstackgerrit | tianhui proposed openstack/nova master: Fix bug to doc:nova-status https://review.openstack.org/573149 | |
| 11:02:24 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | |
| 11:02:25 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | |
| 11:02:26 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | |
| 11:16:38 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Delete duplicate functions in placement test https://review.openstack.org/573193 | |
| 11:17:19 | gibi | tssurya: I'm happy to discuss this on call as well | |
| 11:19:12 | tssurya | gibi: ack :) | |
| 11:37:47 | openstackgerrit | tianhui proposed openstack/nova master: Fix bug to doc:nova-status https://review.openstack.org/573149 | |
| 11:54:17 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: WIP: Optimize member_of check for nested providers https://review.openstack.org/573203 | |
| 12:24:43 | bauzas | jaypipes: thanks for not holding https://review.openstack.org/#/c/557065/6 | |
| 12:25:28 | bauzas | jaypipes: I agree with you about the fact that the vGPU type names are just a vendor silly | |
| 12:25:35 | bauzas | but meh | |
| 12:36:26 | cdent | "a vendor silly" is a good phrase, will need to use it often | |
| 12:39:43 | openstackgerrit | MultipleCrashes proposed openstack/nova master: Retry decorator fix for autoscale delete https://review.openstack.org/563418 | |
| 12:44:00 | bauzas | cdent: sorry, I don't know the english subsantive for the "silly" adjective | |
| 12:44:09 | bauzas | substantive* | |
| 12:46:23 | kashyap | bauzas: No, don't worry. It is still a fun phrase | |
| 12:46:59 | bauzas | lemme try : silliness | |
| 12:47:23 | bauzas | (english is sometimes far easier than french to guess) | |
| 12:49:54 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | |
| 12:49:55 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 12:49:56 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | |
| 12:49:57 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | |
| 12:50:40 | mriedem | mdbooth_: see https://review.openstack.org/#/c/572790/ - you can't attach the source volume to another server while we're swapping because nova changes the volume's status to 'detaching': https://review.openstack.org/#/c/572790/7 | |
| 12:50:43 | mriedem | oops | |
| 12:50:46 | mriedem | http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n723 | |
| 12:51:47 | mdbooth_ | mriedem: That would be convenient | |
| 12:52:19 | mdbooth_ | mriedem: And completely remove the need for the notification thing | |
| 12:53:11 | mriedem | right, so if i remove the libvirt stuff from the commit message are you good? | |
| 12:54:20 | mdbooth_ | mriedem: I wasn't that bothered about the commit message, tbh. | |
| 12:54:38 | mriedem | so what are we going to do to get you into this beautiful car today | |
| 12:54:45 | mdbooth | Hehe | |
| 12:54:53 | mriedem | i'll throw in the undercoat for free | |
| 12:55:15 | mdbooth | Can you gimme a minute to punt the other ping I just got to somebody else? | |
| 12:55:44 | mriedem | sure, i've heard artom is available | |
| 12:55:55 | mriedem | speaking of artom, http://status.openstack.org/elastic-recheck/#1775491 | |
| 12:58:10 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | |
| 13:01:02 | mriedem | dansmith: are you going to propose a backport for this? https://review.openstack.org/#/c/572195/ | |
| 13:02:19 | jmccarthy | Not sure what is going on here, but I'm seeing this issue at the moment https://bugs.launchpad.net/kolla/+bug/1774409 | |
| 13:02:20 | openstack | Launchpad bug 1774409 in kolla "neutron_dhcp_agent accumulates zombie processes " [Undecided,New] | |
| 13:02:46 | jmccarthy | i.e. creating and deleting networks accumulates zombies .. any ideas ? | |
| 13:02:50 | mriedem | jmccarthy: better to ask in the #openstack-neutron channel? | |
| 13:03:04 | jmccarthy | mriedem: Oh ! I'll do that :) Thanks ! | |
| 13:05:11 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | |
| 13:23:05 | openstackgerrit | Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262 | |
| 13:28:48 | mdbooth | mriedem: Related, but not strictly the same thing I think. I was checking how we end up unsetting the 'detaching' state and I came across this: | |
| 13:28:51 | mdbooth | http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n1567 | |
| 13:29:41 | dansmith | mriedem: yep | |
| 13:29:52 | mdbooth | In the tempest test, we swap one of the volumes, then we check if Nova still thinks that the other server is still attached to the original volume, which it apparently does | |
| 13:30:04 | mdbooth | However, I think this code might mean that *cinder* thinks it's not | |
| 13:30:54 | mdbooth | It looks like cinder is doing a detach(old)/attach(new) for all attachments, even though we only migrated one of them. | |
| 13:38:06 | mdbooth | However, the above would only be in the case of the direct nova call, because if it went via cinder there would be a migration record | |
| 13:43:41 | mriedem | mdbooth: that's only if nova calls that migration completion code | |
| 13:44:12 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5626 | |
| 13:44:29 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5661 | |
| 13:44:48 | mriedem | if the swap isn't initiated from cinder for a retype or volume migration, we don't call that callback method in cinder | |
| 13:53:11 | mdbooth | mriedem: Good point. I wonder where we unset that detaching state, then. | |
| 13:53:42 | mdbooth | ..we explicitly delete the attachment | |
| 13:56:04 | openstackgerrit | Dan Smith proposed openstack/nova stable/queens: Change consecutive build failure limit to a weigher https://review.openstack.org/573239 | |
| 13:56:43 | mriedem | mdbooth: i think when the attachment record is deleted http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n2188 | |
| 13:57:05 | mriedem | if there are no more attachments, the volume is available | |
| 13:57:13 | mriedem | if there are other attached attachments, the volume goes back to in-use | |
| 13:57:49 | mriedem | mordred: it's time that we need some API super users to peruse this spec https://review.openstack.org/#/c/557369/ - i'm going to put it on the ML too | |
| 13:58:07 | mriedem | mordred: what nova api should do when listing servers and we have down cells | |
| 14:02:16 | mordred | mriedem: I have it open to read | |
| 14:02:26 | mriedem | <3 | |
| 14:06:33 | openstackgerrit | Dan Smith proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher https://review.openstack.org/573248 | |
| 14:09:05 | mriedem | dansmith: we should backport https://review.openstack.org/#/c/569127/ to queens probably | |
| 14:09:16 | mriedem | it would help with your backport https://review.openstack.org/#/c/573239/ too | |
| 14:09:51 | dansmith | well I already fixed the conflict, but okay | |
| 14:10:58 | dansmith | I guess that means I have to do both of these over | |
| 14:11:29 | mriedem | the pike one shouldn't have any impact from that max attempts thing | |
| 14:11:31 | mriedem | since that was queens only | |
| 14:11:43 | dansmith | the hash will change | |
| 14:11:48 | dansmith | on the queens one | |
| 14:11:50 | mriedem | sure | |
| 14:14:25 | mriedem | you can use this for that max_attempts backport https://bugs.launchpad.net/nova/+bug/1775625 | |
| 14:14:27 | openstack | Launchpad bug 1775625 in OpenStack Compute (nova) queens "nova-scheduler calculates alternative hosts incorrectly" [Medium,Triaged] | |
| 14:14:27 | mriedem | for the paperwork | |
| 14:14:28 | efried | wtf dansmith this is like the second time this week you're online before 8. | |
| 14:15:07 | dansmith | efried: up at 6am every.single.day.dude. | |
| 14:17:20 | mdbooth | Does anything ever create RequestContext except n-api? | |
| 14:18:03 | openstackgerrit | Dan Smith proposed openstack/nova stable/queens: Change consecutive build failure limit to a weigher https://review.openstack.org/573239 | |
| 14:18:04 | openstackgerrit | Dan Smith proposed openstack/nova stable/queens: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/573254 | |
| 14:18:13 | dansmith | mriedem: ^ does that make you happy? | |
| 14:18:45 | mdbooth | dansmith: happy seems like a stretch goal :) | |
| 14:19:32 | dansmith | mdbooth: it was a setup for him to say nothing makes him happy | |
| 14:19:47 | mdbooth | dansmith: And I ruined it... Sorry dude. | |
| 14:19:47 | mriedem | i'm currently unable to stand up in polite company | |