14:51 |
pinesol |
Launchpad bug 1948983 in Evergreen "Reports Administrator Permission to View Other Users Templates, Reports, and Output" [Wishlist,Confirmed] https://launchpad.net/bugs/1948983 |
14:52 |
jihpringle |
Bmagic: is that in angular reports? |
14:52 |
Bmagic |
yes, sorry, 3.13 |
14:53 |
jihpringle |
I've seen that in testing, I thought it got reported but maybe not |
14:53 |
Bmagic |
should it be a bug? Or are "we" (the community) assuming that when you share a template folder, it's fair game for adding templates? |
14:54 |
jihpringle |
for us, it would definitely be a bug |
14:54 |
jihpringle |
it would make our shared consortium template folders unusable if anyone can save into them |
14:59 |
Bmagic |
I'll write a new report for it I guess |
14:59 |
abneiman |
Bmagic: curious for details on this one, as IIRC (and as jihpringle mentioned) this was disovered & fixed during the run up to 3.13 |
15:00 |
Bmagic |
we have a consortium on 3.13.0, and a staff member cloned a template into another staff member's shared folder. Maybe it was fixed later? 3.13.1 or .2? |
15:01 |
abneiman |
on a stock 3.13.1 test system, a CONS-shared folder behaves as expected (i.e. templates can be used to create a new report, or cloned into one of your own folders). I am NOT seeing that a user can choose a shared folder as a destination for a clone. |
15:01 |
Bmagic |
what about simple reports? |
15:01 |
Bmagic |
maybe a loophole |
15:02 |
abneiman |
simple reports doesn't support sharing or folders |
15:02 |
jihpringle |
I think it was happening when you went to clone from shared folder and then canceled it partway through, instead of cancelling the clone it created the template in the shared folder it came from |
15:03 |
jihpringle |
(I tested quite a while ago and haven't tested on a 3.13 server) |
15:05 |
Bmagic |
I'm testing |
15:10 |
Bmagic |
must be something on 3.13.0, because bugsquash.mobiusconsortium.org doesn't allow it |
15:20 |
abneiman |
dunno, the only difference between 3.13.0 and 3.13.1 is a handful of security patches. I had Christine Burns' original comment about the cloning issue (https://bugs.launchpad.net/evergreen/+bug/1993823/comments/8) in my internal feedback list with a note "resolved 4/17" |
15:20 |
pinesol |
Launchpad bug 1993823 in Evergreen "wishlist: Angular Reports Interface" [Wishlist,Fix released] |
15:22 |
abneiman |
anyway I have to run but I'd be interested in other details of how this happened, Bmagic |
15:23 |
Bmagic |
me too! I don't see how they did it. It's not in the UI on production |
15:24 |
Dyrcona |
It's just one of those things, like the purchase orders I'm looking at that silently fail to create EDI messages, though everything looks right to be able to do that. |
15:40 |
Dyrcona |
OK. If I run edi_order_pusher.pl in test mode on one fo the PO ids, it produces output. |
15:54 |
Dyrcona |
So, when I ran the order pusher on each purchase order id, they worked. For some reason, these purchase orders were not getting picked up by the order pusher otherwise, even though they turn up when I run the equivalent SQL query to get all of the active pos. |
16:06 |
Bmagic |
UHG |
16:06 |
Bmagic |
that's annoying |
09:58 |
Dyrcona |
:) |
09:58 |
pinesol |
News from commits: LP#2069472 Conjoined Items Grid Config Not Saving <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=716e81310735d18537d2f0b7b75701a703faf319> |
09:58 |
csharp_ |
pinesol: blame [someone] for Dyrcona lapsing into German |
09:58 |
pinesol |
dluch_ tests their code on the LIVE SERVERS, then blames the user. SAD! for Dyrcona lapsing into German |
09:59 |
Dyrcona |
Whatever.. It just popped into my head because of the praise. It was a tagline I used to use on a MMO years ago: "Entdecke die Insel von Dyrcona." (Yeah, I used it as my gamer tag, too.) |
10:00 |
Dyrcona |
It means, "discover the island." |
10:03 |
csharp_ |
Dyrcona++ |
11:52 |
mmorgan |
redavis: What are you trying to meld together? |
11:52 |
Dyrcona |
redavis: What are you trying to do, and if you it frequently, maybe a view is in order? |
11:54 |
jeff |
speaking of billing: when staff incorrectly record the payment type, how are most folk handling that? manual corrections in the back office, directly correcting the issue in the database, or something else? |
11:54 |
redavis |
I was trying to get a report of anyone that had a bill. I could set up a traditional report (if clark kent is running on the test server), but was hoping I could run into simple reporter and throw something together quickly. |
11:55 |
jeff |
(I can't remember if I've asked that question here recently or not) |
11:56 |
redavis |
I didn't find an easy way to get patron connected to billing in simple reporter. It seems to focus on org units. I suspect if I used the circulation "source" it might be doable, but it went beyond the "quick and easy." |
11:56 |
Dyrcona |
jeff: I don't deal with that, and I don't hear anyone at CW MARS talking about it. If you really want an answer, you might try the general list. |
12:27 |
Dyrcona |
@band add The Reporters |
12:27 |
pinesol |
Dyrcona: Band 'The Reporters' added to list |
12:40 |
|
kmlussier left #evergreen |
12:42 |
eeevil |
(I (I won't blame/out others) call the full reporter Avril internally -- why you gotta make everything so complicated?) |
12:43 |
eeevil |
redavis: jfyi, both simple and full reports depend on clark to generate output, so if you have simple on test, you have full also. |
12:51 |
redavis |
eeevil, acknowledged about clark/avril. It wasn't so much the running as the preparing to run. I was putting more energy into the building than getting on with the testing. |
12:54 |
redavis |
and also acknowledged about SR in relation to billing JBoyer. I just don't generally use SR, and wanted to be a little lazy about building a template in the full reporter...so was hoping someone had done a lot of work so I wouldn't have to. And then I realized, I still don't have to...cuz what I was doing was a choice. So...I quit and ate lunch. The world is better. Also, I love Avril Lavigne and am mostly very happy |
12:54 |
redavis |
with complexity. Even if it makes things more...complicated. |
13:42 |
|
kworstell_isl joined #evergreen |
14:37 |
Dyrcona |
I might have an idea how to fix the 1416 upgrade before it gets run, i.e. replace it. |
14:39 |
Dyrcona |
Instead of grabbing the first record attr defintion from the list of rec_types, we add the rec_type as a function parameter. |
11:14 |
csharp_ |
ehrmagerd berg squersh |
11:15 |
Bmagic |
lol, took me a second |
11:15 |
Bmagic |
csharp_++ |
12:03 |
Bmagic |
what's the magic trick to get the nightwatch tests to run with gecko/firefox? This is the command i'm using: mkdir $HOME/tmp; MOZ_HEADLESS=1 TMPDIR=$HOME/tmp ng e2e |
12:03 |
Bmagic |
error: Failed to connect to GeckoDriver on localhost with port 4444. |
12:07 |
|
jihpringle joined #evergreen |
12:13 |
Bmagic |
just found out that firefox isn't installed, gonna install it |
17:11 |
|
mmorgan left #evergreen |
17:35 |
Bmagic |
lol |
17:35 |
Bmagic |
JBoyer++ |
17:36 |
Bmagic |
for those of us who are still here: I'm running perl live tests via "make livecheck" - and it's all passing, but it's stumbling on the offline interface check. There's this grep command: 'wget --no-check-certificate -m https://localhost/eg/staff/offline-interface 2>&1 |grep -B 2 " 404 "|grep https|grep -v robots.txt|wc -l' |
17:36 |
Bmagic |
which in my case returns 13, bunch of js and css files in the build folder |
17:47 |
Bmagic |
nevermind, figured it out :) |
17:47 |
csharp_ |
what was it? |
17:48 |
csharp_ |
localhost/docker? |
17:48 |
Bmagic |
rsync -L -a --no-owner --no-perms --size-only /home/opensrf/repos/Evergreen-build/Open-ILS/web/js/ui/default/staff/ /openils/var/web/js/ui/default/staff |
17:49 |
Bmagic |
I had performed a step out of order, and the final /openils folder needed the results of "cd /home/opensrf/repos/Evergreen-build/Open-ILS/web/js/ui/default/staff/ && npm run build-prod" |
17:49 |
csharp_ |
ah |
17:50 |
csharp_ |
hooray for tests! |
17:50 |
Bmagic |
yeah, the more I run them, the more I want to write more |
17:50 |
Bmagic |
those nightwatch tests are spectacular because they click on the interface the way humans do |
17:51 |
csharp_ |
still haven't found time to get to that |
17:52 |
Bmagic |
I'm working on bug 2068740 |
17:52 |
pinesol |
Launchpad bug 2068740 in Evergreen "Concerto and Enhanced Concerto user passwords should be updated" [Undecided,Confirmed] https://launchpad.net/bugs/2068740 |
17:52 |
Bmagic |
which, has the nasty side-effect of ruining our test suite. perl, postgres and angular/nightwatch |
17:53 |
Bmagic |
and violla: I got real familiar with our tests |
17:55 |
csharp_ |
can we just pass environment vars or something? |
17:55 |
Bmagic |
yeah, I know what you mean. I had the same thoughts: our tests need to be smarter |
17:56 |
Bmagic |
lots of hardcoded ID numbers and assumptions of the concerto set |
17:56 |
Bmagic |
nightwatch, for example, expects to find "BR1" in the branch list when it's siumating all of the clicks to register a workstation |
17:57 |
csharp_ |
ick |
17:57 |
Bmagic |
siumataing/simulating # lol |
17:57 |
Bmagic |
so, automatically, the nightwatch test requires the old concerto set and doesn't work with enhanced |
18:01 |
csharp_ |
Bmagic++ # doing God's work |
18:05 |
|
kmlussier left #evergreen |
18:11 |
sleary |
I have not had time to play with Nightwatch as much as I would like, but there is a recording of Jane explaining the tests somewhere in the New Devs meeting archives |
18:12 |
sleary |
as soon as I figure out how to make it take screenshots automatically, I will acheive documentation nirvana |
10:27 |
Dyrcona |
If you were hosting Aspen also, I'd suggest getting into Solr. That way there is only 1 to support. But... I suppose one could always add ES to Aspen... :) |
10:28 |
Rogan |
I need a face melting emoji to reply to Dyrcona's comment there |
10:30 |
Rogan |
joking aside, I've been working with Aspen on a mostly application end user level but hoping to dig into the guts more |
10:32 |
* Dyrcona |
has PRs added to Aspen. :) I have yet to set up a test Aspen installation, but it's on my list of things that I'll (probably) never get to. |
10:35 |
redavis |
Y'all are the best ever. Also, pretty sure my husband wasn't expecting a long diatribe aboue databases, indexes, and search when he asked how my studying for the master gardener class was going. |
10:35 |
berick |
ooh, Master Gardener |
10:35 |
berick |
nice |
11:22 |
redavis |
mmorgan, Bmagic, once y'all get your branches, I'll send out an email that the merge pause has been lifted. |
11:22 |
Bmagic |
redavis++ # rock'n roll branch'n emailer |
11:23 |
* mmorgan |
is just going into a meeting, will have to grab it after. |
11:23 |
Dyrcona |
csharp_: I have not seen that on my test systems, but granted, I don't update a whole lot of bibs all at once. We probably should consider forcing serialized updates on bibs. |
11:23 |
berick |
csharp_: see also the description of bug 1880726. |
11:23 |
pinesol |
Launchpad bug 1880726 in Evergreen "MARC Batch Edit Angular Port" [Wishlist,Fix released] https://launchpad.net/bugs/1880726 |
11:23 |
redavis |
I ain't got a lot of skillz, but I can email like a master...typer (with typos included) |
15:02 |
|
halloy1408 joined #evergreen |
15:05 |
|
adam_reid joined #evergreen |
15:09 |
|
Stompro joined #evergreen |
15:17 |
Dyrcona |
Hm... I should probably update the Redis branch on the server where I'm testing it. |
15:24 |
Dyrcona |
I wonder if I can just install it while things are running. Well, I guess I'll find out. |
15:30 |
Dyrcona |
That seemed to work. |
15:30 |
Dyrcona |
Nothing has gone boom so far.... |
15:11 |
mmorgan |
sandbergja++ abneiman++ redavis++ Dyrcona++ |
15:11 |
sleary |
Dyrcona++ |
15:11 |
redavis |
Dyrcona++ sandbergja++ abneiman++ mmorgan++ |
15:11 |
Bmagic |
#info abneiman will review existing EOLI PRs for test plans |
15:11 |
shulabramble |
Dyrcona++ sandbergja++ abneiman++ mmorgan++ |
15:12 |
abneiman |
I reviewd them but did not add test plans to any. Will do so in advance of BSW. |
15:13 |
Bmagic |
abneiman++ |
15:13 |
sandbergja |
abneiman++ # that's awesome |
15:13 |
shulabramble |
abneiman++ |
15:13 |
Bmagic |
want that carried? |
15:14 |
jvwoolf |
abneiman++ |
15:14 |
abneiman |
nah |
15:14 |
Bmagic |
#info abneiman will add a discussion item for next meeting: test plans for big features |
15:14 |
abneiman |
I ninja-edited that into the agenda 10 minutes ago :) |
15:14 |
Bmagic |
I see that, lol |
15:14 |
* redavis |
was also ninja editing |
15:34 |
Bmagic |
#info Needstestplan tag Added - 0 |
15:34 |
Bmagic |
#info Needsrebase tag Added - 5 |
15:34 |
Bmagic |
#info Fix Committed - 22 |
15:34 |
Bmagic |
#topic New Business - (Jane) How can we encourage more automated testing?, What are current barriers to writing and running the automated tests? |
15:34 |
sandbergja |
ok, my turn to paste some stuff: |
15:34 |
Bmagic |
#link https://wiki.evergreen-ils.org/doku.php?id=dev:contributing:qa |
15:34 |
sandbergja |
During the 3.12 and 3.13 release process, we found a number of big issues that we were able to resolve before cutting the release, for example: |
15:34 |
sandbergja |
an unpredictable intermittent failure in search, a bug that prevented users from adding new action trigger event defs, a case where I had committed the wrong patch from a launchpad ticket, one where the holdings view wouldn't load, and a major accessibility regression that detached form inputs from their labels across the client. |
15:35 |
sandbergja |
And the way we found those was by one of our automated tests failing, alerting us to the issue, which is great! Very nice that we discovered those before they were in production. |
15:35 |
sandbergja |
However, it makes me think how many more regressions we could be catching if we'd been writing and running tests more often over the past 9 years. |
15:35 |
sandbergja |
So... I'd like to know: how can we get there? What barriers can we iron out around automated testing? |
15:36 |
redavis |
sandbergja++ |
15:36 |
shulabramble |
sandbergja++ |
15:37 |
Bmagic |
those node nightwatch tests are clever. Automating the process of clicking on the interface like a human would. Not sure what the barriers are? Hard to write? |
15:38 |
sleary |
IIRC there are two (maybe three?) different QA/guideline documents for contributions that have different instructions, and we need to consolidate them into one coherent process. |
15:38 |
Bmagic |
consolidate! +1. Got links? |
15:38 |
Stompro |
For me personally it is just the time it takes to create the tests, but I know if I do it more often it wouldn't take me as long. |
15:39 |
sleary |
working on links |
15:40 |
sandbergja |
Stompro++ sleary++ Bmagic++ |
15:41 |
Bmagic |
giving sleary a minute |
15:43 |
Bmagic |
sleary++ |
15:43 |
sandbergja |
consolidating and refining those seems like a great place to start |
15:43 |
sleary |
https://wiki.evergreen-ils.org/doku.php?id=dev:meetings:common_qa_problems might also be relevant |
15:44 |
terranm |
+1 to a single page just about writing tests |
15:44 |
Bmagic |
do we need to make assignments around this? |
15:44 |
sleary |
sandbergja I will be happy to help with that... about a month from now, lolsob |
15:44 |
Bmagic |
sleary++ |
15:44 |
terranm |
sleary++ sandbergja++ |
15:44 |
shulabramble |
+1 to a page about writing tests |
15:44 |
sandbergja |
a month sounds great! And I'm also happy to help as needed! |
15:45 |
mmorgan |
+1 to a page about writing, and also running them |
15:45 |
jeff |
do we need to do anything to remove barriers to running the tests? i.e., is setting up an environment where you can run the tests challenging or not-well-documented? |
15:45 |
redavis |
sandbergja++ sleary++ |
15:45 |
sleary |
I know that sandbergja has given a couple of talks in New Devs about test-driven development, and we can pull things from those meeting archives |
15:45 |
redavis |
jeff, good question |
15:45 |
Bmagic |
#action sleary and sandbergja will create/consolidate the test writing wiki page(s) |
15:45 |
terranm |
It always makes sense when I watch Jane do it, but then confusion returns |
15:46 |
Bmagic |
jeff: the docker container image now days is including the pre-reqs which is nice |
15:46 |
sleary |
jeff I think there is one quirk around Firefox, but it's mentioned in the eg2 CHEAT SHEET doc |
15:46 |
Bmagic |
Imma preserve those wiki urls |
15:46 |
sandbergja |
for me, reloading a test database from scratch can take a while, and a clean db is necessary for some of the tests |
15:46 |
Bmagic |
#link https://wiki.evergreen-ils.org/doku.php?id=dev:meetings:common_qa_problems |
15:47 |
Bmagic |
#link https://wiki.evergreen-ils.org/doku.php?id=dev:contributing:qa |
15:47 |
Bmagic |
I added a feature to the the dev container, which I find myself using a bunch, that automates the creation of new databases by editing a text file |
15:47 |
sleary |
the e2e tests take a while to run, and the syntax for running them on a single file or directory is not easy to remember |
15:48 |
sandbergja |
true sleary |
15:48 |
sleary |
we can probably script our way out of that problem :) |
15:48 |
sandbergja |
Bmagic I know you've told me this before, but I gotta start using the dev containers! |
15:49 |
Bmagic |
sandbergja: the water is fine, lol |
15:49 |
Bmagic |
the takeaway was consolidating the wiki pages |
15:50 |
Bmagic |
shall we add something else? |
15:50 |
sleary |
maybe step 2 can be writing some simple example tests or pinpointing old ones as good examples for basic tasks |
15:50 |
sandbergja |
maybe as part of that process we can look into the syntax for nightwatch on a single file or directory -- whether it is a docs fix or a script fix? |
15:51 |
sleary |
sandbergja++ |
15:52 |
Bmagic |
making sure I'm understanding: making the test run differently depending on what was committed? |
15:52 |
sleary |
running the tests on only the files you changed rather than the whole Angular project |
15:52 |
Bmagic |
gotcha, and what's missing for that to work? |
15:52 |
sleary |
memorable syntax |
15:53 |
Bmagic |
documentation is missing basically? |
15:53 |
sleary |
sheet |
15:53 |
eeevil |
I would appreciate a how-to, for sure. fwiw |
15:53 |
sandbergja |
so better syntax and better docs perhaps |
15:54 |
Bmagic |
maybe a wrapper that takes some evergreen-community arguments. like ./wrapper --one-test path/to/file |
15:54 |
sleary |
that's what I was thinking, but docs would be simpler :) |
15:54 |
Bmagic |
docs would be simplier :) |
15:55 |
dluch |
docs++ ;-) |
15:56 |
sleary |
that should probably be a separate action from the wiki page consolidation |
15:56 |
Bmagic |
would you like to have it? |
15:56 |
sandbergja |
you could put my name on it |
15:56 |
Bmagic |
#action sandbergja will make it easier for folks to know how to run the tests in single file mode and all-mode |
15:57 |
Bmagic |
sandbergja++ |
15:57 |
sleary |
sandbergja++ |
15:57 |
sandbergja |
thanks everyone! |
15:57 |
dluch |
sandbergja++ |
15:58 |
berick |
count me in for helping with Hatch work. hopefully i can pitch in starting next week |
15:58 |
* berick |
runs off to another meeting |
15:58 |
jeff |
happy to help / help test. |
15:58 |
Bmagic |
berick++ |
15:58 |
Bmagic |
jeff++ |
15:58 |
sandbergja |
berick++ jeff++ |
16:01 |
sleary |
csharp_ most likely. I think I can, but I'm reluctant to touch things without taking time to make my own backups |
16:02 |
Bmagic |
I can help too :) |
16:02 |
Bmagic |
#action Bmagic will work on upgrading the version of DokuWiki |
16:03 |
Bmagic |
#topic (ABN) Test plans for big features? |
16:03 |
Bmagic |
We're almost done... sorry we're going over |
16:03 |
abneiman |
yes, so, briefly |
16:03 |
abneiman |
what does the community want to / need to see out of test plans for new features? |
16:04 |
abneiman |
things that are beyond the scope of "X is broken, install patch Y, follow steps 1 2 3, profit" |
16:04 |
Bmagic |
are you finding that certain things need more than our bsw and bff ? |
16:04 |
abneiman |
asking because several EOLI feature PRs are either forthcoming soon, or carryovers from 3.13, and I want to make sure I'm doing what is needed to facilitate review & commit of those things |
16:05 |
Stompro |
I always appreciate notes on how to configure the system if certain lib settings need to be enabled, etc. |
16:05 |
jeff |
varies, but maybe starting with: "here is a summary of / pointer to description of what this code should add/change/fix; here is a recommended way to test / how we tested; here are special considerations / gotchas"? |
16:05 |
abneiman |
Stompro++ jeff++ |
16:05 |
mmorgan |
What Stompro said. Also if certain data is needed. |
16:05 |
abneiman |
both helpful thanks |
16:05 |
abneiman |
mmorgan++ |
16:06 |
redavis |
Stompro++ jeff++ mmorgan++ |
16:06 |
abneiman |
and Bmagic, regarding BSW, I acknowlege that it's hard to test big features when there's a lot of other fixes flying around ... which is fine, IMO, that's what BSW is about. |
16:06 |
Bmagic |
Stompro++ jeff++ mmorgan++ |
16:07 |
Bmagic |
abneiman++ # I think I took your query the wrong direction after reading Stompro, jeff and mmorgan's ideas |
16:07 |
abneiman |
like I said, I want to make sure we're doing what we can to faciliate merges of features. And to maybe offer similar guidelines to other feature developers |
16:41 |
Bmagic |
csharp_: that sounds super easy |
16:41 |
csharp_ |
yep, and quick |
16:41 |
csharp_ |
just back up the folder for safety |
16:41 |
Bmagic |
do you have an reservations about clicking on it without testing it on a cloned env? |
16:41 |
csharp_ |
nah, I've done it several times |
16:42 |
Bmagic |
ok then, sounds like it's in good hands! |
16:42 |
csharp_ |
we do keep backups for 30 days on that server at ITS also |
10:03 |
|
kworstell_isl joined #evergreen |
10:07 |
Dyrcona |
Stompro: It's all right you missed the meeting. We discussed the 3.next target and some bugs. |
10:07 |
Dyrcona |
Also the Perl MARC code tries to deal with bad files like that. Not sure how it deals with that many nuls in a row, though. |
10:12 |
Stompro |
Dyrcona, if there are any specific bugs that need sign offs or more testing let me know. |
10:13 |
Dyrcona |
Well, there are 12 signed off bugs in total. I was going to have a look at those this week. If you want to take some, feel free. |
10:14 |
Dyrcona |
I'm on vacation next week. |
10:28 |
Stompro |
I'll take a look at them. |
12:09 |
jeffdavis |
disable_email_change rather, but you've found the right setting anyway :) |
12:09 |
* jeffdavis |
-> coffee |
12:27 |
|
collum joined #evergreen |
12:37 |
Stompro |
Dyrcona, do you have your timezone set, that was the test file that failed for me when I had the wrong timezone set. |
13:09 |
|
ian1 joined #evergreen |
13:13 |
Dyrcona |
Stompro: I do have it set. |
13:14 |
Dyrcona |
Reloaded the db and ran make livecheck again and everything passed. |
13:25 |
|
cbrown joined #evergreen |
13:30 |
Dyrcona |
I should not be testing this. I don't know what I'm doing, but anyway.... |
13:32 |
Dyrcona |
Like, how do I make a new call number prefix in the client? |
13:48 |
Dyrcona |
Ugh! Lp madness... There's a bug that has ended up with 2 distinct branches that were applied at different times. |
13:49 |
Dyrcona |
The earlier branch is basically a fix and the latter should have been a new bug, imnsho. |
14:18 |
Bmagic |
thats frustrating |
14:33 |
Dyrcona |
jeff: I am summarily removing your name from "Assigned to" on bugs you don't appear to be actively looking at. |
14:50 |
Dyrcona |
Grabbing 1423. |
14:56 |
pinesol |
News from commits: LP2048425: Increase test coverage for circ limit sets <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=221725a66032d2087481798db6c77b7bd7b7eccc> |
15:00 |
|
mantis left #evergreen |
15:11 |
jeff |
Dyrcona: sounds good. any bugs in particular, or are you just removing me from everything? |
15:26 |
pinesol |
News from commits: LP2065448 Stamp DB upgrade script <https://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=0d306102f3480042647c92cacf01fde329a0bc6b> |
15:36 |
Dyrcona |
Don't think the Lp advanced search URLs survive getting pasted into IRC. |
15:37 |
Dyrcona |
jeff: Turns out it is only 1 bug so far. |
15:38 |
|
SREtds-lib joined #evergreen |
15:52 |
Dyrcona |
JBoyer | Bmagic: Do you know if anyone is using Lp 1613335 in production? I'm going to test it and if it works, I'll push it for 3.14. |
15:53 |
pinesol |
Launchpad bug 1613335 in Evergreen "SIP Return Screen Message OK" [Low,Confirmed] https://launchpad.net/bugs/1613335 - Assigned to Jason Stephenson (jstephenson) |
15:54 |
Bmagic |
awesome, I'm using it in production |
15:56 |
Dyrcona |
Bmagic: Your branch or JBoyer's? I'm going to push the one that makes it optional. |
16:02 |
jeff |
(behind a config option for existing installs, i should say) |
16:02 |
Dyrcona |
jeff: Yeah, it's a setting on the login element. |
16:02 |
jeff |
yup, reviewed. :-) |
16:02 |
Dyrcona |
I'm going to test with two of ours copied from production: one I'll add the setting the other I won't. |
16:05 |
Dyrcona |
Oof. I might not have my SIP test code on this vm any more... |
16:16 |
Dyrcona |
Then, I have to modify the script for different file locations.... :) |
16:21 |
Dyrcona |
And, it works for me! |
16:21 |
Dyrcona |
After all these years.... :) |
10:58 |
|
ian1 joined #evergreen |
11:32 |
|
ian1 joined #evergreen |
11:32 |
|
mantis joined #evergreen |
11:32 |
mantis |
could use some guidance on this error I got when applying 3.11.7 to our servers from 3.11.4 |
11:32 |
mantis |
/usr/share/perl5/Error.pm:465', ilsevent: '', textcode: 'RESOURCE_IN_USE', desc: 'Resource is in use at this time'} |
11:33 |
mantis |
I ran a browser test after compiling the Angular files |
11:33 |
mantis |
We never use the booking module so this is kind of new to me |
11:33 |
mantis |
'/usr/local/share/perl/5.30.0/OpenILS/Application/Booking.pm:213 /usr/local/share/perl/5.30.0/OpenSRF/Application.pm:628 |
11:35 |
Dyrcona |
mantis: Disable booking in the configuration. |
11:35 |
Dyrcona |
If you don't use it. |
11:35 |
mantis |
thanks! |
11:52 |
mantis |
Firefox 128.0 (Ubuntu 0.0.0) CashReportsComponent alerts the user if end date is before start date FAILED |
11:52 |
mantis |
NullInjectorError: R3InjectorError(DynamicTestModule)[PrintService -> PrintService]: |
11:52 |
mantis |
NullInjectorError: No provider for PrintService! in vendor.js (line 66717) |
11:52 |
mantis |
error properties: Object({ ngTempTokenPath: null, ngTokenPath: [ 'PrintService', 'PrintService' ] }) |
12:05 |
|
jihpringle joined #evergreen |
12:07 |
mantis |
I'm also testing out Hatch and the Print button within Cash Reports |
12:07 |
Dyrcona |
Don't know. Open a bug on Lp. |
12:07 |
mantis |
okie doke |
12:08 |
mantis |
Dyrcona++ |
12:08 |
Dyrcona |
I am almost never run those tests. |
12:13 |
mantis |
looks like we need a template for the cash reports now because as far as I know, with Hatch enabled, it doesn't print |
12:50 |
Stompro |
Anyone here work with Cloudlibrary? |
12:51 |
Stompro |
I've been asked to setup auth, and I'm having trouble with them not being willing to explain how they want to encrypt sip2. |
15:27 |
Dyrcona |
No, commit isn't necessary with cherry-pick. |
15:29 |
mantis |
thank you ! Dyrcona++ |
15:30 |
Dyrcona |
Oh. I just noticed it looks like your original branch was based on something else... |
15:32 |
Dyrcona |
hey! That's already a lot of signoffs. I wonder if I should even bother testing this... :) |
15:32 |
Dyrcona |
mantis++ |
15:32 |
mantis |
ah I see |
15:32 |
mantis |
that must had been it then |
10:03 |
mantis |
this is the command |
10:03 |
mantis |
ng build --configuration production |
10:04 |
JBoyer |
The npm install step is what I'm wondering about, that's what pulls down things like angular core and friends. |
10:04 |
mantis |
I know it's technically outdated |
10:05 |
mantis |
also run into this after running CHROME_BIN=/usr/bin/chromium-browser npm run test |
10:05 |
mantis |
23 07 2024 09:59:20.061:ERROR [karma-server]: Error: Found 1 load error |
10:05 |
mantis |
at Server.<anonymous> (/home/opensrf/Evergreen-ILS-3.11.6-biblio/Open-ILS/src/eg2/node_modules/karma/lib/server.js:243:26) |
10:05 |
mantis |
at Object.onceWrapper (node:events:631:28) |
10:05 |
mantis |
at Server.emit (node:events:529:35) |
10:05 |
mantis |
at Server.emit (node:domain:489:12) |
10:05 |
mantis |
at emitListeningNT (node:net:1851:10) |
10:05 |
|
ian1 joined #evergreen |
10:06 |
JBoyer |
is there anything in /home/opensrf/Evergreen-ILS-3.11.6-biblio/Open-ILS/src/eg2/node_modules and was a different version of Evergreen installed previously? |
10:06 |
mantis |
I'll check |
12:19 |
JBoyer |
To this day I do not understand rooibos tea. (At least that one, I know there's more than one type.) |
12:40 |
Dyrcona |
Looks like my problem was nginx needed a restart. |
12:41 |
* Dyrcona |
has never had rooibos tea. |
12:42 |
Dyrcona |
Ok. gonna test my fake upgrade from 3.14.0 to 3.14.1, then I'll do 3.14.1 to 3.14.2.... :) |
12:46 |
Dyrcona |
Think I'll do somethin' unorthodox... I'll reinstall the 3.14.0 schema and data, then run the db upgrade, and then install 3.14.1. After that, I'll run the tests. |
13:01 |
|
cbrown_isl joined #evergreen |
13:03 |
Dyrcona |
So, that works... |
13:05 |
|
cbrown joined #evergreen |
14:18 |
sandbergja |
It looks good to me! |
14:19 |
Dyrcona |
OK. I moderately prefer deleting the line over modifying it. |
14:19 |
sandbergja |
agree, it does seem cleaner |
14:21 |
Dyrcona |
OK. That's the one we'll go with. I'm basically done. I'll update the Lp bug and add some instructions for testing it in a comment. I exercised just about everything it does. |
14:24 |
Dyrcona |
I'm going to give that commit another test or two first. |
14:25 |
sandbergja |
woo hoo! Thanks so much, Dyrcona! |
14:33 |
Dyrcona |
sandbergja++ This branch will make certain aspects of the releases so much easier. |
14:34 |
sandbergja |
I'm glad! And we as a community can keep chipping away at it. |