Time |
Nick |
Message |
04:30 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
07:00 |
|
agoben joined #evergreen |
07:07 |
|
rjackson_isl joined #evergreen |
07:39 |
|
bdljohn joined #evergreen |
07:44 |
|
bdljohn1 joined #evergreen |
08:24 |
|
bos20k joined #evergreen |
08:40 |
|
mmorgan joined #evergreen |
08:47 |
|
bdljohn joined #evergreen |
08:48 |
|
bdljohn joined #evergreen |
09:05 |
|
aabbee joined #evergreen |
09:40 |
cesardv |
berick: yes, of course! Thanks for pushing #1737800 |
09:40 |
* cesardv |
replies 20 hours later... |
09:54 |
berick |
:) |
10:02 |
|
jvwoolf joined #evergreen |
10:04 |
|
jvwoolf joined #evergreen |
10:05 |
|
sandbergja joined #evergreen |
10:09 |
sandbergja |
Is anybody available to review my branch for https://bugs.launchpad.net/evergreen/+bug/1805856? There was a big discussion of this bug on the Evergreen cataloging email list recently, so I'm hoping that we can get a fix released before too long. |
10:09 |
pinesol |
Launchpad bug 1805856 in Evergreen "WebClient: would like certain results to open in a new tab" [High,Confirmed] |
10:30 |
|
Dyrcona joined #evergreen |
10:30 |
bshum |
sandbergja: gmcharlt is assigned to the bug, best to check with him on the review state. |
10:30 |
sandbergja |
gmcharlt++ |
10:30 |
bshum |
(that's also my way of asking, by mentioning their name) :) |
10:30 |
sandbergja |
bshum: thanks for pointing that out! |
10:31 |
gmcharlt |
yep, I just assigned to myself and will put the patch through its paces today |
10:31 |
sandbergja |
thanks so much -- looking forward to it :-) |
10:40 |
* JBoyer |
will also poke more at that patch, several folks here will love it. |
10:50 |
|
khuckins joined #evergreen |
11:00 |
|
khuckins_ joined #evergreen |
11:32 |
dbwells |
FYI, cutoff time for commits for today's point releases is 2:00pm Eastern. Thanks! |
11:50 |
|
sandbergja joined #evergreen |
11:58 |
|
khuckins joined #evergreen |
12:07 |
|
jihpringle joined #evergreen |
12:13 |
JBoyer |
gmcharlt, depending on how far you've made it looking into bug 1805856 you might want to load the branch I referenced in the latest comment before testing further. |
12:13 |
pinesol |
Launchpad bug 1805856 in Evergreen "WebClient: would like certain results to open in a new tab" [High,Confirmed] https://launchpad.net/bugs/1805856 - Assigned to Galen Charlton (gmc) |
12:14 |
gmcharlt |
JBoyer: thanks for the heads-up |
12:15 |
sandbergja |
JBoyer: thanks for the testing and correction! |
12:15 |
sandbergja |
JBoyer++ |
12:15 |
JBoyer |
sandbergja++ |
12:15 |
JBoyer |
Thanks for putting it out there to test! |
12:23 |
mmorgan |
Just mentioning I added my signoff branch to bug 1806709 |
12:23 |
pinesol |
Launchpad bug 1806709 in Evergreen 3.2 "Add Billing History grid persist-key to server-side db settings" [High,Confirmed] https://launchpad.net/bugs/1806709 |
12:29 |
|
Christineb joined #evergreen |
12:36 |
|
jvwoolf joined #evergreen |
12:47 |
jeffdavis |
JBoyer++ # https://bugs.launchpad.net/evergreen/+bug/1742521/comments/3 |
12:47 |
pinesol |
Launchpad bug 1742521 in Evergreen "Web Client: Bill Receipt on Payment ignores # Copies" [Medium,Confirmed] |
13:17 |
pinesol |
[evergreen|Jeff Davis] LP1804716: grab recall-able copies before they are filtered out - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=66656c9> |
13:29 |
|
hbrennan joined #evergreen |
13:42 |
JBoyer |
jeffdavis, What's the point if you can't have a little fun now and then, I say. |
13:43 |
JBoyer |
It does take effort not to go so far as to be obnoxious about it sometimes though, heh. |
13:46 |
csharp |
could someone with Angular/JS-fu please let me know why I might be getting this error in the patron registration UI? : https://pastebin.com/P745UeUk |
13:46 |
csharp |
symptoms are that we can't save and those errors pop up in the console |
13:47 |
csharp |
this is in PINES 3.2, though I just tried adding in the stock 3.2 files (regctl.js and t_edit.tt2) to see if it made a difference - nope |
13:48 |
JBoyer |
csharp, do you see it appear when trying to save, or is it already there when you pull up the console? |
13:48 |
csharp |
JBoyer: already there when I pull up the console |
13:49 |
csharp |
in this particular instance, it's when I'm registering a new patron |
13:49 |
csharp |
but the original report was that they couldn't save a change they'd made |
13:49 |
berick |
getting same error when I pull up page to register new patron |
13:50 |
JBoyer |
I'm running rel_3_2 here and our line numbers don't match up. |
13:50 |
csharp |
hemm |
13:50 |
JBoyer |
(rel_3_2 as of yesterday that is) |
13:50 |
csharp |
sec... |
13:50 |
berick |
ah, simple sanity check fix |
13:54 |
berick |
pushing a patch in a sec |
13:54 |
JBoyer |
So, here we get that error about 4 times as soon as the page loads, but can save new patrons just fine. |
13:54 |
csharp |
ok -hmm |
13:54 |
berick |
yeah, may be unrelated |
13:54 |
JBoyer |
If you're not able to save I guess something else is unhappy. |
13:54 |
|
Dyrcona joined #evergreen |
13:55 |
csharp |
when trying to change a patron we get a pop-up event "USERNAME_EXISTS" |
13:55 |
* csharp |
just learned that Chrome doesn't let you select and copy text from alert boxes |
13:57 |
csharp |
and thanks to Firefox: |
13:57 |
csharp |
https://pastebin.com/xrj2nCCg |
13:57 |
csharp |
that's what we get when trying to save after a quick update |
13:58 |
csharp |
in this case I changed a digit of day phone and clicked Save |
13:58 |
Dyrcona |
Were you trying to change the usrname? |
13:58 |
csharp |
no |
13:58 |
berick |
if anyone wants to test/merge: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/patron-create-load-js-errors |
13:59 |
berick |
hm, no trouble saving modified patrons here |
13:59 |
berick |
in master |
14:00 |
csharp |
berick++ # your fix stopped the console errors - thanks |
14:02 |
Dyrcona |
csharp: Somehow, you're ending up in the code to create a new patron. |
14:02 |
JBoyer |
I was about to be confused as to how that usrname check could possibly work, but then I noticed that it's in _add_patron... So something about an isnew check (maybe not the ones berick fixed?) seems to be failing somewhere. |
14:03 |
Dyrcona |
/usr/local/share/perl/5.22.1/OpenILS/Application/Actor.pm:588 is right in the middle of _add_patron {} |
14:04 |
csharp |
here's what I get when I click save when trying to save a *new* patron: https://pastebin.com/8RPsfuSw |
14:05 |
Dyrcona |
Yeahp. I think what JBoyer suggested about isnew fields is going on. |
14:06 |
|
jvwoolf left #evergreen |
14:06 |
JBoyer |
As in csharp's appear to be completely backward, which is wild. |
14:06 |
Dyrcona |
The failure on the "new" patron is right in the middle of a check for "! $p->isnew" or in other words the patron does not have isnew set. |
14:06 |
Dyrcona |
yeah. |
14:07 |
berick |
isnew field shuffling from IDL inconsistencies? |
14:07 |
berick |
though, hm, I think those fields are autogenerated |
14:07 |
Dyrcona |
berick: That could be it. |
14:08 |
berick |
not stored in the idl proper |
14:08 |
Dyrcona |
csharp: When was the last time you ran autogen.sh? |
14:08 |
JBoyer |
csharp, maybe diff regctl.js against master or the rel_3_2_x version of the same? Since line 2008 isn't that check on any version I'm looking at I wonder if something that's changed is weird. |
14:08 |
csharp |
okay - that seems like the right direction to go it |
14:08 |
csharp |
in |
14:08 |
Dyrcona |
berick: Right, but JavaScript gets the fields from autogen.sh, no? |
14:09 |
Dyrcona |
I think berick has it with the IDL. |
14:09 |
berick |
aha, isnew, etc. is appended to the end of the fields list |
14:09 |
csharp |
JBoyer: this is rel_3_2 with that check on 2008, right?: http://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js;h=35fbfde460d8a84293bacb6b5fd1c3030f446abb;hb=refs/heads/rel_3_2#l2008 |
14:09 |
berick |
so a field variation could modify the position of isnew, etc. |
14:09 |
Dyrcona |
At least, that's the easiest thing to try. |
14:10 |
berick |
so, yeah, i'm back on team IDL |
14:10 |
* csharp |
is sleep-deprived and grumpy, so is almost certainly missing obvious things |
14:10 |
berick |
autogen + apache reload + browser shift-reload |
14:10 |
berick |
and confirm both IDL files match (conf / reporter) |
14:11 |
JBoyer |
Woops. Maybe I've got something changed on my version... (Master moves things around too) |
14:11 |
dbwells |
That reporter IDL will get you every time... |
14:12 |
dbwells |
That is the "reporter" IDL. |
14:15 |
pinesol |
[evergreen|Jason Boyer] LP1742521: Receipt Count Repair - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=bc9c9d3> |
14:18 |
dbwells |
We're a few minutes past the cutoff for release today, and release notes are now being produced. If you are still scratching at a last minute push, just give a heads up so we don't get too far along. Otherwise, please consider the release window as closed. Thank you! |
14:20 |
csharp |
berick++ Dyrcona++ # yep, that worked |
14:20 |
csharp |
JBoyer++ # helpin' |
14:20 |
berick |
yay |
14:20 |
csharp |
whew |
14:22 |
csharp |
gotta say, this channel has saved my... *looks around*... ass so many times :-) |
14:33 |
Bmagic |
evergreen++ |
14:34 |
Bmagic |
I'm looking over the conversation last year on IRC about AppArmor http://irc.evergreen-ils.org/evergreen/2018-12-18#i_388768 |
14:34 |
Bmagic |
Suddenly, I find myself with the same issue on our utility server but that fix isn't fixing |
14:35 |
Bmagic |
apparmor="DENIED" operation="connect" profile="/usr/sbin/ejabberdctl//su" name="/run/dbus/system_bus_socket" comm="su" requested_mask="wr" denied_mask="wr" fsuid=0 ouid=0 |
14:35 |
Bmagic |
mean anything to anyone? |
14:36 |
Dyrcona |
So, Bmagic: Two (well, three) things usually fix that: 1) update apparmor, 2) use sudo instead of su, 3) use sudo -u ejabberd. |
14:36 |
bshum |
Well, googling that shows me a bunch of ejabberd bugs and issues. So I guess first question... what distro is your utility server on? |
14:36 |
Bmagic |
16.04.1 |
14:37 |
Bmagic |
I think I might have guessed and fixed it |
14:37 |
bshum |
And what version is your ejabberd? |
14:37 |
Bmagic |
ejabberd is 16.01-2 |
14:37 |
Bmagic |
editing this file /etc/apparmor.d/usr.sbin.ejabberdctl |
14:38 |
Bmagic |
changing this line: /bin/su r, |
14:38 |
Bmagic |
to "rw" seemed to do the trick. Even though the article said it needed to be "rm" |
14:38 |
Dyrcona |
Bmagic: Ubuntu 16.04 is at 16.04.5, now. Updating apparmor and rebooting should fix it. |
14:39 |
Bmagic |
weird that this would just crop up on a machine that's been running for years with the versions it has |
14:40 |
Dyrcona |
No, it is not weird. Apparmor is broken for some apps in 16.04.1. :) |
14:40 |
Dyrcona |
You really should sudo apt upgrade followed by a reboot regularly |
14:40 |
Dyrcona |
"You" being everyone. |
14:40 |
Bmagic |
gotcha, yeah, that's not been done lately |
14:41 |
bshum |
I'll always remember that day we said, "hey what's this server in the rack?" and we checked the uptime and it hadn't been logged into or restarted in almost a year |
14:42 |
JBoyer |
At least it wasn't the Netware server that was walled in for something like 6 years. (total uptime over 10 I believe.) :) |
14:42 |
Bmagic |
I had one at my old job with uptime of over 1500 days |
14:49 |
JBoyer |
dbwells++ # release managing |
14:55 |
|
khuckins_ joined #evergreen |
14:57 |
|
jvwoolf joined #evergreen |
14:58 |
sandbergja |
Check my understanding: Hatch is deprecated for local data storage. But it will be used for printing for the foreseeable future? |
14:58 |
sandbergja |
And there's not anything that Hatch does other than local data storage and printing, right? |
15:02 |
mmorgan |
sandbergja: I was waiting for someone else to chime in, but that's my understanding, too. |
15:07 |
berick |
sandbergja: correct |
15:12 |
sandbergja |
Thanks! |
15:21 |
pinesol |
[evergreen|Jane Sandberg] Docs: adding 3.1.9 release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=6a81402> |
15:21 |
pinesol |
[evergreen|Jane Sandberg] Docs: adding release notes for 3.2.3 - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=5bcfc08> |
15:23 |
pinesol |
[evergreen|Jane Sandberg] LP1805856: Open records in new tabs - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=7717f63> |
15:23 |
pinesol |
[evergreen|Jason Boyer] LP1805856: Open new windows for record operations - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=68ceb66> |
15:31 |
pinesol |
[evergreen|Jane Sandberg] Docs: adding one more note to 3.2.3 release notes - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=55a0e12> |
15:36 |
hbrennan |
bshum: It WAS a custom template issue! Ha! bshum++ for always knowing what to guess next. |
15:56 |
|
jamesrf joined #evergreen |
16:00 |
|
jvwoolf1 joined #evergreen |
16:10 |
bshum |
hbrennan++ # persistence pays off :) |
16:19 |
hbrennan |
bshum: Helps to have equinox++ on call when I run out of energy! :) |
16:21 |
miker |
:) |
16:21 |
miker |
hbrennan++ |
16:22 |
hbrennan |
miker: Sometimes it just takes another set of eyes! |
16:23 |
miker |
they make all bugs shallow |
16:23 |
Dyrcona |
Well, most bugs...anyway. |
16:25 |
Bmagic |
this apparmor error is happening on a fresh install of 16.04 |
16:26 |
Dyrcona |
Bmagic: Is it a freshly downloaded image or is it still an older image? |
16:26 |
* Dyrcona |
has not had that problem because I use sudo, and not su. |
16:26 |
Bmagic |
16.04.5 |
16:27 |
Bmagic |
just installed the os, and loaded OpenSRF up to ejabberd |
16:27 |
* Dyrcona |
shrugs. Use sudo instead of su, and specify the ejabberd user if you still have a problem. |
16:28 |
* Dyrcona |
considers su to be antiquated. :) |
16:28 |
Bmagic |
"ejabberdctl unregister router private.localhost" is as root |
16:28 |
bshum |
Yeah, ejabberd user is what berick and I have been using in the ansible scripting for awhile now |
16:28 |
Bmagic |
throws the error. haven't tried it as any other user... let me see |
16:29 |
bshum |
Might be time to put that in the OpenSRF readme somewhere |
16:29 |
Dyrcona |
Everything in the READMEs that tells you to use su, works if done with sudo. |
16:30 |
Dyrcona |
Maybe we should change to sudo in the documentation? |
16:30 |
Bmagic |
sheesh |
16:30 |
Bmagic |
it works as ejabberd |
16:30 |
bshum |
Well "sudo" doesn't come with Debian right? |
16:30 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
16:30 |
bshum |
You'd have to add sudo as a package prereq or something for the blind followers :) |
16:31 |
Dyrcona |
Um, I get sudo by default. |
16:31 |
bshum |
Maybe it's an old Debian problem |
16:31 |
bshum |
I haven't used anything newer than Squeeze clearly :) |
16:32 |
Dyrcona |
Y'know. I may have had to install it on Debian last time I tried. |
16:32 |
bshum |
I always had to install it in the past. |
16:32 |
bshum |
If I really wanted it, which I tended to want it |
16:32 |
Dyrcona |
Been a while since I tried Jessie or Stretch, so I don't remember. |
16:33 |
|
khuckins joined #evergreen |
16:33 |
* bshum |
hugs his Trusty Tahr and spits on the other VMs in his laptop |
16:33 |
Dyrcona |
Anyway, 'bout time for me to go. I may or may not be in the channel over the next few days. |
16:33 |
* Dyrcona |
recently deleted his Trusty Tahr VM. |
16:35 |
bshum |
Yeah I guess it's 2019 now. Time to let it go. |
16:35 |
Dyrcona |
Yeahp. |
16:35 |
* Dyrcona |
waves and dissolves away. |
17:06 |
|
mmorgan left #evergreen |
18:07 |
|
jamesrf joined #evergreen |
18:21 |
csharp |
terran++ |
18:22 |
|
jamesrf joined #evergreen |
18:30 |
csharp |
7891c024 |
18:30 |
pinesol |
csharp: [evergreen|Bill Erickson] LP#1750894 Workstation & Cascade settings - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=7891c02> |
20:24 |
|
jvwoolf joined #evergreen |
21:19 |
|
devted joined #evergreen |
21:19 |
|
Bmagic joined #evergreen |
21:36 |
|
tsadok joined #evergreen |
21:38 |
|
phasefx_ joined #evergreen |
21:38 |
|
csharp_ joined #evergreen |
21:43 |
|
_bott_1 joined #evergreen |