| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-14 | |||
| 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 | openstack | Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,New] | |
| 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: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 | Volume size extension is not supported by the hypervisor.: ExtendVolumeNotSupported: Volume size extension is not supported by the hypervisor. | |
| 11:03:12 | pvc | hi guys | |
| 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 | anyone | |
| 11:22:12 | pvc | hi guys | |
| 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 | |
| 12:47:45 | gibi | btw I will take Thursday and Friday afternoon off and then Monday is public holiday here and I will take Tuesday out as well | |
| 13:00:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder https://review.openstack.org/585262 | |
| 13:14:03 | openstackgerrit | Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041 | |
| 13:20:41 | efried | mriedem: I was approaching it from "I just happened to notice this while I was doing something else," not, "Hey, we need to clean up a big thing and this is one of the pieces." I'm sure it makes sense to go looking for the chunk of code that guy tests... | |
| 13:22:17 | efried | jaypipes: Did you run that locally? I'm wondering if you need to do list(traits.keys()) else you'd wind up with output like <Generator at 0x0a7e0a> etc. | |
| 13:23:02 | efried | whoops, wrong channel. | |
| 13:25:53 | openstackgerrit | Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041 | |
| 13:29:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041 | |
| 13:33:57 | openstackgerrit | Merged openstack/nova master: api-ref: fix GET /flavors?is_public description https://review.openstack.org/588092 | |
| 13:36:28 | efried | mriedem: Why are we waiting until rocky rc2 to merge things into stein? | |
| 13:36:42 | efried | dansmith: maybe you know this answer? | |
| 13:36:57 | dansmith | efried: um we're not? | |
| 13:37:07 | dansmith | stable/rocky exists which means master is stein | |
| 13:37:10 | efried | okay. Matt said something weird earlier. | |
| 13:37:19 | dansmith | which was what? | |
| 13:37:33 | efried | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-08-15.log.html#t2018-08-15T00:39:03 | |
| 13:37:53 | dansmith | well, first off, | |
| 13:38:12 | dansmith | he said master was stein, | |
| 13:38:13 | dansmith | but his point was that if you do big removals or refactors, | |
| 13:38:14 | dansmith | and we then suddenly need to backport a fix, | |
| 13:38:15 | dansmith | it may be much uglier | |