| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-14 | |||
| 18:51:28 | jroll | :) | |
| 20:19:31 | cfriesen | maybe odd question, but if I boot an instance from a flavor with trait:HW_CPU_X86_AVX2=required, the exact CPU model chosen seems to depend on the nova-compute config. Suppose I then try to live-migrate...I'm guaranteed that the dest node will support AVX2, but Is there somewhere that we verify that the dest node can support the same CPU model as is currently being used by the running VM? | |
| 20:31:23 | cfriesen | never mind, found the code. the scheduler filters don't check, but check_can_live_migrate_destination() should catch it. | |
| 20:42:23 | openstackgerrit | Jay Pipes proposed openstack/nova master: DNM: test possible deadlock cause https://review.openstack.org/591845 | |
| 21:15:52 | openstackgerrit | Chris Dent proposed openstack/nova master: Add explanatory prefix to post_test_perf output https://review.openstack.org/591850 | |
| 21:19:32 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: test possible deadlock cause https://review.openstack.org/591845 | |
| 21:21:10 | efried | cdent: Love it. | |
| 21:22:03 | cdent | I suppose once we've all memorized those words we can read right over them | |
| 21:25:11 | efried | cdent: One concern is tying that behavior to a specific version of placeload. | |
| 21:25:33 | efried | Perhaps a TODO to vet that periodically. | |
| 21:26:14 | cdent | but where will we put the todo to remind us to look for todos? | |
| 21:27:26 | openstackgerrit | Dmitry Sutyagin proposed openstack/nova master: Implement disabling KSM via flavor's extra specs https://review.openstack.org/591854 | |
| 21:30:31 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM: update_provider_tree() (compatible) https://review.openstack.org/468560 | |
| 21:30:39 | efried | edmondsw, jaypipes: ^ | |
| 21:32:15 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources https://review.openstack.org/565841 | |
| 21:32:54 | jroll | efried: jaypipes: ^ had a syntax error in release note :( | |
| 21:33:11 | efried | jroll: Wish somebody had been paying attention to your release note | |
| 21:33:17 | jroll | heh | |
| 21:33:24 | jroll | I did your suggestion too :) | |
| 21:33:53 | efried | d'oh, I should have caught that. | |
| 21:33:55 | efried | thanks. | |
| 21:34:16 | efried | jroll: +A | |
| 21:35:05 | jroll | thanks | |
| 22:35:28 | melwitt | mriedem: fyi added a question to the RC todos etherpad about translations | |
| 22:40:11 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863 | |
| #openstack-nova - 2018-08-15 | |||
| 00:20:31 | openstackgerrit | xhzhf proposed openstack/nova master: [vmware] add missing os types in vSphere sdk 6.5 https://review.openstack.org/591771 | |
| 00:29:55 | mriedem | melwitt: ack, just got online | |
| 00:35:16 | mriedem | dansmith: lyarwood: fwiw, i backported the live migration revert from master to stable/rocky b/c i was sure that would be clean since we just cut stable/rocky, but assumed we'd have to revert per-branch for queens and pike | |
| 00:39:03 | mriedem | efried: re: https://github.com/openstack/nova/blob/61f854ff6445dd7bc797fdde7361e47f3dc3b1bd/nova/tests/functional/compute/test_resource_tracker.py#L215 have we removed the compat code in the RT for that yet? or are you saying we can now because master is stein? i would hold off on merging any big code removals this early until we've cut rc2 | |
| 00:43:39 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/591872 | |
| 00:44:55 | mriedem | melwitt: on backporting https://review.openstack.org/#/c/591129/ to rc2, idk, would ask the release team (smcginnis or dhellmann) | |
| 00:45:27 | mriedem | or maybe ajaegar | |
| 00:59:02 | mriedem | melwitt: we don't need to backport the translations | |
| 02:01:13 | openstackgerrit | Merged openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061 | |
| 02:01:20 | openstackgerrit | Merged openstack/nova master: Update really old comments about vmware hosts managing multiple nodes https://review.openstack.org/589666 | |
| 02:01:28 | openstackgerrit | Merged openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources https://review.openstack.org/565841 | |
| 02:04:10 | mriedem | efried: ok i guess https://review.openstack.org/#/c/565841/ is merged... note there are other things to cleanup related to this from the ptg etherpad, ~L25 https://etherpad.openstack.org/p/nova-ptg-stein | |
| 02:46:58 | alex_xu | Kevin_Zheng: few comments on https://review.openstack.org/#/c/486850 | |
| 02:48:24 | alex_xu | Kevin_Zheng: I think using PATH_INFO is right. just need refine the code a little bit. And I found we will strip any work start with 'v', if we have a url called '/volume', then it will be stripped. although we don't have it, but it is trap for any url start with 'v' | |
| 02:48:47 | alex_xu | in the future | |
| 03:14:20 | melwitt | mriedem: thanks. I guess I don't know what all the hubbub about "rc2 translations" used to be about then | |
| 03:14:53 | Kevin_Zheng | alex_xu, thanks alot, I will look into that | |
| 03:22:49 | sapd1 | melwitt: Can I workaround to attach SR-IOV to exist instance? :D | |
| 03:26:04 | melwitt | sapd1: not that I know of. I haven't gotten around to asking artom about the bug https://bugs.launchpad.net/nova/+bug/1708433 | |
| 03:26:04 | openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,New] | |
| 03:31:13 | alex_xu | mriedem: enjoy...Xi'an food :) | |
| 03:32:06 | mriedem | alex_xu: biang biang noodle! | |
| 03:32:11 | mriedem | chinese hamburger | |
| 03:32:27 | alex_xu | mriedem: hahaha | |
| 03:33:32 | mriedem | sapd1: you can't attach sriov ports to existing instances | |
| 03:33:43 | mriedem | beyond just the KeyError | |
| 03:34:00 | mriedem | we should likely just 400 those types of requestse | |
| 03:34:49 | melwitt | yeah, sean-k-mooney was saying he thought it used to 400 but that artom might know about patch(es) that might have inadvertently changed it? | |
| 03:34:56 | sapd1 | mriedem: so we don't have any way to attach sriov ports, do you? | |
| 03:36:21 | mriedem | sapd1: it's not currently supported no | |
| 03:36:32 | mriedem | see https://review.openstack.org/#/c/139910/ | |
| 03:38:03 | mriedem | it would definitely be a microversion to start saying we support that type of attach | |
| 03:39:22 | mriedem | so i think we essentially need to check the vnic_type on the port and if it's not 'normal' we need to fail | |
| 03:41:57 | mriedem | sean-k-mooney: nova never pre-creates sriov ports right because you have to specify that in the binding:vnic_type when creating the port | |
| 03:42:23 | mriedem | therefore if a user tries to attach a port to a server by passing a network id rather than a port id and nova creates the port on the network, it will always be vnic_type 'normal' today | |
| 03:48:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Force refresh instance info_cache during heal https://review.openstack.org/591607 | |
| 04:11:59 | openstackgerrit | Chen proposed openstack/nova master: Revisons on notifications doc https://review.openstack.org/591621 | |
| 04:12:16 | sapd1 | mriedem: I saw in neutron-sriov-agent log, When launch instance It update profile of sr-iov port, But when attach port to exist instance It does not. | |
| 04:58:45 | gmann | alex_xu: do you have anything for today API office hour ? i am little busy on QA releases so if nothing much i would like to skip it. | |
| 05:04:14 | mriedem | sapd1: because attaching sriov interfaces to existing instances is not supported in nova | |
| 05:04:38 | mriedem | the api might say the interface is attached, but nova-compute doesn't actually allocate any pci device | |
| 05:31:02 | alex_xu | gmann: we can skip it today | |
| 05:40:01 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/591898 | |
| 05:48:13 | gmann | ok | |
| 06:54:32 | mriedem | so quiet in here | |
| 06:54:36 | mriedem | is everyone on wechat? | |
| 07:09:06 | sapd1 | mriedem: I'm still here | |
| 07:10:22 | openstackgerrit | Merged openstack/nova master: Remove blacklisted py3 xen tests https://review.openstack.org/591419 | |
| 07:10:34 | brinzhang | mriedem: If we use wechat, we can speak Chinese? :) | |
| 07:14:21 | brinzhang | anyone know a project of masakari? It has too few document. | |
| 08:07:48 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: consumer gen: move_allocations https://review.openstack.org/591810 | |
| 08:07:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases https://review.openstack.org/591811 | |
| 08:07:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: consumer gen: support claim_resources https://review.openstack.org/583667 | |
| 08:59:29 | sean-k-mooney | mriedem i noticed the same thing for the last week or so | |
| 09:00:33 | sean-k-mooney | basically since we started requireing registered nics that chat has been much more quite, and there have been less nics i have not seen before | |
| 09:04:47 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | |
| 10:14:01 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | |
| 10:39:26 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976 | |
| 11:03:12 | pvc | hi guys | |
| 11:03:12 | pvc | Volume size extension is not supported by the hypervisor.: ExtendVolumeNotSupported: Volume size extension is not supported by the hypervisor. | |
| 11:15:47 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872 | |
| 11:16:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872 | |
| 11:22:12 | pvc | hi guys | |
| 11:22:12 | pvc | anyone | |
| 11:22:13 | pvc | here | |
| 11:49:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller https://review.openstack.org/502859 | |
| 12:03:14 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223 | |
| 12:18:19 | sean-k-mooney | sorry for the delay, alot of people are on vacation in europe and most of the us folks are not online yet | |
| 12:18:24 | sean-k-mooney | pvc: ^ | |
| 12:19:12 | sean-k-mooney | pvc: alot of eu people that are working today are also gone to lunch | |
| 12:19:51 | sean-k-mooney | pvc: you were getting an error with exetending voluems "ExtendVolumeNotSupported" | |
| 12:21:15 | sean-k-mooney | pvc: im not sure i can help but if you have a specifc question/issue i might at least be able to point you in the right direction | |
| 12:31:50 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response https://review.openstack.org/584590 | |
| 12:47:13 | edmondsw | pvc which virt driver are you using? | |
| 12:47:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge keypair extension response into server view builder https://review.openstack.org/584748 | |