| mordred | mnasiadka, yoctozepto: we should also likely do something in sdk so that calls to create_endpoint get a system-scoped token for you for their calls so that you don't have to have a separate clouds.yaml entry | 00:02 |
|---|---|---|
| mordred | (the separate entry is a good way to ensure this all works for now - but that seems like a pretty terrible UI to ask you to do for this) | 00:02 |
| openstackgerrit | Merged openstack/python-openstackclient master: common: autogenerate docs https://review.opendev.org/691989 | 00:33 |
| openstackgerrit | Merged openstack/python-openstackclient master: openstack.cli: autogenerate docs https://review.opendev.org/692914 | 00:33 |
| openstackgerrit | Merged openstack/python-openstackclient master: compute: autogenerate docs https://review.opendev.org/692916 | 00:34 |
| *** slaweq__ has joined #openstack-sdks | 00:40 | |
| *** slaweq has joined #openstack-sdks | 00:42 | |
| *** slaweq__ has quit IRC | 00:45 | |
| *** ricolin has joined #openstack-sdks | 00:47 | |
| *** slaweq_ has joined #openstack-sdks | 00:49 | |
| *** slaweq has quit IRC | 00:50 | |
| *** ricolin has quit IRC | 00:56 | |
| *** slaweq_ has quit IRC | 01:06 | |
| *** slaweq_ has joined #openstack-sdks | 01:08 | |
| *** dayou has quit IRC | 01:18 | |
| *** slaweq__ has joined #openstack-sdks | 01:20 | |
| *** slaweq__ has quit IRC | 01:20 | |
| *** slaweq has joined #openstack-sdks | 01:20 | |
| *** slaweq_ has quit IRC | 01:20 | |
| *** slaweq has quit IRC | 01:20 | |
| *** slaweq has joined #openstack-sdks | 01:21 | |
| *** slaweq has quit IRC | 01:22 | |
| *** slaweq has joined #openstack-sdks | 01:22 | |
| openstackgerrit | Bence Romsics proposed openstack/openstacksdk master: Add router add/remove route operations https://review.opendev.org/674324 | 01:27 |
| openstackgerrit | Bence Romsics proposed openstack/openstacksdk master: Handle HTTP errors in add/remove router interface calls https://review.opendev.org/687304 | 01:27 |
| *** ricolin has joined #openstack-sdks | 01:30 | |
| *** openstackstatus has joined #openstack-sdks | 01:44 | |
| *** ChanServ sets mode: +v openstackstatus | 01:44 | |
| *** dayou has joined #openstack-sdks | 02:14 | |
| *** ricolin has quit IRC | 02:19 | |
| *** ricolin has joined #openstack-sdks | 02:52 | |
| *** ricolin has quit IRC | 03:08 | |
| openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325 | 03:30 |
| *** ricolin has joined #openstack-sdks | 03:37 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Add redirect testing https://review.opendev.org/692929 | 03:55 |
| openstackgerrit | Merged openstack/python-openstackclient master: Deflate .htaccess https://review.opendev.org/692930 | 03:55 |
| *** slaweq has quit IRC | 04:19 | |
| *** ricolin_ has joined #openstack-sdks | 04:22 | |
| *** ricolin has quit IRC | 04:25 | |
| *** ricolin_ has quit IRC | 04:35 | |
| *** goldyfruit_ has quit IRC | 04:47 | |
| *** slaweq has joined #openstack-sdks | 05:06 | |
| *** irclogbot_3 has quit IRC | 05:34 | |
| *** irclogbot_1 has joined #openstack-sdks | 05:38 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Refactor AggregateTests https://review.opendev.org/693073 | 05:59 |
| mnasiadka | mordred: agreed, now it doesn't really work at all - when I do not supply domain in clouds.yaml - keystone complains it needs one, if I do - the token is not system scoped. So I guess in both cases it doesn't really do a system scoped token... I'll raise a bug. | 06:24 |
| *** ricolin_ has joined #openstack-sdks | 07:02 | |
| *** jawad_axd has joined #openstack-sdks | 07:17 | |
| *** slaweq_ has joined #openstack-sdks | 07:34 | |
| *** slaweq has quit IRC | 07:36 | |
| *** yolanda has quit IRC | 07:55 | |
| *** gtema has joined #openstack-sdks | 07:59 | |
| *** tosky has joined #openstack-sdks | 08:10 | |
| *** yolanda has joined #openstack-sdks | 08:11 | |
| *** ricolin_ is now known as ricolin | 08:28 | |
| *** ralonsoh has joined #openstack-sdks | 08:46 | |
| *** jpena|off is now known as jpena | 08:48 | |
| *** slaweq_ has quit IRC | 08:55 | |
| *** ricolin has quit IRC | 08:57 | |
| *** jpich has joined #openstack-sdks | 09:12 | |
| *** gtema has quit IRC | 09:13 | |
| *** gtema has joined #openstack-sdks | 09:49 | |
| *** gtema has quit IRC | 09:54 | |
| *** cdent has joined #openstack-sdks | 09:58 | |
| *** slaweq_ has joined #openstack-sdks | 10:00 | |
| *** slaweq__ has joined #openstack-sdks | 10:04 | |
| *** slaweq_ has quit IRC | 10:05 | |
| *** slaweq__ has quit IRC | 10:10 | |
| *** gtema has joined #openstack-sdks | 10:17 | |
| frickler | python-novaclient 16.0 seems to be breaking heat and therefore sdk testing https://zuul.opendev.org/t/openstack/build/2c83d0168fda4384837b676d336694ed/log/controller/logs/screen-h-eng.txt.gz#1379-1410 | 11:37 |
| frickler | https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack | 11:38 |
| *** gtema_ has joined #openstack-sdks | 12:26 | |
| *** jpena is now known as jpena|lunch | 12:27 | |
| *** jpena|lunch is now known as jpena | 13:21 | |
| efried | frickler: anything I can do about that novaclient breakage? | 13:28 |
| efried | (I haven't looked at it yet, but I was following the steps that led up to that requirements patch being mergeable) | 13:29 |
| *** noama has joined #openstack-sdks | 13:35 | |
| *** slaweq__ has joined #openstack-sdks | 13:39 | |
| *** slaweq has joined #openstack-sdks | 13:43 | |
| *** slaweq__ has quit IRC | 13:44 | |
| *** gtema_ has quit IRC | 14:04 | |
| *** goldyfruit_ has joined #openstack-sdks | 14:10 | |
| *** slaweq has quit IRC | 14:31 | |
| *** goldyfruit___ has joined #openstack-sdks | 14:38 | |
| *** goldyfruit_ has quit IRC | 14:40 | |
| openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Fixes OIDC authentication with multiple IdPs https://review.opendev.org/692140 | 14:43 |
| *** dave-mccowan has joined #openstack-sdks | 14:49 | |
| *** slaweq has joined #openstack-sdks | 14:51 | |
| *** dave-mccowan has quit IRC | 14:54 | |
| *** slaweq has quit IRC | 14:55 | |
| *** mriedem has joined #openstack-sdks | 15:00 | |
| frickler | efried: there's a fix proposed in heat now https://review.opendev.org/693165 but I see you already reviewed it | 15:01 |
| efried | yup | 15:01 |
| efried | if a quicker fix were needed, I guess I would suggest capping the novaclient requirement there | 15:02 |
| efried | but that fix seems straightforward enough. | 15:02 |
| frickler | efried: yes, I think I'd be fine waiting a couple of days for heat to get this merged, but it would be good to add some test/job into reqs to prevent this from happening again | 15:04 |
| efried | getting novaclient to gate on all possible consumers though... | 15:05 |
| mriedem | the reqs job failed on horizon and i fixed osc since i noticed it was going to break but assumed the reqs job tested heat | 15:05 |
| efried | oh, you're talking about getting reqs to gate on heat | 15:05 |
| mriedem | yeah | 15:05 |
| efried | that's probably not a bad plan. | 15:05 |
| mriedem | https://review.opendev.org/#/c/690097/ | 15:05 |
| mriedem | it's definitely a slippery slope, because watcher, designate, senlin, blazar, etc | 15:06 |
| efried | would heat's CI have caught this though? I thought we first noticed when sdk was running heat running novaclient... | 15:06 |
| efried | yeah, that | 15:06 |
| frickler | efried: heat func tests are broken now, too | 15:06 |
| efried | okay. | 15:06 |
| mriedem | heat's ci is gating on upper-constraints using novaclient | 15:07 |
| mriedem | and the reqs for u-c just recently merged | 15:07 |
| efried | I mean, if it wasn't, that would point to a hole in heat's CI | 15:07 |
| frickler | see e.g. https://review.opendev.org/#/c/693134/ | 15:07 |
| efried | okay, so is someone going to propose a reqs zuul.yaml change to make it run some heat? | 15:07 |
| frickler | heat might install some -tips job for their deps | 15:07 |
| frickler | so they notice upcoming breakage earlier | 15:08 |
| efried | also not a terrible plan | 15:08 |
| frickler | the other option would be to drop heat tests from the sdk jobs and transitively from devstack | 15:08 |
| efried | wouldn't that just lead to field escapes? | 15:09 |
| frickler | or move them into a seperate non-voting jobs at least | 15:09 |
| frickler | what does "field escapes" mean in this context? | 15:09 |
| efried | "we don't notice the problem until a customer hits it" | 15:09 |
| efried | i.e. "bug escapes to the field" | 15:10 |
| efried | maybe I'm misunderstanding, but the point isn't for us to be able to continue to merge heat code despite a broken dep. The point is to avoid the broken dep, as early as possible. | 15:10 |
| frickler | for me the point would be to be able to merge sdk and devstack patches even while heat isn't keeping up with things. though I admit I'm slightly biased on this because it's also heat blocking my pet "drop identity admin endpoint" project | 15:12 |
| mriedem | ha | 15:12 |
| mriedem | the truth comes out | 15:13 |
| frickler | but o.k., I can propose to add some heat job, or maybe even better the same sdk job that is also running on devstack, to reqs, then see what comes out of that | 15:15 |
| *** gtema has quit IRC | 15:17 | |
| frickler | if you want to watch or comment: https://review.opendev.org/693197 Add openstacksdk-functional-devstack job | 15:22 |
| *** gtema has joined #openstack-sdks | 15:32 | |
| *** slaweq has joined #openstack-sdks | 16:01 | |
| *** slaweq has quit IRC | 16:07 | |
| *** cdent has quit IRC | 17:01 | |
| *** gtema has quit IRC | 17:01 | |
| *** efried is now known as efried_rollin | 17:11 | |
| *** jawad_axd has quit IRC | 17:23 | |
| *** jawad_axd has joined #openstack-sdks | 17:29 | |
| openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Fixes OIDC authentication with multiple IdPs https://review.opendev.org/692140 | 17:29 |
| *** melwitt is now known as jgwentworth | 17:31 | |
| *** jawad_axd has quit IRC | 17:32 | |
| *** jpich has quit IRC | 17:34 | |
| *** dtantsur is now known as dtantsur|afk | 17:38 | |
| *** jpena is now known as jpena|off | 18:06 | |
| *** mriedem is now known as mriedem_lunch | 18:15 | |
| umbSublime | Hi again, would this be the right place to discuss about ospurge ? | 18:23 |
| *** jgwentworth is now known as melwitt | 18:36 | |
| *** tosky has quit IRC | 18:48 | |
| *** mriedem_lunch is now known as mriedem | 18:57 | |
| openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add documentation about login with federation https://review.opendev.org/693232 | 19:10 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: "qos_network_policy_id" attribute added to port resource https://review.opendev.org/693247 | 20:06 |
| *** gtema has joined #openstack-sdks | 20:15 | |
| *** ralonsoh has quit IRC | 20:19 | |
| *** gtema has quit IRC | 20:19 | |
| *** efried_rollin is now known as efried | 20:24 | |
| efried | umbSublime: Shanghai would probably be the right place this week | 20:26 |
| umbSublime | efried: I wish I was there | 20:27 |
| efried | that makes one of us | 20:27 |
| umbSublime | Last summit I was too was the one in Boston | 20:27 |
| efried | that might have been my first one | 20:28 |
| efried | no, that can't be right | 20:28 |
| * umbSublime 's first and only summit | 20:29 | |
| efried | Austin or Boston | 20:29 |
| *** goldyfruit___ has quit IRC | 21:16 | |
| *** efried is now known as efried_afk | 21:19 | |
| *** efried_afk is now known as efried | 21:41 | |
| openstackgerrit | Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864 | 21:50 |
| johnsom | Would really be good to merge ^^^^ as changes are being made to the old _tag.py that this removes. | 21:51 |
| johnsom | efried So, I see your change to _tag merged that added the "enhance_help parameter to some of the tags methods. I am guessing those need to be added to the new osc_lib version? | 21:55 |
| johnsom | https://www.irccloud.com/pastebin/Gj2aSRX5/ | 21:56 |
| efried | johnsom: ugh, that's an ugly enough hack kept buried in python-openstackclient, I would hate for it to have to bleed over to a different repo | 21:56 |
| johnsom | efried I had removed _tag.py from pythonclient, it has just been sitting for review for a while. | 21:56 |
| efried | I didn't know about that moving to osc-lib | 21:56 |
| johnsom | We moved all of that code to osc_lib/utils/tags.py | 21:57 |
| johnsom | So that other projects could use it | 21:57 |
| efried | Out of curiosity, what other projects would need to use it? | 21:57 |
| efried | Since it's you talking, I'm guessing octavia :P | 21:57 |
| johnsom | Octavia wants it for example | 21:57 |
| johnsom | It would be nice if we all implemented --tags the same... grin | 21:58 |
| johnsom | FYI: https://review.opendev.org/#/c/662859/ | 21:59 |
| johnsom | I'm not sure I quite understand why the lambda is needed, but I just found that the removal patch was in conflict | 22:00 |
| johnsom | This is the patch I just rebased that will likely fail as osc_lib doesn't have the "enhance_help" param: https://review.opendev.org/#/c/662864/ | 22:05 |
| efried | johnsom: I can at least explain how this came about. | 22:08 |
| efried | The networking commands are kind of a mess. The commands and options for neutron and nova-network are interleaved. | 22:09 |
| johnsom | Ok. I am guessing it is part of getting the docs working | 22:09 |
| johnsom | Joy | 22:10 |
| efried | When you're running on the CLI, the client tries to detect which network you're running by looking for the neutron service. | 22:10 |
| efried | and if it doesn't find it, assumes you're running nova-net | 22:10 |
| efried | then it loads up only the commands/options for whichever service it decided you're running. | 22:10 |
| *** tosky has joined #openstack-sdks | 22:10 | |
| efried | this of course relies on you being authorized to ask the right question to look for the service. But I guess chances are if you can't look at the catalog, you can't do anything else on the CLI, so it may not matter | 22:11 |
| efried | anyway, that didn't affect the docs when they were hardcoded... | 22:11 |
| johnsom | Well, if I remember right, you could run both at the same time | 22:11 |
| efried | oh, well, osc wouldn't let you | 22:11 |
| efried | because if it detected neutron, it would only load up the neutron options and operations. | 22:12 |
| efried | I'm asserting this based on code inspection, not experience btw | 22:12 |
| efried | so my goal was to get the docs generated | 22:13 |
| efried | BUT | 22:13 |
| efried | if you're looking at the HTML docs, on a web server somewhere, we have NO way of knowing whether you care about neutron or nova-net. | 22:13 |
| efried | so we have to show docs for both | 22:13 |
| johnsom | So nova networking is EOL I think, maybe we just have the docs render the neutron version? | 22:13 |
| efried | except osc has to continue to work against old clouds. Forever. | 22:14 |
| efried | ...so I had to come up with a way to make cliff generate both sets of docs | 22:14 |
| efried | and this was before I got to the keystone and glance docs, where I actually ended up completely separating them into different pages. | 22:14 |
| johnsom | It would still *work*... wink, wink | 22:14 |
| efried | ...for e.g. identity v2 vs v3, glance v1 vs v2 | 22:15 |
| efried | but that was a much easier sell in that case because the different versions were handled by completely different code, separated cleanly by package. | 22:15 |
| efried | in order to do that for the networking pieces, one would first have to split them into at least separate classes. | 22:16 |
| efried | In any case, what I ended up doing was implementing this three-way code path: We look for neutron, and if we don't find it because it's not there we assume nova-net, but if we don't find it because we can't even ask the question, we assume we're building docs. | 22:17 |
| efried | If neutron, we only run the methods that add neutron opts. If nova-net, likewise. | 22:19 |
| efried | But for the docs fork, we run both; and everywhere we register an option, we decorate its help with "neutron only" or "nova net only" kind of thing. | 22:19 |
| efried | To come full circle, this means that any code path that registers an option used by networking commands needs a way to understand *when* and *how* to add those decorations. | 22:20 |
| efried | the easiest way was with a callable kwarg | 22:20 |
| efried | so that in fact only the caller has to know when/how. | 22:20 |
| efried | And making the default a no-op lambda was easier than having | 22:21 |
| efried | if enhance_help is None: | 22:21 |
| efried | help=_(normal) | 22:21 |
| efried | else: | 22:21 |
| efried | help=enhance_help(_(normal)) | 22:21 |
| efried | in each of the methods (tags wasn't the only one) | 22:21 |
| johnsom | So, the odd thing is looking at the nova networks API, it doesn't even have tags | 22:21 |
| johnsom | https://docs.openstack.org/api-ref/compute/?expanded=create-allocate-floating-ip-address-detail#create-allocate-floating-ip-address | 22:21 |
| johnsom | Right, ok, so that is this use case. | 22:23 |
| efried | please also keep in mind that I was trying to make $new_docs look as much like $old_docs as possible with this series. I wasn't getting in the business of figuring out where $old_docs were incorrect or whatever. | 22:24 |
| johnsom | Right | 22:24 |
| efried | ...except insofar as $old_docs had gotten left behind when new options were added, kind of thing. | 22:25 |
| johnsom | There is a bunch of bit-rot in both the nova and neutron docs from what I have seen.. Fixing the API-ref for load balancing was a chore | 22:25 |
| efried | which is one of the purposes of this transition: so you don't have to worry about maintaining two places. | 22:25 |
| efried | Is dtroyer the/a maintainer of osc-lib? | 22:26 |
| efried | Probably need to ask him whether he's willing to let this enhance_help kwarg cross the project boundary | 22:27 |
| johnsom | Yeah I think so. He is the one I talked to when I did the tags move to osc_lib | 22:27 |
| efried | Since the default is a no-op, it's probably not the worst thing in the world, shouldn't hurt callers who don't care about it. | 22:27 |
| efried | I would want to see it documented. | 22:28 |
| efried | ...but it looks like you put nice docstrings in the osc-lib incarnations, so yeah, just following that. | 22:28 |
| johnsom | Yeah, I tried to make it better. | 22:29 |
| efried | ++ | 22:29 |
| efried | I guess the way to go about this is to propose that modification to the osc-lib side and use that vehicle to get dtroyer's yea/nay. | 22:29 |
| efried | You want to do that since you did the original, or you want me to do it since I made this mess? (inadvertently, and sorry) | 22:30 |
| johnsom | Well, I didn't do a good job getting his attention to get the python-openstackclient patch merged, so.... | 22:30 |
| dtroyer | tbh I really don't like that approach but I don't have the time to do anything differrent. the entire generated docs is a huge compromise anyway so wtf | 22:30 |
| johnsom | dtroyer Hi Dean | 22:30 |
| dtroyer | johnsom: my apologies for that | 22:30 |
| johnsom | I am wondering for tags if we can make a parameter that is more generic and optional and move forward that way. That way for these edge cases we can pass something over to have an annotation. | 22:32 |
| johnsom | Ok, well, at least I understand the issue now. | 22:32 |
| johnsom | efried Let me noodle on it a bit. | 22:32 |
| efried | dtroyer: Yeah, the right thing is probably to split the neutron/nova-net commands/opts into distinct classes (preferably in distinct packages), but I'm not sure how the CLI would decide which entry points to load up. | 22:34 |
| efried | uhh, maybe the framework is already there, because of how we use --os-*-api-versionZ | 22:34 |
| dtroyer | efried: the problem there is there is nobody to do that work. That also needs to be done for volume v2/v3 and has sat there for 2-3 years now | 22:36 |
| dtroyer | and I am now stuck trying to sort out how to deal with the SDK release breaking the snot out of nodepool and cleaning up the mess that made in my 3ci stacks | 22:37 |
| johnsom | I think we can make the tags code in osc_lib take a version qualifier and produce the same result, just by handling the tags params special on the OSC side. I think a generic parameter like "version_qualifier" would be ok for the shared tags code. | 22:39 |
| efried | johnsom: but then you have to duplicate the text, translation, formatting, and conditioning in osc-lib. | 22:41 |
| efried | I suppose you could just *move* it in there, shrug. | 22:41 |
| johnsom | One question I have is we don't call out api versions on any of the other commands right? We just assume the "not implemented" response is clear enough. | 22:43 |
| efried | what do you mean? | 22:44 |
| johnsom | Well, here: https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/network.html#network-create | 22:44 |
| johnsom | --dns-domain only exists on certain versions of neutron API. We don't call that out here. | 22:45 |
| johnsom | Same with --qos-policy which is only available in certain versions of neutron API and with the neutron extension loaded. | 22:45 |
| efried | Yeah, that would be nice. Not a reason to move backwards on neutron vs nova-net though | 22:46 |
| efried | and btw, the "not implemented" response is not necessarily clear. | 22:47 |
| efried | as an osc noob I had a hell of a time figuring out that my under-the-covers microversion wasn't adequate, and how to remedy it. | 22:47 |
| johnsom | I'm just noticing that this is the first time we have added any version information to the python-openstackclient docs. | 22:48 |
| efried | Can't imagine someone trying to figure out why they can't use the --subnet flag | 22:48 |
| efried | oh, no it's not at all | 22:48 |
| efried | The previous version of these same docs had these same decorations. | 22:48 |
| efried | stand by | 22:48 |
| johnsom | Weil, in server create, the micro-versions aren't called out for those commands | 22:49 |
| efried | https://docs.openstack.org/python-openstackclient/train/cli/command-objects/network.html#network-create | 22:49 |
| efried | no, microversions aren't. That would arguably be a nice enhancement. Ideally it would somehow tie back into the client code to discover that automatically, rather than being hardcoded. | 22:49 |
| efried | I'm just saying for neutron vs nova-net, it *was* called out, which is why I didn't see it being a viable path forward to stop doing that when the docs were generated. | 22:50 |
| efried | also fwiw https://docs.openstack.org/python-openstackclient/train/cli/command-objects/role.html identity docs had v2 vs v3 markers. | 22:51 |
| johnsom | were those static pages before? | 22:51 |
| efried | that's major version of course, not microversion. | 22:51 |
| efried | yes | 22:51 |
| johnsom | ah | 22:51 |
| efried | oh, and the way I found out this was even a thing was by trying to build the docs by running both paths. That gives errors on conflicting opts. If you look at the rest of the offending patch, you'll see where I had to put in some pretty weird "if docs build" conditions in some places. | 22:52 |
| johnsom | Yeah, I saw those | 22:53 |
| efried | anyway, as dtroyer implied, there's a right way and there's an expedient way. Unless you're signing up to do the former (I'm not) we're pretty much stuck doing the latter, even though it's kind of distasteful. | 22:53 |
| johnsom | I think we can leave that nastiness in OSC. I'm just working on a way to allow passing that stuff into osc_lib in a more clean way, focused on the tags library for now. | 22:54 |
| johnsom | Well, no good dead goes unpunished. I volunteered to do the tags library work... lol | 22:54 |
| efried | for what it's worth, I played around with other "more clean" options when I was authoring the patch. | 22:55 |
| efried | Believe it or not, this turned out to be the cleanest, from a DRY standpoint. | 22:55 |
| johnsom | So you are advocating for leaving duplicate code in OSC and osc_lib? | 22:56 |
| efried | oh, sorry, no; I'm advocating adding the enhance_help kwarg in osc-lib, properly documented. | 22:56 |
| efried | Yeah, I guess there's two "expedient" solutions, but I was assuming "leave it like it is" (and abandon the removal patch) wasn't on the table. | 22:57 |
| johnsom | Ugh, I really hate having to modify osc_lib again and start this cycle over and I'm not a fan of pulling forward this setup. | 23:14 |
| openstackgerrit | Eric Fried proposed openstack/osc-lib master: WIP: Add enhance_help kwarg to tags option generators https://review.opendev.org/693267 | 23:14 |
| efried | johnsom: well, fwiw there's the code side ^ | 23:15 |
| efried | I'll let you sleep on it :) | 23:16 |
| efried | I'm out o/ | 23:16 |
| openstackgerrit | Michael Johnson proposed openstack/python-openstackclient master: Switch to using osc_lib.utils.tags https://review.opendev.org/662864 | 23:24 |
| johnsom | Not sure those jobs are going to pick up that tags library hack patch, so mine is probably going to fail out even with the depends-on. | 23:25 |
| *** mriedem has quit IRC | 23:29 | |
| *** noama has quit IRC | 23:35 | |
| *** tosky has quit IRC | 23:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!