| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-20 | |||
| 13:45:20 | opendevreview | Rabi Mishra proposed openstack/python-openstackclient master: WIP Skip ndp_proxy tests with older openstacksdk https://review.opendev.org/c/openstack/python-openstackclient/+/858533 | |
| 14:02:19 | jm1 | gtema, frickler: ^ | |
| 14:03:57 | jm1 | gtema, frickler: intention is to get openstackclient compatible with both <0.99.0 and >= 0.99.0 | |
| 14:06:45 | frickler | jm1: ack, will have a look later, but would also be good to have an actual job testing the former scenario | |
| 14:08:24 | jm1 | frickler: please put your suggestion as comments to the patch. i am not really deep inside osc 😅 | |
| 14:38:49 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update address scope to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848635 | |
| 14:46:36 | opendevreview | Jakob Meng proposed openstack/openstacksdk master: Drop query parameter 'id' from identity mapping https://review.opendev.org/c/openstack/openstacksdk/+/858522 | |
| 15:15:00 | opendevreview | JaeSeong-Shin proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | |
| 15:23:01 | opendevreview | Polina Gubina proposed openstack/ansible-collections-openstack master: Group assignment update https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858543 | |
| 16:13:52 | opendevreview | Stephen Finucane proposed openstack/cliff master: columns: Useful __str__, __repr__ implementation https://review.opendev.org/c/openstack/cliff/+/858550 | |
| 16:14:39 | opendevreview | Stephen Finucane proposed openstack/cliff master: columns: Useful __str__, __repr__ implementation https://review.opendev.org/c/openstack/cliff/+/858550 | |
| 16:22:33 | opendevreview | Merged openstack/python-openstackclient master: Adding missing command mapping in docs https://review.opendev.org/c/openstack/python-openstackclient/+/857728 | |
| 17:23:02 | opendevreview | Polina Gubina proposed openstack/ansible-collections-openstack master: Group assignment update https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858543 | |
| 18:24:46 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Run swift in -tips job https://review.opendev.org/c/openstack/python-openstackclient/+/858568 | |
| 18:24:46 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test job with capped sdk https://review.opendev.org/c/openstack/python-openstackclient/+/858569 | |
| #openstack-sdks - 2022-09-21 | |||
| 00:23:42 | opendevreview | Merged openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 00:27:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Makes security_group_rule_info compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846148 | |
| 02:09:57 | diablo_rojo | More than happy to let gtema do it, stephenfin :D | |
| 02:39:09 | opendevreview | Merged openstack/ansible-collections-openstack master: Makes security_group_rule_info compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846148 | |
| 04:10:31 | opendevreview | JaeSeong-Shin proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858604 | |
| 05:17:01 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test job with capped sdk https://review.opendev.org/c/openstack/python-openstackclient/+/858569 | |
| 08:39:40 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated network module docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/858642 | |
| 08:53:04 | frickler | jm1: gtema: stephenfin: 9 failures with sdk from stable/yoga, all look like recently added new features to me, makes me wonder whether we should consider backporting those sdk additions instead of just conditionalizing the osc tests https://zuul.opendev.org/t/openstack/build/6e2faca3ff0b45729d74e72eb7191260 | |
| 08:54:25 | gtema | frickler, we always say both sdk and osc are backward compatible | |
| 08:55:14 | gtema | I am now also working on adding "functional" tests running on real clouds, and there we simply have no other option rather then improving tests detecting features | |
| 09:03:24 | frickler | backwards compatible against older clouds, yes, but should current osc also work with older sdk or not? | |
| 09:04:01 | gtema | it should work, maybe complaining the "feature" is not supported by the cloud | |
| 09:04:21 | gtema | and tests correspondingly should detect feature presence and skip when it is not | |
| 09:04:59 | gtema | ah, the question was about osc vs sdk? | |
| 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 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 | |