| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-21 | |||
| 09:28:56 | gtema | I want to run against the wall because of this mess. And users should be also frustrated | |
| 09:29:14 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | |
| 09:29:22 | gtema | I am not opposed to backporting, but I don't really think this is the best approach | |
| 09:30:29 | frickler | gtema: so what is a better approach. add a "needs newer sdk" error to OSC? | |
| 09:30:38 | gtema | from my pov the best case is if osc can detect whether feature is available on cloud and raise error for the command if it is not. The same to do with SDK saying: you need to use newer SDK | |
| 09:30:57 | gtema | this is also what ansible modules are doing | |
| 09:31:10 | frickler | hmm, o.k. | |
| 09:31:31 | gtema | but the worst is necessity to implement and backport it | |
| 09:31:37 | gtema | it's a real mess | |
| 09:41:08 | frickler | hmm, the ndp proxy commands also fail to check whether Neutron has the required extension enabled. not directly related, but also less than optimal | |
| 09:42:22 | gtema | why is that? is it not reported as extension? | |
| 09:43:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Add 'image task show' commands https://review.opendev.org/c/openstack/python-openstackclient/+/813436 | |
| 09:43:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Add 'image task list' command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 09:52:28 | frickler | it is, the check just hasn't been added to the code | |
| 09:52:54 | gtema | okay | |
| 10:14:19 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 10:15:58 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | |
| 10:16:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | |
| 10:27:09 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 10:30:37 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 10:48:53 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.101.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858670 | |
| 12:05:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 12:19:13 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 12:20:17 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 12:22:09 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 12:29:30 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update federation_mapping_info module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |
| 13:08:54 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update address scope to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848635 | |
| 13:25:22 | opendevreview | Bence Romsics proposed openstack/python-openstackclient master: Improve help text for network create --external https://review.opendev.org/c/openstack/python-openstackclient/+/858708 | |
| 13:28:02 | opendevreview | Bence Romsics proposed openstack/python-openstackclient master: Improve help text for network create --external https://review.opendev.org/c/openstack/python-openstackclient/+/858708 | |
| 13:47:27 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update address scope to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848635 | |
| 14:53:29 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Specifically build master in tripleo periodic job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858760 | |
| 14:54:08 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Specifically build master in tripleo periodic job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858760 | |
| 16:35:26 | Daviey | https://review.opendev.org/c/openstack/openstacksdk/+/853703 Could really do with another Core review please? | |
| 16:38:14 | opendevreview | Merged openstack/ansible-collections-openstack master: Update address scope to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848635 | |
| 17:07:17 | frickler | got the same failure twice now on openstacksdk-functional-devstack for yoga https://zuul.opendev.org/t/openstack/build/935a8d58f509404f852297c68e2bd5fe/ | |
| 17:22:09 | gtema | There's cleanup fix | |
| 17:22:20 | gtema | It helped on yoga branch of nova | |
| 17:42:23 | opendevreview | Merged openstack/ansible-collections-openstack master: Update federation_mapping_info module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |
| 17:43:28 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.101.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858670 | |
| 18:00:30 | frickler | gtema: ah, there's already a backport proposed https://review.opendev.org/c/openstack/openstacksdk/+/856509 , I'll look into the failures | |
| 18:01:57 | gtema | frickler: https://review.opendev.org/c/openstack/openstacksdk/+/858268 | |
| 18:02:07 | gtema | This is the real fix | |
| 18:33:21 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update quota for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/851483 | |
| 19:31:26 | opendevreview | Merged openstack/openstacksdk master: Improve project cleanup for cinder https://review.opendev.org/c/openstack/openstacksdk/+/858268 | |
| 20:09:02 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updated network module docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858642 | |
| 20:37:17 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | |
| 23:11:22 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858834 | |
| 23:14:29 | opendevreview | Merged openstack/ansible-collections-openstack master: Updated network module docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858642 | |
| #openstack-sdks - 2022-09-22 | |||
| 04:49:41 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.101.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858670 | |
| 05:55:07 | frickler | now I'm confused, why does the failing job on yoga use sdk from master? but maybe then it will be fixed now and we don't actually need the backport | |
| 06:11:57 | gtema | frickler, some month ago it was noticed that devstack on stable branches was using sdk from same branch and we said it was wrong. I guess back those days it was changed to be using master one | |
| 06:12:10 | gtema | and yes, I guess the fix should help | |
| 06:20:38 | frickler | hmm, o.k., maybe then sdk master should also run jobs against stable devstack to avoid such regressions | |
| 06:22:57 | gtema | oh, another 5-6 long running jobs? ;-) | |
| 06:33:01 | frickler | well the alternative is to make those jobs non-voting in devstack and essentially ignore them that way. if you want to claim that sdk is backwards compatible, you either need to test it or that claim doesn't have much substance | |
| 06:33:24 | gtema | yeah, right | |
| 07:11:46 | opendevreview | Merged openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.101.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858670 | |
| 11:11:11 | opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Add support for updated_at field for volume snapshots https://review.opendev.org/c/openstack/openstacksdk/+/858289 | |
| 11:19:44 | opendevreview | Bence Romsics proposed openstack/python-openstackclient master: Improve help text for network create --external https://review.opendev.org/c/openstack/python-openstackclient/+/858708 | |
| 12:26:31 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update quota for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/851483 | |
| 13:11:31 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | |
| 14:11:42 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Specifically build master in tripleo periodic job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858760 | |
| 14:14:53 | opendevreview | Merged openstack/openstacksdk master: Add support for updated_at field for volume snapshots https://review.opendev.org/c/openstack/openstacksdk/+/858289 | |
| 14:44:56 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858834 | |
| 14:45:25 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858834 | |
| 16:04:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: doc: Update nova command mapping https://review.opendev.org/c/openstack/python-openstackclient/+/858952 | |
| 16:24:15 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Conditionally skip ndp_proxy tests https://review.opendev.org/c/openstack/python-openstackclient/+/858955 | |
| 16:25:12 | gtema | frickler, jm1: I think ^^ can be used to address ndp_proxy tests of master osc with older sdk | |
| 16:53:55 | opendevreview | Merged openstack/ansible-collections-openstack master: Specifically build master in tripleo periodic job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858760 | |
| 17:31:07 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates server_volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858834 | |
| 17:43:39 | jm1 | gtema: great :) | |
| 20:23:38 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test job with capped sdk https://review.opendev.org/c/openstack/python-openstackclient/+/858569 | |
| 21:07:12 | Daviey | Hi, please could some review https://review.opendev.org/c/openstack/openstacksdk/+/853703 , thanks | |
| #openstack-sdks - 2022-09-23 | |||
| 05:03:58 | frickler | gtema: o.k., that's a step forward at least. one down, two to go ;) https://zuul.opendev.org/t/openstack/build/4e67f635ac524001a010ef2fa4d46ef7 | |
| 07:12:37 | opendevreview | Merged openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 07:19:03 | Daviey | (Thanks gtema!) | |
| 08:20:49 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Initialize tests of real clouds https://review.opendev.org/c/openstack/openstacksdk/+/859026 | |
| 08:47:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Force major version bump in pbr https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859053 | |
| 08:57:57 | gtema | @frickler you are in the infra team, aren't you? | |
| 09:01:00 | frickler | gtema: pleading guilty | |
| 09:14:57 | gtema | so, back from meeting | |
| 09:15:19 | gtema | I am working on the concept of running sdk/osc functional tests against real clouds | |
| 09:15:24 | gtema | I have now creds for Cleura | |
| 09:15:43 | gtema | and I want to make non voting jobs for running those | |
| 09:16:00 | gtema | now the point (we did it in our zuul this way) | |
| 09:16:28 | gtema | in order to run this tests in check I need to define jobs in config project (meaning project-config) | |
| 09:17:25 | gtema | here I would need to put credentials of the clouds, define jobs that prepare clouds.yaml for sdk/osc (and I do this with token and not pwd to reduce chance of leak) and revokation of the token afterwards | |
| 09:18:07 | gtema | would this be accepted by infra (to put such jobs into the project-config) or we should better consider setting up a 3rdparty CI for all of that? | |
| 09:34:58 | gtema | frickler: https://review.opendev.org/c/openstack/project-config/+/859060 + https://review.opendev.org/c/openstack/openstacksdk/+/859026 are wips for that | |
| 09:54:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Initialize tests of real clouds https://review.opendev.org/c/openstack/openstacksdk/+/859026 | |
| 09:55:40 | frickler | gtema: you could put the secrets into the sdk repo, too, I'm not sure that project-config is needed for that. but I'm not sure how to circumvent the restriction that secrets can only be used in post-merge jobs | |
| 09:56:10 | frickler | also, do we need to serialize jobs? or could multiple jobs run with the same credentials in parallel? | |
| 09:56:22 | gtema | by placing them in the project-config repo, this is for sure. That way sdk would be able to use those jobs in check | |
| 09:56:56 | gtema | parallelization is a bit different question where I have no answer yet. cleanup is the similar one | |
| 09:57:09 | gtema | but first we should have some base for running them at all | |
| 09:57:52 | gtema | for the reference how we do it in our setup: | |
| 09:57:54 | gtema | https://github.com/opentelekomcloud-infra/zuul-project-config/blob/master/zuul.d/jobs.yaml#L72 | |
| 09:58:17 | gtema | and used i.e. here: https://github.com/opentelekomcloud/python-otcextensions/blob/master/.zuul.yaml#L40 | |
| 09:59:13 | gtema | in our setup zuul-project-config is also a config repo. And jobs with secrets defined in a config repo can be used in non post-review pipelines | |
| 09:59:43 | frickler | o.k., I'll have a look later. what I've done in a similar situation is use a static node that gets pre-configured with the credentials out-of-band | |