| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-01-26 | |||
| 12:35:36 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored federation_idp{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870313 | |
| 12:35:54 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Pivoted docs to generic resource{,s} modules and StateMachine class https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870529 | |
| 12:36:11 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored tests for keystone_federation_protocol{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870531 | |
| 12:36:24 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Sorted Ansible roles with ci integration tests https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870532 | |
| 12:36:37 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored {group,role}_assignment modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870671 | |
| 12:36:51 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/871815 | |
| 13:29:44 | opendevreview | Merged openstack/ansible-collections-openstack master: Added resource{,s} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869822 | |
| #openstack-sdks - 2023-01-27 | |||
| 11:35:55 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| 13:34:54 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/871815 | |
| 13:39:27 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped unmaintained, obsolete and broken inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872017 | |
| 16:38:01 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 16:44:56 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Add magnum cluster templates resource https://review.opendev.org/c/openstack/openstacksdk/+/871648 | |
| 17:11:31 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 17:40:02 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| 17:41:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/871815 | |
| 17:42:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped unmaintained, obsolete and broken inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872017 | |
| 17:47:18 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| 17:47:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 17:47:28 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 18:36:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | |
| 18:39:40 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Add magnum cluster templates resource https://review.opendev.org/c/openstack/openstacksdk/+/871648 | |
| 20:47:41 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| 20:47:47 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| #openstack-sdks - 2023-01-28 | |||
| 09:44:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | |
| 10:40:32 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| 12:19:09 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 13:12:58 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored identity_domain{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870047 | |
| 13:13:00 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored identity_group{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870070 | |
| 13:15:10 | opendevreview | Merged openstack/openstacksdk master: Add BGP Speakers and Peers to SDK https://review.opendev.org/c/openstack/openstacksdk/+/869485 | |
| 13:17:41 | opendevreview | Merged openstack/openstacksdk master: image: Reformat proxy modules https://review.opendev.org/c/openstack/openstacksdk/+/867040 | |
| 13:17:44 | opendevreview | Merged openstack/openstacksdk master: image: Remove _base_proxy module https://review.opendev.org/c/openstack/openstacksdk/+/867041 | |
| 13:17:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/871815 | |
| 13:18:27 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped unmaintained, obsolete and broken inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872017 | |
| 13:19:21 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored identity_role{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870072 | |
| 13:19:22 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored identity_user{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870087 | |
| 13:19:23 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored federation_idp{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870313 | |
| 13:19:26 | opendevreview | Merged openstack/ansible-collections-openstack master: Pivoted docs to generic resource{,s} modules and StateMachine class https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870529 | |
| 13:19:27 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored tests for keystone_federation_protocol{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870531 | |
| 13:19:28 | opendevreview | Merged openstack/ansible-collections-openstack master: Sorted Ansible roles with ci integration tests https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870532 | |
| 13:19:29 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored {group,role}_assignment modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/870671 | |
| 13:21:21 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Add Tap Services and Flows to SDK https://review.opendev.org/c/openstack/openstacksdk/+/871287 | |
| 13:25:33 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | |
| 13:25:53 | opendevreview | Merged openstack/openstacksdk master: image: Remove unnecessary abstractions https://review.opendev.org/c/openstack/openstacksdk/+/867042 | |
| 13:25:56 | opendevreview | Merged openstack/openstacksdk master: image: Modify signatures of various image methods https://review.opendev.org/c/openstack/openstacksdk/+/867043 | |
| 13:25:59 | opendevreview | Merged openstack/openstacksdk master: image: Remove unsupported parameters from v1 proxy https://review.opendev.org/c/openstack/openstacksdk/+/867044 | |
| 13:26:01 | opendevreview | Merged openstack/openstacksdk master: image: Prevent passing conflicts args to stage_image https://review.opendev.org/c/openstack/openstacksdk/+/867045 | |
| 13:45:04 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 1.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872066 | |
| 15:35:31 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/871815 | |
| 16:38:17 | opendevreview | Merged openstack/openstacksdk master: Add Tap Services and Flows to SDK https://review.opendev.org/c/openstack/openstacksdk/+/871287 | |
| 17:27:02 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | |
| 18:48:35 | opendevreview | Merged openstack/ansible-collections-openstack master: Dropped unmaintained, obsolete and broken inventory script https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872017 | |
| 18:51:43 | opendevreview | Vasyl Saienko proposed openstack/openstacksdk master: Allow to disable prometheus metrics collection https://review.opendev.org/c/openstack/openstacksdk/+/872067 | |
| 19:13:17 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | |
| 20:44:19 | opendevreview | Merged openstack/openstacksdk master: Add magnum cluster templates resource https://review.opendev.org/c/openstack/openstacksdk/+/871648 | |
| 20:48:35 | opendevreview | Merged openstack/openstacksdk master: Finish Magnum rework https://review.opendev.org/c/openstack/openstacksdk/+/871987 | |
| #openstack-sdks - 2023-01-29 | |||
| 08:03:00 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 10:25:31 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Do not create prometheus_client registry by default https://review.opendev.org/c/openstack/openstacksdk/+/872075 | |
| 12:00:56 | opendevreview | Merged openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 12:11:19 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 12:14:53 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Add 'details' parameter to various 'find' baremetal proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/869370 | |
| 12:35:23 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 12:50:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| #openstack-sdks - 2023-01-30 | |||
| 00:54:52 | opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 10:30:34 | opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 10:57:56 | gtema | stephenfin: diablo_rojo_phone willing to give a go for https://review.opendev.org/c/openstack/openstacksdk/+/872030? | |
| 11:01:07 | stephenfin | gtema: 🥳 Looking | |
| 11:01:07 | stephenfin | gtema: 🥳 Looking | |
| 11:07:59 | gtema | thanks stephenfin. What have you meant with comments on "Resource" and "Munch"? | |
| 11:08:22 | stephenfin | Oh, I was just suggesting to format them as ``literals`` | |
| 11:08:29 | stephenfin | double backticks | |
| 11:08:33 | gtema | got it | |
| 11:11:00 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 11:13:06 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 11:22:50 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Do not create prometheus_client registry by default https://review.opendev.org/c/openstack/openstacksdk/+/872075 | |
| 13:05:45 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 13:32:27 | gtema | stephenfin: another look at https://review.opendev.org/c/openstack/openstacksdk/+/872030? | |
| 13:33:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 15:33:25 | stephenfin | gtema: one small typo there but otherwise good (y) | |
| 15:33:43 | stephenfin | gtema: Unrelated: are you happy to backport https://review.opendev.org/c/openstack/python-openstackclient/+/869832/ ? | |
| 15:33:50 | gtema | another typo (facepalm) | |
| 15:35:18 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 15:36:04 | gtema | fixed. I seem to make lot of typos in a hurry to make this happen | |
| 15:37:05 | stephenfin | ta, +2 | |
| 15:37:26 | gtema | stephenfin, your opinion on https://review.opendev.org/c/openstack/openstacksdk/+/872075 is also welcome. Would be nice to get it into R1.0 since it can be treated as backward incompatible | |
| 15:38:50 | stephenfin | looking | |
| 15:47:38 | stephenfin | gtema: I prefer Vasyl's approach, personally, though I think we should do it for all collectors (i.e. statsd and InfluxDB too) | |
| 15:47:52 | stephenfin | but I can be swayed either way | |
| 15:48:04 | gtema | hmmm | |
| 15:48:54 | gtema | adding "enabled: true" to them feel real awkward for me. Either section is present (which means we gather metrics) or absent - means not | |
| 15:54:20 | stephenfin | I'll defer to you then. At least what you've done is backwards compatible. Could we improve documentation around this, beyond the release note? | |
| 15:55:00 | stephenfin | i.e. add a worked example to doc/source/user/guides/stats.rst | |
| 16:02:15 | gtema | in my eyes this is matching currently to https://docs.openstack.org/openstacksdk/latest/user/guides/stats.html | |
| 16:35:55 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: volume: Allow more versions https://review.opendev.org/c/openstack/python-openstackclient/+/869832 | |
| 18:29:30 | opendevreview | Merged openstack/openstacksdk master: Prepare release note for R1.0 https://review.opendev.org/c/openstack/openstacksdk/+/872030 | |
| 18:43:18 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 18:55:53 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| #openstack-sdks - 2023-01-31 | |||
| 08:18:55 | lajoskatona | gtema, stephenfin: Hi, if you have some free time could you please review this one for Neutron trunks: https://review.opendev.org/c/openstack/python-openstackclient/+/869447 , thanks in advance | |
| 08:19:00 | lajoskatona | gtema, stephenfin: if it is merged we can delete the trunk command related code from neutronclient :-) | |
| 08:19:23 | gtema | great. Looking now | |
| 08:23:25 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |