| *** dasp has joined #openstack-sdks | 00:16 | |
| *** dasp_ has quit IRC | 00:18 | |
| *** factor has joined #openstack-sdks | 01:00 | |
| *** tkajinam has quit IRC | 01:24 | |
| *** tkajinam has joined #openstack-sdks | 01:25 | |
| *** slaweq has joined #openstack-sdks | 04:17 | |
| *** slaweq has quit IRC | 04:22 | |
| *** slaweq has joined #openstack-sdks | 05:20 | |
| *** dave-mccowan has quit IRC | 05:24 | |
| *** slaweq has quit IRC | 05:24 | |
| *** evrardjp has quit IRC | 05:35 | |
| *** evrardjp has joined #openstack-sdks | 05:35 | |
| *** slaweq has joined #openstack-sdks | 05:35 | |
| *** slaweq has quit IRC | 05:40 | |
| *** yolanda has joined #openstack-sdks | 06:47 | |
| *** iurygregory has joined #openstack-sdks | 07:28 | |
| *** slaweq has joined #openstack-sdks | 07:53 | |
| *** tosky has joined #openstack-sdks | 08:23 | |
| *** jpena|off is now known as jpena | 08:23 | |
| *** tkajinam has quit IRC | 08:24 | |
| *** mgoddard has quit IRC | 08:27 | |
| *** mgoddard has joined #openstack-sdks | 08:33 | |
| *** ralonsoh has joined #openstack-sdks | 08:34 | |
| *** jpich has joined #openstack-sdks | 08:59 | |
| *** dmellado has quit IRC | 09:23 | |
| *** dmellado has joined #openstack-sdks | 09:26 | |
| *** dmellado has quit IRC | 09:33 | |
| *** dmellado has joined #openstack-sdks | 09:35 | |
| *** tkajinam has joined #openstack-sdks | 09:39 | |
| *** sshnaidm is now known as sshnaidm|pto | 09:59 | |
| *** dtantsur|afk is now known as dtantsur | 10:08 | |
| *** tkajinam has quit IRC | 10:18 | |
| *** sshnaidm|pto has quit IRC | 10:26 | |
| *** sshnaidm has joined #openstack-sdks | 10:42 | |
| *** sshnaidm_ has joined #openstack-sdks | 11:12 | |
| *** sshnaidm has quit IRC | 11:12 | |
| *** sshnaidm_ is now known as sshnaidm|pto | 11:15 | |
| *** sshnaidm|pto has quit IRC | 11:23 | |
| *** sshnaidm has joined #openstack-sdks | 11:23 | |
| *** sshnaidm is now known as sshnaidm|pto | 11:24 | |
| *** stingrayza has quit IRC | 11:28 | |
| *** stingrayza has joined #openstack-sdks | 11:34 | |
| *** tkajinam has joined #openstack-sdks | 11:47 | |
| *** Luzi has joined #openstack-sdks | 11:53 | |
| *** yolanda has quit IRC | 12:10 | |
| *** yolanda has joined #openstack-sdks | 12:11 | |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** tkajinam has quit IRC | 12:45 | |
| *** dave-mccowan has joined #openstack-sdks | 12:47 | |
| *** larsks has joined #openstack-sdks | 12:53 | |
| larsks | How does one pass all_tenants=true to the list_volumes method in python-openstacksdk? | 12:53 |
|---|---|---|
| *** jpich has quit IRC | 13:06 | |
| *** jpich has joined #openstack-sdks | 13:06 | |
| *** enriquetaso has joined #openstack-sdks | 13:16 | |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Remove old artifacts when building new ones https://review.opendev.org/710293 | 13:27 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Run flake8 in linters https://review.opendev.org/710294 | 13:31 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix F841 and remove exclusion https://review.opendev.org/698063 | 13:31 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Remove F403 and F405 exclusions https://review.opendev.org/710295 | 13:31 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix E128 and remove exclusion https://review.opendev.org/698064 | 13:31 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix W504 and remove exclusion https://review.opendev.org/710296 | 13:31 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix F401 and remove exclusion https://review.opendev.org/698065 | 13:31 |
| *** jpena|lunch is now known as jpena | 13:32 | |
| *** tosky_ has joined #openstack-sdks | 13:47 | |
| *** tosky has quit IRC | 13:47 | |
| *** tosky_ is now known as tosky | 13:47 | |
| *** jpich has quit IRC | 14:09 | |
| *** jpich has joined #openstack-sdks | 14:10 | |
| *** jawad_axd has joined #openstack-sdks | 14:33 | |
| *** jawad_axd has quit IRC | 14:37 | |
| *** Luzi has quit IRC | 15:01 | |
| *** __ministry has quit IRC | 15:15 | |
| *** __ministry has joined #openstack-sdks | 15:16 | |
| dtantsur | mordred: http://lists.openstack.org/pipermail/openstack-discuss/2020-March/012928.html Oo | 15:38 |
| mordred | dtantsur: uhm | 15:38 |
| brtknr | mordred: since when did openstacksdk start to replace uuid keys with id? | 15:39 |
| brtknr | dtantsur: ^ | 15:39 |
| brtknr | for get requests | 15:39 |
| *** sshnaidm|pto has quit IRC | 15:41 | |
| mordred | brtknr: since always. the REST interactions should use uuid as the key name if it needs to, and uuid _should_ be available as an alias - but it's a primary design of SDK that every object returned has an "id" field that is the id of the object | 15:43 |
| mordred | dtantsur: I have also responded to the thread, possibly a bit tersely | 15:44 |
| dtantsur | cool, that's what I expected :) | 15:45 |
| *** sshnaidm|pto has joined #openstack-sdks | 15:55 | |
| *** dmellado has quit IRC | 15:57 | |
| mordred | dtantsur: if you get bored, I left a disagreement on https://review.opendev.org/#/c/700219/ - I think we should probably come to an agreement amongst ourselves | 15:59 |
| *** dmellado has joined #openstack-sdks | 16:00 | |
| dtantsur | mordred: I like the idea of a global executor and more or less okay with defaulting to threads as long as it can be completely overridden | 16:09 |
| tosky | just to bring the discussion in the right place, one of the blocker raised by the cinder people was and is https://review.opendev.org/#/c/590807/ | 16:10 |
| tosky | and the lack of updates on it | 16:10 |
| tosky | is there any update on it? | 16:10 |
| mordred | dtantsur: cool! | 16:11 |
| mordred | tosky: looking | 16:12 |
| mordred | tosky: 3 things. a) I'd still prefer migrating to sdk there, since microversion support is solid and we use it in other places and all. HOWEVER b) I don't think we should block forward progress on that, we can certainly migrate later c) that patch should probably be fixed to pass tests - all of the previous blockers that existed at the time of that patch originally should not exist anymore | 16:15 |
| *** dmellado has quit IRC | 16:18 | |
| tosky | mordred: that's good to hear (all of it) - could you please comment on the patch? :) | 16:18 |
| brtknr | mordred: so uuid not being available too is possibly a bug? | 16:19 |
| *** dmellado has joined #openstack-sdks | 16:20 | |
| mordred | brtknr: yeah, possibly so. where are you seeing the issue? | 16:20 |
| brtknr | for os_coe_cluster and os_coe_cluster_template modules, only id is returned, not uuid | 16:20 |
| brtknr | looks like the corresponding ansible module expects to see uuid field which leads to issues with idempotency | 16:21 |
| mordred | brtknr: that's super weird :) | 16:21 |
| brtknr | sorry the get_coe_cluster and get_coe_cluster_template functions | 16:21 |
| mordred | brtknr: oh - that's even weirder. that's teh cloud layer, becuase it's for ansible. I'm not sure that's _ever_ returned uuid - so I have no idea how those ansible modules worked? looking further though .,.. | 16:22 |
| brtknr | mordred: wait, only get_coe_cluster has this problem | 16:23 |
| brtknr | get_coe_cluster_template seems to return uuid as an alias | 16:23 |
| mordred | brtknr: yes - I agree with you | 16:24 |
| mordred | cluster template adds it ba k | 16:24 |
| brtknr | mordred: can you show me the code? | 16:24 |
| brtknr | mordred: dont worry, i will try to find it myself, i should stop being lazy | 16:25 |
| mordred | brtknr: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_normalize.py#L1026-L1032 is where cluster_template adds it back if strict == False | 16:25 |
| mordred | and you can see that https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_normalize.py#L971-L997 doens't do a similar thing | 16:25 |
| brtknr | ouch | 16:26 |
| brtknr | who'd like to propose a patch? | 16:26 |
| brtknr | I'm happy to if people are busy :) | 16:26 |
| brtknr | Actually I am not sure what this strict_mode is about | 16:28 |
| brtknr | found the reno: https://github.com/openstack/openstacksdk/blob/master/releasenotes/notes/strict-mode-d493abc0c3e87945.yaml | 16:31 |
| mordred | brtknr: the idea was that we'd have a consistent interface peopel could count on - as well as fallback values | 16:35 |
| mordred | however - that concept pre-dates the merge of shade and sdk. with the sdk objects we have the opportunity to define aliases - and that seems to be a better long-term experience for users anyway | 16:36 |
| mordred | we've been working on shifting the shade layer to using the sdk resource objects and getting rid of that normalize layer - but havne't made it to the coe methods yet | 16:37 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Include "fields" to "SecurityGroup" query parameters https://review.opendev.org/710820 | 16:37 |
| mordred | brtknr: if you have a sec, you could probably just cargo-cult in an if not strict_mode: add back uuid patch to keep it simple for the time being - and we can see about migrating the methods as a followup | 16:37 |
| larsks | mordred: Do you know if there is a particular reason that volume.volumes(details=True) doesn't return the user_id information from the remote response? | 16:39 |
| openstackgerrit | Bharat Kunwar proposed openstack/openstacksdk master: Return uuid alias for coe_cluster in non strict mode https://review.opendev.org/710823 | 16:41 |
| brtknr | mordred: ^ | 16:41 |
| mordred | larsks: looking | 16:41 |
| mordred | brtknr: looks great. it's possible a test mock will need to get updated as a result - but let's see how it goes | 16:42 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query https://review.opendev.org/710825 | 16:42 |
| mordred | larsks: there is no mention of user_id in openstack/block_storage/v3/volume.py - so I think it's purely that it's missing and not anything on purpose | 16:43 |
| larsks | mordred: thanks. I was just curious if the omission was intentional or not. I'll submit a change. | 16:43 |
| brtknr | mordred: locally, all my tox -e py36 failures are related to cloud devstack not found as I export OS_CLOUD=devstack to my env vars | 16:45 |
| mordred | brtknr: hah. nice | 16:46 |
| mordred | brtknr: well - let's see what zuul tells us :) | 16:46 |
| brtknr | mordred: also TIL what cargo cult means | 16:46 |
| mordred | brtknr: :) | 16:46 |
| * mordred is going to grab the patch and run it just to check | 16:46 | |
| *** dmellado has quit IRC | 16:46 | |
| brtknr | If this patch goes through, it will void this patch: https://review.opendev.org/#/c/709734/ | 16:47 |
| brtknr | py36: commands succeeded | 16:48 |
| brtknr | congratulations :) | 16:48 |
| brtknr | after unsetting OS_CLOUD | 16:48 |
| *** dmellado has joined #openstack-sdks | 16:49 | |
| mordred | brtknr: I think we should land that patch too - it describes the current state - and somone might have those modules and _not_ latest sdk - and the id codepath you did is always going to work | 16:49 |
| mordred | when the uuid codepath lands in sdk it'll improve things for people and then we can add a doc that says uuid will be an alias in the return value if sdk is new enough | 16:50 |
| brtknr | hmm this seems to highlight an underlying problem, why is tox -e py36 fail when I export OS_CLOUD= | 16:50 |
| brtknr | mordred: sounds good to me | 16:50 |
| brtknr | I will revert the change to the doc | 16:50 |
| brtknr | btw how the hell do you use ansible-collection-openstack? | 16:50 |
| brtknr | Its not available on galaxy yet so cant use ansible-galaxy to install it | 16:51 |
| smcginnis | brtknr: I haven't been following the conversation here, but sound like the OS_CLOUD issue might be a side effect of this: https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/tox.ini#L10 | 16:51 |
| mordred | brtknr: no - don't revert it - I think it's solid | 16:51 |
| mordred | brtknr: it's the right thing | 16:52 |
| mordred | brtknr: you need to build a tarball from source then you can install the tarball | 16:52 |
| mordred | we're working on getting galaxy pub working | 16:52 |
| mordred | brtknr: I recommend using tox -ebuild to make the tarball | 16:52 |
| mordred | otherwise it's going to have a sad | 16:52 |
| brtknr | revert line 190 here: https://review.opendev.org/#/c/709734/4/plugins/modules/os_coe_cluster.py | 16:52 |
| mordred | no - don't revert that | 16:53 |
| mordred | your comment there is correct | 16:53 |
| brtknr | Ah okay fair enough | 16:53 |
| mordred | that _is_ the situation most users will encounter - they won't see uuid unless they install an as-yet-unreleased sdk :) | 16:53 |
| brtknr | mordred: is it correct that when a cluster gets *created* then uuid is returned? | 16:54 |
| brtknr | shouldn't the uuid get normalised in the same way for create requests too? | 16:54 |
| mordred | it shouldn't - it should be the same data structure all the time | 16:54 |
| mordred | yup | 16:54 |
| brtknr | ok so thats another bummer, it returns uuid at present | 16:55 |
| brtknr | when cluster get created | 16:55 |
| mordred | wow really? | 16:55 |
| * mordred cries | 16:55 | |
| brtknr | yes sorry | 16:56 |
| mordred | so that means we're not running normalize on the result of create - which is sadmaking. maybe we need to do a workaround in the ansible module itself | 16:56 |
| mordred | to grab the id in both cases with a get('id', get('uuid') - then make sure both id and uuid are set in the dict before we return to teh ansible user | 16:56 |
| mordred | cause otherwise this is just a mess for the user | 16:57 |
| brtknr | that is the premise of the patch | 16:58 |
| brtknr | its what i am already proposing we do with that patch | 16:58 |
| mordred | cool. then yes - I think we should land that patch asap | 17:00 |
| mordred | Shrews: ^^ feel like an easy +A? I know you like patches with no context | 17:00 |
| Shrews | so much | 17:00 |
| brtknr | :D | 17:00 |
| mordred | brtknr: oh - actually - one more thing... | 17:00 |
| *** dmellado has quit IRC | 17:01 | |
| * brtknr stops pauses stuffing his bag with his belongings | 17:01 | |
| * brtknr pauses stuffing his bag with his belongings | 17:01 | |
| mordred | brtknr: real quick ... or I can update the patch if you're packing up | 17:01 |
| mordred | brtknr: (left inline comment) | 17:01 |
| Shrews | mordred: let's pretend i haven't read the backscroll yet. is 709734 the thing you referred to? | 17:02 |
| Shrews | that looks doubtful. /me just reads bs | 17:03 |
| mordred | Shrews: yes. and the tl;dr is that normalization of cluster_template dicts is blown and we forgot to add the 'uuid' field back to the dict, but the ansible docs and module refer to uuid not id | 17:03 |
| mordred | there's an sdk fix to add uuid - but of course latest module doesn't mean latest sdk - so there's also a patch to the module to work around the fact that it might not get uuid back - but a user might be expecting it to exist because of the docs saying it's there | 17:04 |
| brtknr | mordred: done | 17:07 |
| openstackgerrit | Bharat Kunwar proposed openstack/ansible-collections-openstack master: os_coe_cluster: Retrive id/uuid correctly https://review.opendev.org/709734 | 17:07 |
| *** dmellado has joined #openstack-sdks | 17:07 | |
| mordred | brtknr: thanks! | 17:08 |
| brtknr | is cluster['id'] = cluster['uuid'] = cluster_id okay? | 17:08 |
| Shrews | shouldn't the changelog fragment at least mention the module being fixed? not sure that that is clear when just reading the resulting log | 17:08 |
| brtknr | mordred: or is that bad practice? | 17:08 |
| brtknr | Shrews: done | 17:09 |
| openstackgerrit | Bharat Kunwar proposed openstack/ansible-collections-openstack master: os_coe_cluster: Retrive id/uuid correctly https://review.opendev.org/709734 | 17:09 |
| mordred | brtknr: I think it's fine in this case - not sure if pep8 will agree or not :) | 17:09 |
| brtknr | I'm running pep8 locally | 17:10 |
| brtknr | band it succeeds | 17:10 |
| brtknr | Thanks folks | 17:10 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/openstacksdk master: Include user_id attribute in volume information https://review.opendev.org/710834 | 17:11 |
| mordred | woot. thanks brtknr | 17:11 |
| mordred | larsks: thanks | 17:11 |
| mordred | I think when that and the coe patch above land we should cut a bugfix release | 17:12 |
| brtknr | would be nice to fix the create case too on openstacksdk side for consistency? | 17:12 |
| Shrews | mordred: you'll need to reapply your +2 to 709734 | 17:13 |
| mordred | brtknr: yeah, it would | 17:13 |
| mordred | Shrews: done. thanks | 17:13 |
| mordred | brtknr: if you want to do it - it would be adding a normalize call in openstack/cloud/_coe.py in create_cluster_template (and create_cluster) - | 17:14 |
| mordred | brtknr: so like return self._normalize_cluster_template(cluster_template) | 17:15 |
| *** jpich has quit IRC | 17:18 | |
| *** dmellado has quit IRC | 17:19 | |
| *** dmellado has joined #openstack-sdks | 17:25 | |
| *** stingrayza has quit IRC | 17:31 | |
| *** evrardjp has quit IRC | 17:35 | |
| *** evrardjp has joined #openstack-sdks | 17:35 | |
| *** jpena is now known as jpena|off | 17:39 | |
| openstackgerrit | Adam Harwell proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374 | 17:40 |
| openstackgerrit | Adam Harwell proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416 | 17:40 |
| *** efried has quit IRC | 17:40 | |
| *** efried has joined #openstack-sdks | 17:42 | |
| mordred | rm_work: \o/ | 17:48 |
| rm_work | ... i believe i still owe something with ospurge too <_< | 17:48 |
| rm_work | my volunteer backlog with SDK/OSC is high T_T | 17:48 |
| mordred | rm_work: well - https://review.opendev.org/#/c/700219/ | 17:49 |
| rm_work | yes, but i believe that was superseded by the decisions we made in the meetup in Shanghai :D | 17:50 |
| rm_work | (or is this something else?) | 17:51 |
| mordred | I think this is still aiming at that - but it's entirely possible it's going down the wrong road | 17:51 |
| rm_work | I believe what we said was "just drop-in the ospurge codebase into SDK" | 17:52 |
| mordred | oh right | 17:52 |
| rm_work | and then drop the mic and walk away :D | 17:52 |
| mordred | and I think we keep getting busy ... | 17:52 |
| rm_work | yes. | 17:52 |
| mordred | stupid busy | 17:53 |
| mordred | ok - let me wrap my head around that whole thing this wweek | 17:53 |
| rm_work | my current quest is fixing octavia's AZ handling and compatibility with nova/neutron routed-network stuff, which requires also ... fixing nova/neutron routed-network stuff :D | 17:53 |
| mordred | this is what we get for gtema not being able to come to shanghai - and has nothing to do with me forgetting things at all | 17:53 |
| rm_work | well, it's also not nearly a top priority | 17:53 |
| rm_work | which is the other reason it's been sidelined thus far for me | 17:53 |
| mordred | yeah | 17:54 |
| mordred | and oh goodie - it sounds like you have a wonderful rabbit hole to go down with routed networks :) | 17:54 |
| rm_work | ugh yes | 17:54 |
| rm_work | i was hoping i could take the easy route but i was denied :D | 17:54 |
| mordred | where would the fun in that be? | 17:55 |
| * dtroyer peeks in to see if vegetables will get thrown this direction… | 17:56 | |
| dtroyer | damn, you guys have been busy | 17:56 |
| mordred | it's a dtroyer! | 17:56 |
| dtroyer | where? | 17:56 |
| mordred | ssssh. it's a sekrit | 17:56 |
| dtroyer | oh good, don't let him out of the asylum | 17:56 |
| mordred | I hear he's bringing vegetables | 17:56 |
| dtroyer | he hates vegetables, which is why being vegetarian for 8 years was, um, interesting | 17:57 |
| mordred | mmm. eggs | 17:57 |
| mordred | could I have some eggs with those eggs? maybe with a side of eggs? | 17:58 |
| dtroyer | So I think it is time to acknowledge the clear truth that I'm not going to continue to be working on OSC, not OpenStack for that matter. I owe an apology and major thanks to everyone who has kept the lights on. | 17:59 |
| mordred | dtroyer: that is sad panda - but thanks for your many years of keeping the lights on yourself! | 17:59 |
| dtroyer | I don't have a new job yet, but nothing I am looking at includes room here, unfortunately | 17:59 |
| rm_work | hmm | 18:06 |
| rm_work | is that because you're explicitly not looking at stuff that's related, because you need to GTFO for your sanity? :D | 18:06 |
| rm_work | or just coincidental | 18:06 |
| dtroyer | more coincidental. dis-investment in OpenStack is a real thing and more than just Intel | 18:07 |
| rm_work | I feel like it's just ... shifting | 18:08 |
| dtroyer | but it may be time, too. I almost made it to Nova's 10th birthday, but that's a long time in one place | 18:08 |
| rm_work | but yes | 18:08 |
| mordred | dtroyer: tell me about it - I have certainly never worked 10 years on anything else | 18:12 |
| *** iurygregory has quit IRC | 18:19 | |
| rm_work | me either! | 18:21 |
| * rm_work is now 10 years out of college | 18:21 | |
| rm_work | I think I'm going on 7 years on openstack now tho :) | 18:22 |
| dtantsur | tosky: dumped some microversion microthoughts to the patch :) | 18:24 |
| mordred | Shrews: https://review.opendev.org/#/c/710823 is the sdk-side of that uuid thing from earlier | 18:25 |
| dtantsur | mordred: will it be too much to ask for a unit test on ^^^? | 18:26 |
| dtantsur | I guess it's a question for brtknr really | 18:26 |
| rm_work | oh hey, good place for this since i'm already here: https://review.opendev.org/#/c/710452/ | 18:26 |
| rm_work | need that for senlin ASAP, and will need that to do a conversion for OSC to SDK | 18:27 |
| dtantsur | rm_work: does this resource even handle microversions? otherwise you'll always see None there | 18:27 |
| rm_work | which, the one i just linked? | 18:28 |
| rm_work | or the glance thing | 18:28 |
| dtantsur | rm_work: the one you just linked | 18:29 |
| rm_work | no microversioned involved | 18:30 |
| rm_work | there's just a note in the commit message about which octavia version it appears in | 18:30 |
| dtantsur | mmm, so 2.14 is not a microversion | 18:30 |
| dtantsur | ? | 18:30 |
| rm_work | there's no code around verifying that (though the octavia API will let you see what version it's on, yes) | 18:30 |
| rm_work | but it's not strictly microversions | 18:30 |
| mordred | so - it'll be there if octavia supports it - and none otherwise yeah? | 18:30 |
| dtantsur | ah, similar to what ironic-inspector does, information-only microversions | 18:31 |
| rm_work | https://github.com/openstack/octavia/blob/master/octavia/api/root_controller.py#L93-L95 | 18:31 |
| rm_work | err, in a get? it'd be missing in a GET if the octavia version is older | 18:32 |
| mordred | kinda like what glance does | 18:32 |
| rm_work | non-present on the octavia side, illustrated by None on the SDK object side | 18:32 |
| mordred | yeah | 18:33 |
| Shrews | mordred: should there be a non-strict-mode test to validate that? We have an entire TestNormalize/TestStrictNormalize suite of tests for such things currently | 18:33 |
| rm_work | and I guess a 400 if passed in fromt he SDK on a create | 18:33 |
| johnsom | dtroyer Thanks for all of your excellent work on OSC. Good luck on your next adventure. | 18:33 |
| Shrews | but apparently for not all things :/ | 18:33 |
| mordred | Shrews: yeah, probably. I think you and dtantsur had the same thought | 18:33 |
| mordred | it's almost like you value tests or something | 18:33 |
| dtantsur | rm_work: +A then | 18:34 |
| rm_work | whaaaaaaat, tests are important? who knew :D | 18:34 |
| Shrews | mordred: we're such silly people folk | 18:34 |
| dtantsur | can confirm ^^ | 18:34 |
| rm_work | Oh my, yes.</farnsworthy> | 18:34 |
| *** KeithMnemonic has joined #openstack-sdks | 18:35 | |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: os_coe_cluster: Retrive id/uuid correctly https://review.opendev.org/709734 | 18:36 |
| KeithMnemonic | mordred: so is there anyway to get this merged or is it too late now? https://review.opendev.org/703683 | 18:36 |
| *** ralonsoh has quit IRC | 18:42 | |
| *** dtantsur is now known as dtantsur|afk | 19:04 | |
| mordred | KeithMnemonic: there you go | 19:26 |
| KeithMnemonic | thank you!! | 19:28 |
| KeithMnemonic | mordred: while you are in the giving mood, care to also help on this one for train? https://review.opendev.org/#/c/701299/ | 19:40 |
| mordred | KeithMnemonic: done | 19:41 |
| openstackgerrit | Merged openstack/openstacksdk master: Add availability_zone param to load balancer https://review.opendev.org/710452 | 20:01 |
| *** tobiash has quit IRC | 21:13 | |
| *** tobiash has joined #openstack-sdks | 21:15 | |
| openstackgerrit | Merged openstack/python-openstackclient stable/train: Fix openstack server list --deleted --marker option https://review.opendev.org/701299 | 21:19 |
| KeithMnemonic | looking for review on this backport now that the train branch merged. thank you https://review.opendev.org/#/c/702260/4 | 21:22 |
| *** slaweq has quit IRC | 22:09 | |
| *** slaweq has joined #openstack-sdks | 22:11 | |
| *** slaweq has quit IRC | 22:16 | |
| *** tkajinam has joined #openstack-sdks | 22:28 | |
| *** slaweq has joined #openstack-sdks | 22:33 | |
| *** slaweq has quit IRC | 22:39 | |
| *** slaweq has joined #openstack-sdks | 22:41 | |
| *** slaweq has quit IRC | 22:46 | |
| *** slaweq has joined #openstack-sdks | 23:11 | |
| *** slaweq has quit IRC | 23:16 | |
| *** enriquetaso has quit IRC | 23:25 | |
| *** tosky has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!