| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-02 | |||
| 17:13:09 | sean-k-mooney | rambo_li: at the moment im not working on it, the patch i submited shoudl fix it but they dont work anymore due to rebases and refacoting | |
| 17:13:25 | mordred | it is that way for hysterical raisins and I think those raisins are no longer useful to us | |
| 17:14:04 | sean-k-mooney | mordred: if we default to oldest that support feature ex it keeps the stablity and if you want the new behavior you ask for it | |
| 17:14:07 | mordred | I am planning on writing something up so we can discuss it around PTG time to make sure it meets what people are wanting | |
| 17:14:26 | sean-k-mooney | but we dont do that today in many cases we say to use option x you need to pass version y | |
| 17:15:18 | sean-k-mooney | i have noticed that lately with osc-placment when trying ot use some operation but im sure we have the same behviaor in the nova supprot in osc | |
| 17:15:22 | mordred | right - I think there's actually a third option which can be more aggressive about new things while not breaking people - but I need to write it up | |
| 17:16:04 | mordred | from my end-user pov - if I *ever* have to give osc a version for a reason other than "I am a developer debugging the rest api" something has completely failed | |
| 17:16:23 | dansmith | mordred: ++ | |
| 17:16:26 | mordred | which is not to say that this failure isn't happening consistently today :) | |
| 17:16:37 | mordred | but - it's a failure from an UX perspective and I want to fix it | |
| 17:16:51 | mordred | s/want/am going to/ | |
| 17:17:12 | mordred | I will know I'm successful when everyone here thinks it's behaving properly ;) | |
| 17:17:14 | dansmith | "if you've ever had to pass a version to OSC in the last six months, you get an ATC pass to the summit" | |
| 17:17:22 | mordred | dansmith: ++ | |
| 17:18:09 | sean-k-mooney | mordred: you will have to get the ironic folks on board too | |
| 17:18:36 | sean-k-mooney | mordred: sicne they went ahead and impleemnted the same version negoction logic in there plugin they had in the legacy client | |
| 17:19:10 | mordred | sean-k-mooney: yeah - I've already got good ironic support done by the ironic team in sdk though - so I think I'm closer with them | |
| 17:19:34 | sean-k-mooney | mordred: i think one thing to condiser is moveing everything after nova v2.0 out of osc and into a nova plugin | |
| 17:19:44 | mordred | I want to get rid of all plugins | |
| 17:20:02 | sean-k-mooney | mordred: well part of the issue currently is geting things landed in osc | |
| 17:20:17 | mordred | right. that's based on the past - we're making some massive changes | |
| 17:20:25 | sean-k-mooney | so project wich plugisn could make progress but "core" project had an issue | |
| 17:20:26 | mordred | so - gimme a little bit to let those changes percolate | |
| 17:20:52 | sean-k-mooney | sure im glad the sdk and ocs team are combined/combinging | |
| 17:31:45 | sean-k-mooney | mordred: honestly if we could deprecate the nova clinet and deliver changes as easily as we can today to osc for V that would be awsome but there are still a long list of gaps to close before that is realistic | |
| 17:33:02 | artom | sean-k-mooney, I don't think anyone disagrees - but to paraphrase mriedem, and directly quote Nike, "just do it" :) | |
| 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 | |