19:10:58 #startmeeting Development IRC meeting 6 May 2020 19:10:58 Meeting started Wed May 6 19:10:58 2020 UTC. The chair is tcohen. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:10:58 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 19:10:58 The meeting name has been set to 'development_irc_meeting_6_may_2020' 19:11:03 ah :) 19:11:12 #link https://wiki.koha-community.org/wiki/Development_IRC_meeting_6_May_2020 Agenda 19:11:12 #topc Introductions 19:11:31 #info Katrin Fischer, BSZ, Germany 19:11:33 #info Tomas Cohen Arazi, Javier and Manuel's father 19:11:33 #info Owen Leonard, Athens County Public Libraries, Ohio, USA 19:11:44 #topic Introductions 19:12:05 #info David Nind, Wlelington, New Zealand 19:12:13 #info tuxayo/Victor Grousset, France 19:12:18 #chair tuxayo 19:12:18 Current chairs: tcohen tuxayo 19:12:27 do not overlap 19:12:28 whaaaat 19:12:30 #info Thomas Dukleth, Agogme, New York City [still world virus capital] 19:12:34 but if you want to do it, be my guest 19:12:40 I just wanted to fix a typo >_< 19:12:51 ah 19:12:54 haha 19:13:23 oops 19:13:37 topc→topic. Maybe it was not a big deal. Anyways ^^" 19:13:38 #topic Introductions 19:13:53 You don't need to introduce yourself again 19:14:11 #topic Announcements 19:14:16 tuxayo: only chairs can set topics 19:14:29 Anyone has any announcement to make? 19:14:48 #info We are in feature freeze, heading into String freeze end of week 19:15:13 cait: And if some tries, they get a place in the chair instead, hue hue 19:15:18 can't think of something else :) 19:16:05 move on? 19:16:13 #topc Update from the Release manager (20.05) 19:16:28 just messing with you 19:16:30 Just join by phone 19:16:31 tcohen: topc→topic 19:16:35 hooo >_< 19:16:36 Sorry, computer issues 19:16:39 #topic Update from the Release manager (20.05) 19:16:47 just in time, ashimema[m] 19:16:53 #info Martin Renvoize, PTFS-Europe 19:17:00 o/ ashimema 19:17:10 o/ 19:17:50 #info We are loving at a pace readying ourselves for release.. QA team are great and bugs are getting squashed. We're now in feature freeze and string freeze is this coming Friday. 19:18:15 Please excuse a higher rate of typos.. no idea why I can't sign in on the laptop or pc 19:18:19 \_o< quack! 19:18:19 *click* 19:18:24 !bang 19:18:24 \_x< tcohen: 2 (5.15 seconds) 19:18:25 [('tcohen', 2), ('tuxayo', 1)] 19:18:26 Best time: tcohen with 5.15 seconds 19:18:27 Longest time: tuxayo with 845444.47 seconds (this is your new longest time in this channel! Your previous longest time was 386334.35) 19:18:28 tcohen took the lead for the week over kidclamp with 2 points. 19:18:33 !stop 19:18:34 Not a single duck was shot during this hunt! 19:18:35 Nothing to stop: there's no hunt right now. 19:18:54 ashimema[m]++ 19:19:00 very excited about this release! 19:19:11 me too :) 19:19:21 moving on then 19:19:29 good pace overall - just no slacking off after release please heh 19:19:51 I'm sure we will all be busy next cycle 19:19:59 #topic Updates from the Release Maintainers 19:20:14 rmaints? 19:20:14 hmmm... rmaints is talljoy, lukeG, hayley 19:21:27 yeay.. back in.. I can type again :) 19:21:37 #info Joy Nelson ByWater 19:21:45 o/ 19:21:49 o/ 19:21:58 it is the rmaints report topic 19:22:22 Jenkins is failing on me and aborted on two runs. Anyone able to help with that (19.11.x) 19:22:51 other than that, trying to catch up to ashimema with stuff being pushed to 20.05 19:22:52 \o 19:23:07 I will Joy, sorry I forgot this morning 19:23:24 thanks! 19:23:50 #info some tests are failing on 19.11, no news from other rmaints 19:24:00 * ashimema[m] isn't sure what he did to upset Jenkins.. noever seen him to red and mad looking 19:24:19 we need a bigger server 19:24:26 well.. once maybe.. I got he devil response once 19:24:26 I will take care of that 19:24:36 and probably move into gitlab-ci 19:24:44 tcohen: that's the cause of failures? 19:24:54 sometimes jenkins is causing OOM 19:25:04 I haven't found the plugin that is to blame 19:25:15 Ho, that's bad. 19:25:17 if you wanna chat about that we can on pm 19:25:26 ok! 19:25:26 ideas are welcome 19:25:40 +1 19:25:57 :) 19:26:07 can we skip the next topic? cait will kick our butts I think 19:26:16 lol 19:26:24 why would i? qa team is great the rM said... ;) 19:26:24 #topic Updates from the QA team 19:26:42 .... well, QA team IS great, but we can always do better 19:26:54 Joubu++ # his numbers are impressive 19:27:19 #info QA Team please jump on any bugs popping up now - there is still some things we should include in release to make things go smoothly especially with the new feature sintroduced and highlighted 19:28:15 I wanted to mention some recent changes on the API front 19:28:19 so please, all hands on deck, don't fiddle with your shiny features for a bit, but hunt bugs :) 19:28:26 ok, go for it 19:28:28 have broken some behaviour on the plugins that implement routes 19:28:33 * oleonard pouts 19:29:13 it only highlighted that on the plugins front, the 'anonymous', 'public' and 'privileged' access paths hadn't been thought much 19:29:19 * ashimema[m] sends less than subliminal messge to devs to contribute to the technical release notes pad 19:29:35 oh, did i miss the link for that? 19:29:43 in a minute 19:30:18 cait: email "[Koha-devel] New technical changes for 20.05?" 19:30:26 * cait doesn't get koha-devel recently, it looks like my web.de email is blocked soemwhere :( 19:30:26 that's it tuxayo 19:30:46 any help with that appreciated by the way... 19:31:26 cait: that shouldn't block incoming mail though. Maybe it's web.de that blocks lists.koha-community.org 19:31:27 I will finish what I was writing 19:32:20 #link https://hebdo.framapad.org/p/9ga9-koha-tech-release-notes tech notes 19:32:21 what I meant to say is that before the release, we need to replicate the behavioiur for the core API, on the plugins front, with a /public path, and honouring the 'RESTAnnonymous...' syspref as well 19:32:33 I will work as fast as possible on that 19:32:42 thanks tcohen 19:32:52 and write on that technical notes so any dev knows about this change 19:33:18 * ashimema[m] has lost track of topics 19:33:24 anyway, we don't catch this things in QA 19:33:28 technical_notes++ 19:33:30 and we need to work on that as well 19:33:40 there is info on the wiki about api - but a note on the outdated pages would be great too 19:33:47 #topic General development discussion (trends, ideas, ...) 19:34:08 #info 1) What's the state of ES 7 support 19:34:11 tuxayo? 19:34:11 tuxayo is probably on a role 19:34:23 I'll detail 19:34:34 can I ask a question? 19:34:36 There doesn't seem to be anything related to ES 7 other than Bug 22520 19:34:36 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22520 enhancement, P5 - low, ---, koha-bugs, NEW , Be Elastic compliant 7.x and 8.x (_doc) 19:34:44 Should we have something like this? «Bug 20196 - [Omnibus] Prepare Koha to ElasticSearch6 - ES6» 19:34:44 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20196 enhancement, P5 - low, ---, alex.arnaud, RESOLVED FIXED, [Omnibus] Prepare Koha to ElasticSearch6 - ES6 19:34:56 done. (for this question) 19:35:09 oops, sorry tcohen , go on. 19:35:17 its ok 19:35:35 I wanted to ask if there's a concept of LTS for ES 19:35:47 tcohen: nope 19:35:52 because we are always running way behind 19:35:52 good question 19:36:01 lets move back to SolR then :-D 19:36:12 No, too late 19:36:14 19:36:21 After looking a lot about to try to find the end of life of ES 6, there is definitely no LTS 19:36:23 ? 19:36:43 it's hard to keep up with a fast moving project like ES 19:36:55 good and bad... 19:37:19 It may be possible to artificially create a long term stable version at least somewhat. 19:37:25 From what I get, ES has a new major release every 12 or 18 months (variable) 19:37:52 And a version is supported during until the next next release 19:38:00 what does 'Major' constitute? 19:38:10 6.X 5.X 19:38:18 compared to our own cycle for example 19:38:30 One could standardise on some Elastic Search release in some Long Term Stable GNU/Linux distribution for example. 19:38:32 So between 2 to 3 year of support for each major version. 19:38:36 number of breaking changes I supose it my question 19:39:08 that's LTS-ish 19:39:51 LTS-ish++ 19:40:11 It should be easier to keep up when we have broader use, the slow point right now is not a lot of devs or users 19:40:19 > number of breaking changes 19:40:19 Breaking changes are only in major release. And with deprecation notices on the previous version. That helps. 19:40:37 mmm 19:40:57 is it hard to install an 'older' version? 19:41:21 two years is ok 19:41:25 we don't as a community have any form of centrally lead roadmaps so it's hard to build in regular updates for dependancies like this 19:41:28 the issue here is lack of devs 19:41:48 that's something i would have loved to try and change during my time as RM but alas it was a mountain too far.. 19:42:03 > is it hard to install an 'older' version? 19:42:03 I guess not because we are still installing 5.X which is EOL. 19:42:10 ok 19:42:16 I am however, happy to work on such a scheme next cycle or too if the next RM is keen for me to do so 19:42:22 so at least if we miss one, it will not break stuff for people immediately 19:42:37 i think if we had a group of three that helped each other out, that oculd go far 19:42:58 indeed 19:43:20 once we got a core group testing the last ES upgrade.. it went pretty quick 19:43:37 just needs focus 19:44:01 So, that might be useful to have something like «Bug20196 - [Omnibus] Prepare Koha to ElasticSearch6 - ES6» but for ES 7. 19:44:24 I'll create it to link new bugs if they pop. 19:44:35 yup 19:44:37 tuxayo++ 19:44:52 tuxayo :) 19:44:53 The next questions might help also. Moving on to them? 19:45:12 #info 2) Default ES versions of koha-testing-docker and KohaDevBox 19:45:23 what shape are we in for ES unit tests and running them on Jenkins? 19:45:53 Are there much unit/UI tests using ES for now? 19:46:18 I'm seeing another whole raft of CI work in need :(.. we hugely improved our coverage the last couple of cycle by introducing a bigger mix of OS + DB combinations 19:46:30 so we also need to start testing ES combinations too 19:46:50 koha-testing-docker currently can be started with es6 19:47:00 nice to learn about these CI improvement! :D 19:47:05 with just a param change 19:47:18 Oh, yes so the question was. 19:47:21 in the case of KohaDevBox 19:47:23 2) Default ES versions of koha-testing-docker and KohaDevBox [tuxayo] 19:47:26 Should they move to ES 6 by default? ES 5 is end of life since 2019-03 (ES 7 release) 19:47:32 that's alreay configurable in the user.yml file 19:47:39 And should they move to ES 7 by default after a few months ? 19:48:22 tcohen: if ES6 is officially supported having it as default for a new one would make sense to me 19:48:25 tcohen: the default is how devs/SO/QA can discover new bugs right? 19:49:25 totally 19:49:32 kidclamp: maybe? 19:49:33 Otherwise it's only on user reports and the devs/SO/QA tjat choose to use ES 6. 19:49:49 changing the default is ok 19:50:04 but we need to make sure CI is testing against 5.x 19:50:11 I will admit.. I only really pay attention to ES when I'm looking specifically at and ES bug 19:50:15 Good :D 19:50:20 agreed 19:50:26 info? 19:50:26 somebody said info was largely out there.. just not especially well summarised 19:50:33 or agreed, but log :) 19:50:46 I don't know how to start a vote 19:51:06 do we even need to vote on that? 19:51:12 feels like a no brainer to be honest 19:51:12 #agreed We will make ES 6.x the default in our dev tools (koha-testing-docker and KohaDevBox) 19:51:22 hooray! 19:51:22 :) 19:51:24 +1 19:51:26 not vote, but log it :) 19:51:33 for the minutes 19:51:34 Next question (related) 19:51:36 #info tcohen will make sure our CI is still running tests against ES 5.x 19:51:47 tcohen++ 19:51:52 #info We will make ES 6.x the default in our dev tools (koha-testing-docker and KohaDevBox) 19:51:53 tcohen++ 19:52:05 should we be running against both ES5 and ES6 tcohen? 19:52:18 yes, and why not 7? 19:52:26 then have a deprecation point for dropping ES5 19:52:30 sounds good to me.. 19:52:32 Yes, +1 to 5, 6 and 7 19:52:34 poor Jenkins.. 19:52:37 haha 19:52:39 it's probably my fault he hurts 19:52:51 and drop 5 probably with 20.11 19:53:01 I will ask mtj[m] to talk about his setup for running more tasks simultaneously on his node 19:53:19 :) 19:53:21 he's node is big and has resources 19:53:55 #info 3) Should koha-testing-docker and KohaDevBox have ES enabled by default instead of Zebra? 19:54:03 awsome 19:54:15 Again, to find more bugs. 19:54:24 And it would make sense if we want to move toward making ES 19:54:25 the recommended search engine in Koha. 19:54:33 done. 19:54:52 koha-testing-docker has ES set by default 19:54:53 tuxayo++ # raising great talking points 19:55:13 it is only a matter of enabling it in misc4dev, probably 19:55:21 tcohen: But one must set the SearchEngine syspref and index ES right? 19:55:22 in the case of KohaDevBox 19:55:29 tuxayo yes 19:55:53 in the case of KohaDevBox the thing is it would require VirtualBox to assign more resources (RAM) 19:55:57 Sorry, with kid so not here, we need to build actual search tests, but we need support for testing in a dev env that won't leave records in our indeed 19:56:00 4GB 19:56:01 Index 19:56:18 > in the case of KohaDevBox the thing is it would require VirtualBox to assign more resources 19:56:18 Indeed 19:56:46 kidclamp didn't we solve taht already by having a separate index? 19:56:47 > we need support for testing in a dev env that won't leave records in our 19:56:47 what do you mean kidclamp ? 19:57:00 or we didn't finish that... 19:58:17 > one must set the SearchEngine syspref and index ES 19:58:19 This means that many of our dev, SO, QA work in still done on Zebra so discovery on new bugs will be low. 19:58:25 our general tests that create bibs end up in the index 19:58:38 i thnk we can't break Zebra either 19:58:40 because we don't have a queue, we just index upon creation/change 19:58:40 i regularly switch 19:58:49 when your devbox is set up, you can just throw the pref 19:59:07 it's not an either/or choice 19:59:43 hmmm, so how could we have a mixed usage of ES or Zebra? 19:59:46 I find it easy enough to switch when testing elastic search bugs (koha-testing-docker) 20:00:17 tuxayo: i think we already have 20:00:32 bug 24119 20:00:32 04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24119 major, P5 - low, ---, chris, NEW , Records indexed into ES during tests are not removed by rollback 20:01:23 cait, davidnind . Good, I was worried that the usage was low and that many ES 20:01:26 bugs could sneak to end user. 20:01:51 i think the ones we find now are too specific often, ES works 'well enough' i don#t notice when i forget to turn it off 20:01:59 but i don't do the real hard stuff in testing 20:02:00 nice! 20:02:16 like importing, merging etc. - unless i test a bug in that area 20:02:20 tuxayo: yes, we are trying to get a broad base of our users testing right now - once we upgrade to 19.11 we will work on switching to ES6 and finding those bugs, then can focus on support ofr 7/8 20:02:37 Though if we want to catch up with ES versions. That means bugs than the current pace. 20:03:00 kidclamp: very good :D 20:03:28 Would it be worth it to make docker-testing-docker and the DevBox to automatically index ES on startup? 20:03:29 Then it's only a matter of syspref. 20:03:42 ptfs-e are starting to migrte customers to es with 19.11 too 20:04:03 es indexing should be added to 'do all you can do' 20:04:08 and need support in sandbox 20:04:13 for indexing es 20:04:23 do we have bugs for thoseß 20:04:24 ? 20:04:25 yes.. sandbox support would be good.. 20:04:30 we could tag them priority at least 20:04:35 well or gitlab issues 20:04:37 but I imagine that could increase system requirements for sandbxoes significantly? 20:04:56 ashimema[m] sandboxes should share an ES instance 20:05:14 good idea, less resources 20:05:19 I think they might already.. I can't remember what state it's currently in 20:05:37 khall is king there.. I just dable with patches now and then 20:05:57 afaik they support es, but users cannot index it 20:06:00 So the proposals would be. 20:06:00 - es indexing should be added to 'do all you can do' 20:06:05 - add support in sandboxes for ES 20:06:05 - ES support in KohaDevBox 20:06:06 Any thing more? 20:06:36 I don't get the KohaDevBox item 20:07:00 tcohen: I though KohaDevBox didn't use ES. 20:07:18 it does, if you launch vagrant with KOHA_ELASTICSEARCH=1 20:07:37 and vars/user.yml lets you change the ES version 20:07:59 but someone needs to try setting 6,x and see if the repo works, or needs to be changed in the config 20:08:15 i know the default works 20:08:17 i use it all the time 20:08:31 yes, I'm talking about making 6.x the default 20:08:37 and what changes are needed 20:08:50 re: install repositories, for example 20:09:05 tcohen: that was to tuxayo's question :) 20:09:30 tcohen: Good. So nothing to do about DevBox except the default ES version (when choosing ES, Zebra stays by default) 20:09:45 I'm just suggesting someone should volunteer to try and file an issue in Gitlab if we need to change something 20:09:52 I have been using ES6, works in testing docker well 20:10:21 kidclamp would you suggest we make it the default? 20:10:24 yes 20:10:26 100% 20:10:29 definitely 20:10:31 absolutely 20:10:35 :-) 20:10:40 tcohen: okay, I'm noting that. So: 20:10:40 - es indexing should be added to 'do all you can do' 20:10:41 #info tcohen will make ES6 the default in koha-testing-docker 20:10:42 - add support in sandboxes for ES 20:10:42 And that's all? 20:11:23 maybe 20:11:38 :) 20:11:43 Great :D 20:11:50 gotta run 20:11:55 thanks kidclamp 20:11:58 next topic? 20:12:00 kidclamp++ 20:12:05 yup 20:12:19 #topic Review of coding guidelines 20:12:47 #info There's a proposal to add aria-hidden="true" in the guidelines 20:13:04 #link https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25166 20:13:04 04Bug 25166: enhancement, P5 - low, ---, oleonard, Pushed to master , Add aria-hidden = "true" to Font Awesome icons in the OPAC 20:13:53 * tuxayo tries to understand 20:13:54 #info https://fontawesome.com/v4.7.0/accessibility/ 20:13:57 https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-hidden_attribute 20:14:07 "If you're using an icon to add some extra decoration or branding, it does not need to be announced to users as they are navigating your site or app aurally" 20:14:33 Adding aria-hidden="true" to an element removes that element and all of its children from the accessibility tree. 20:14:35 This can improve the experience for assistive technology users by hiding: 20:14:38 - purely decorative content, such as icons or images 20:14:39 Having worked with people using screen readers, anything to reduce the auditory noise in screen reading should be done. 20:14:41 - duplicated content, such as repeated text 20:14:41 - offscreen or collapsed content, such as menus 20:14:45 so if we have an edit button that is labelled added with a pencil icon,.. we don't need the icon 20:15:17 and what about role="button" and aria-label? 20:15:48 ....and if we have an icon which is the entire control (no label) we can add aria-label to the