Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-22
15:58:38 melwitt make sense now, aaarrrggghh
15:58:49 stephenfin mriedem: question on https://review.opendev.org/#/c/686810/7/api-ref/source/os-tenant-network.inc@2
15:59:07 stephenfin ?
15:59:07 stephenfin does that actually make sense, since some of the APIs have been removed but others have not
16:05:54 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: Filter hypervisors by service host https://review.opendev.org/695716
16:07:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'USE_NEUTRON' from functional tests https://review.opendev.org/686811
16:07:43 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove '/os-tenant-networks' REST API https://review.opendev.org/686810
16:07:44 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812
16:07:45 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814
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

Earlier   Later