Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-29
17:17:36 stephenfin and now watch
17:18:57 stephenfin hmm, turns out you can't put emoji in commit messages
17:19:00 stephenfin gerrit--
17:19:06 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
17:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group' https://review.opendev.org/696746
17:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745
17:19:21 sean-k-mooney you cant? im pretty sure i have see them there before
17:19:37 stephenfin and I've broken Gerrit. Hurrah!
17:19:50 stephenfin I see
17:19:51 stephenfin ! [remote rejected] HEAD -> refs/for/master%topic=bp/remove-nova-network-ussuri (internal server error: Error inserting change/patchset)
17:20:36 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Blast final references to nova-network https://review.opendev.org/696498
17:20:36 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove references to nova-net service from tests https://review.opendev.org/686818
17:20:37 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove 'enable_hairpin' https://review.opendev.org/696500
17:20:37 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove final references to nova-net from functional tests https://review.opendev.org/696499
17:20:38 openstackgerrit Stephen Finucane proposed openstack/nova master: hyperv: Remove vestigial nova-network support https://review.opendev.org/696502
17:20:38 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove vestigial nova-network support https://review.opendev.org/696501
17:20:39 openstackgerrit Stephen Finucane proposed openstack/nova master: zvm: Remove vestigial nova-network support https://review.opendev.org/696504
17:20:39 openstackgerrit Stephen Finucane proposed openstack/nova master: vmware: Remove vestigial nova-network support https://review.opendev.org/696503
17:20:40 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Flatten class hierarchy for neutron SG tests https://review.opendev.org/696506
17:20:40 openstackgerrit Stephen Finucane proposed openstack/nova master: xenapi: Remove vestigial nova-network support https://review.opendev.org/696505
17:20:41 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Drop nova-network-base security group tests https://review.opendev.org/696508
17:20:41 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove SG tests that don't apply to neutron https://review.opendev.org/696507
17:20:42 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Convert remaining unit tests to neutron https://review.opendev.org/696510
17:20:42 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Convert remaining API tests to use neutron https://review.opendev.org/696509
17:20:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove nova-network security group driver https://review.opendev.org/696512
17:20:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'is_neutron_security_groups' function https://review.opendev.org/696511
17:20:44 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove firewall support https://review.opendev.org/696514
17:20:44 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Correct some broken VIF tests https://review.opendev.org/696513
17:20:56 stephenfin there we go
17:21:21 stephenfin sean-k-mooney: https://review.opendev.org/#/c/696518/ :)
17:21:43 stephenfin There's something cathartic about those stats: +112, -15960
17:22:20 sean-k-mooney was the title orogint nova-net: kill it :)
17:22:31 sean-k-mooney *orgininally
17:22:35 stephenfin there was also a celebration emoji
17:22:40 sean-k-mooney ah ok
17:22:55 stephenfin damn you, Gerrit
17:23:31 sean-k-mooney so we will still have the neutron code in the nova/network/neutronv2 folder right
17:23:39 sean-k-mooney and some interface files in the folder above
17:23:44 stephenfin https://review.opendev.org/#/c/696745/
17:23:52 stephenfin I'm hoping not
17:24:06 stephenfin but that's way off. It'll be weeks before the CI lets us get that far
17:24:10 sean-k-mooney ok because i would like to move those to os-vif i think
17:24:19 sean-k-mooney but i wanted to see what was left after you were done
17:24:29 stephenfin yeah, we can probably kill nova.network.models next
17:24:34 stephenfin in favour of os-vif stuff
17:24:49 sean-k-mooney maybe but im not sure
17:24:54 sean-k-mooney we put those in teh network info cache
17:24:54 stephenfin but I need to move offices again, so I best go
17:25:01 stephenfin have a good weekend o/
17:25:16 sean-k-mooney we can serailise os-vif object but it would require an db migration
17:25:21 sean-k-mooney enjoy o/
18:28:20 openstackgerrit Merged openstack/nova stable/rocky: Fix false ERROR message at compute restart https://review.opendev.org/694581
18:28:26 openstackgerrit Merged openstack/nova master: Add test coverage of existing os-services policies https://review.opendev.org/669181
18:50:13 openstackgerrit Merged openstack/nova master: Introduce scope_types in os-services https://review.opendev.org/645427
19:41:09 openstackgerrit Vlad Gusev proposed openstack/nova stable/queens: Fix false ERROR message at compute restart https://review.opendev.org/696733
20:56:19 openstackgerrit Merged openstack/nova master: Follow up I18d73212f9d98bc75974a024cf6fd872fdfb1ca4 https://review.opendev.org/696740
23:11:28 openstackgerrit Merged openstack/nova stable/train: Reset instance to current vm_state if rolling back in resize_instance https://review.opendev.org/696085
#openstack-nova - 2019-11-30
01:39:32 eandersson When using the openstacksdk why to get a server. Why would flavor_id ever be None?
03:22:46 openstackgerrit Merged openstack/nova stable/queens: Fix false ERROR message at compute restart https://review.opendev.org/696733
04:24:47 openstackgerrit Brin Zhang proposed openstack/nova master: PoC: Support re-configure the delete_on_termination in server https://review.opendev.org/693828
04:47:55 eandersson Do I really need to do this starting with nova 2.47? https://review.opendev.org/#/c/696767/1/senlin/profiles/os/nova/server.py
04:57:37 eandersson I wonder if this has been broken in senlin for 2 years, or if micro version was always set lower or something
06:22:34 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
11:41:50 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove final references to nova-network https://review.opendev.org/696516
11:41:51 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs to remove security group-related objects https://review.opendev.org/696517
11:41:51 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'MetadataManager' https://review.opendev.org/696748
11:41:52 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
11:41:52 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove dependency on nova-net from fake cache https://review.opendev.org/696749
11:41:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group' https://review.opendev.org/696746
11:41:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745
16:04:17 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group' https://review.opendev.org/696746
16:04:17 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745
16:04:18 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776
16:57:10 openstackgerrit Merged openstack/nova master: nova-net: Remove references to nova-net service from tests https://review.opendev.org/686818
16:57:15 openstackgerrit Merged openstack/nova master: docs: Blast final references to nova-network https://review.opendev.org/696498
16:57:23 openstackgerrit Merged openstack/nova master: nova-net: Remove final references to nova-net from functional tests https://review.opendev.org/696499
18:13:11 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group' https://review.opendev.org/696746
18:13:12 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776
19:16:46 openstackgerrit Merged openstack/nova stable/stein: Add --instance option to heal_allocations https://review.opendev.org/693200
19:16:52 openstackgerrit Merged openstack/nova stable/stein: Add BFV wrinkle to TestNovaManagePlacementHealAllocations https://review.opendev.org/693201
19:16:58 openstackgerrit Merged openstack/nova stable/rocky: Nova compute: add in log exception to help debug failures https://review.opendev.org/693009
20:13:58 openstackgerrit Merged openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
20:14:05 openstackgerrit Merged openstack/nova master: libvirt: Remove 'enable_hairpin' https://review.opendev.org/696500
22:30:09 openstackgerrit Merged openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013
#openstack-nova - 2019-12-01
00:51:52 openstackgerrit Merged openstack/nova master: libvirt: Remove vestigial nova-network support https://review.opendev.org/696501
00:51:58 openstackgerrit Merged openstack/nova master: hyperv: Remove vestigial nova-network support https://review.opendev.org/696502
02:04:57 eandersson mriedem https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2075
02:05:15 eandersson This code path can also raise an Eventlet Timeout exception
02:05:29 eandersson Which wouldn't be caught and leads to a UnboundLocalError
02:06:22 eandersson Since eventlet.Timeout is based on BaseException and not Exception.
02:07:00 eandersson http://paste.openstack.org/show/786939/
02:08:59 eandersson Probably almost impossible to hit that timeout, but still :p
06:17:55 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
07:50:08 openstackgerrit Adit Sarfaty proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247
#openstack-nova - 2019-12-02
09:10:02 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Use virDomainBlockCopy to swap volumes with >= 5.10.0 https://review.opendev.org/696834
09:59:06 openstackgerrit Dirk Mueller proposed openstack/nova master: Remove rootwrap filters for nova network https://review.opendev.org/696844
10:11:41 openstackgerrit Guo Jingyu proposed openstack/nova master: WIP:Drop unreliable host_aggregate_map https://review.opendev.org/696851
10:36:16 openstackgerrit Guo Jingyu proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120
11:12:56 stephenfin sean-k-mooney: Hit https://review.opendev.org/#/c/689861/ and it's predecessor, FYI. Sorry for the delay
11:46:27 sean-k-mooney ah cool ill address the comments on that today

Earlier   Later