| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-11-30 | |||
| 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 | |
| 16:51:41 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 16:54:47 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:32:01 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:39:29 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:45:39 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| #openstack-sdks - 2021-12-05 | |||
| 15:40:00 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 22:10:51 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server dump create to using sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816296 | |
| 22:42:23 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| #openstack-sdks - 2021-12-06 | |||
| 02:00:43 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Fixing issue with functional test https://review.opendev.org/c/openstack/python-openstackclient/+/820466 | |
| 08:30:12 | slaweq | gtema hi, can You check https://review.opendev.org/c/openstack/python-openstackclient/+/819024 once again when You will have some time? thx in advance | |
| 08:45:32 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 10:27:59 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Improve logging for executed commands https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 13:45:44 | sshnaidm | when is "files" in config-drive deprecated? https://github.com/openstack/openstacksdk/blob/215403731db17e14506d2d83b17afd6df6d257d4/openstack/cloud/_compute.py#L687 | |
| 13:45:57 | sshnaidm | is there replacement? | |
| 14:35:23 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Improve logging for executed commands https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 14:43:44 | opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | |
| 17:34:21 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch openstack server show to using sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/817871 | |
| #openstack-sdks - 2021-12-07 | |||
| 10:22:29 | frickler | gtema: could you have another look at https://review.opendev.org/c/openstack/python-openstackclient/+/736973 pls? I had messed up pep8. slaweq this may also be interesting for you | |
| 10:23:08 | gtema | sure, but first need to find reason for one functest failure we currently have | |
| 10:24:25 | frickler | gtema: I haven't seen that, do you have a pointer? | |
| 10:24:52 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 10:26:27 | frickler | volumes once again ... sigh ... will take a look | |
| 10:26:38 | gtema | thks a lot | |