| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-21 | |||
| 09:05:29 | frickler | yes | |
| 09:05:40 | gtema | okay | |
| 09:06:01 | frickler | osc from master, sdk from stable/yoga, testing against master devstack | |
| 09:06:27 | gtema | we were discussing, but never really did - change the release model to not to have stable branches anymore. This will partially help to get rid of this "compatibility" | |
| 09:06:35 | frickler | (sdk from stable/yoga was easier and more flexible than just pinning to 0.61.0) | |
| 09:08:25 | frickler | maybe we don't need a general solution, just one for the current ansible-collection situation | |
| 09:09:08 | gtema | tja, this is just a 3rd dimension of compatibility issues | |
| 09:10:11 | gtema | I am actually wondering why "osc from master, sdk from stable/yoga, testing against master devstack"? | |
| 09:10:36 | gtema | this is wrong in my eyes | |
| 09:11:25 | frickler | the question is whether people can co-install ansible-collections and OSC | |
| 09:11:37 | frickler | s/can/should be able to/ | |
| 09:11:45 | frickler | currently they can't | |
| 09:11:54 | gtema | I would say the should be able to | |
| 09:12:08 | gtema | we have few cases on our side requiring that | |
| 09:12:12 | jm1 | gtema: "osc from master, sdk from stable/yoga and other openstack components from master" is what we might have to fallback to in rdo/tripleo. aoc is not compatible to new sdk, hence tripleo is not, too... | |
| 09:12:14 | frickler | ansible-collections still depends on sdk<0.99 | |
| 09:12:45 | frickler | osc requires sdk>=0.99, at least for testing to pass | |
| 09:16:03 | jm1 | gtema, frickler: we are currently discussing how to solve this sdk version deadlock for rdo zed release: one solution is "sdk+aoc+tripleo from yoga, everything else from zed". another solution is to use everything from zed, including a half-broken aoc release and a has-to-be-ported-to-new-aoc-until-zed-release tripleo. difficult tradeoff.. | |
| 09:23:41 | jm1 | gtema, frickler: zed's osc being backward compatible with yoga's sdk would help with the first approach | |
| 09:26:06 | gtema | tja, and how to make zed osc to yoga sdk without backporting mess? | |
| 09:27:25 | frickler | so there are three features involved: server_group, router_ndp_proxy and network_qos_rule. we can either make OSC just ignore these features with older sdk or backport to stable/yoga sdk. the former solution might be more difficult from a UI perspective. like you will still have the commands listed in the help output, but cannot use them | |
| 09:28:24 | 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 | |
| 09:28:49 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | |
| 09:28:53 | frickler | gtema: are you completly opposed to backporting? if not, I'd start by taking a look at ndp_proxy and see how much work that would be | |
| 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 list' command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 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: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!) | |