| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-11-04 | |||
| 19:09:05 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored volume_snapshot{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863498 | |
| 19:09:29 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored config module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863512 | |
| 19:09:49 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored integration test of openstacksdk logging feature https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863529 | |
| 19:10:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored project_access module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| #openstack-sdks - 2022-11-05 | |||
| 00:34:14 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: after adding get_server_action and providing the necessary inputs the return from sever event show is a openstack error https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 09:09:17 | opendevreview | Merged openstack/python-openstackclient master: quota: Allow showing project-specific quotas https://review.opendev.org/c/openstack/python-openstackclient/+/859957 | |
| 09:09:19 | opendevreview | Merged openstack/python-openstackclient master: quota: Deprecate 'quota show --class', 'quota set --class' arguments https://review.opendev.org/c/openstack/python-openstackclient/+/859958 | |
| 09:09:22 | opendevreview | Merged openstack/python-openstackclient master: quota: Trivial style fixups https://review.opendev.org/c/openstack/python-openstackclient/+/859959 | |
| 09:09:24 | opendevreview | Merged openstack/python-openstackclient master: tests: Move fake clients to top of file https://review.opendev.org/c/openstack/python-openstackclient/+/859960 | |
| 09:10:12 | opendevreview | Merged openstack/python-openstackclient master: tests: Remove duplicate FakeImagev1Client https://review.opendev.org/c/openstack/python-openstackclient/+/859961 | |
| 09:10:15 | opendevreview | Merged openstack/python-openstackclient master: tests: Remove unnecessary nesting of volume resources https://review.opendev.org/c/openstack/python-openstackclient/+/859962 | |
| 09:24:50 | opendevreview | Merged openstack/python-openstackclient master: quota: Fix issues with delete quota command https://review.opendev.org/c/openstack/python-openstackclient/+/860721 | |
| 09:24:52 | opendevreview | Merged openstack/python-openstackclient master: quota: Deprecate "force" behavior for network quotas https://review.opendev.org/c/openstack/python-openstackclient/+/819627 | |
| 19:45:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored volume_snapshot{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863498 | |
| 19:45:41 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored config module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863512 | |
| 19:46:01 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored integration test of openstacksdk logging feature https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863529 | |
| 19:46:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored project_access module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| #openstack-sdks - 2022-11-07 | |||
| 09:12:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored config module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863512 | |
| 09:16:46 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored integration test of openstacksdk logging feature https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863529 | |
| 09:54:21 | opendevreview | Hervé Beraud proposed openstack/js-openstack-lib master: Remove python-dev from bindep https://review.opendev.org/c/openstack/js-openstack-lib/+/863833 | |
| 09:54:44 | opendevreview | Hervé Beraud proposed openstack/keystoneauth master: Remove python-dev from bindep https://review.opendev.org/c/openstack/keystoneauth/+/863834 | |
| 10:02:30 | opendevreview | Hervé Beraud proposed openstack/openstacksdk master: Remove python-dev from bindep https://review.opendev.org/c/openstack/openstacksdk/+/863839 | |
| 10:08:15 | opendevreview | Hervé Beraud proposed openstack/shade master: Remove python-dev from bindep https://review.opendev.org/c/openstack/shade/+/863851 | |
| 11:42:30 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored compute_flavor module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863868 | |
| 13:33:39 | opendevreview | Merged openstack/openstacksdk master: Remove python-dev from bindep https://review.opendev.org/c/openstack/openstacksdk/+/863839 | |
| 14:10:38 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Split project_access into {compute_flavor,volume_type}_access modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| 16:24:41 | opendevreview | Artem Goncharov proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/c/openstack/cliff/+/733961 | |
| 16:24:41 | opendevreview | Artem Goncharov proposed openstack/cliff master: Add method to load builtins by string https://review.opendev.org/c/openstack/cliff/+/735465 | |
| 16:26:43 | opendevreview | Artem Goncharov proposed openstack/cliff master: Add method to load builtins by string https://review.opendev.org/c/openstack/cliff/+/735465 | |
| 16:27:13 | opendevreview | Artem Goncharov proposed openstack/osc-lib master: WIP Add support for deferred plugin loading https://review.opendev.org/c/openstack/osc-lib/+/735430 | |
| 16:28:01 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: WIP Use deferred plugin loading https://review.opendev.org/c/openstack/python-openstackclient/+/735431 | |
| 16:45:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Remove nova-network tests https://review.opendev.org/c/openstack/python-openstackclient/+/863897 | |
| 16:45:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Merge osc-functional-devstack-base into only child https://review.opendev.org/c/openstack/python-openstackclient/+/863898 | |
| 16:45:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Enable Swift tests https://review.opendev.org/c/openstack/python-openstackclient/+/863899 | |
| 16:46:00 | stephenfin | gtema: Are you picking up mordred's patches? | |
| 16:46:06 | gtema | yes | |
| 16:46:10 | stephenfin | nice | |
| 16:46:23 | stephenfin | will review once they're ready (showing merge conflict messages rn) | |
| 16:46:37 | gtema | so far was able to have something like 200ms - 1s improvement depending on the command and conditions | |
| 16:47:16 | gtema | one really dirty thing however is how to get rid of reading own entrypoints - possible only by having terribly long dictionary in the code | |
| 16:51:00 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Register base commands as builtins https://review.opendev.org/c/openstack/python-openstackclient/+/735434 | |
| 17:14:46 | frickler | that sounds interesting, makes me want to have more time to look at this | |
| 17:15:26 | gtema | not yet, I still try to figure out all the edges (and I can't put to WIP not my patches) | |
| 17:15:43 | gtema | but in principle - sure, maybe you notice what I have missed | |
| 17:15:48 | frickler | can that dict be autogenerated and placed into a dedicated file? | |
| 17:16:23 | gtema | well, we either put this data in entrypoints or as dict | |
| 17:16:40 | gtema | I was thinking to actually move those into individual clients to keep it a bit lighter | |
| 17:22:12 | frickler | ah, that's kind of 1:1 moved from setup.cfg, that seems manageable. I'll take a closer look later | |
| 17:23:01 | opendevreview | Artem Goncharov proposed openstack/cliff master: Load plugins in the interactive mode https://review.opendev.org/c/openstack/cliff/+/863907 | |
| 17:24:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Move json decoding to base test class https://review.opendev.org/c/openstack/python-openstackclient/+/863908 | |
| 17:24:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert more functional tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863909 | |
| 17:26:23 | opendevreview | Artem Goncharov proposed openstack/cliff master: Load plugins in the interactive mode https://review.opendev.org/c/openstack/cliff/+/863907 | |
| 18:41:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored config module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863512 | |
| 19:05:58 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863924 | |
| 19:06:34 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Split project_access into {compute_flavor,volume_type}_access modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| 19:21:15 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_metadata for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863178 | |
| 19:23:27 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_metadata for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863178 | |
| 21:15:43 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored volume_backup{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863486 | |
| 22:48:31 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_metadata for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863178 | |
| 22:53:04 | opendevreview | Merged openstack/python-openstackclient master: Run swift in -tips job https://review.opendev.org/c/openstack/python-openstackclient/+/858568 | |
| 23:05:01 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add note about microversion 2.87 in server rescue help https://review.opendev.org/c/openstack/python-openstackclient/+/863949 | |
| #openstack-sdks - 2022-11-08 | |||
| 06:48:23 | opendevreview | Dr. Jens Harbott proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/c/openstack/cliff/+/733961 | |
| 06:49:31 | frickler | gtema: ^^ that should at least fix stestr, let's see how for this gets us | |
| 06:50:05 | frickler | s/for/far | |
| 06:51:45 | gtema | Cool, thks. I was not really looking at tests so far, was focused on making it work first | |
| 06:53:36 | frickler | well if even stestr (which uses cliff) doesn't work, not sure if anything else could work | |
| 06:54:42 | frickler | ah, fancy, next patch in the stack fixed this by moving from a list to a set https://review.opendev.org/c/openstack/cliff/+/735465/4/cliff/commandmanager.py | |
| 07:07:34 | opendevreview | Dr. Jens Harbott proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/c/openstack/cliff/+/733961 | |
| 07:07:53 | opendevreview | Dr. Jens Harbott proposed openstack/cliff master: Add method to load builtins by string https://review.opendev.org/c/openstack/cliff/+/735465 | |
| 07:08:39 | opendevreview | Dr. Jens Harbott proposed openstack/cliff master: Load plugins in the interactive mode https://review.opendev.org/c/openstack/cliff/+/863907 | |
| 07:16:16 | frickler | we would need to add cliff as required-project in order to make the depends-on in https://review.opendev.org/c/openstack/osc-lib/+/735430/ work for the unit tests | |
| 07:16:29 | frickler | but I'm first going to spin up some devstack testing now | |
| 07:17:43 | gtema | frickler: overall I could say that in comparison to few years ago this series does not bring a huge improvement anymore. python entrypoints got better itself. But it is still definitely better when running outside venv with lots of other entrypoints | |
| 08:14:36 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863924 | |
| 08:16:13 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updates server_metadata for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863178 | |
| 08:17:55 | gtema | frickler stephenfin - getting rid of munch in SDK helps getting another 150ms win - that is surprising honestly speaking. Something what is not doing much is having such an impact | |
| 08:21:17 | gtema | in most of the cases things are slowing at import pkg_resources which takes around 120ms (on my machine) | |
| 09:52:04 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863924 | |
| 09:54:32 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Split project_access into {compute_flavor,volume_type}_access modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| 10:20:42 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Remove nova-network tests https://review.opendev.org/c/openstack/python-openstackclient/+/863897 | |
| 10:20:43 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Merge osc-functional-devstack-base into only child https://review.opendev.org/c/openstack/python-openstackclient/+/863898 | |
| 10:25:33 | opendevreview | Taehyun Park proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | |
| 10:48:37 | opendevreview | Merged openstack/python-openstackclient master: compute: Add '--no-network', '--auto-network' flags https://review.opendev.org/c/openstack/python-openstackclient/+/862680 | |
| 10:55:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use module defaults groups in ci integration tests to reduce code bloat https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863985 | |
| 11:24:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Move json decoding to base test class https://review.opendev.org/c/openstack/python-openstackclient/+/863908 | |
| 11:24:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert more functional tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863909 | |
| 11:24:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Resolve TODO in image tests https://review.opendev.org/c/openstack/python-openstackclient/+/863986 | |
| 11:24:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert image tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863987 | |
| 11:24:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert volume tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863988 | |
| 11:24:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert identity tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863989 | |
| 11:24:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert compute tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863990 | |
| 11:24:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Convert network tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863991 | |
| 11:28:37 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix '--network none/auto' handling https://review.opendev.org/c/openstack/python-openstackclient/+/862730 | |
| 11:28:41 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Add missing microversion check for networks https://review.opendev.org/c/openstack/python-openstackclient/+/862731 | |
| 12:22:17 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863924 | |
| 12:22:41 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use module defaults groups in ci integration tests to reduce code bloat https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863985 | |
| 12:25:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Split project_access into {compute_flavor,volume_type}_access modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863696 | |
| 12:48:27 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Fixed check mode https://review.opendev.org/c/openstack/ansible-collections-openstack/+/864001 | |
| 12:49:12 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored volume_snapshot{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863498 | |
| 12:49:38 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863924 | |