| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-12-31 | |||
| 02:03:58 | opendevreview | Violet Kurtz proposed openstack/python-openstackclient master: Finish switching server migration to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/867334 | |
| #openstack-sdks - 2023-01-01 | |||
| 12:57:16 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Improved compatibility with both tox>3,<4 and tox>=4 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868776 | |
| #openstack-sdks - 2023-01-02 | |||
| 08:13:49 | opendevreview | Jiri Podivin proposed openstack/cliff master: Removing helper functions providing Python < 3.3 compatibility https://review.opendev.org/c/openstack/cliff/+/862689 | |
| 10:46:25 | opendevreview | Felix Huettner proposed openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | |
| 10:53:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored security_group{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867740 | |
| 10:54:11 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored loadbalancer module breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867558 | |
| 10:54:25 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored lb_{health_monitor,listener,member,pool} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867560 | |
| 12:21:20 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored loadbalancer module breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867558 | |
| 12:22:03 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored lb_{health_monitor,listener,member,pool} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867560 | |
| 12:23:04 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 18:35:40 | opendevreview | Merged openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | |
| 21:43:24 | opendevreview | Merged openstack/openstacksdk master: Normalise query strings passed to 'find_*' methods https://review.opendev.org/c/openstack/openstacksdk/+/868094 | |
| #openstack-sdks - 2023-01-03 | |||
| 00:59:32 | opendevreview | Merged openstack/openstacksdk master: Deprecate all of the compute image proxy APIs https://review.opendev.org/c/openstack/openstacksdk/+/868095 | |
| 09:01:52 | opendevreview | Merged openstack/openstacksdk master: Add missing block storage v2 'find_*' methods https://review.opendev.org/c/openstack/openstacksdk/+/868096 | |
| 09:01:54 | opendevreview | Merged openstack/openstacksdk master: Remove unnecessary mocks https://review.opendev.org/c/openstack/openstacksdk/+/868097 | |
| 09:12:03 | opendevreview | Merged openstack/openstacksdk master: Add 'all_projects' support to proxy layers https://review.opendev.org/c/openstack/openstacksdk/+/868098 | |
| 12:51:50 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move normalize security group to sg class https://review.opendev.org/c/openstack/openstacksdk/+/865025 | |
| 12:52:40 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move normalize_server to compute mixin https://review.opendev.org/c/openstack/openstacksdk/+/866567 | |
| 12:52:41 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop _normalize class https://review.opendev.org/c/openstack/openstacksdk/+/865044 | |
| 12:52:41 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move _normalize_coe_* into _coe class https://review.opendev.org/c/openstack/openstacksdk/+/865043 | |
| 12:52:42 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 13:32:14 | opendevreview | Christian Rohmann proposed openstack/openstacksdk stable/yoga: Replace deprecated inspect.getargspec call https://review.opendev.org/c/openstack/openstacksdk/+/869081 | |
| 14:20:15 | opendevreview | Merged openstack/openstacksdk master: Whitelist cloud functional tests in acceptance https://review.opendev.org/c/openstack/openstacksdk/+/860009 | |
| 21:36:21 | opendevreview | Merged openstack/openstacksdk master: Add 'details' parameter to various 'find' proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/868099 | |
| #openstack-sdks - 2023-01-04 | |||
| 08:45:41 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 10:03:11 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 10:30:09 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored compute_service_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869167 | |
| 11:00:47 | opendevreview | Jiri Podivin proposed openstack/python-openstackclient master: Excluding test code from coverage reports https://review.opendev.org/c/openstack/python-openstackclient/+/856237 | |
| 12:15:59 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tox: Trivial fixes https://review.opendev.org/c/openstack/openstacksdk/+/869199 | |
| 12:17:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/zed: Update tox.ini for tox v4 compatibility https://review.opendev.org/c/openstack/openstacksdk/+/869200 | |
| 12:18:02 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/yoga: Update tox.ini for tox v4 compatibility https://review.opendev.org/c/openstack/openstacksdk/+/869201 | |
| 12:18:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/yoga: Replace deprecated inspect.getargspec call https://review.opendev.org/c/openstack/openstacksdk/+/869081 | |
| 12:26:47 | stephenfin | gtema: When you've time, https://review.opendev.org/c/openstack/openstacksdk/+/867890 is an important fix for various OSC changes | |
| 12:27:28 | gtema | Ack | |
| 12:40:43 | opendevreview | Polina Gubina proposed openstack/ansible-collections-openstack master: Volume module: - Add 'is_multiattach' parameter. When it is true, volume will be shareble https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865477 | |
| 13:29:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Use fake SDK Migration object https://review.opendev.org/c/openstack/python-openstackclient/+/869211 | |
| 13:29:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Finish switching server migration to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/867334 | |
| 13:36:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove unnecessary nesting of compute resources https://review.opendev.org/c/openstack/python-openstackclient/+/867626 | |
| 13:36:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Use SDK objects where expected https://review.opendev.org/c/openstack/python-openstackclient/+/867627 | |
| 13:54:09 | opendevreview | Merged openstack/openstacksdk master: compute: Pass microversion for actions https://review.opendev.org/c/openstack/openstacksdk/+/867890 | |
| 13:54:11 | opendevreview | Merged openstack/python-openstackclient master: Excluding test code from coverage reports https://review.opendev.org/c/openstack/python-openstackclient/+/856237 | |
| 16:51:33 | opendevreview | Merged openstack/openstacksdk master: tox: Trivial fixes https://review.opendev.org/c/openstack/openstacksdk/+/869199 | |
| 18:56:48 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 19:06:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored project{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869254 | |
| 19:43:44 | opendevreview | Benjamin Alan Beasley proposed openstack/cliff master: Use importlib.metadata on Python 3.10+ https://review.opendev.org/c/openstack/cliff/+/859579 | |
| 20:28:22 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated contributor docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868923 | |
| 20:34:00 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored project{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869254 | |
| #openstack-sdks - 2023-01-05 | |||
| 07:59:40 | jm1 | gtema: hey :) openstacksdk's remove_floating_ip_from_server() is failing randomly in aoc ci tests. often a simple recheck solves the issue. looking at nova's api doc it looks like the action being called is deprecated, so maybe we shall mark that sdk function as deprecated as well? https://docs.openstack.org/api-ref/compute/#remove-disassociate-floating-ip-removefloatingip-action-deprecated | |
| 08:01:42 | gtema | I will have a deeper look. Can you see any dependency whether this fails when running on certain cloud? | |
| 08:09:28 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use Neutron instead of Nova when detaching floating ips if available https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869284 | |
| 08:11:09 | jm1 | gtema: we run tests only against devstack https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_921/869167/1/check/ansible-collections-openstack-functional-devstack/9219698/job-output.txt | |
| 08:11:23 | jm1 | gtema: error at 2023-01-04 21:23:49.821932 | |
| 08:12:10 | opendevreview | Felix Huettner proposed openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | |
| 08:13:03 | jm1 | gtema: ah, its failing now because of this: https://review.opendev.org/c/openstack/openstacksdk/+/867890 | |
| 08:13:16 | gtema | Ok, will check. Even it is devstack things sometimes run differently depending on whether they run in ovh or rax or vex (timeouts and co) | |
| 08:13:49 | jm1 | gtema: stephenfin's patch is fine, its just our code which is using deprecated code. submitted a patch to aoc already | |
| 08:14:56 | gtema | But why then recheck helps, this is what disturbs me | |
| 08:15:09 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use Neutron instead of Nova when detaching floating ips if available https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869284 | |
| 08:16:11 | jm1 | gtema: forget about my recheck comment. that was about the other floating ip error we are chasing in cloud layer | |
| 08:18:52 | jm1 | gtema: when checking ci output i was first assuming its the same random floating ip error which happens from time to time. but its a different issue | |
| 08:19:23 | gtema | Ok | |
| 09:21:12 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored security_group{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867740 | |
| 10:36:01 | opendevreview | Arnaud Morin proposed openstack/ansible-collections-openstack master: Rename reboot into reboot_soft https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869343 | |
| 11:12:01 | sshnaidm | dtantsur, hi, probably https://review.opendev.org/c/openstack/openstacksdk/+/868099 broke baremetal node module and SDK,it fails in bifrost job: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_edf/869284/2/check/bifrost-keystone-collections-src/edf27b6/job-output.txt | |
| 12:10:58 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored security_group_rule{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869355 | |
| 12:46:57 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Revert "Add 'details' parameter to various 'find' proxy methods" https://review.opendev.org/c/openstack/openstacksdk/+/869315 | |
| 12:47:46 | frickler | gtema: stephenfin: ^^ see sshnaidm's comment, this was shown in the non-voting jobs on that patch already, I guess we should make at least the ironic job voting? | |
| 12:48:23 | frickler | also take better care of hard broken n-v jobs like legacy so that people do not get used to simply ignoring them | |
| 12:48:30 | gtema | ack | |
| 12:48:59 | frickler | if you can fix it instead of reverting, that would of course be fine, too | |
| 12:49:25 | stephenfin | Yeah, better to fix. I'll take a look now | |
| 12:52:37 | opendevreview | Merged openstack/ansible-collections-openstack master: Use Neutron instead of Nova when detaching floating ips if available https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869284 | |
| 13:04:35 | opendevreview | Taehyun Park proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | |
| 13:09:54 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored compute_service_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869167 | |
| 13:10:17 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored project{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869254 | |
| 13:10:56 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored loadbalancer module breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867558 | |
| 13:11:12 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored lb_{health_monitor,listener,member,pool} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867560 | |
| 13:11:45 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 13:45:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Rework 'details' parameter for baremetal proxy https://review.opendev.org/c/openstack/openstacksdk/+/869370 | |
| 13:45:27 | stephenfin | frickler: ^ | |
| 14:23:25 | frickler | stephenfin: thx, I'll mostly have to defer to what the CI says though | |
| 15:10:10 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move _normalize_coe_* into _coe class https://review.opendev.org/c/openstack/openstacksdk/+/865043 | |
| 15:50:08 | lajoskatona | stephenfin, frickler: Hi, a question to think about regarding networking stadium projects CLI and python binding code | |
| 15:50:33 | lajoskatona | stephenfin, frickler: back in December we discussed it on team meeting: https://meetings.opendev.org/meetings/networking/2022/networking.2022-12-20-14.01.log.html#l-108 | |
| 15:51:14 | lajoskatona | stephenfin, frickler: frickler already shared his opinion but I think it is better to have the discussion here also perhaps with more people | |
| 15:52:07 | gtema | can you please reiterate what is stadium and what you want to achieve? | |
| 15:53:22 | lajoskatona | gtema: stadium projects for Neutron are projects that implement some networking feature/functionality with extra API that is not in core Neutron (like neutron-fwaas for Firewalls, neutron-dynamic-routing for BGP etc...) | |
| 15:54:31 | gtema | vpnaas as well? | |
| 15:54:33 | lajoskatona | gtema: so these are not core networking features, but now we have the CLI code (both old neutron commands and OSC plugin) in most cases in python-neutronclient repo | |
| 15:54:49 | lajoskatona | gtema yes vpnaas is also a stadium project | |
| 15:54:59 | gtema | okay | |
| 15:56:21 | lajoskatona | gtema: and we already deprecated the CLI code in python-neutronclient but the python bining code is also there, and even some OSC plugins call into the python binding of neutronclient instead of SDK | |
| 15:58:22 | gtema | ok, and so we should add those into the sdk and cli | |
| 15:59:00 | lajoskatona | gtema: and the question is that can we put these SDK methods (like add_bgp_speaker_to_dragent) to openstacksdk or keep them in neutronclient, and the same for the CLI part, shall we add these to osc or keep them as plugins in neutronclient? | |
| 15:59:40 | gtema | I would really like to see us starting adding those to sdk so that later also ansible modules can make use of that | |
| 15:59:59 | gtema | and once sdk is done also osc | |
| 16:00:02 | lajoskatona | gtema: if you can check the meeting log from December which I linked, the argument was to move only openstack network * commands, but that was just a first glance idea at that time | |
| 16:00:23 | gtema | or alternatively keep those in neutronclient (cli part), but use sdk as base | |
| 16:00:35 | lajoskatona | gtema: ok | |
| 16:01:18 | gtema | from my pov it is really important to have sdk coverage | |