Time  Nick            Message
00:06 mtompset        oh sucky. git pull's aren't working for me.
00:07 mtompset        the stupid problem I've been fighting all afternoon is going to happen.
00:08 mtompset        I'm going to blame symantec end point. I hate it. :)
00:11 wizzycray       mtompset, have you tried git:// protocol instead of http?
00:11 wizzycray       like, what's your clone command
00:11 wizzycray       git clone git://blah
00:11 wizzycray       or git clone http://blah
00:12 mtj             hey wiz, good idea ^ :0)
00:12 mtompset        pretty sure it was git://
00:13 wizzycray       you might try it the other way? it's probably nothing but you mentioned your firewall
00:13 mtompset        Xenial worked. *whew*
00:13 mtompset        Just painful long.
00:13 mtompset        And now...
00:13 mtompset        sorry, mtj, same failure.
00:14 mtompset        what command line are you using on your kohadevbox?
00:15 mtj             default
00:16 mtompset        $ default
00:16 mtompset        ?!
00:16 mtompset        BTW...
00:16 mtompset        $ sudo koha-shell -c "prove t/db_dependent/00-strict.t" kohadev
00:16 mtompset        failed on my xenial.
00:16 mtompset        SAME error.
00:17 mtj             paste error pls
00:17 mtj             hmm, ok
00:18 pastebot        "mtompset" at pasted "I commented all the tests except for admin and acqui to shorten the run." (17 lines) at http://paste.koha-community.org/305
00:18 mtompset        you must have some TAP harness hack implemented.
00:18 wizzycray       can you prove -v it might tell you more
00:18 mtompset        wizzycray: No need...
00:19 wizzycray       well ok then
00:19 mtompset        It's the SAME error.
00:19 mtompset        acqui runs 1-43.
00:19 mtompset        then admn starts running 1-X
00:19 mtompset        and the tap parsing barfs because the counts don't line up.
00:21 mtompset        mtj...
00:21 mtompset        I'd like to ask you to try a test for me.
00:21 mtompset        find a file... any file... that 00-strict.t would look at.
00:21 mtj             'you must have some TAP harness hack implemented.'
00:21 mtompset        and add "make_this_blow_up()" into the file.
00:21 mtj             ^ ..no, it works on a fresh kohadevbox
00:22 mtompset        Well, my jessie will be fresh in a couple hours.
00:22 mtompset        I'm 1/3 through the clone.
00:22 mtj             vagrant destroy ; vagrant provision
00:22 mtj             works on that, for me ^
00:23 mtompset        Which reminds me...
00:23 mtompset        I should submit my patch.
00:23 mtompset        I figured out the cause of the Test::DBIx::Class failures.
00:23 mtompset        recursive meeting of dependencies is poorly implemented.
00:24 mtompset        just instlal DBD::SQLite and the Test::DBIx::Class installs smoothly.
00:24 mtompset        Though, the misc2dev copying is in the wrong place now.
00:25 mtompset        had to comment that out.
00:25 mtompset        I'll need to manually run it later.
00:45 mtj             oops, sorry ... i need to swap to some other stuff now mtompset
00:45 mtompset        No problem.
00:45 mtj             thanks for testing the patch
00:46 mtj             ..i guess my only suggestion would be to test on a pristine kohadevbox
00:47 mtj             its a really curious bug :)
00:49 mtj             the threads doco looks to have a good workaround.. ->  http://perldoc.perl.org/threads.html
00:50 mtj             my $can_use_threads = eval 'use threads; 1';
00:50 mtj             ..a test for threads ^
00:51 mtj             if that fails, then dont use threads :)
00:57 * kidclamp      waves
01:01 mtj             hiya kidclamp
01:02 kidclamp        how goes it mtj?
01:02 mtj             hum, i realise that fix wont work for your situ mtompset :(
01:03 mtj             ..threads seem to run just fine, its the order of the output thats causing TAP problems
01:03 mtj             pretty good here kidclamp, thanks :)
01:03 mtj             and you?
01:04 kidclamp        not too shabby
01:07 mtompset        mtj: No, it isn't the order. It's the count.
01:07 mtompset        ok x, ok x+1, ok x+2... the okay's need to be sequential and increasing.
01:08 mtompset        but acqui and admin are started at 1.
01:08 mtompset        I was thinking what if it were all files...
01:09 mtompset        the problem is ok 1 ok 1 ok 1 ok 1...
01:09 mtompset        still not increasing.
01:11 mtompset        now... all_perl_files_ok is the same as strict_ok, warnings_ok, and syntax_ok... you could thread those into subtests and it would work.
01:15 mtj             yes, either way worked fine on my systems
01:16 mtj             but i think you'll hit the same problem
01:16 mtompset        probably.
01:28 mtj             i'll add some other thread-ish tests - perhaps they will show why its failing
01:48 Harold          Hello, everyone
01:48 Harold          Nice to see you here
01:48 Harold          is there someone who cpuld help me please?
01:49 Harold          i'm facing a problem trying to add users to koha 16.11
01:49 Harold          both staff client and OPAC
01:50 Harold          first, when I get into Patrons module, intranet-error.log is added with a new entry: [Mon Feb 06 21:39:54.522722 2017] [cgi:error] [pid 2538] [client] AH01215: [Mon Feb  6 21:39:54 2017] members-home.pl: No owner passed in or defined! at /usr/share/koha/intranet/cgi-bin/members/members-home.pl line 70.: /usr/share/koha/intranet/cgi-bin/members/members-home.pl, referer:
01:50 Harold          http://biblioteca-admin/cgi-bin/koha/mainpage.pl
01:51 wizzycray       hi Harold, are you logged in with the database user?
01:51 Harold          I've reviewed previows chat logs, and it says that it is because I'm using database user. And yes, I am. So I'm trying to create a new user, and I face these errors
01:52 wizzycray       right ok. Do you have a patron category set up?
01:53 Harold          Then, when I get into add new patron form, I get all this: [Mon Feb 06 21:52:42.529541 2017] [cgi:error] [pid 2574] [client] AH01215: [Mon Feb  6 21:52:42 2017] memberentry.pl: DBD::mysql::st execute failed: Expression #1 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'koha_biblioteca.authorised_values.id' which is not functionally dependent on
01:53 Harold          columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by [for Statement ": /usr/share/koha/intranet/cgi-bin/members/memberentry.pl, referer: http://biblioteca-admin/cgi-bin/koha/members/members-home.pl
01:53 Harold          [Mon Feb 06 21:52:42.529720 2017] [cgi:error] [pid 2574] [client] AH01215: [Mon Feb  6 21:52:42 2017] memberentry.pl:         SELECT *: /usr/share/koha/intranet/cgi-bin/members/memberentry.pl, referer: http://biblioteca-admin/cgi-bin/koha/members/members-home.pl
01:53 Harold          [Mon Feb 06 21:52:42.529806 2017] [cgi:error] [pid 2574] [client] AH01215: [Mon Feb  6 21:52:42 2017] memberentry.pl:         FROM authorised_values: /usr/share/koha/intranet/cgi-bin/members/memberentry.pl, referer: http://biblioteca-admin/cgi-bin/koha/members/members-home.pl
01:53 Harold          [Mon Feb 06 21:52:42.529890 2017] [cgi:error] [pid 2574] [client] AH01215: [Mon Feb  6 21:52:42 2017] memberentry.pl:     : /usr/share/koha/intranet/cgi-bin/members/memberentry.pl, referer: http://biblioteca-admin/cgi-bin/koha/members/members-home.pl
01:54 wizzycray       Harold: did you install any of the sample data/categories at all?
01:54 Harold          ys, I do have categories on it
01:54 Harold          Staff is the one I want to add my user to
01:54 wizzycray       are you using ubuntu 16, with mysql 5.7?
01:55 Harold          yes
01:55 wizzycray       that's why
01:55 Harold          oops
01:55 Harold          how can be it?
01:55 wizzycray       https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17258
01:55 huginn          04Bug 17258: major, P5 - low, ---, gmcharlt, NEW , [OMIBUS] MySQL 5.7
01:55 wizzycray       ^ see all those bugs.
01:55 wizzycray       that's why ;)
01:55 Harold          ok... I'll check
01:56 wizzycray       there's a work around https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17258#c6
01:56 huginn          04Bug 17258: major, P5 - low, ---, gmcharlt, NEW , [OMIBUS] MySQL 5.7
01:57 Harold          looks like Maria DB is the one I must use
01:58 wizzycray       you could do that, or you could do the work around which edits the mysql config
01:58 Harold          Thank you so much!
01:58 wizzycray       you're welcome, good luck!
01:58 Harold          really?
01:58 wahanui         really is more trouble than she's worth.
01:58 Harold          how could I?
01:58 mtompset        If you are willing, you could change your config.
01:59 mtompset        https://wiki.koha-community.org/w/index.php?title=Koha_on_ubuntu_-_packages#Ubuntu_16.04_and_MySQL_5.7
02:00 mtompset        Speaking ubuntu and mysql 5.7 -- bug 17260
02:00 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17260 major, P5 - low, ---, philippe.blouin, Needs Signoff , updatedatabase.pl fails on invalid entries in ENUM and BOOLEAN columns
02:05 Harold          OK... changed mysqld.cnf
02:05 Harold          restarted server
02:05 Harold          no longer that looong message at opening user add form
02:06 Harold          but still error 500 at save button in the form
02:06 Harold          same error than before on log file:
02:06 Harold          [Mon Feb 06 22:04:33.420411 2017] [cgi:error] [pid 2496] [client] End of script output before headers: memberentry.pl, referer: http://biblioteca-admin/cgi-bin/koha/members/memberentry.pl?op=add&categorycode=S
02:06 Harold          same problem in user self register on OPAC
02:07 Harold          and I find no way to debug and find the error source by myself :(
02:12 mtompset        Harold, if you don't have data yet, drop the database, create it, and run the web install again.
02:13 mtompset        if you do have data, sadly, I am going for the night.
02:13 Harold          Yes... will do
02:13 Harold          thanks!
02:13 mtompset        Have a good day (24 hour period), #koha Harold wizzycray mtj
03:45 bag             hello
03:45 wahanui         hello, bag
05:50 Harold          Updates about my issue on add patron giving error:
05:50 Harold          I installed everithing again, but with MariaDB instead of MySQL
05:50 Harold          same error...
06:30 * cait          waves
06:36 reiveune        hello
07:07 LibraryClaire   morning #koha
07:09 drojf           hi LibraryClaire
07:09 LibraryClaire   moin drojf
07:19 alex_a          bonjour
07:19 wahanui         kia ora, alex_a
07:20 LibraryClaire   salut alex_a
07:26 alex_a          hello LibraryClaire
07:29 Joubu           hello
07:29 LibraryClaire   hey joubu
07:30 * cait          waves bbiab
08:09 * magnuse       waves
08:10 magnuse         @wunder enbo
08:10 huginn          magnuse: The current temperature in Bodo, Norway is -1.0°C (8:50 AM CET on February 07, 2017). Conditions: Clear. Humidity: 47%. Dew Point: -11.0°C. Windchill: -8.0°C. Pressure: 30.57 in 1035 hPa (Steady).
08:10 magnuse         the sun is shining in Lofoten!
08:16 LibraryClaire   hei magnuse
08:16 LibraryClaire   @wunder konstanz
08:16 huginn          LibraryClaire: The current temperature in Mainaustraße, Konstanz, Germany is 5.8°C (9:13 AM CET on February 07, 2017). Conditions: Mostly Cloudy. Humidity: 85%. Dew Point: 3.0°C. Windchill: 6.0°C. Pressure: 30.04 in 1017 hPa (Falling).
08:19 * cait          waves
08:20 cait            magnuse: on the Lofoten again? :)
08:20 magnuse         i'm not there, but i can see it
08:27 fridolin        hie there
08:31 LibraryClaire   hey fridolin
08:31 wahanui         i heard fridolin was busy at the moment, I asked him to backport the bug fix
08:32 fridolin        LibraryClaire: hie watsup ?
08:32 fridolin        wahanui: go to bed
08:32 wahanui         http://cdn4.list25.com/wp-content/uploads/2012/04/penguin.gif
08:33 LibraryClaire   lol
08:34 LibraryClaire   fridolin: not a lot ;) work work
08:34 fridolin        ;)
09:17 eythian         hi
09:17 wahanui         niihau, eythian
09:17 eythian         @wunder ams
09:17 huginn          eythian: The current temperature in Schiphol, Badhoevedorp, Netherlands is 2.7°C (10:10 AM CET on February 07, 2017). Conditions: Drizzle. Humidity: 90%. Dew Point: 1.0°C. Windchill: 1.0°C. Pressure: 30.02 in 1017 hPa (Rising).
09:18 eythian         fo shizzle it's drizzle
10:34 lds             hello Joubu
10:34 Joubu           ;)
10:43 magnuse         @wunder enbo
10:43 huginn          magnuse: The current temperature in Bodo, Norway is -1.0°C (11:20 AM CET on February 07, 2017). Conditions: Clear. Humidity: 44%. Dew Point: -12.0°C. Windchill: -9.0°C. Pressure: 30.60 in 1036 hPa (Steady).
12:25 AndrewIsh       hi #koha!
12:25 AndrewIsh       i'm looking for some help with devbox
12:26 AndrewIsh       i've got it set up on my local machine and it's fine. i'm now trying to get it running on a headless server. i'm running into a problem while trying to bring it up for the first time
12:26 AndrewIsh       it gets to "==> jessie: Waiting for machine to boot. This may take a few minutes..."
12:27 AndrewIsh       then i get the following: "The guest machine entered an invalid state while waiting for it to boot. Valid states are 'starting, running'. The machine is in the 'poweroff' state. Please verify everything is configured properly and try again."
12:27 AndrewIsh       has anyone else seen this?
12:27 AndrewIsh       i checked the irc logs and the only reference i could find was someone trying to run it on a 32 bit machine, my server is definintely not 32 bit. so i'm a bit stumped
12:31 Harold          good day, #koha
12:34 tcohen          morning
12:35 AndrewIsh       tcohen:  good morning, the very person i was hoping would appear! :)
12:35 AndrewIsh       i have a devbox problem ;-0
12:41 oleonard        Hi all
12:41 cait            hi AndrewIsh and oleonard
12:42 AndrewIsh       hi cait
12:43 * magnuse       waves again
12:44 * LibraryClaire waves
12:45 tcohen          hi AndrewIsh
12:45 * eythian       makes waves
12:45 tcohen          hey eythian
12:45 wahanui         go back to bed, eythian
12:45 AndrewIsh       hi tcohen
12:45 tcohen          whats up
12:46 AndrewIsh       tcohen:  to save me creating more noise in the channel, have a look here: http://irc.koha-community.org/koha/2017-02-07#i_1904237
12:46 * ashimema      seeks out ☕
12:46 AndrewIsh       it's a really weird one, i'm sure there's a simple explanation though
12:47 eythian         AndrewIsh: is your vm set to be 64 bit?
12:47 eythian         not just your actual hardware
12:47 AndrewIsh       eythian:  oooh, good question, i've not checked that. is that in the devbox config?
12:47 eythian         I dunno
12:47 eythian         I'm just making noises :)
12:47 AndrewIsh       noises are good, you never know, they could be the correct noises ;-)
12:47 tcohen          AndrewIsh: the devbox uses the debian/jessie64 image, which is 64 bits
12:48 AndrewIsh       ok
12:48 tcohen          you need to check if your machine supports 64 bit guests
12:48 tcohen          i.e. if virtualbox supports 64 bit guests
12:48 eythian         yes, but if your vm config is trying to virtualise a 32 bit machine, it still will fail
12:48 tcohen          hi nengard
12:49 nengard         howdy :)
12:49 AndrewIsh       tcohen:  are there any logs i could look at? i've not been able to find any
12:50 tcohen          "Start VirtualBox, select the virtual machine where you want to run your 64-Bit OS and go into the settings of this VM. Look for "Settings -> System -> Acceleration" and make sure that "Enable VT-x/AMD-V" is activated."
12:50 tcohen          that's a way to check
12:51 AndrewIsh       it's a headless server, i don't have a ui in which to do that :(
12:52 AndrewIsh       i'm sure there must be a way of doing that headless, i'll check
12:53 eythian         you can do everything headless in vbox. It's not fun, but is possible.
12:54 eythian         you can also probably have it present a vpn or similar endpoint which you could connect to to see what's going on
12:54 tcohen          VBoxManage is the command
12:55 AndrewIsh       tcohen:  indeed, it *appears* to start, but doesn't:
12:55 AndrewIsh       andrew@aidev:~/kohadevbox (master *)$ VBoxManage startvm kohadevbox_jessie_1486469724673_38409 --type headless
12:55 AndrewIsh       Waiting for VM "kohadevbox_jessie_1486469724673_38409" to power on...
12:55 AndrewIsh       VM "kohadevbox_jessie_1486469724673_38409" has been successfully started.
12:55 AndrewIsh       andrew@aidev:~/kohadevbox (master *)$ VBoxManage controlvm kohadevbox_jessie_1486469724673_38409 poweroff
12:55 AndrewIsh       VBoxManage: error: Invalid machine state: PoweredOff (must be Running, Paused or Stuck)
12:55 AndrewIsh       strange...
12:56 tcohen          all indicates that you are trying to boot the 64 bit image on an unsupported platform
12:56 tcohen          what is the processor you have?
12:56 eythian         use vrdp
12:56 AndrewIsh       it's a qemu vm
12:57 eythian         what
12:57 eythian         you're trying to run vbox inside qemu?
12:57 tcohen          a dream inside a dream
12:57 eythian         you're putting a bag of holding into a bag of holding, man
12:57 AndrewIsh       it's all very inception, i know, but i thought it might work...
12:58 eythian         I wouldn't be surprised if it doesn't
12:58 eythian         however, vrdp will give you a screen
12:58 tcohen          qemu probably doesn't provide what Vbox needs to run
12:58 AndrewIsh       mmmm, that would certainly appear to be the case
12:59 tcohen          cat /proc/cpuinfo | grep -e lm -e vmx -e svm
12:59 eythian         I just don't think you can nest virtualisation like that
12:59 eythian         use lxc or something instead
12:59 AndrewIsh       flags           : fpu de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pse36 clflush mmx fxsr sse sse2 syscall nx lm rep_good nopl pni cx16 popcnt hypervisor lahf_lm
12:59 * ashimema      would suggest you try using the ansible playbooks from the vm to install directly instead of using virtualbox and vagrnt?
12:59 eythian         or see what's going on using vrdp
13:00 tcohen          I agree with ashimema
13:00 tcohen          the playbook is designed to be ran without vagrant
13:00 AndrewIsh       eythian:  i'll see if vrdp can shed any light, one tic
13:00 tcohen          AndrewIsh: you don't  have vmx or svm, so it won't work with the 64 bit image
13:01 tcohen          you can try changing debian/jessie64 for debian/jessie32 in the Vagrant file
13:01 tcohen          but ashimema is right about using the playbook directly
13:01 * ashimema      would like to point out he doesn't know how to do this.. but would like to lear ;)
13:01 AndrewIsh       i'm happy to give ashimema's suggestion a go, not sure what it entails though
13:01 tcohen          ashimema: you need to create an inventory file
13:01 tcohen          like
13:01 AndrewIsh       my ansible/vagrant knowledge isn't good
13:02 tcohen          echo "IP_OF_QEMU_MACHINE"
13:02 tcohen          > inventory.qemu
13:02 tcohen          and then
13:02 tcohen          run ansible-playbook -i inventory.qemu site.yml
13:03 tcohen          there might be some caveats
13:03 tcohen          because some functionalities from the playbook are controlled by env variables
13:03 tcohen          so instead of ENV you need to set them in vars/user.yml
13:03 tcohen          he
13:04 ashimema        interesting
13:04 wahanui         interesting is sometimes good and sometimes bad
13:04 * ashimema      wishes he had mroe time to play with ansible
13:04 AndrewIsh       i just tried it, i'll paste the output
13:05 pastebot        "AndrewIsh" at pasted "andrew@aidev:~/kohadevbox (mas" (10 lines) at http://paste.koha-community.org/306
13:05 AndrewIsh       it's clear what the problem is, but i'm not sure what's trying to ssh and where to get it's public key from
13:06 * AndrewIsh     realised he just pasted the ip address of his dev server, ignore that everyone...
13:06 ashimema        you should run the playbook from your local machine I believe..
13:06 AndrewIsh       ashimema:  oh really?
13:06 wahanui         i heard really was more trouble than she's worth.
13:06 marcelr         hi #koha
13:06 ashimema        ansible will then connect to the remote using the IP you gave in your inventory.qemu?
13:07 ashimema        I'm assuming your qemu isn't actually a local qemu.. it's te remote vm from BigV ;)?
13:07 AndrewIsh       yes, it must be
13:07 ashimema        so yeah..
13:07 AndrewIsh       ah, ok, i'll try that
13:08 ashimema        the playbook is simply a set of commands to run in sequence on a box somewhere via ssh at it's most basic (as far as I understand)
13:08 ashimema        so.. in the kohadevbox sense.. we're running up a vagrant based virtual machine then ansible connects to it and installs everything we need for us
13:09 ashimema        in your case.. you don't need the vagrant virtualbox server as you already have a remote debian server (that happens to be in a remote qemu)
13:10 * AndrewIsh     tries to parse what ashimema just wrote...
13:10 ashimema        feel free to slack call me ;)
13:10 AndrewIsh       ta
13:31 * kidclamp      waves
13:34 * LibraryClaire waves at kidclamp
13:34 kidclamp        hi LibraryClaire
13:34 tcohen          hi kidclamp LibraryClaire
13:34 LibraryClaire   hey tcohen
13:34 kidclamp        morning tcohen
13:36 tcohen          if anyone has some time, I would like someone to test bug 13757
13:36 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13757 enhancement, P5 - low, ---, tomascohen, Signed Off , Make patron attributes editable in the opac if set to 'editable in OPAC'
13:36 tcohen          it is a big refactoring work
13:36 tcohen          on a simple feature
13:36 tcohen          it is easy to test
13:36 tcohen          i think I covered all edge cases
14:09 magnuse         @wunder enbo
14:09 huginn          magnuse: The current temperature in Bodo, Norway is -1.0°C (2:50 PM CET on February 07, 2017). Conditions: Clear. Humidity: 40%. Dew Point: -13.0°C. Windchill: -8.0°C. Pressure: 30.66 in 1038 hPa (Rising).
14:18 ebegin          Hi everybody !
14:19 ebegin          Just showing the IRC channel during a training session ! :)
14:19 kellym          Hi !
14:19 ebegin          Cheers !
14:42 ashimema        hello
14:42 wahanui         bonjour, ashimema
14:57 pastebot        "marcelr" at pasted "kohadevbox" (5 lines) at http://paste.koha-community.org/307
14:58 marcelr         tcohen, Joubu:  http://paste.koha-community.org/307
15:03 Joubu           hum?
15:03 Joubu           ha
15:03 marcelr         chmod 755 on the script ?
15:04 Joubu           marcelr: but does it try to execute cp_debian_files.pl before cloning misc4dev?
15:05 marcelr         no but i tried chmod between two runs
15:05 marcelr         and got stuck
15:07 Joubu           marcelr: shell: perl misc4dev/cp_debian_files.pl
15:07 Joubu           you should not need x
15:08 marcelr         yeah but this is something from ansible
15:08 marcelr         in the kohadevbox stuff
15:08 marcelr         i wanted to try kohadevbox, but my enthusiasm is diminishing when running provision run #381
15:08 marcelr         :)
15:11 Joubu           I'd suggest to replace misc4dev/cp_debian_files.pl with the full path
15:11 Joubu           {{ vagrant_home_dir }}/misc4dev/cp_debian_files.pl
15:12 Joubu           tcohen: do you confirm? ^
15:17 tcohen          hi
15:17 wahanui         hello, tcohen
15:17 tcohen          en la carpeta kohadevbox
15:17 tcohen          Joubu: yes
15:17 tcohen          that makes sense
15:17 cait            tuesday--
15:18 * tcohen        goes destroy his box and tries
15:19 tcohen          Joubu, marcelr: are you sending a pull request?
15:19 marcelr         no sir
15:19 tcohen          lazy
15:20 marcelr         as always
15:20 marcelr         but i have a rambling? kohadevbox
15:20 Joubu           nope, I can but later
15:21 tcohen          marcelr give me a minute
15:21 tcohen          and git pull
15:21 wahanui         git pull is really simple.
15:21 tcohen          he
15:22 marcelr         git pull time
15:22 tcohen          one more minute ;:-_P
15:45 reiveune        bye
15:55 tcohen          mtompset++ # taking the time debug the annoying Test::DBIx::Class build failure
15:57 magnuse         mtompset++
15:57 * magnuse       wanders off to make pancakes
16:01 * LibraryClaire invites herself over to magnuse's for dinner again
16:09 LibraryClaire   laters #koga
16:09 LibraryClaire   hm
16:09 LibraryClaire   also koha...
16:23 huginn          New commit(s) kohagit: Bug 18033: Remove duplicate code in paycollect.pl <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=0b2393cd655eb43e2d9893b40fb48abf1e786618>
16:58 CrispyBran      https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18075
16:58 huginn          04Bug 18075: critical, P5 - low, ---, koha-bugs, NEW , Hold modal buttons do not work at destination after transfer
17:23 huginn          New commit(s) kohagit: Bug 17255 - Upgrade Elastic Search code to work with version 5.1 <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=4f178fad53fb748bd1ccf67680e02411918c9584> / Bug 17255 - Upgrade Elastic Search code to work with version 2.4+ - rebased wip <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=d1077b5ae6aa5c375ef91e6150a7d39b60513325>
17:53 huginn          New commit(s) kohagit: Bug 17929 - You can't edit indicators in the cataloging screen <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=18529d7cc5a6334e45d977da9ddfe5429f402c2b> / Bug 17922: Use correct number of digits when replacing date placeholders <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=95e94766af653de4bc721af64981140cacecf567> / Bug 18015 - On shelf holds allowed > "If all unavailable" ignores notforl
18:03 huginn          New commit(s) kohagit: Bug 17927: (QA followup) Fix timestamp nullable in hold.json <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=2f64fd3918a205ac21630657160f61200c791320> / Bug 17927: (QA followup) Fix boolean types <http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=105b82b8461786ac6695e90aa8a2c1d4c36b4190> / Bug 17927: (followup) Fix /cities cityid type <http://git.koha-community.org/gitweb/?p=koha.git;a=commitd
18:07 jenkins         Project Koha_Master_D8 build #105: STILL UNSTABLE in 1 hr 39 min: https://jenkins.koha-community.org/job/Koha_Master_D8/105/
18:07 jenkins         kyle: Bug 18033: Remove duplicate code in paycollect.pl
18:07 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18033 enhancement, P5 - low, ---, josef.moravec, Pushed to Master , If/else sometimes does not make sense after koha account  system refactoring
19:29 tcohen          so, budgets tables are called aqbudgetperiods
19:29 tcohen          and funds, aqbudgets
19:30 oleonard        tcohen: Yeah we didn't have the terminology pinned down when the db structure was added :|
19:30 tcohen          =D
19:37 cait            yeah
19:37 cait            basically it was switched later for the interface
19:37 cait            but not in the code
19:38 tcohen          yeah, and we started with budgets, and then generalized
19:38 tcohen          budgets > funds but kept the names
19:38 rangi           i stil think those are backwards
19:39 tcohen          ?
19:39 rangi           you should have a fund
19:39 rangi           and you define budgets for it
19:39 rangi           not the other way round
19:39 rangi           but that would be a massive change
19:39 * tcohen        won't discuss terminology as it is not his best
19:39 rangi           one day, when i win lotto and retire, i will build a whole new acquisitions system
19:40 tcohen          hehe
19:40 * oleonard      buys rangi a lottery ticket
19:40 * tcohen        too
19:40 * ibeardslee    buys one with plans to donate some winnings to rangi
19:41 oleonard        "some" :)
19:41 * aleisha       has no money but offers support
19:41 ibeardslee      students eh?
19:41 aleisha         you know how it is
19:41 tcohen          rangi, according to google budget is well defined
19:42 tcohen          'an estimate of income and expediture set for a period of time'
19:42 rangi           yeah
19:42 tcohen          i think it is fund that doesn't fit
19:42 rangi           so you have a fund
19:42 rangi           Fiction books
19:42 rangi           say
19:42 rangi           and you define a budget for 2017
19:42 rangi           for it
19:42 rangi           thats how most accounting systems work
19:43 tcohen          hm
19:43 rangi           then yuo can see how much you have spent over the years etc
19:43 rangi           and you can roll over
19:43 tcohen          we might need a third category
19:43 aleisha         that makes way more sense
19:43 aleisha         our acq module is all backwards
19:43 jenkins         Project Koha_Master_D8 build #106: STILL UNSTABLE in 1 hr 35 min: https://jenkins.koha-community.org/job/Koha_Master_D8/106/
19:43 jenkins         * kyle: Bug 17255 - Upgrade Elastic Search code to work with version 2.4+ -
19:43 jenkins         * kyle: Bug 17255 - Upgrade Elastic Search code to work with version 5.1
19:43 jenkins         * kyle: Bug 15030 - Certain values in serials' items are lost on next edit
19:43 jenkins         * kyle: Bug 15030 - Fixes the serials fields associated with a plugin, to not
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17255 enhancement, P5 - low, ---, olli-antti.kivilahti, Pushed to Master , Upgrade Elastic Search code to work with version 5.1
19:43 jenkins         * kyle: Bug 15030 - continue. The enumchron value was overwritten by 'header'
19:43 jenkins         * kyle: Bug 15030: Add tests
19:43 jenkins         * kyle: Bug 17775 - Add new user with LDAP not works under Plack
19:43 jenkins         * kyle: Bug 18044: Label Batches not displaying
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15030 blocker, P5 - low, ---, philippe.blouin, Pushed to Master , Certain values in serials' items are lost on next edit
19:43 jenkins         * kyle: Bug 18044: Add a test
19:43 jenkins         * kyle: Bug 18013 - acqui/transferorder.pl typo in find method
19:43 jenkins         * kyle: Bug 18005: Re-styled pagination on search results with Bootstrap
19:43 jenkins         * kyle: Bug 17871: Remove zebra::snippet to allow access to facets in YAZ 5.8.1+
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17775 blocker, P5 - low, ---, liz, Pushed to Master , Add new user with LDAP not works under Plack
19:43 jenkins         * kyle: Bug 17871: (followup) Remove zebra::* from the packages templates
19:43 jenkins         * kyle: Bug 18015 - On shelf holds allowed > "If all unavailable" ignores
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18044 critical, P2, ---, cnighswonger, Pushed to Master , Label Batches not displaying
19:43 jenkins         * kyle: Bug 17922: Use correct number of digits when replacing date placeholders
19:43 jenkins         * kyle: Bug 17929 - You can't edit indicators in the cataloging screen
19:43 jenkins         * kyle: Bug 16387: Fix default shortened loan period time
19:43 jenkins         * kyle: Bug 17782 - Patron updated_on field should be set to current timestamp
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18013 critical, P5 - low, ---, dpavlin, Pushed to Master , acqui/transferorder.pl typo in find method
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18005 major, P5 - low, ---, josef.moravec, Pushed to Master , Pagination of the search result displayed wrong
19:43 jenkins         * kyle: Bug 17782 - (QA Followup)
19:43 jenkins         * kyle: Bug 17782: Fix tests
19:43 jenkins         * kyle: Bug 17927: Fix /holds and /patrons data types
19:43 jenkins         * kyle: Bug 17927: (followup) Fix /cities cityid type
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17871 major, P5 - low, ---, dcook, Pushed to Master , Can't retrieve facets (or zebra::snippet) from Zebra with YAZ 5.8.1
19:43 jenkins         * kyle: Bug 17927: (QA followup) Fix boolean types
19:43 jenkins         * kyle: Bug 17927: (QA followup) Fix timestamp nullable in hold.json
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 major, P5 - low, ---, mirko, Pushed to Master , On shelf holds allowed > "If all unavailable" ignores notforloan
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17922 major, P5 - low, ---, julian.maurice, Pushed to Master , Default value substitution for month and day should be fixed length
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17929 major, P5 - low, ---, josef.moravec, Pushed to Master , You can't edit indicators in the cataloging screen
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16387 major, P5 - low, ---, lari.taskula, Pushed to Master , Incorrect loan period calculation when using  decreaseLoanHighHolds feature
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17782 major, P5 - low, ---, nick, Pushed to Master , Patron updated_on field should be set to current timestamp when borrower is deleted
19:43 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927 normal, P5 - low, ---, tomascohen, Pushed to Master , REST API: /holds and /patrons attributes have wrong types
19:44 rangi           jeez bots, get a room
19:44 tcohen          rangi: jenkins is doing its best
19:44 aleisha         im glad bug 18044 went through
19:44 aleisha         that was messing me up yesterday
19:45 * tcohen        is happy 17255 is in, and we can now move to making it even better
19:46 tcohen          can anyone take a look at 17932? :-D
19:46 rangi           yep, and the stuff i was doing for search overdrive and stuff can get submitted soon
19:47 tcohen          take a look == sign it
19:47 rangi           (it pulls in a libraries overdrive collection into their elasticsearch index, so can search over both easily)
19:47 tcohen          rangi: awesome!
19:47 wahanui         That'll be $1 for the awesome jar, tcohen
19:47 tcohen          oops
19:47 tcohen          need to pick Manuel soon!
19:47 tcohen          bye!
20:06 * cait          waves to alexbuckley_
20:07 alexbuckley_    Hi cait
20:07 oleonard        rangi: that overdrive feature sounds really useful
20:07 cait            we need to do more ES testing, it gets cooler the minute :)
20:09 rangi           yeah, working on recordedbooks too, combine them with the patches that are already there that allow users to circulate from koha
20:09 rangi           then your users can search and borrow stuff from overdrive without leaving koha
20:10 rangi           should be super handy
20:14 oleonard        Bye all
20:17 eythian         rangi: Dave from HZ is visiting me later in the month
20:18 eythian         rangi: surely you need to consult with marcelr some time :)
20:21 rangi           :) ill float that idea past him
20:26 eythian         I have a couch bed should catalyst want to do it cheaper...
20:30 rangi           :)
21:29 cait            ah no tcohen
21:52 cait            hm it looks like i can't place holds in my 16.11.x branch
21:52 cait            plack error log shows: Template process failed: undef error - WHILE loop terminated (> 1000 iterations)
21:52 cait            when I search for a patron to place a hold on, then internal server error...
21:53 wizzycray       I feel like I have seen this
21:54 wizzycray       oh wait I have
21:54 cait            i have gone backon my branch, couldn't find a 'good' spot
21:54 wizzycray       it is something to do with....
21:54 wizzycray       oh damn I looked at this just last week
21:54 cait            so I am thinking it might be a config difference somehow
21:55 wizzycray       this is staff client right
21:55 cait            yep
21:56 tcohen          hi
21:57 cait            hi tcohen
21:57 cait            have you seen this by chance:
21:57 cait            (22:52:22) cait: plack error log shows: Template process failed: undef error - WHILE loop terminated (> 1000 iterations)
21:57 cait            (22:52:22) cait:  when I search for a patron to place a hold on, then internal server error...
21:57 cait            in staff
21:58 cait            also - bug 17927 - backport?
21:58 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17927 normal, P5 - low, ---, tomascohen, Pushed to Master , REST API: /holds and /patrons attributes have wrong types
21:58 cait            i have been able to test patrons - but not holds
21:58 wizzycray       yeah cait I"m getting that too
21:58 cait            not getting the path right maybe
21:58 cait            wizzyrea: on master?
21:58 cait            ugh
21:58 wizzycray       in 16.11.03
21:59 cait            but not in master?
21:59 cait            i went back about 100 commits
21:59 cait            still seeing it
21:59 wizzycray       idk I need to check that
21:59 cait            so i think not something i pushed recently hm
22:00 wizzycray       Template process failed: undef error - WHILE loop terminated (> 1000 iterations)
22:00 tcohen          cait: template cache?
22:00 cait            not set
22:00 cait            or what do you mean?
22:00 wizzycray       yes, explain :P
22:00 wizzycray       please
22:00 tcohen          you went back and the problem remains
22:01 wizzycray       I am surprised that no one is screaming about this bug, actually
22:02 tcohen          so, on the patron search when placing a hold?
22:02 cait            tcohen: i tihnk i did yes - about 100 commits back, i wanted to try a git bisect, but couldn't find a good commit
22:02 cait            yeah, look for a record with an item
22:02 cait            go tot he holds tab
22:02 cait            search for a patron - auto-suggest works ok
22:02 cait            when you send the search or click one of the suggested - internal server error
22:03 cait            and the error i posted earlier about the loop
22:03 tcohen          cannot reproduce
22:03 wahanui         cannot reproduce is https://xkcd.com/583/
22:03 cait            i wonder what it could be
22:04 wizzycray       I see the same thing
22:04 cait            no clue where to start searching even
22:04 cait            there is nothing in the intranet error logs
22:04 wizzycray       the config for this one where I see it is 16.11 native
22:04 tcohen          https://snag.gy/fBvdnI.jpg
22:05 tcohen          i'm on master
22:06 tcohen          cait, the REST api is WIP in general
22:06 tcohen          on 16.11 it has the wrong data types
22:06 cait            actually, i had that happen
22:06 cait            can you try adding a circ rule that will allow you to place on hold?
22:06 tcohen          it is your call, but I'd backport the type fixes
22:07 cait            ugh
22:07 cait            wizzyrea: try changing holds per record count
22:07 cait            i set it to 0 and the error went away
22:07 wizzycray       !
22:07 cait            tcohen: could you try setting the setting to something like 99?
22:08 cait            to see if it breaks then?
22:08 wizzycray       oh, this lib has 999 holds allowed
22:08 wizzycray       per record.
22:08 wizzycray       which is not 1000 iterations but close to.
22:09 tcohen          still works
22:09 tcohen          with 99
22:09 cait            999
22:09 wizzycray       I changed mine from 999 to 10 and now it is working
22:09 cait            kills it for me
22:09 wizzycray       999 kills it for me too
22:09 cait            cool
22:09 cait            so we got that 'solved'
22:09 cait            i will file a bug
22:09 cait            thx all for helping :)
22:10 wizzycray       w. t. f. ;)
22:10 cait            so it's not something i pushed at least heh
22:10 tcohen          https://snag.gy/1TMVy5.jpg
22:10 tcohen          999 killed it
22:10 cait            it fits today
22:10 cait            today was a day of odd things
22:11 tcohen          question is why the template has such a big loop
22:11 tcohen          long
22:12 cait            yeah, certainly something strange there
22:12 cait            but at least we have a step by step to break things onw
22:12 cait            tcohen: can you give me an example of how to test the holds rest api?
22:12 cait            i am not getting it right i think
22:12 cait            this looks http://localhost:8081/api/v1/patrons/31 good
22:13 wizzycray       if I had to guess
22:13 wizzycray       it's the multiple holds per title part
22:13 cait            http://localhost:8081/api/v1/patrons too, but /holds seems not to work
22:15 tcohen          cait: does it work without the patch?
22:16 cait            so it should work?
22:16 cait            will try in a sec, just finishing the bug
22:16 tcohen          maybe you shouldn't backport it
22:16 tcohen          because it fixes the types
22:17 tcohen          which doesn't mean it will work
22:17 tcohen          because we have a situation with integers and booleans
22:17 cait            bug 18076
22:17 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18076 normal, P5 - low, ---, koha-bugs, NEW , Plack error when placing a hold and holds per record is set to 999
22:17 tcohen          that requires TO_JSON
22:17 tcohen          bug 17932
22:17 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17932 enhancement, P5 - low, ---, tomascohen, Signed Off , Koha::Object should provide a TO_JSON method
22:18 cait            yeha
22:18 cait            it works without hte patches
22:18 cait            hm
22:18 cait            wonder if the error before affected it
22:18 cait            sec
22:18 tcohen          cait, it shouldn't work with the patch
22:19 cait            nope
22:19 cait            it breaks
22:19 tcohen          yeap
22:19 cait            ok, so not backporting
22:19 tcohen          it changes
22:19 tcohen          suspend and lowestPriority
22:19 tcohen          into booleans
22:19 cait            left a note
22:20 tcohen          which is not handled without 17932
22:20 * tcohen        hopes someone QAes it
22:20 cait            hm so broken on master now too?
22:21 tcohen          the holds endpoint?
22:21 tcohen          yes
22:22 tcohen          we need more people to QA
22:22 tcohen          the patches are there
22:23 * cait          pushes so jenkins can work overnight
22:23 tcohen          hehe
22:25 tcohen          cait: about 18076
22:25 tcohen          i'm not sure plack is guilty
22:25 cait            ah
22:25 cait            there was no error in the intenral logs - that's why i assumed
22:25 cait            but i haven't tried turning it off
22:26 tcohen          when plack is enabled (on packages) the logs are on /var/log/koha/kohadev/plack-error.log
22:26 tcohen          in kohadevbox theres a shortcut
22:26 tcohen          koha-plack-error
22:26 wizzycray       tcohen, a slight grizzle I have about that
22:26 tcohen          koha-plack-err
22:27 tcohen          if you disable plack
22:27 tcohen          you will get the same error the logs show
22:27 tcohen          but on the browser
22:27 wizzycray       it chucks all of the errors in there, instead of for their respective interfaces. Maybe I"m being sentimental for the compartmental logs of ye olden tymes
22:27 tcohen          sudo koha-plack —disable kohadev
22:27 tcohen          sudo service apache2 restart
22:27 tcohen          wizzycray: what's because we lack womanpower
22:28 tcohen          to properly use Koha::Logger
22:28 tcohen          in plack.psgi
22:28 cait            feel fre to correct/amend the bug
22:28 cait            getting too late for me tonight - gotta get ready for bed :)
22:29 wizzycray       tcohen: touche
22:30 tcohen          wizzycray: maybe you can file a bug? :-D
22:30 wizzycray       ;)
22:30 tcohen          I think ther's a Plack::Middleware::Log4perl thing that we could reuse
22:31 tcohen          or steal and adapt to Koha::Logger
22:32 tcohen          is  academy over?
22:32 tcohen          the log stuff could be an easy one
22:35 wizzycray       it is over
22:57 bag             @wunder pdx
22:57 huginn          bag: The current temperature in Portland, Oregon is 3.0°C (2:53 PM PST on February 07, 2017). Conditions: Light Rain. Humidity: 89%. Dew Point: 2.0°C. Windchill: 0.0°C. Pressure: 29.78 in 1008 hPa (Rising).  Winter Weather Advisory in effect until 1 PM PST Wednesday... 
22:59 wizzycray       WOO great on 17775
23:00 cait            bug 17775
23:00 huginn          04Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17775 blocker, P5 - low, ---, liz, Pushed to Stable , Add new user with LDAP not works under Plack
23:00 cait            ah :)
23:00 cait            that one i couldn't test, but change looked safe and wizzyrea tested :)
23:01 wizzycray       I have an excellent setup for LDAP testing now
23:01 cait            :)
23:01 wizzycray       with a real LDAP server and everything!
23:01 * cait          is envious
23:01 cait            :)
23:01 cait            but now... sleep
23:01 cait            good night #koha
23:01 wizzycray       zzz's for you
23:17 tcohen          wizzycray: don't forget to file that bug :-P
23:42 tcohen          good night #koha
23:57 eythian         @later tell LibraryClaire https://youtu.be/wuCy_UkKBcs
23:57 huginn          eythian: The operation succeeded.