Earlier  
Posted Nick Remark
#openstack-nova - 2020-04-02
17:37:39 mordred sean-k-mooney: I agree ... but I'm going to make a concerted push to see how many of those gaps I can close and how much I can ease the situation
17:38:37 mordred might not work, obviously- but otoh - getting novaclient parity and being able to deprecate it would be a _huge_ step forward in terms of gravity, so I think it's worth extra attention
17:39:35 openstackgerrit Merged openstack/nova master: Add test coverage of existing server diagnostics policies https://review.opendev.org/716800
17:40:19 sean-k-mooney well i almost never need to use nova client for day to day things but it does happen.
17:40:31 sean-k-mooney i should really just chagne https://review.opendev.org/#/c/643578/ back to evacuate too
17:44:12 sean-k-mooney mordred: should i just abandon ^ by the way or if i rebase it and change the command back to evacuate will it get reviews
17:44:58 sean-k-mooney i kindof gave up on this since i was hoping it would be a non contoversionl quick gap to close and it ended up being the opisite
17:51:45 mordred sean-k-mooney: I will totally review it
17:51:52 mordred actually. lemme read the scrollback on it real quick
17:53:12 sean-k-mooney mordred: context was at the fourm sesseion we express frustration that evacuate did not match with peopel expcations of keeping data at least not unless you are on shared storage
17:53:26 sean-k-mooney so it was suggested to change the name to resurect or recreate
17:53:46 sean-k-mooney so i did but there is also the consernt that using a different name
17:54:04 sean-k-mooney would confuse people that think its somehting new or different
17:54:38 mordred sean-k-mooney: hrm. well - I like the idea of the term ressurrect
17:54:47 mordred (for all the reasons you mention in the patch)
17:55:19 sean-k-mooney i kind of feel like we should really change how the api works
17:55:34 sean-k-mooney and then have two seperate commands
17:55:41 mordred sean-k-mooney: I kinda want it to be resurrect and then just have evacuate work as an alias for it - so that people with muscle memory will get what they're expecting, but new people will learn something new
17:55:45 mordred or maybe that
17:55:50 sean-k-mooney one that will always preserve your data and refuse to work if not on shared stroate
17:56:01 sean-k-mooney and one that always recretes the vm
17:56:09 sean-k-mooney basicaly just rebuild on a differnet host
17:56:17 mordred yeah - but with the same volumes and ports
17:56:41 sean-k-mooney ya so litrally rebuild on different host reimabing the root disk
17:56:55 sean-k-mooney *reimageing
17:58:01 sean-k-mooney see its tricky and once its in a cli we cant really remove it
18:30:27 openstackgerrit Merged openstack/nova master: Introduce scope_types in server diagnostics https://review.opendev.org/716803
18:39:12 openstackgerrit Merged openstack/nova master: Add new default roles in server diagnostics policies https://review.opendev.org/716805
18:39:19 openstackgerrit Merged openstack/nova master: func tests: move _run_periodics() into base class https://review.opendev.org/714228
18:39:27 openstackgerrit Merged openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546
19:55:22 _erlon_ bauzas: ping
19:56:11 _erlon_ bauzas: so, you mentioned that there is a way to add a custom filter without having to change the egg files?
20:07:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
20:07:51 mriedem hello nova cores, ^ is pretty simple and would be nice to get in before that stuff is all refactored to nova-audit
20:36:02 artom _erlon_, since bauzas is in France is done for the day, maybe https://docs.openstack.org/nova/latest/user/filter-scheduler.html#writing-your-own-filter can help you?
20:39:46 _erlon_ artom: hey, thanks for helping, my question was more about the import error that I was getting the last time I tried to add a class that was not in the built in filters
20:42:08 _erlon_ artom: let's say I create a new file my_filter.py, with TheFilter, inside. If I put my_filter,py inside the nova/scheduler/filters, and change enabled_filters=+TheFilter, it use to give me an import error
20:42:50 _erlon_ artom: so, we needed to add an entry_point in the nova-egg
21:01:48 mordred _erlon_: don't put it in nova/scheduler/filters - check that link
21:07:10 _erlon_ available_filters = myfilter.MyFilter
21:07:10 _erlon_ available_filters = nova.scheduler.filters.all_filters
21:07:10 _erlon_ [filter_scheduler]
21:07:10 _erlon_ see that thereis 2 instances under the same stanza:
21:07:10 _erlon_ mordred: got it! As he puts in the config file, isnt that overwriting the default available filters?
21:09:31 mriedem no, it's multi-valued https://docs.openstack.org/nova/latest/configuration/config.html#filter_scheduler.available_filters
22:23:45 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
22:29:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct security groups policy check_str https://review.opendev.org/716782
22:29:51 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783
22:29:52 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/708785
22:29:52 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784
22:29:53 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787
22:29:53 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786
22:29:54 openstackgerrit melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788
22:37:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
22:43:58 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
22:48:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
23:08:08 openstackgerrit Merged openstack/nova master: FUP for Add a placement audit command https://review.opendev.org/715797
23:08:22 openstackgerrit Merged openstack/nova master: Introduce scope_types in list migrations https://review.opendev.org/716141
23:08:30 openstackgerrit Merged openstack/nova master: Add new default roles in migrations policies https://review.opendev.org/716145
23:08:38 openstackgerrit Merged openstack/nova master: Fix assertEqual param order in Accelerator tests https://review.opendev.org/717069
#openstack-nova - 2020-04-03
00:11:00 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server external events policies https://review.opendev.org/717155
00:13:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
00:13:54 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
00:20:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
00:21:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
00:24:02 alex_xu gmann: dansmith just want to double check with you, there is API behaviour change https://review.opendev.org/#/c/693828/25/nova/api/openstack/compute/volumes.py@495, but it is ok for new microversion. Since it is API change, I want to double check with you before +w
00:39:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server diagnostics policy https://review.opendev.org/716810
01:09:18 rambo_li sean-k-mooney Are we plan to fix this bug before Ussuri release or later?https://bugs.launchpad.net/neutron/+bug/1815989
01:09:21 openstack Launchpad bug 1815989 in OpenStack Compute (nova) "OVS drops RARP packets by QEMU upon live-migration causes up to 40s ping pause in Rocky" [Medium,In progress] - Assigned to sean mooney (sean-k-mooney)
01:10:16 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct security groups policy check_str https://review.opendev.org/716782
01:10:45 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
01:11:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in security groups policy https://review.opendev.org/716786
01:18:09 brinzhang_ alex_xu: thanks
01:18:27 alex_xu brinzhang_: you are welcome
01:20:17 brinzhang_ gmann: I reviewed you and dansmith's chat record, I am sorry not to participate in your discussion. But I have not got more clear thing what I need to do, I tend to wait for dansmith rebase and than update.
01:26:26 openstackgerrit sean mooney proposed openstack/nova master: cyborg evacuate support https://review.opendev.org/715326
02:14:36 openstackgerrit Merged openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
02:18:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793
02:20:49 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server external events policies https://review.opendev.org/717155
02:21:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
02:22:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
02:22:29 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
02:38:43 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server external events https://review.opendev.org/717167
02:44:38 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168
02:47:48 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169
03:52:50 openstackgerrit Merged openstack/nova master: Pass the actual target in migrations policy https://review.opendev.org/716147
04:28:25 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
04:28:26 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
04:41:45 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server group policies https://review.opendev.org/717173
04:48:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168
04:49:25 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169
04:58:36 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server group policy https://review.opendev.org/717174
05:21:36 openstackgerrit Merged openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828
05:33:25 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
05:38:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
06:25:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
06:43:57 bauzas _erlon_: apologies, I was done for the day
06:50:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix server metadata policy to be admin_or_owner https://review.opendev.org/717189
07:01:49 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
07:01:50 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186

Earlier   Later