Time Nick Message 07:09 marcelr o/ 07:13 magnuse__ \o/ 07:48 Joubu hi #koha 07:51 marcelr hi Joubu 08:15 alex_ Bonjour 08:18 cait1 hi alex_ :) 08:27 zimbo_[m] good morning :) 08:28 cait1 oh a new 'face' :) 08:29 ashimema morning 08:33 ashimema where are you based zimbo_ ? 08:33 ashimema interesting to see you here late last night for me and early this morning π 08:34 zimbo_[m] ashimema: IΒ΄m in Austria in a library :) 08:34 cait1 so just another night owl then :) 08:34 cait1 IRC regulars? 08:34 cait1 oh no wahanui 08:34 zimbo_[m] ashimema: Yeah IΒ΄m just happy to be able to read the messages again hahah :D 08:34 zimbo_[m] cait1: I was still working at that time :) 08:35 cait1 I was trying to get the bot to give this link, but the bot is not there: https://wiki.koha-community.org/wiki/IRC_Regulars 08:35 ashimema π 08:35 ashimema well.. it's lovely to have someone else to chat to π 08:37 cait1 welcome :) 08:38 zimbo_[m] ahh thank u c: 08:38 ashimema school run time for me now in england.. back in half hour :() 08:51 Joubu tcohen: ktd "Issue #365: Only chown if needed" - what did you do there? Do you mean we should not set LOCAL_USER_ID if we are using uid=1000? 09:16 magnuse Oak 09:24 magnuse gah! shibboleth is driving me up the walls. i get reidrected to the idp, sign in there, an returned to koha, but the opac does not show me being logged in. it looks like a shib session is created, but koha does not recognize it. when i look at /Shibboleth.sso/Session there is some "Miscellaneous" info, but no "Attributes". could that be the problem, that the idp is not sending the attribute i am mapping to userid in koha? 09:24 ashimema yes 09:24 ashimema Koha matches an attribute to get the local account 09:25 ashimema there is an auto-create option.. but if you're not mapping anything that won't really help 09:27 pastebot "magnuse" at 127.0.0.1 pasted "shib config" (48 lines) at https://paste.koha-community.org/2255 09:27 magnuse ashimema: see any glaring mistakes there? 09:28 magnuse patrons have already been created in koha, they should only use shib to log in, not to create new patrons 09:30 magnuse i restared shibd, apache and plack 09:30 ashimema nothing obvious 09:30 ashimema but i'm a little out of touch 09:33 magnuse thanks for looking :-) 09:39 cait1 ashimema: pm? :) 09:39 cait1 I need something that says: this issue is at the bindery / out for binding 09:39 cait1 or maybe unavailable for binding? 09:40 ashimema Β¬out for bindingΒ¬ sounds good to me 09:40 cait1 thx! 09:43 cait1 ashimema++ 10:21 * cait waves 10:50 tcohen hola #koha o/ 10:55 marcelr hola tcohen 10:55 cait hola tcohen and hallo marcelr 10:55 marcelr o. 10:55 marcelr o/ 10:56 tcohen hey marcelr, will you be around long this morning? 10:56 marcelr no did you see my last message on irc yesterday? 10:56 marcelr about the api logging 11:01 tcohen you mentioned we need to change the logging level 11:03 marcelr yes, it might be needed depending context 11:03 marcelr we have to resume this next week 11:04 marcelr see you #koha 11:09 Joubu tcohen: you missed a follow-up on 32983 11:17 tcohen we need to communicat ebetter 11:18 Joubu you told us you were watching the additional_work keyword ;) 11:19 tcohen yeah, but this has been an eclectic week 11:21 tcohen Joubu: I will patch ktd so it sets a good logging level for the API 11:22 Joubu tcohen: did you see my question earlier? It was ktd related, but nothing to do with logging. 11:22 tcohen I haven't seen it, sorry 11:23 Joubu tcohen: ktd "Issue #365: Only chown if needed" - what did you do there? Do you mean we should not set LOCAL_USER_ID if we are using uid=1000? 11:24 tcohen I pushed so see if things worked (they do!) 11:24 tcohen next step is changing if $LOCAL_USER_ID for a comparisson with 1000 11:25 tcohen && mv /root/.cache/Cypress /kohadevbox && chown -R 1000 /kohadevbox/Cypress 11:25 tcohen we now have that in the Dockerfile 11:25 tcohen I was trying to fix what you filed on gitlab about speed 11:26 tcohen I will bring the compaints book to Marseille, will be available on O'Malleys front desk 11:27 Joubu complaint? I am not complaining, just trying to follow what I missed 11:30 tcohen I know, was a general comment 11:31 Joubu kidclamp: ping bug 32594 comment 36 and 37 11:31 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32594 critical, P5 - low, ---, nick, Pushed to stable , Add a dedicated ES indexing background worker 11:39 tcohen Joubu: https://gitlab.com/koha-community/koha-testing-docker/-/commit/22e3001ca9b8b3e2409d4e3e524cde3c4cb6ff6d 12:49 cait Joubu: it looks like the git log option for sandboxes is broken :( 12:52 cait and the css is wonky, especially on the biblibre ones 12:52 cait item form is totally broken on the newly set up one 12:58 cait we are one week pre Marseille and the sandboxes seemto be broken overal - all 3 servers 12:59 cait khall ? 13:00 Joubu cait: talking about serialstat? 13:01 cait yeah 13:01 Joubu the sandbox is actually not there 13:01 cait we just created that one on all 3 servers 13:01 cait it disappeared 13:01 cait it wqas there before, but I think something goes wrong 13:01 Joubu TASK [Install YAML::Syck, CGI::Session::Serialize::YAML, needed for Koha <= 21.05] *** 13:01 cait on bywater we had css from another library in the newly created box 13:01 Joubu fatal: [localhost -> koha-serialstat]: FAILED! => {"changed": false, "module_stderr": "", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 137} 13:01 cait as if it was using another one 13:01 Joubu did you request something else than master? 13:02 Joubu cait: it appears on the interface, but it has not been created correctly 13:02 cait master with the bug applied on creation 13:02 cait it appears to be a general issue 13:03 cait actiong > git log brings up an empty page 13:03 Joubu tcohen: did we push something about YAML to ktd recently? 13:03 tcohen API-related? 13:03 tcohen more context? 13:03 Joubu see above 13:03 Joubu YAML, 21.05, ktd 13:04 tcohen doesn't ring any bell, sorry 13:04 tcohen we abandoned yaml::Syck but a while ago 13:04 Joubu it's koha-sandboxes-docker actually 13:05 Joubu or we removed it from our debian repo 13:08 Joubu cait: https://gitlab.com/koha-community/koha-sandboxes-docker/-/issues/87 13:10 cait thanks Joubu 13:10 cait Joubu++ 13:17 Joubu and.. another problem. 13:17 Joubu tcohen: "usermod: invalid user ID 'test-koha'" looks like this one may be related to the last push to ktd 13:18 tcohen related, but is it highlighting an underlying issue? 13:18 tcohen koha-create is supposed to have created the user? 13:21 caroline ashimema, can you ping aude and lucy for me? 13:22 Joubu tcohen: my guess it that LOCAL_USER_ID is empty 13:22 tcohen oh 13:23 Joubu and "" != "1000" 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:26 $ export HOLA="" 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:34 $ if [ "${HOLA}" != 1 ]; then echo "woo"; fi 13:23 tcohen woo 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:35 $ export HOLA="1" 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:35 $ if [ "${HOLA}" != 1 ]; then echo "woo"; fi 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:35 $ tig 13:23 tcohen β ~/git/koha-testing-docker [master|β 2] 13:23 tcohen 08:35 $ if [ "${HOLA}" != "1" ]; then echo "woo"; fi 13:23 tcohen β ~/git/koha-testing-docker [master|β 1β 2] 13:23 tcohen 08:36 $ if [ "${HOLA}" != "1000" ]; then echo "woo"; fi 13:23 tcohen woo 13:23 tcohen this were my tests before submission 13:24 tcohen so I missed the case "" != "1000" 13:24 tcohen didn't spot it 13:24 tcohen ok 13:34 tcohen that will be fixed shortly on the next build 13:35 Joubu thanks 13:35 tcohen I still think LOCAL_USER_ID should always be set, though 13:36 tcohen it is a very particular case in which it should not be set: a machine with a single user, which is the one that will run things. 13:36 tcohen if a sysadmin sets a generic admin user by default, and then the sandboxes user, permissions will get borked 13:37 tcohen not saying things will break, but seems fragile to me 13:38 cait thanks to you both for look8ing into it 13:38 cait could you let me know when it makes sense to test again? 13:38 tcohen https://gitlab.com/koha-community/koha-testing-docker/-/jobs/3955412058 13:38 tcohen once that finishes successfuly 13:39 tcohen at least for the last error Jonathan mentioned 13:40 cait I am not srue waht effects what 13:40 cait Joubu? 13:41 cait what about https://gitlab.com/koha-community/koha-sandboxes-docker/-/issues/87 ? 13:43 Joubu I will let you know when it's fixed 13:43 cait thanks! 13:53 ashimema driving at the moment Kate 13:54 cait don't chat and drive! 13:54 ashimema modern Marvels, Google 13:57 caroline (I read that in a Robin voice... "Holy Marvels, Batman!") 13:57 cait hehe 13:58 Joubu cait: https://staff-test.sandboxes.biblibre.eu/cgi-bin/koha/mainpage.pl - should be fixed now 13:58 cait i'll give it a try 14:02 cait Joubu: applying the patches on the existing sandbox worked, just spinning up a new one to test that too 14:08 Joubu tcohen: follow-up added to 32923 14:08 Joubu MatthewBlenkinsop[m]: same, see bug 32923 14:08 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32923 normal, P5 - low, ---, jonathan.druart+koha, RESOLVED FIXED, x-koha-embed must a header of collectionFormat csv 14:10 tcohen Joubu: this is related to the exception we added, right? 14:10 cait Joubu: sandbox looking great, thank you 14:10 cait Joubu++ 14:11 Joubu tcohen: yes 14:23 kidclamp Joubu: I put a patch on 32594 - let me know your thoughts 14:34 cait @seen oleonard 14:34 huginn cait: oleonard was last seen in #koha 1 week, 0 days, 20 hours, 14 minutes, and 10 seconds ago: <oleonard> yeah probably making it too complicated 14:35 cait a week... 14:35 PedroAmorim[m] tcohen: around? 14:35 tcohen indeed 14:36 PedroAmorim[m] think I found a weird bug in 21.11.17 14:36 PedroAmorim[m] this commit is changing 1 line 14:36 PedroAmorim[m] https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=31245&attachment=138969 14:36 cait kidclamp++ thanks or QA :) 14:36 tcohen weird bugs ar a myth 14:36 tcohen Koha doesn't have weird bugs 14:36 PedroAmorim[m] haha 14:36 PedroAmorim[m] but the pushed commit to 22.11.x has more than that line 14:36 PedroAmorim[m] https://github.com/Koha-Community/Koha/commit/ca7cb4896d1dee0f266f12d778a3cb841f05afaf 14:36 tcohen looking 14:36 PedroAmorim[m] and I think it's causing the bug I'm experiencing, I also wouldn't expect these 2 commits to differ? from bugzilla -> github 14:37 PedroAmorim[m] s/22/21/ 14:38 tcohen I recall we stopped being silly by passing the job_id and re-fetching the object. 14:38 tcohen but I don't remember why we added the return unless... 14:38 PedroAmorim[m] I understand that, but why do these 2 patches differ? it should be the same patch 14:39 tcohen maybe I cheated? 14:39 PedroAmorim[m] the code I'm working on is in this weird limbo, where the additional_report is on the stage where $self is being used instead of $job 14:39 PedroAmorim[m] but the rest of the code is still using $job instead of $self 14:39 tcohen oh, that's a rebased patch Arthur cherry-picked 14:39 tcohen that's a bad rebase 14:42 PedroAmorim[m] can confirm, I've solved the issue by reverting those 2 extra lines and using revert with job instead of self 14:42 PedroAmorim[m] but damn, wasn't easy to pinpoint 14:54 PedroAmorim[m] * can confirm, I've solved the issue by reverting those 2 extra lines and using return with $job instead of $self 15:01 adminucd[m] hello i where i can finde koha uml ? 15:02 adminucd[m] admin ucd: hello where i can finde koha uml 15:02 Joubu https://schema.koha-community.org/ 15:38 cait adminucd[m]: ^ 15:38 cait for the database schema 16:00 cait bug 33268 16:00 huginn 04Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33268 major, P5 - low, ---, koha-bugs, NEW , Overlay rules don't work correctly when source is set to * 17:54 cait have a nice weekend #koha!