Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-07
09:35:55 openstackgerrit Stephen Finucane proposed openstack/nova master: fix tox python3 overrides https://review.openstack.org/572974
09:37:51 openstackgerrit Stephen Finucane proposed openstack/nova master: fix tox python3 overrides https://review.openstack.org/572974
09:45:37 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383
09:45:38 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Speed things up and document them https://review.openstack.org/534382
10:09:56 openstackgerrit tianhui proposed openstack/nova master: Fix bug to doc:nova-status https://review.openstack.org/573149
10:12:12 openstackgerrit Merged openstack/osc-placement master: fix tox python3 overrides https://review.openstack.org/572917
10:14:11 openstackgerrit Merged openstack/os-vif master: fix tox python3 overrides https://review.openstack.org/572922
10:16:06 alexchadin hi nova team. how can I see which scheduler hints have been used during instance creation?
10:16:43 openstackgerrit tianhui proposed openstack/nova master: Fix bug to doc:nova-status https://review.openstack.org/573149
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 mriedem for the paperwork
14:14:27 openstack Launchpad bug 1775625 in OpenStack Compute (nova) queens "nova-scheduler calculates alternative hosts incorrectly" [Medium,Triaged]
14:14:28 efried wtf dansmith this is like the second time this week you're online before 8.

Earlier   Later