| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-11-29 | |||
| 18:48:17 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 20:58:18 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 21:54:49 | opendevreview | Merged openstack/python-openstackclient master: image: Remove dead test helper methods https://review.opendev.org/c/openstack/python-openstackclient/+/818223 | |
| 21:54:52 | opendevreview | Merged openstack/openstacksdk master: compute: Add support for server lock reason https://review.opendev.org/c/openstack/openstacksdk/+/819293 | |
| #openstack-sdks - 2021-11-30 | |||
| 05:59:11 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server start and server stop to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/819753 | |
| 07:37:54 | isabek | gtema: Good morning! Can you please approve this patch [1]? Thanks in advance! 1) https://review.opendev.org/c/openstack/openstacksdk/+/819585 | |
| 08:23:27 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Allow setting gateway when creating a router https://review.opendev.org/c/openstack/python-openstackclient/+/736973 | |
| 08:26:29 | frickler | gtema: amotoki: slaweq: ^^ dug out some old stuff if you have a moment | |
| 11:29:41 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Allow setting gateway when creating a router https://review.opendev.org/c/openstack/python-openstackclient/+/736973 | |
| 11:51:39 | opendevreview | Manuel Holtgrewe proposed openstack/ansible-collections-openstack master: Adding "files/config_drive_files" to server module. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/819811 | |
| 14:39:54 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/772641 | |
| 14:39:54 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/761772 | |
| 16:30:07 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/761772 | |
| 16:46:44 | stephenfin | gtema: Can you look at https://review.opendev.org/c/openstack/openstacksdk/+/819585 and https://review.opendev.org/c/openstack/openstacksdk/+/817997, please? | |
| 16:47:00 | stephenfin | Once those are merged, we can cut 0.60, I think | |
| 16:47:18 | stephenfin | *0.61 | |
| 16:47:56 | gtema | I have issue with vpn one. It was created by us, but there are no functests, since in our biggest devstack job there is no vpn. So this one is pretty much untested under real conditions | |
| 16:48:11 | gtema | we tested it on our cloud, but not on devstack | |
| 16:52:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/761772 | |
| 16:56:31 | stephenfin | gtema: Well it's in feature/r1, so it's eventually going to make it to master, no? Unless there's a chance it'll cause regressions elsewhere, I don't think we're losing anything by releasing it earlier (which is what cherry-picking to master does) | |
| 16:57:09 | gtema | it will come to master. But I was really hoping I can find devstack config with VPN to enable testing of it | |
| 16:57:36 | gtema | but ok, you convinced me | |
| 17:00:04 | stephenfin | \o/ | |
| 18:00:14 | opendevreview | Merged openstack/cliff stable/wallaby: Add conflict_handler parameter as attribut in Command class https://review.opendev.org/c/openstack/cliff/+/813767 | |
| 19:07:53 | opendevreview | Merged openstack/python-openstackclient master: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/772641 | |
| 19:07:56 | opendevreview | Merged openstack/python-openstackclient master: Switch server pause and server unpause to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817853 | |
| 19:07:59 | opendevreview | Merged openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 19:08:02 | opendevreview | Merged openstack/python-openstackclient master: Switch openstack server remove port/network to using sdk https://review.opendev.org/c/openstack/python-openstackclient/+/817838 | |
| 19:08:08 | opendevreview | Merged openstack/openstacksdk master: Add "check_limit" to network Quota class https://review.opendev.org/c/openstack/openstacksdk/+/806254 | |
| 19:44:58 | opendevreview | Merged openstack/openstacksdk master: Vpn ike policy resource https://review.opendev.org/c/openstack/openstacksdk/+/819585 | |
| 19:45:01 | opendevreview | Merged openstack/openstacksdk master: Splits class `TestNetworkProxy` https://review.opendev.org/c/openstack/openstacksdk/+/819586 | |
| 19:45:04 | opendevreview | Merged openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 19:45:11 | opendevreview | Merged openstack/python-openstackclient master: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/761772 | |
| 20:16:58 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| #openstack-sdks - 2021-12-01 | |||
| 00:13:26 | opendevreview | Merged openstack/openstacksdk master: compute: Add support for microversion 2.89 https://review.opendev.org/c/openstack/openstacksdk/+/817997 | |
| 00:42:00 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch compute service list, delete and set to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/818434 | |
| 01:33:22 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/xena: Fix typo error in listing server's column name https://review.opendev.org/c/openstack/python-openstackclient/+/819933 | |
| 01:33:28 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/xena: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819934 | |
| 01:33:29 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/xena: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/819935 | |
| 01:35:24 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/wallaby: Fix typo error in listing server's column name https://review.opendev.org/c/openstack/python-openstackclient/+/819936 | |
| 01:35:30 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/wallaby: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819937 | |
| 01:35:31 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/wallaby: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/819938 | |
| 06:04:27 | isabek | stephenfin: Good morning! I have proposed patch for sdk release [1]. We can cut or need wait for some other patches? 1) https://review.opendev.org/c/openstack/releases/+/819952 | |
| 09:56:37 | stephenfin | isabek: nope, looks good. Just need gtema to sign off on it | |
| 09:59:19 | isabek | stephenfin: Thanks! | |
| 11:06:41 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Default to 2.48 for server diagnostics https://review.opendev.org/c/openstack/openstacksdk/+/820016 | |
| 11:27:36 | opendevreview | Merged openstack/openstacksdk master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/openstacksdk/+/793729 | |
| 12:35:52 | opendevreview | Merged openstack/openstacksdk master: Fix some docstrings, and a small bug https://review.opendev.org/c/openstack/openstacksdk/+/805020 | |
| 13:00:12 | opendevreview | Merged openstack/openstacksdk master: Remove misspelled speccing arguments https://review.opendev.org/c/openstack/openstacksdk/+/796706 | |
| 16:14:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Default to 2.48 for server diagnostics https://review.opendev.org/c/openstack/openstacksdk/+/820016 | |
| 16:55:21 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed ip to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 18:02:22 | opendevreview | Merged openstack/openstacksdk master: Adds "test_update_zone" test case https://review.opendev.org/c/openstack/openstacksdk/+/816842 | |
| 21:17:08 | opendevreview | Ujjval proposed openstack/python-openstackclient master: compute: Switch server group to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820070 | |
| 21:35:28 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed ip and remove fixed ip to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 21:37:51 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed ip `to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| #openstack-sdks - 2021-12-02 | |||
| 08:04:43 | opendevreview | Manuel Holtgrewe proposed openstack/ansible-collections-openstack master: Adding "files/config_drive_files" to server module. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/819657 | |
| 09:26:26 | isabek | gtema: Hi! Can you please take a look to release patch [1] when you have a time? Thanks in advance! 1) https://review.opendev.org/c/openstack/releases/+/819952 | |
| 11:12:48 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add --security-group to port list https://review.opendev.org/c/openstack/python-openstackclient/+/815506 | |
| 13:17:43 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add network update quota "limit_check" parameter https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| 18:45:09 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed ip `to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| #openstack-sdks - 2021-12-03 | |||
| 08:24:53 | thomasb06 | gtema: Hello. At the moment, I'm modifying `load_balancer/v2/_proxy.py`, but it looks like the docstrings indentation is broken back again. Maybe I missed something? | |
| 08:25:37 | gtema | maybe there was some additional change that broke it again | |
| 08:26:26 | gtema | actually I do not see any change from you landed there | |
| 08:27:02 | gtema | and I do not see any your change fixing load_balancer | |
| 08:43:56 | thomasb06 | maybe I forgot to commit the changes on this file, I do it now | |
| 08:48:56 | thomasb06 | what is the line length limit, 82 characters? | |
| 08:49:02 | gtema | 80 | |
| 08:49:22 | thomasb06 | arg... Good thing I asked | |
| 08:56:50 | thomasb06 | so 81 triggers an error but 80 is ok? | |
| 08:57:22 | gtema | actually 79 is ok, but 80 is not good anymore | |
| 08:57:35 | thomasb06 | alright | |
| 09:01:45 | thomasb06 | line 1097 or so, there is ":class:`~openstack.load_balancer.v2.availability_zone.AvailabilityZone`," but it can fit 79 characters | |
| 09:02:30 | thomasb06 | maybe for the docstrings, the 80 limit is not blocking? | |
| 09:06:29 | thomasb06 | *can't fit | |
| 09:10:06 | gtema | those particular lines must stay on the same line, and pep should not actually complain about it - it should know what to ignore | |
| 09:10:19 | thomasb06 | great, thank you | |
| 09:10:31 | gtema | but just ensure that :class:`blabla` stays on the single line | |
| 09:11:11 | thomasb06 | so ":rtype: :class:`~openstack.load_balancer.v2.availability_zone_profile.AvailabilityZoneProfile`" takes to be split in two lines? | |
| 09:12:27 | gtema | yes, ":rtype: \n :class:`blabla`" | |
| 09:17:47 | thomasb06 | here it is: https://review.opendev.org/c/openstack/openstacksdk/+/820357 | |
| 09:24:26 | gtema | yeah, let's see what checks will say | |
| 09:25:23 | gtema | oh, I missed in the reviews list. -https://review.opendev.org/c/openstack/openstacksdk/+/816809 - this was the initial change for fixing it in the LB, but checks failed | |
| 09:27:45 | thomasb06 | so maybe we wait for this one? It doesn't seem related to the docstrings, is it? | |
| 09:28:18 | gtema | we take the latest one once it passes | |
| 09:28:46 | thomasb06 | so I abandon number 816809? | |
| 09:30:04 | gtema | yeah, perhaps | |
| 09:30:09 | thomasb06 | sure | |
| 09:34:44 | thomasb06 | Orchestration is ready too: https://review.opendev.org/c/openstack/openstacksdk/+/820360 | |
| 11:26:39 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 12:53:08 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Adding "files/config_drive_files" to server module. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/819657 | |
| 18:44:33 | opendevreview | Suneeth Ravi proposed openstack/python-openstackclient master: Switch server lock to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/819517 | |
| 19:02:18 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 22:51:46 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 22:53:44 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 22:55:53 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| #openstack-sdks - 2021-12-04 | |||
| 00:19:32 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 00:22:43 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 01:44:41 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |
| 16:12:16 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed ip to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 16:15:11 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch add fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820050 | |