Time
S
Nick
Message
00:37 pameyer joined #dataverse
00:40 pameyer
pdurbin , poikilotherm - if I'm remebering right, one of the ORCID 2.0 requirements involved saving something (scopes or grants, not completely sure) persistently, even though dataverse wasn't going to be using them
00:41 pameyer
wasn't an OAuth2 functional requirement, but came up in review
01:01 sivoais joined #dataverse
10:47 pdurbin
they had a lot of requirements
10:47 pdurbin
around branding, etc
11:04 poikilotherm joined #dataverse
11:15 pdurbin
if that part of the review was written down (and I could find it), I'd show it to poikilotherm
11:15 poikilotherm
Someone mentioned my name
11:16 poikilotherm
pdurbin: did you create a branch in IQSS/dataverse?
11:17 poikilotherm
I couldn't find one and can't remember what was the outcome...
11:18 poikilotherm
Digging into Payara today...
11:30 pdurbin
poikilotherm: is there anything you can drag to "doing" at https://github.com/orgs/IQSS/projects/16 ?
11:33 poikilotherm
Dunno. Those are some old issues and we are trying to look for https://github.com/IQSS/dataverse/pull/5933
11:33 poikilotherm
We want to dig for the root cause
11:33 poikilotherm
Getting nearer...
11:33 pdurbin
You can also create a "note" under "doing" if that's easier.
11:34 pdurbin
And covert it to an issue later.
11:34 pdurbin
I'm not sure what branch you're talking about. I create a lot of branches. :)
11:37 poikilotherm
the payara branch where we can do breaking changes
11:43 pdurbin
I have not. Is there any plan? Any strategy? Or do we just start hacking?
11:48 pdurbin
https://github.com/poikilotherm/dataverse/tree/5292-small-container is 2313 commits behind develop. I think there were some good ideas over there and in https://github.com/IQSS/dataverse/issues/5292 but maybe you could start a fresh branch?
11:58 poikilotherm
Lemme create my issue and talk later
11:58 poikilotherm
Ok?
11:58 pdurbin
good idea :)
12:11 donsizemore joined #dataverse
12:20 poikilotherm
https://github.com/orgs/IQSS/projects/16#card-26908484
12:21 poikilotherm
We are still digging into this
12:21 poikilotherm
And I was a maker again :-D Created a label for Payara
12:29 pdurbin
thanks!
12:32 pdurbin
I can't tell from your screenshots if you are logged in to Dataverse or not. In https://github.com/IQSS/dataverse/issues/5907 I was logged in.
12:32 pdurbin
Your screenshots in https://github.com/IQSS/dataverse/issues/6216 I mean.
12:34 poikilotherm
It doesn't matter if you are logged in or not
12:34 poikilotherm
This is all happening way before there is any kind of authn or authz happening
12:35 poikilotherm
I added a commen
12:59 pdurbin
donsizemore: mornin! I see you're hacking away. Need anything?
12:59 donsizemore
@pdurbin morning. Q?
12:59 donsizemore
should createsequence.sql be run only in case of the API test suite, or with any branch build / warfile deploy?
13:00 pdurbin
To make things simple I would just always run it. Up to you.
13:04 donsizemore
should :IdentifierGenerationStyle be a switch, to allow a choice of sequentialNumber or randomString?
13:05 donsizemore
(or does this SQL just allow sequentialNumbers?)
13:07 donsizemore
(it looks like the latter)
13:19 pdurbin_m joined #dataverse
13:19 pdurbin_m
donsizemore: yeah, the latter. Enables.
13:44 pdurbin
I don't think we need a switch right now.
13:44 pdurbin
Defer. :)
13:45 pdurbin
poikilotherm: I don't see your commen
13:56 donsizemore
@pdurbin oops i pushed to master again https://github.com/IQSS/dataverse-ansible/commit/61d6b7d459c389f979384e9df27ccda4c5c8109f
13:58 pdurbin
donsizemore: no worries, should I give it a try?
13:59 donsizemore
let me merge it into branch #65
14:03 donsizemore
okay, try it now?
14:19 poikilotherm
Alright guys, pair programming done for today
14:40 pdurbin
poikilotherm: Java? Javascript?
14:40 poikilotherm
Java :-)
14:41 poikilotherm
We worked on #6216 to evaluate necessary manpower
14:41 poikilotherm
We do have a solution and it might even run on GF 4.1 :-D
14:41 pdurbin
We? Do you have a partner in crime?
14:41 poikilotherm
Aye.
14:42 pdurbin
Nice. The more the merrier.
14:42 poikilotherm
Dirk, he is developing the Open Access Monitor.
14:42 poikilotherm
https://open-access-monitor.de/#/
14:42 poikilotherm
He was into Java before and know a bit about JSF etc
14:43 poikilotherm
On my way to Berlin I will try to craft a PR to talk about.
14:43 pdurbin
He knows about JSF?!? Can he teach me?
14:43 poikilotherm
Nope, he's not very much above me :-D
14:43 poikilotherm
He quited that a while ago
14:43 pdurbin
T-T
14:44 poikilotherm
OK guys, gotta run. Catch a train ;-)
14:44 pdurbin
o/
14:44 poikilotherm
Maybe read you later. Depends on WLAN in ICE train
14:44 pdurbin
donsizemore: sorry, I'm a little turned around. Should I be testing master or 65_api_test_suite?
14:45 donsizemore
65
14:45 pdurbin
can do, thanks!
14:50 pdurbin
Giving this a whirl: ./ec2-create-instance.sh -g ec2config.yaml -r https://github.com/IQSS/dataverse.git -b develop -a 65_api_test_suite
14:59 pdurbin
fingers crossed as "TASK [dataverse : run test suite]" runs :)
15:00 pdurbin
I'm happy to see these above it:
15:00 pdurbin
- TASK [dataverse : allow authenticated users to add to root]
15:00 pdurbin
- TASK [dataverse : the root dataverse must be published]
15:04 pdurbin
donsizemore: it almost passed! [ERROR] Tests run: 103, Failures: 1, Errors: 1, Skipped: 4
15:04 donsizemore
what was the error?
15:04 donsizemore
(and the failure?)
15:08 pdurbin
donsizemore: posted to https://github.com/IQSS/dataverse-ansible/issues/65#issuecomment-535068370
15:08 pdurbin
I saw this one a lot yesterday too: SearchIT.testNestedSubtree:749
15:09 pdurbin
Huh, this one is new: ExternalToolsIT.testFileLevelTool1:130
15:18 donsizemore
do i have a next step there? or is this all deeper debugging
15:34 pdurbin
Well, you can go ahead and merge that 65 branch. Great stuff.
15:35 pdurbin
And standup I just said I'll add some sleep or whatever it takes to get the test suite passing.
16:49 donsizemore
everything works better well-rested!
17:02 donsizemore
@pdurbin do you want me to open an issue about the failing tests above?
17:18 poikilotherm joined #dataverse
18:04 pdurbin
donsizemore: maybe? I should go check if it's failing on phoenix first I guess.
18:04 poikilotherm
Aloa pdurbin
18:05 donsizemore
aloha awakea!
18:07 poikilotherm
Any news from over the ocean?
19:03 pdurbin
poikilotherm: Gustavo and I are about to have a call with donsizemore about automated testing. Would you like to join us? I think it's on Google Hangouts.
19:04 poikilotherm
Sry, I can't. Riding my train to Berlin :-D
19:04 poikilotherm
No WLAN, running on LTE tethering
19:04 poikilotherm
:-(
19:04 pdurbin
oh well
19:04 pdurbin
It's always this time. Every Wednesday.
19:05 poikilotherm
When do you start?
19:05 pdurbin
It's 3pm here.
19:05 poikilotherm
So you are late to the party?
19:05 poikilotherm
:-D
19:05 poikilotherm
Is it always around that time?
19:06 poikilotherm
Its late enough so kids are in bed, so I might join next week
19:33 pdurbin
poikilotherm: yes, always 3pm Wednesdays
19:33 poikilotherm
:-)
19:34 pdurbin
after sprint planning which is at 2pm
19:35 pdurbin
donsizemore: to answer your question from before, I don't think we need a new issue about failing tests because I *just* ran them against phoenix and they passed. So I'll continue to troubleshoot in EC2. (phoenix is a centos 6 VMware VM in our basement.)
20:25 poikilotherm
pdurbin you might consider taking a look at https://github.com/IQSS/dataverse/pull/6220
20:33 pdurbin
poikilotherm: I'm spinning it up with zipurl set to latest payara while we speak. :)
20:33 poikilotherm
Hihihi
20:33 pdurbin
I forget, does dataverse-kubernetes have a switch for Payara?
20:34 poikilotherm
Yes, but I didn't push it yet.
20:34 poikilotherm
I found some issues
20:34 poikilotherm
Like eating all of my 16G RAMfor breakfast
20:34 pdurbin
!
20:34 poikilotherm
(Was my fault)
20:34 pdurbin
:)
20:34 poikilotherm
(Didn't provide a memory limit)
20:35 pdurbin
Huh, the Perl installer should set a memory limit.
20:35 poikilotherm
I'm going to do add this to my PR for #63, but I can't test my patches right now - no WLAN :-/
20:35 poikilotherm
No perl installer around...
20:35 pdurbin
oh
20:35 poikilotherm
And those settings are the root of all evil
20:35 pdurbin
ok
20:36 poikilotherm
You do have native container memory limit support these days in Java 8
20:36 poikilotherm
You should use 'em ;-)
20:36 pdurbin
I'm not using containers. :)
20:36 poikilotherm
I know :-D
20:37 poikilotherm
Since I limited the Payara container to 2 gigs, it behaves like a pussycat
20:38 poikilotherm
Oh and I need to create an issue tomorrow... When creating a Dataverse with dataverse.siteUrl set, that unexpanded string is shown to the user
20:39 poikilotherm
At least I saw that on Payara 5
20:39 pdurbin
screenshot please
20:39 poikilotherm
Yeah, that's why I'll do it tomorrow
20:39 pdurbin
thanks!
20:39 poikilotherm
Need a testcluster online :-D
20:39 poikilotherm
(see above ;-) )
20:39 pdurbin
glassfish-setup.sh is taking a suspiciously long time to run
20:40 poikilotherm
You might try this on GF 4.1, too
20:40 poikilotherm
Had no chance to test yet
20:40 poikilotherm
In theory my changes so far should be ok with GF 4.1, too
20:40 poikilotherm
But this will need discussion as I lined out in the PR description
20:44 pdurbin
glassfish-setup.sh is still hanging :(
20:44 poikilotherm
Should I request a review from other devs, too? Like landreev/...?
20:44 poikilotherm
That doesn't sound nice
20:44 poikilotherm
Although this doesn't seem related to my changes
20:45 poikilotherm
Those are just about JSF things. The installer doesn't use those, right?
20:45 pdurbin
Sure, why not. Leonid has been using the spin up scripts. And Gustavo talked a lot about getting off Glassfish 4 in tech hours yesterday. I mentioned then that the spin up scripts support Payara.
20:46 pdurbin
Hmm, it finally timed out. asadmin did.
20:46 pdurbin
asadmin timed out with this... Unable to access the server to execute the command remotely. Verify the server is available.", "Command set-log-file-format failed."
20:48 poikilotherm
I have this command in neither init script of dataverse-k8s
20:49 poikilotherm
Not in Glassfish, neither in Payara
20:49 poikilotherm
Link?
20:52 pdurbin
https://github.com/IQSS/dataverse/blob/v4.16/scripts/installer/glassfish-setup.sh
20:53 pdurbin
Waiting for domain1 to start ....
20:53 pdurbin
No response from the Domain Administration Server (domain1) after 600 seconds.
20:53 poikilotherm
Strange
20:54 poikilotherm
I really try to remember what the differences where
20:54 poikilotherm
-h
20:55 pdurbin
I'm getting tired. I just typed `chown 644`.
20:56 poikilotherm
Yeah, that's really sth :-D
20:56 poikilotherm
Its 22:56 here and I wont be in Berlin till 1:00
20:56 poikilotherm
Train evac ahead
21:00 pdurbin
poikilotherm: I just left a comment with some attachments in your pull request. Heading home. Tired. Have a good night!
21:01 poikilotherm
Okidoki!
21:01 poikilotherm
Sleep well!
21:01 poikilotherm
And thx!
21:15 jkurtz-asu joined #dataverse
21:17 jkurtz-asu
hi all, quick question - would there be any sense / value in us attempting install a dv pilot on centos 8? or would that be completely insane?
22:57 pdurbin_m joined #dataverse
22:57 pdurbin_m
jkurtz-asu: hi! We talked about CentOS 8 a bit yesterday
22:58 pdurbin_m
Please see http://irclog.iq.harvard.edu/dataverse/2019-09-24#i_106785
