| *** diablo_rojo has joined #openstack-sdks | 00:15 | |
| *** enriquetaso has quit IRC | 00:49 | |
| *** tkajinam has quit IRC | 01:29 | |
| *** tkajinam has joined #openstack-sdks | 01:29 | |
| *** diablo_rojo has quit IRC | 04:25 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-sdks | 04:36 | |
| *** nightmare_unreal has joined #openstack-sdks | 06:14 | |
| *** dayou has quit IRC | 06:28 | |
| *** tkajinam has quit IRC | 06:35 | |
| *** tkajinam has joined #openstack-sdks | 06:36 | |
| *** ttsiouts has joined #openstack-sdks | 06:39 | |
| *** gtema has joined #openstack-sdks | 06:39 | |
| *** gtema_ has joined #openstack-sdks | 06:41 | |
| *** slaweq has joined #openstack-sdks | 07:11 | |
| frickler | donnyd: I didn't have any issue when I tested app creds some time ago, can you show your command in detail? do you use a clouds.yaml config or environment variables? in the latter case, make sure you don't have too many of them | 07:27 |
|---|---|---|
| *** vishakha has joined #openstack-sdks | 07:28 | |
| *** ralonsoh has joined #openstack-sdks | 07:33 | |
| *** tosky has joined #openstack-sdks | 07:35 | |
| *** dayou has joined #openstack-sdks | 07:40 | |
| *** dtantsur|afk is now known as dtantsur | 07:54 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** rpittau|afk is now known as rpittau | 07:58 | |
| *** jpich has joined #openstack-sdks | 08:01 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097 | 08:06 |
| *** ttsiouts has quit IRC | 08:32 | |
| *** ttsiouts_ has joined #openstack-sdks | 08:32 | |
| *** ttsiouts has joined #openstack-sdks | 08:36 | |
| *** ttsiouts_ has quit IRC | 08:38 | |
| *** ttsiouts has quit IRC | 08:39 | |
| *** ttsiouts has joined #openstack-sdks | 08:39 | |
| *** sshnaidm|afk is now known as sshnaidm | 08:52 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097 | 09:00 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952 | 09:06 |
| *** ttsiouts_ has joined #openstack-sdks | 09:07 | |
| *** ttsiouts has quit IRC | 09:09 | |
| *** mgariepy has quit IRC | 09:10 | |
| *** mgariepy has joined #openstack-sdks | 09:15 | |
| *** ttsiouts_ has quit IRC | 09:18 | |
| *** ttsiouts has joined #openstack-sdks | 09:19 | |
| gtema | what has happened to pep? Who made gods angry? | 10:13 |
| *** rpittau is now known as rpittau|bbl | 10:19 | |
| *** ttsiouts_ has joined #openstack-sdks | 10:19 | |
| *** ttsiouts has quit IRC | 10:19 | |
| *** ttsiouts_ has quit IRC | 10:34 | |
| *** ttsiouts has joined #openstack-sdks | 10:34 | |
| AJaeger | gtema: see #openstack-qa, new release of flake8 | 10:46 |
| *** ttsiouts has quit IRC | 10:51 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Extend statistics reporting https://review.opendev.org/727097 | 10:53 |
| *** jpich has quit IRC | 10:57 | |
| *** jpich has joined #openstack-sdks | 10:58 | |
| *** ttsiouts has joined #openstack-sdks | 11:00 | |
| *** ttsiouts_ has joined #openstack-sdks | 11:03 | |
| *** ttsiouts has quit IRC | 11:04 | |
| *** gtema_ has quit IRC | 11:22 | |
| *** samueldmq_ has joined #openstack-sdks | 11:28 | |
| *** jpena is now known as jpena|lunch | 11:33 | |
| *** samueldmq_ has quit IRC | 11:34 | |
| *** samueldmq_ has joined #openstack-sdks | 11:36 | |
| *** jpich has quit IRC | 11:36 | |
| *** jpich has joined #openstack-sdks | 11:36 | |
| *** samueldmq_ is now known as samueldmq | 11:40 | |
| *** gtema_ has joined #openstack-sdks | 11:59 | |
| *** gtema_ has quit IRC | 12:04 | |
| *** ttsiouts_ has quit IRC | 12:20 | |
| *** ttsiouts has joined #openstack-sdks | 12:21 | |
| *** rpittau|bbl is now known as rpittau | 12:21 | |
| *** ttsiouts_ has joined #openstack-sdks | 12:31 | |
| *** ttsiouts has quit IRC | 12:31 | |
| *** ttsiouts_ has quit IRC | 12:38 | |
| *** ttsiouts has joined #openstack-sdks | 12:38 | |
| *** jpena|lunch is now known as jpena | 12:40 | |
| *** ttsiouts has quit IRC | 12:42 | |
| *** gtema_ has joined #openstack-sdks | 12:50 | |
| *** gtema_ has quit IRC | 12:53 | |
| *** gtema_ has joined #openstack-sdks | 12:53 | |
| donnyd | frickler: turns out I had some env variables set. It was a layer 8 problem | 12:59 |
| *** gtema_ has quit IRC | 13:05 | |
| *** ttsiouts has joined #openstack-sdks | 13:06 | |
| rm_work | ah damn, sdk got hit by the flake8 thing | 13:18 |
| *** ttsiouts has quit IRC | 13:21 | |
| *** ttsiouts has joined #openstack-sdks | 13:21 | |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056 | 13:25 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538 | 13:25 |
| *** ttsiouts has quit IRC | 13:25 | |
| *** ttsiouts has joined #openstack-sdks | 13:26 | |
| *** enriquetaso has joined #openstack-sdks | 13:30 | |
| *** openstackstatus has quit IRC | 13:53 | |
| *** openstackstatus has joined #openstack-sdks | 13:54 | |
| *** ChanServ sets mode: +v openstackstatus | 13:54 | |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056 | 13:55 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538 | 13:55 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: WIP Use stub modules for deprecated aliases https://review.opendev.org/727197 | 13:55 |
| mordred | gtema, AJaeger : patch on the way to fix the new flake8 issues | 14:09 |
| AJaeger | mordred: there's a plan to release a new hacking version to cap flake8 - but fixing them is better ;) Thanks! | 14:10 |
| mordred | AJaeger: yeah - these are all good errors | 14:13 |
| mordred | like - this is literally the sort of thing flake8 should be checking | 14:13 |
| AJaeger | then let's fix them ;) | 14:14 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix issues found by latest flake8 https://review.opendev.org/727204 | 14:15 |
| mordred | gtema, mnaser, dtantsur: if you have a second - it's a quick patch but will unbreak the gate here: && | 14:15 |
| mordred | I meant ^^ :) | 14:15 |
| dtantsur | mordred: FYI we're working on releasing a new hacking that caps flake8 | 14:15 |
| mordred | dtantsur: totally - but the errors are all good | 14:16 |
| mordred | so we should fix them | 14:16 |
| dtantsur | mordred: well, the nit-picking around "l" variables is so bad that we disabled that check in ironic | 14:16 |
| *** ttsiouts has quit IRC | 14:16 | |
| mordred | I thnik it's great :) | 14:16 |
| dtantsur | *shrug* telling people how to name variables in list/dist comprehensions | 14:16 |
| mordred | yeah - sure | 14:17 |
| mordred | I mean - I'm not going to argue the case broadly - just saying here it was 2 comprehensions in the hole code base, and I'd be happy if we never used single-letter variables for anything in sdk | 14:17 |
| mordred | so I don't think it's an issue - but I can understand it being frustrating in other paces | 14:17 |
| mordred | I care more about the unused format stuff - it actually caught an actual bug | 14:18 |
| dtantsur | yep, the format check is awesome | 14:18 |
| dtantsur | there will be a hacking release uncapping it again, so don't worry | 14:18 |
| mordred | (we were passing an argument to the format that should have been an argument to the surrounding method) | 14:18 |
| dtantsur | and note that it's not about single-letter variables, it's about "l is easy to confuse with I" | 14:18 |
| dtantsur | aka "we don't know how to set up fonts" :) | 14:19 |
| mordred | :) | 14:19 |
| dtantsur | actually, I'm using the default Fedora fonts and l looks nothing alike I | 14:20 |
| dtantsur | but dunno | 14:20 |
| mordred | l and 1 are teh ones I've seen be more confusing on bad fonts | 14:20 |
| mordred | I use the hack font - so it is not a problem for me | 14:20 |
| dtantsur | okay, these ones are a bit alike, but not identical (and 1 cannot be a variable name) | 14:21 |
| *** ttsiouts has joined #openstack-sdks | 14:22 | |
| *** mgariepy has quit IRC | 14:31 | |
| *** ttsiouts_ has joined #openstack-sdks | 14:31 | |
| *** ttsiouts has quit IRC | 14:35 | |
| *** tkajinam has quit IRC | 14:37 | |
| *** ralonsoh has quit IRC | 14:40 | |
| *** ralonsoh has joined #openstack-sdks | 14:40 | |
| *** ttsiouts_ has quit IRC | 14:41 | |
| *** ttsiouts has joined #openstack-sdks | 14:41 | |
| *** mgariepy has joined #openstack-sdks | 14:44 | |
| *** ralonsoh has quit IRC | 14:44 | |
| *** ttsiouts has quit IRC | 14:46 | |
| gtema | oh, after moving temporarily to another machine I do not get notifications from IRC. | 14:48 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add template for generation of artibtrary module https://review.opendev.org/727216 | 14:53 |
| noonedeadpunk | hey folks. has a question - was there any recent issue related to nova client and instance filtering by project name? | 15:02 |
| mordred | gtema: "awesome" | 15:03 |
| noonedeadpunk | just have a horizon (and openstack) installation that while filtering instances by project name returns all available isntances. | 15:04 |
| noonedeadpunk | I was told that most likely that panel don't do any filtering itsef but relies on client | 15:05 |
| noonedeadpunk | so decided to ask if you probably heard about smth like this | 15:05 |
| *** dtantsur is now known as dtantsur|afk | 15:08 | |
| *** dasp_ has quit IRC | 15:14 | |
| *** dasp has joined #openstack-sdks | 15:16 | |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056 | 15:20 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538 | 15:20 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Remove incorrect os_ prefix in documentation https://review.opendev.org/727226 | 15:20 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538 | 15:20 |
| mordred | noonedeadpunk: I haven't - but I try to stay as far away from python-*client as I can, so I'm not much help | 15:25 |
| noonedeadpunk | mordred: ok, I see:) I didn't found anything there as well actualy:( so maybe it's in horizon... | 15:32 |
| mordred | noonedeadpunk: we should add to our already-way-too-large-todo-list updating horizon to use sdk :) | 15:35 |
| noonedeadpunk | that would be great... but tough:) | 15:36 |
| *** dasp has quit IRC | 15:41 | |
| *** dasp has joined #openstack-sdks | 15:42 | |
| mordred | noonedeadpunk: yeah. tell me about it | 15:49 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Run linters in a temp dir instead of an install https://review.opendev.org/726952 | 16:03 |
| *** rpittau is now known as rpittau|afk | 16:10 | |
| *** jpich has quit IRC | 16:12 | |
| *** enriquetaso has quit IRC | 16:33 | |
| *** ttsiouts has joined #openstack-sdks | 16:35 | |
| *** evrardjp has quit IRC | 16:36 | |
| *** evrardjp has joined #openstack-sdks | 16:36 | |
| *** ttsiouts has quit IRC | 16:50 | |
| *** ttsiouts has joined #openstack-sdks | 16:51 | |
| *** gtema has quit IRC | 16:53 | |
| *** gtema has joined #openstack-sdks | 16:54 | |
| *** ttsiouts has quit IRC | 16:55 | |
| *** gtema has quit IRC | 16:55 | |
| *** jpena is now known as jpena|off | 17:14 | |
| *** ralonsoh has joined #openstack-sdks | 17:21 | |
| *** iurygregory has quit IRC | 17:27 | |
| *** ralonsoh has quit IRC | 17:28 | |
| *** yolanda has quit IRC | 17:35 | |
| *** ralonsoh has joined #openstack-sdks | 17:48 | |
| *** nightmare_unreal has quit IRC | 17:54 | |
| *** dustinc has joined #openstack-sdks | 18:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352 | 18:13 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Remove incorrect os_ prefix in documentation https://review.opendev.org/727226 | 18:16 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056 | 18:16 |
| *** ralonsoh has quit IRC | 18:17 | |
| *** ralonsoh has joined #openstack-sdks | 18:22 | |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Rename all of the modules https://review.opendev.org/726538 | 18:24 |
| *** iurygregory has joined #openstack-sdks | 18:31 | |
| openstackgerrit | Merged openstack/openstacksdk master: Fix issues found by latest flake8 https://review.opendev.org/727204 | 18:39 |
| *** ralonsoh has quit IRC | 18:43 | |
| *** tremble has joined #openstack-sdks | 18:55 | |
| *** diablo_rojo has joined #openstack-sdks | 19:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352 | 19:22 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Revert "Run linters in a temp dir instead of an install" https://review.opendev.org/727400 | 19:34 |
| *** camelCaser has quit IRC | 19:35 | |
| *** camelCaser has joined #openstack-sdks | 19:35 | |
| *** enriquetaso has joined #openstack-sdks | 19:39 | |
| tremble | mordred, baremetal_node_action (Ansible module) - We talked about fixing some of the 'bool's that are currently strings when I did the big batch of cleanup, technically right now we allow several values that aren't 'bool's (and aren't in the description as valid) would you rather a formal deprecation cycle, or just clean it up for v1 of the collection? | 19:40 |
| mordred | tremble: I say just clean it up for v1 of the collection ... sshnaidm what do you think? | 19:40 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352 | 19:41 |
| sshnaidm | mordred, yeah, i think it's ok | 19:42 |
| tremble | ok | 19:43 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411 | 19:52 |
| AJaeger | mordred: https://review.opendev.org/#/c/727352/ removes pbrx, hope that the jobs now passes. could you review later, please? | 19:54 |
| mordred | AJaeger: oh thanks! yes - that looks good | 19:56 |
| *** slaweq has quit IRC | 19:57 | |
| AJaeger | mordred: doesn't work, it's failing ;( | 19:58 |
| AJaeger | do we need to remove the repos from the constraints list first? | 19:58 |
| tremble | mordred, Do you happen to know the history of https://github.com/openstack/ansible-collections-openstack/blob/master/plugins/modules/baremetal_node_action.py#L212 I think that's actually unreachable because of the preceding if statement | 20:01 |
| *** slaweq has joined #openstack-sdks | 20:01 | |
| AJaeger | mordred: yeah, this basically needs rewrite of the siblings role ;( should we keep pbrx or move it over? | 20:01 |
| mordred | tremble: I don't - I think we'd want to ask dtantsur|afk or TheJulia | 20:02 |
| AJaeger | mordred: we could also do some ansible magic... | 20:02 |
| mordred | AJaeger: hrm | 20:02 |
| AJaeger | cp constraitns-file and then iterate with lineinfile to remove the lines | 20:03 |
| tremble | mordred, Ok, I'll try dtantsur|afk in our morning | 20:04 |
| *** slaweq has quit IRC | 20:06 | |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackclient master: Remove pbrx https://review.opendev.org/727352 | 20:08 |
| AJaeger | mordred: let's try again - if you have a better idea or a fix, feel free to take over. I'm signing off for today | 20:09 |
| mordred | AJaeger: :) thank you! | 20:09 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Improve ansible-test sanity for collection https://review.opendev.org/727411 | 20:10 |
| *** sshnaidm is now known as sshnaidm|afk | 20:18 | |
| TheJulia | tremble: I have this fear I may have been on a third whiskey at a hotel bar in downtown seattle when I wrote that | 20:20 |
| tremble | Think I just nuke the lines? | 20:21 |
| TheJulia | I remember feeling inspired and working out such caseI feel like your likely safe doing so. | 20:21 |
| * TheJulia needs to reboot this macbook because of gui fun | 20:21 | |
| * tremble tries to follow some of this logic and wonders if he should start on the whisky first... | 20:26 | |
| * tremble heads to bed instead | 20:29 | |
| *** slaweq has joined #openstack-sdks | 20:35 | |
| TheJulia | tremble: may not be a bad idea. I can alloc brain cells tomorrow because the module was basically overloaded | 20:35 |
| mordred | TheJulia: we might also be in a position now where we have some of this logic covered down in sdk but we never got around to nuking it from the modules | 20:43 |
| *** vishakha has quit IRC | 20:51 | |
| *** slaweq has quit IRC | 21:13 | |
| *** slaweq has joined #openstack-sdks | 21:25 | |
| *** slaweq has quit IRC | 21:30 | |
| *** slaweq has joined #openstack-sdks | 21:40 | |
| TheJulia | mordred: yeah, I think a lot of it was to cover "you should try to do this and that at the same time" kind of stuff | 21:43 |
| *** slaweq has quit IRC | 21:44 | |
| openstackgerrit | Merged openstack/openstacksdk master: Set BaseImageProxy.create_image validate_checksum default to False https://review.opendev.org/726465 | 21:53 |
| *** tkajinam has joined #openstack-sdks | 22:54 | |
| *** tosky has quit IRC | 23:14 | |
| *** enriquetaso has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!