| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-11-29 | |||
| 15:24:42 | stephenfin | isabek: You can cherry-pick the following three changes to get it released sooner: 1909df1f4219fbf2a1bb0f9ba087ef39758ffded 93f69e42f125fb87c3269881509b3913b9374f34 f5b60d2b03e611854822f17bd5487d322ca0b71f | |
| 15:25:18 | stephenfin | isabek: cherry-pick those and propose them to master and we can merge and cut a new release (cc gtema) | |
| 15:25:44 | isabek | stephenfin: Thank you! I will do it right now | |
| 15:26:13 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Vpn ike policy resource https://review.opendev.org/c/openstack/openstacksdk/+/819585 | |
| 15:28:16 | stephenfin | isabek: Maybe drop the '-x' option | |
| 15:28:34 | stephenfin | given this isn't really a "backport" | |
| 15:28:51 | stephenfin | (i.e. the thing that adds the "cherry picked from" lines) | |
| 15:30:15 | isabek | stephenfin: I cherry picked from UI, should i drop commit message? | |
| 15:30:52 | isabek | * drop from commit message | |
| 15:33:06 | stephenfin | yes | |
| 15:33:24 | stephenfin | fwiw, it would be easier to do this locally since you've got to stack a few changes | |
| 15:33:26 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Splits class `TestNetworkProxy` https://review.opendev.org/c/openstack/openstacksdk/+/819586 | |
| 15:34:20 | stephenfin | They need to be stacked on top of each other in the order I indicated above | |
| 15:34:37 | stephenfin | otherwise you'll get the merge conflicts you're seeing at https://review.opendev.org/c/openstack/openstacksdk/+/819586 | |
| 15:35:05 | isabek | Sorry, you mean as related patches? | |
| 15:35:26 | isabek | Yes there merge conflicts now | |
| 15:35:33 | stephenfin | git checkout master | |
| 15:35:44 | stephenfin | git reset --hard origin/master | |
| 15:36:03 | stephenfin | git checkout -b bug/1930200 | |
| 15:36:16 | stephenfin | git cherry-pick 1909df1f4219fbf2a1bb0f9ba087ef39758ffded | |
| 15:36:22 | stephenfin | git cherry-pick 93f69e42f125fb87c3269881509b3913b9374f34 | |
| 15:36:26 | stephenfin | git cherry-pick f5b60d2b03e611854822f17bd5487d322ca0b71f | |
| 15:36:28 | stephenfin | git review -fy | |
| 15:36:30 | stephenfin | ^ that | |
| 15:36:36 | stephenfin | or something similar | |
| 15:36:44 | isabek | :-) ok I will do it | |
| 15:37:02 | isabek | Should I abandon that cherrypicked patches first? | |
| 15:37:50 | stephenfin | nope, they'll be updated when you push the local changes (since there's the same Change ID) | |
| 15:43:20 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Vpn ike policy resource https://review.opendev.org/c/openstack/openstacksdk/+/819585 | |
| 15:43:21 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Splits class `TestNetworkProxy` https://review.opendev.org/c/openstack/openstacksdk/+/819586 | |
| 15:43:21 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 15:44:43 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 15:46:40 | isabek | stephenfin: Thanks a lot! | |
| 15:50:02 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 16:16:12 | 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 | |
| 16:19:17 | opendevreview | Suneeth Ravi proposed openstack/python-openstackclient master: Switch server lock to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/819517 | |
| 16:48:14 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server pause and server unpause to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817853 | |
| 16:48:33 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 16:50:24 | opendevreview | Thrivikram Mudunuri proposed openstack/python-openstackclient master: Switch server suspend and server resume to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/817860 | |
| 17:43:18 | diablo_rojo | stephenfin, gtema before you both run away for the day. This Friday is the 'deadline' for submitting a proposal for a project for students from NDSU after new years through spring. I am happy to start throwing stuff together if we want to? Otherwise I will try to mobilize another group of mentors somewhere in OpenStack to get a proposal in. | |
| 17:43:49 | gtema | facepalm | |
| 17:43:56 | gtema | :) | |
| 17:44:10 | gtema | I would suggest - not this time | |
| 17:46:18 | diablo_rojo | gtema, fair enough :) | |
| 17:46:23 | diablo_rojo | stephenfin, you agree? | |
| 17:46:42 | diablo_rojo | I was kind of thinking that myself gtema, just wanted to check before I gave it away to another group. | |
| 17:46:58 | gtema | :) | |
| 17:47:19 | stephenfin | Yeah, agreed | |
| 17:47:43 | diablo_rojo | Okay cool. Will find another group to hand it to then :) | |
| 17:47:51 | gtema | thanks a lot | |
| 18:39:50 | 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 | |
| 18:39:51 | 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 | |
| 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 | |