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