| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-01-28 | |||
| 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 | |
| 08:23:25 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 08:23:41 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 08:23:41 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 10:45:39 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 10:45:39 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 11:25:08 | stephenfin | gtema: I'm going to propose 6.1.0 of OSC, if that's okay with you? In the interim, can you look at https://review.opendev.org/c/openstack/releases/+/870732 ? | |
| 11:27:04 | gtema | Yes, go for it | |
| 11:35:04 | stephenfin | lajoskatona: That had been on my list. Done! | |
| 11:35:50 | stephenfin | gtema: Actually, before we cut that we might merge https://review.opendev.org/q/project:openstack/python-openstackclient+topic:add-missing-cinderclient-commands for whoami-rajat ? | |
| 11:36:44 | gtema | looking on those | |
| 11:36:52 | whoami-rajat | stephenfin, we've 2 weeks for client release right? or am i missing something | |
| 11:37:24 | stephenfin | Yeah, but we're cycle with intermediary so we can cut effectively whenever so long as we have *a* release per-cycle | |
| 11:38:00 | stephenfin | there are a lot of changes queued up (especially for glance - coverage is looking sooo much better there now) so let's get the bulk of them out before the panic | |
| 11:38:13 | whoami-rajat | ok, I've 3 more commands to add and then cinder will be fully in sync with OSC, at least for the commands part | |
| 11:38:40 | whoami-rajat | so are we planning to release again then with other client releases? | |
| 11:38:50 | stephenfin | Perfect. Ping me if/when you've those up and I'll happily review them. | |
| 11:39:01 | stephenfin | Yeah. We can release whenever it suits us | |
| 11:39:17 | whoami-rajat | cool, I will work on them then, thanks! | |
| 11:39:21 | stephenfin | Let's aim for a 6.2.0 in the next few weeks if you can get cinder coverage closed out | |
| 11:39:48 | whoami-rajat | stephenfin, do you want me to update this? just noticed your comments https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | |
| 11:40:03 | whoami-rajat | or better as a followup? | |
| 11:40:11 | stephenfin | follow-up would be better | |
| 11:40:23 | stephenfin | they're nits at best | |
| 11:41:00 | whoami-rajat | ok, sounds good | |
| 11:58:21 | opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 12:18:55 | lajoskatona | stephenfin, gtema: thanks | |