| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-06 | |||
| 17:32:56 | dansmith | mriedem: bottom patch got hit by the mirror thing so I rebased and re-pushed.. so it needs a re-=W | |
| 17:33:00 | dansmith | er, re-+W | |
| 17:57:17 | mriedem | dansmith: got it, so the first 5 are ready | |
| 17:57:33 | mriedem | looks like typo city in https://review.openstack.org/#/c/442736/ | |
| 17:58:04 | dansmith | yeah | |
| 17:58:05 | dansmith | thanks | |
| 17:58:57 | cfriesen | does anyone know why we default to not returning the userdata when showing server details to a normal user, but show it for an admin user? | |
| 18:03:16 | mriedem | nope | |
| 18:04:31 | EmilienM | mriedem: hey, I think I'll abandon https://review.openstack.org/#/c/453338/ if nobody approves it, it needed to land before you tag next week but since're approaching next week, I can we can wait now for the tag... | |
| 18:05:13 | mriedem | wfm | |
| 18:11:30 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 18:11:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 18:11:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 18:14:35 | cdent | melwitt: do I recall that you were going to adjust the project/user association in placement to use usages? | |
| 18:15:05 | melwitt | cdent: yes, I had intended to propose an amendment to the spec yesterday but didn't get to it yet | |
| 18:15:24 | cdent | ah, cool, that would explain why I can't find it :) thanks | |
| 18:15:35 | melwitt | heh, sorry about that | |
| 18:16:14 | cdent | no problem, you're the one who is making with the good works | |
| 18:51:32 | openstackgerrit | Sean Dague proposed openstack/nova master: Remove the extension check for os-networks in servers API https://review.openstack.org/450837 | |
| 18:51:56 | openstackgerrit | Sean Dague proposed openstack/nova master: Remove the json-schema extension point of server create https://review.openstack.org/451194 | |
| 18:52:12 | openstackgerrit | Sean Dague proposed openstack/nova master: Remove the stevedore extension point for server create https://review.openstack.org/451195 | |
| 18:54:42 | sdague | mriedem: you want in on - https://review.openstack.org/#/c/448882 before I approve (nova cert removal) | |
| 18:54:46 | sdague | ? | |
| 18:59:57 | diana_clarke | Anyone have time to put my last nova patch out of its misery? https://review.openstack.org/#/c/447688/ thx! | |
| 19:01:38 | dansmith | I bet sdague wants that one | |
| 19:01:53 | mriedem | sdague: probably | |
| 19:02:19 | stevemar | mriedem: why you gotta break osc for! | |
| 19:02:35 | mriedem | stevemar: novaclient 8.0? | |
| 19:02:41 | stevemar | yah | |
| 19:02:42 | mriedem | stevemar: upper-constraints? | |
| 19:03:04 | stevemar | i assume the gate is fine, someone pinged me from BB about latest pip install of osc is broken | |
| 19:03:08 | stevemar | cause they don't UC | |
| 19:03:18 | mriedem | nuts to them | |
| 19:03:23 | mriedem | i don't work for BB anymore sucka | |
| 19:03:26 | stevemar | :) | |
| 19:03:31 | stevemar | we'll fix it up | |
| 19:04:13 | mriedem | btw, more than 6 months to notice the deprecation warning | |
| 19:04:58 | mriedem | sdague: did anyone from ec2api ever chime in? | |
| 19:05:01 | mriedem | b/c this will break them | |
| 19:05:12 | mriedem | oh i see ftersin did | |
| 19:10:38 | mriedem | sdague: +W | |
| 19:12:36 | mriedem | when can we remove os-cloudpipe? | |
| 19:13:06 | mriedem | macsz: https://blueprints.launchpad.net/nova/+spec/remove-nova-cert | |
| 19:13:10 | mriedem | :( | |
| 19:16:01 | mriedem | i created it | |
| 19:16:15 | macsz | mriedem: ? | |
| 19:16:23 | mriedem | you hadn't created the actual bp in lp | |
| 19:16:47 | macsz | oh, i lost something | |
| 19:18:05 | mriedem | who wants it? https://review.openstack.org/#/c/453859/ | |
| 19:18:18 | mriedem | dansmith: i'll trade you ^ for your workaround patch | |
| 19:19:05 | dansmith | k | |
| 19:19:08 | macsz | waswas wondering what happened but it seems that once more you were faster than me and created the bp, mriedem | |
| 19:19:35 | dansmith | mriedem: heh, that conflicts with my hostapi patch.. damn you | |
| 19:20:02 | mriedem | it shouldn't | |
| 19:20:09 | mriedem | yours doesn't touch the delete path | |
| 19:20:54 | sdague | mriedem: when ever anyone writes the deprecation spec | |
| 19:21:03 | sdague | I am sure that it hasn't worked in years | |
| 19:21:35 | mriedem | i would, but (1) i don't want to and (2) i'm not sure i even understand what it is to explain it and why we want to remove it, except it's not supported | |
| 19:21:44 | mriedem | some kind of direct vpn image download to the vm or something? | |
| 19:22:20 | mriedem | ooo https://www.mirantis.com/blog/cloudpipe-image-creation-automation/ | |
| 19:22:21 | dansmith | mriedem: well, gerrit says it does.. it's fine anyway | |
| 19:22:58 | mriedem | damn the link to the cactus release docs don't work anymore... | |
| 19:25:11 | sdague | mriedem: it all depends on vlan nova network stuff right? which means whenever the great nova-network delete happens it's all definitely bust | |
| 19:26:02 | openstackgerrit | Sujitha proposed openstack/nova master: Add sync_instances action in os-cells API https://review.openstack.org/453366 | |
| 19:26:04 | mriedem | yeah i guess | |
| 19:27:05 | mriedem | so cloudpipe just basically creates a vm and then you ssh into that vm to get into the internal tenant network? | |
| 19:28:27 | mriedem | yeah anyway, nova-network plus unmaintained plus neutron vpnaas | |
| 19:28:30 | mriedem | equals death | |
| 19:33:46 | mriedem | ooo https://www.sebastien-han.fr/blog/2012/06/20/setup-cloud-pipe-vpn-in-openstack/ talks about using nova x509-create-cert which relies on none other than the nova-cert service... | |
| 19:33:52 | mriedem | which we just said we were going to kill | |
| 19:34:18 | mriedem | so, as part of the nova-cert removal, we probably also need to deprecate os-cloudpipe | |
| 19:51:32 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 20:01:27 | openstackgerrit | Sean Dague proposed openstack/nova master: Change livesnapshot to true by default https://review.openstack.org/454323 | |
| 20:08:14 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Deprecate certs commands and APIs https://review.openstack.org/454326 | |
| 20:08:17 | mriedem | macsz: ^ | |
| 20:09:01 | macsz | mriedem: should i draft a spec for it? | |
| 20:12:11 | mriedem | macsz: for what? | |
| 20:12:17 | mriedem | cloudpipe/ | |
| 20:12:18 | mriedem | sure | |
| 20:12:35 | macsz | or can we smuggle it under the cert deprecation banner? | |
| 20:12:47 | sdague | macsz: I'm pro smuggle | |
| 20:13:06 | mriedem | if we smuggle, we should probably amend the spec | |
| 20:13:11 | sdague | mriedem: yeh | |
| 20:13:14 | macsz | the spec was merged, but i can also do a follow-up patch that modifies the current spec | |
| 20:13:26 | sdague | macsz: ++ an ammend would be great | |
| 20:13:27 | macsz | yeah, shorter way to say what i meant | |
| 20:20:47 | openstackgerrit | Sihan Wang proposed openstack/nova master: Update checking vm state in vmware driver https://review.openstack.org/454328 | |
| 20:22:01 | macsz | mriedem: it seems that os-cloudpipe is already deprecated | |
| 20:22:02 | macsz | https://github.com/openstack/nova/blob/master/api-ref/source/os-cloudpipe.inc | |
| 20:23:13 | mriedem | just in the docs | |
| 20:23:31 | mriedem | it doesn't return an error though on any microversion like 2.36 would for listing networks | |
| 20:24:14 | mriedem | i don't know if we're proposing to just remove it, like os-certificates, or deprecating it with a microversion, but if we remove nova-network and nova-cert, then it seems kind of pointless to do the microversoin | |
| 20:24:16 | mriedem | sdague: ^ thoughts? | |
| 20:26:20 | macsz | doc mentions that os-cloudpipe works only with nova-net, so there is really no point in having it around, with or without nova-cert, it seems | |
| 20:27:16 | mriedem | we still have nova-net | |
| 20:27:20 | mriedem | although | |
| 20:27:24 | mriedem | nova-net is cells v1 only now | |
| 20:27:33 | mriedem | and i assume os-cloudpipe doesn't work with cells v1 | |
| 20:29:06 | mriedem | it might, it's a weird orchestration api that creates an instance for you | |
| 20:29:10 | mriedem | with a configured vpn image | |