| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-22 | |||
| 16:07:45 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813 | |
| 16:07:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816 | |
| 16:07:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815 | |
| 16:07:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818 | |
| 16:07:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817 | |
| 16:08:13 | stephenfin | melwitt, gibi: fixed that silly typo in the reno if you want to re +2 https://review.opendev.org/686812 | |
| 16:08:21 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support service_host filtering for os-hypervisor API https://review.opendev.org/695708 | |
| 16:09:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'os-consoles' API https://review.opendev.org/687907 | |
| 16:09:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909 | |
| 16:09:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'nova-console' service, 'console' RPC API https://review.opendev.org/687908 | |
| 16:10:40 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Filter hypervisors by service host https://review.opendev.org/695716 | |
| 16:10:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support service_host filtering for os-hypervisor API https://review.opendev.org/695708 | |
| 16:13:01 | bauzas | stephenfin: ask others for your series, sorry | |
| 16:13:14 | stephenfin | have a good weekend o/ | |
| 16:13:36 | melwitt | stephenfin: why did you update to say "which has been removed"? it's not going to removed until the end of the series right? I guess it makes sense to write it as thought it's already happened though since it will be by the time anyone reads that | |
| 16:13:43 | melwitt | *though | |
| 16:14:13 | stephenfin | yeah, I'm writing from the perspective of the end of the series | |
| 16:14:43 | melwitt | yeah, ok that makes sense | |
| 16:14:49 | stephenfin | and in reality, we removed nova-network support when we dropped cells v1 support https://github.com/openstack/nova/blob/master/nova/cmd/network.py#L46 | |
| 16:15:14 | melwitt | oh yeah, I didn't think about that | |
| 16:15:15 | gibi | stephenfin: sorry, I have to take that next week | |
| 16:15:36 | stephenfin | gibi: 'sall good (y) | |
| 16:16:05 | gibi | have a nice weekend folks! | |
| 16:16:26 | melwitt | bye, happy weekend o/ | |
| 16:27:28 | mriedem | stephenfin: a couple of thoughts/questions in the bottom change https://review.opendev.org/#/c/686810/ | |
| 16:27:37 | stephenfin | looking | |
| 16:27:42 | mriedem | i don't know if we have precedent for 410ing some methods in a route but not all | |
| 16:28:25 | stephenfin | why would it be 500? | |
| 16:29:09 | mriedem | i was thinking if you hit something that's not neutron, but i guess if you upgrade and the nova-network binary is literally gone, then yeah i guess that's not possible | |
| 16:29:15 | mriedem | b/c you can only have neutron or you can't upgrade | |
| 16:29:48 | stephenfin | yeah, the nova-network service literally can't be started since we dropped the cells v1 code | |
| 16:30:27 | stephenfin | so people have to be using neutron in Train, never mind Ussuri | |
| 16:42:20 | mriedem | should update the reno here https://review.opendev.org/#/c/686812/8/releasenotes/notes/remove-nova-network-c02953ba72a1795d.yaml | |
| 16:44:12 | stephenfin | damn it | |
| 16:44:17 | stephenfin | yup | |
| 16:44:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812 | |
| 16:45:02 | stephenfin | done | |
| 16:48:19 | mriedem | sorry one more in there - you forgot os-quota-class-sets in the reno | |
| 16:48:28 | mriedem | the quota apis are super fun! | |
| 16:48:34 | mriedem | because there are 100 of them | |
| 16:49:50 | melwitt | this patch is a stats machine | |
| 16:49:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812 | |
| 16:50:06 | stephenfin | Do I get a prize if I win? | |
| 16:50:15 | stephenfin | Can someone "win"? | |
| 16:50:31 | stephenfin | I want the plush tiger | |
| 16:50:35 | melwitt | haha | |
| 16:51:32 | melwitt | I think we're all winning cause we're talking about quotas | |
| 16:51:34 | melwitt | right?? | |
| 16:52:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813 | |
| 16:52:31 | stephenfin | so long as you don't get more "winning" than you're paying for | |
| 16:53:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814 | |
| 16:53:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815 | |
| 16:53:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816 | |
| 16:53:26 | melwitt | badum csh! | |
| 16:53:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817 | |
| 16:53:47 | melwitt | man, I really can't type anything today | |
| 16:54:10 | mriedem | have a kit kat | |
| 16:54:39 | melwitt | hm, I could. still have some halloween candy | |
| 16:55:34 | efried | stephenfin: did you see the conversation yesterday with johnthetubaguy about adding a "signal microversion" so clients can discover that nova-net is gone? | |
| 16:55:58 | stephenfin | efried: I did. mriedem is probably the person you want to discuss that with, tbh | |
| 16:56:23 | mriedem | -1 | |
| 16:56:32 | efried | oh? | |
| 16:56:50 | mriedem | oh. | |
| 16:56:51 | mriedem | yeah. | |
| 16:57:10 | stephenfin | I realise why he suggested it but I'm not sure about the idea of using a microversion when we can't actually provide the previous behavior on old microversions | |
| 16:57:15 | mriedem | the deprecation microversions were the signals | |
| 16:57:30 | stephenfin | we didn't do it for things like os-cells, os-fping, etc. | |
| 16:57:36 | mriedem | if you want to add a microversion for each api we've removed now, you're talking about all of these https://docs.openstack.org/api-ref/compute/#obsolete-apis | |
| 16:57:37 | mriedem | so -1 | |
| 16:58:13 | efried | johnthetubaguy: ^ | |
| 16:58:14 | efried | you have five minutes to respond | |
| 16:58:23 | mriedem | i mean, i'm sure we can deal with someone that shows up and is like, "WTF HAPPENED TO CLOUDPIPE?!" | |
| 16:59:39 | melwitt | the good old days | |
| 17:00:06 | mriedem | https://docs.openstack.org/nova/latest/contributor/api.html#removing-deprecated-apis | |
| 17:00:16 | mriedem | i think i wrote that after we did the first 410s | |
| 17:01:20 | efried | dig | |
| 17:02:05 | openstackgerrit | Merged openstack/nova stable/train: Block deleting compute services with in-progress migrations https://review.opendev.org/694546 | |
| 17:02:11 | openstackgerrit | Merged openstack/nova stable/train: Improve metadata server performance with large security groups https://review.opendev.org/694409 | |
| 17:05:28 | efried | mriedem: I got confused where it was suggested that GET /os-hypervisors*?service_host=CONF.host would ever return more than one entry. | |
| 17:05:40 | efried | It would still return a list, but it would have (at most) one entry, right? | |
| 17:05:46 | johnthetubaguy | mriedem: OK, I buy that. If you see the deprecation micro version, expect to be disappointed with HTTP Gone | |
| 17:05:48 | efried | even for ironic | |
| 17:05:48 | mriedem | WRONG | |
| 17:06:06 | mriedem | efried: compute service : compute node (hypervisor) is 1:M with ironic | |
| 17:06:21 | gregwork | is it expected that when an instance is booted into a non nova az, that volumes created by cinder in the nova AZ (successfully as part of the stack create) will fail to attach to the instance? | |
| 17:06:21 | openstack | Launchpad bug 1852446 in OpenStack Compute (nova) "Hypervisors in nova - no subpage details for ironic" [Undecided,New] | |
| 17:06:21 | mriedem | please to be enjoying https://bugs.launchpad.net/nova/+bug/1852446 | |
| 17:06:28 | gregwork | also cinder doesnt seem to know about the az's i created in nova | |
| 17:06:42 | gregwork | trying to pass availability_zone: mynovaAZ returns an unknown az error | |
| 17:07:05 | mriedem | gregwork: see cross_az_attach in https://docs.openstack.org/nova/latest/admin/availability-zones.html | |
| 17:07:15 | efried | mriedem: ack | |
| 17:07:37 | gregwork | oh interesting | |
| 17:08:06 | mriedem | gregwork: also https://www.openstack.org/videos/summits/vancouver-2018/curse-your-bones-availability-zones-1 | |
| 17:08:27 | mriedem | before you go too deep down the AZ rabbit hole you might want to watch that | |
| 17:08:33 | gregwork | i was at that summit, i wish i attended that session | |
| 17:08:39 | gregwork | ill check it out | |
| 17:09:08 | gregwork | wait cross_az_attach defaults to true | |
| 17:09:16 | gregwork | at least thats how it looks? | |
| 17:09:47 | gregwork | "By default there is no availability zone restruction on volume attach." | |
| 17:09:51 | mriedem | correct | |