| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 02:15 |
|---|---|---|
| gtema | ok headphoneJames, will have a look today | 08:00 |
| opendevreview | Jakob Meng proposed openstack/osc-lib master: Respect 'interface' key from clouds.yaml https://review.opendev.org/c/openstack/osc-lib/+/784559 | 09:38 |
| jm1 | hello :) who would i have to ask for a workflow+1 on this osc-lib patch? https://review.opendev.org/c/openstack/osc-lib/+/784559 | 09:42 |
| opendevreview | Tom Weininger proposed openstack/openstacksdk master: Correct documentation about Load Balancer API https://review.opendev.org/c/openstack/openstacksdk/+/828689 | 10:35 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow retrieval of migration by UUID https://review.opendev.org/c/openstack/python-openstackclient/+/828785 | 10:59 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Move server migrations commands to their own file https://review.opendev.org/c/openstack/python-openstackclient/+/828855 | 10:59 |
| frickler | jm1: I've put it into my review list, but won't get to it today. from a quick glance, I'm wondering whether it would help to have a test for that scenario that would avoid regressing again | 11:08 |
| opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | 12:46 |
| jm1 | frickler: thanks :) | 13:17 |
| jm1 | frickler: regarding the tests.. not sure but i think the tests did not catch this flaw because it asserts 'public' which is exactly DEFAULT_INTERFACE https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/tests/test_shell.py#L356 | 13:18 |
| opendevreview | Jakob Meng proposed openstack/osc-lib master: Fixed test for interface key https://review.opendev.org/c/openstack/osc-lib/+/828876 | 13:31 |
| jm1 | frickler: submitted a patch which fixes the test. of course zuul now fails but this is to be expected as the other patch has not been merged yet. https://review.opendev.org/c/openstack/osc-lib/+/828876 | 13:41 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 13:52 |
| Adri2000 | about the in-use/attached volume resize issue I mentioned here yesterday, here is a bug report I just filed: https://storyboard.openstack.org/#!/story/2009850 | 13:57 |
| lajoskatona | Hi, a quick question, we have a feature for Neutron (actually huge parts in Nova/Placement also): QoS min pps rule | 15:03 |
| lajoskatona | The client patch is here: https://review.opendev.org/c/openstack/python-openstackclient/+/810559 which needs the SDK one which is merged to featrue/r1 (https://review.opendev.org/c/openstack/openstacksdk/+/810364 ) | 15:04 |
| lajoskatona | is there a chance that we will have a release of things hanging on feature/r1 in Yoga timeframe, or better to cherry-pick to master ? | 15:05 |
| frickler | lajoskatona: that is already merged to master, see the "included in" link in gerrit. but I don't know when this will be released, gtema? | 15:21 |
| gtema | well, master is now not in the state to be released | 15:21 |
| gtema | I fear it may take another month or so to polish what r1 broke (and what other projects depend on) | 15:22 |
| gtema | but there is still target to make r1 till mid spring | 15:22 |
| lajoskatona | frickler, gtema: ok thanks, so it will not be Yoga-3 :-(, Anyway we keep an eye on it, and thanks for the hard work with it | 15:27 |
| gtema | wlcm | 15:27 |
| ralonsoh | frickler, https://3ef969af3b83bcedd744-0f0c0c25612519a3a781fc7a48e209af.ssl.cf1.rackcdn.com/828877/2/check/openstack-tox-bashate/b597d4a/job-output.txt do you know what is this error? | 15:38 |
| frickler | ralonsoh: yes, split the "local var" and "var=..." into two lines. also that's for -qa and not -sdk ;) | 15:40 |
| ralonsoh | frickler, ahhhhh | 15:40 |
| ralonsoh | thanks a lot | 15:40 |
| *** artom__ is now known as artom | 16:56 | |
| opendevreview | Merged openstack/osc-lib master: parseractions: Make key validation reusable https://review.opendev.org/c/openstack/osc-lib/+/784110 | 18:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!