Log of the #fcrepo channel on chat.freenode.net

Using timezone: Eastern Standard Time
* benpennell leaves00:04
* dbernstein leaves01:26
* bryjbrown joins06:49
* bryjbrown leaves07:55
* awoods joins08:23
* dhlamb joins08:34
* dwilcox joins08:37
* benpennell joins09:15
* yamil joins09:18
* bseeger joins09:25
* peichman joins09:44
* escowles_ leaves09:58
* bryjbrown joins10:01
* bryjbrown leaves10:32
* whikloj joins10:46
awoods/peichman: good morning folks, did we all figure out the NullPointer issue or was it just me?10:47
<awoods>whikloj: no news here
<whikloj>awoods: https://github.com/hibernate/hibernate-ogm/blob/master/core/src/test/resources/jbossts-properties.xml#L4810:48
<peichman>whikloj: nothing new from me
<whikloj>Add a jbossts-properties.xml beside logback.xml inside fcrepo with only that key defined to a writeable location and BOOM solved10:49
<awoods>well well well
<whikloj>So this is essentially a file permission issue
* dbernstein joins
<whikloj>There must be some way to define that variable without the jbossts-properties.xml file, but I haven't dug into it too far.10:50
But I am running 4.7.5-RC-1 on my CLAW vagrant now
which would explain why some different setups work and others don't. Could depend on the user running tomcat or the user.home defined or permissions10:51
I did also find this about defining the location of that file. http://docs.jboss.org/jbosstm/5.0.0.M1/guides/narayana-jta-installation_guide/chap-JBossJTA_Installation_Guide-Test_Chapter.html10:52
Lastly, I was finding the NullPointer in my localhost.2018-01-18.log file, but catalina.out was showing the error described here.10:54
https://forum.hibernate.org/viewtopic.php?f=31&t=1038012
Which is how I resolved it as they suggested adding the config file because of permissions
* bseeger1 joins11:03
* bseeger leaves11:06
* dwilcox leaves11:47
* dwilcox joins11:49
* f4jenkins leaves11:52
* f4jenkins joins11:53
* bseeger1 leaves12:01
* bseeger joins12:04
* bseeger leaves
* github-ff joins12:30
[fcrepo-specification] awoods pushed 1 new commit to master: https://git.io/vNuM4
fcrepo-specification/master fe7275e Simeon Warner: Add Ben Pennell as contributor (#322)
* github-ff leaves
* peichman leaves12:55
* github-ff joins12:56
[fcrepo4-vagrant] dbernstein tagged fcrepo4-vagrant-4.7.5-RC-1 at 4.7.5-RC: https://git.io/vN3PN
* github-ff leaves
* peichman joins12:57
<dbernstein>whikloj and peichman: So regarding the naranaya issue, it seems that we have a potential fix? ie adding the jbossts-properties.xml into fcrepo-webapp? Or is whikloj persuing other approach that doesn’t require the new xml file?13:04
<whikloj>dbernstein: I was hoping not to have to add this file because then we need to worry about another configuration option. But I'm not clear if we could configure this some other way13:05
dbernstein: like do we create a new variable for this, or should we just have it default to the fcrepo.home location?13:07
<dbernstein>whikloj: if we put it alongside logback.xml, won’t it get picked up automatically? ie if we can do it without adding any extra configuration that would obviously be optimal.13:11
<whikloj>dbernstein: right but we are setting where narayana puts its transaction log, so we need it to be configurable.
dbernstein: for instance mine was <entry key="ObjectStoreEnvironmentBean.objectStoreDir">/opt/fcrepo/jaredsObjectStore</entry>13:12
dbernstein: I guess putting it under the fcrepo.home system property is the easiest13:13
<dbernstein>whikloj: That seems straightforward. Do you see any downside in going that route?13:14
<whikloj>dbernstein: not sure, I'll put together a PR and we can hash it out.
<dbernstein>whikloj++
<whikloj>dbernstein: Do you think I should put all the configurable options in the jbossts-properties.xml file or just the one we need?13:16
dbernstein: actually we already define a couple system properties that seem close. I'll reuse one of them13:24
https://github.com/fcrepo4/fcrepo4/blob/bcbf9d4509dca40848b5db68d849edf938eeae75/fcrepo-kernel-modeshape/src/main/java/org/fcrepo/kernel/modeshape/spring/DefaultPropertiesLoader.java#L51-L52
* bseeger joins13:32
<dbernstein>whikloj++
benpennell: I saw your comment about failing sanity building on windows and copied the contents in the sanity builds matrix.13:44
<benpennell>okay thanks
<dbernstein>benpennell: can you do me a favor and check if you’re seeing the same problem on 4.7.4?13:45
It would be good if we can determine whether this is a new issue with 4.7.5, an issue with windows 10 or something in your environment.13:46
<benpennell>dbernstein: yes, i checked and that fails too. Some but not all the same tests fail in 4.7.0 as well. aaron b indicated that its an issue that other people have run into before (I believe he said yinlin had the same problem). He said it seems like the tests pass in windows in a virtualized environment (which is how aaron ran the tests), but not in a direct windows install13:47
<dbernstein>benpennell: good to know.13:49
<benpennell>dbernstein: it sounded like they tried to figure out the issue before but weren't successful
i'm not sure how much time to invest in it, but it is a bit worrisome13:51
* bseeger leaves
* dwilcox leaves
<dbernstein>bpennell: is there a record of yinlin and aaron’s discussion? I’d like to review it...13:53
If not we can wait for aaron to get on IRC...13:55
<benpennell>dbernstein: sent you the conversation from yesterday, in slack since it'd be too long a message for irc13:57
<dbernstein>thanks - I’m looking at it now.
* travis-ci joins14:01
fcrepo4-labs/Fedora-API-Test-Suite#53 (Http-Head-Test-Modifications - 4bd737e : Jorge Abrego): The build passed.
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/commit/4bd737e5cf40
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/330946119
* travis-ci leaves
<awoods>dbernstein: fyi: http://irclogs.fcrepo.org/
* github-ff joins
[fcrepo4] whikloj opened pull request #1277: Add jbossts-properties.xml file (4.7-maintenance...fcrepo-2664) https://git.io/vNuFe
* github-ff leaves
<whikloj>People who have used this property before -> https://github.com/fcrepo4/fcrepo4/blob/4.7-maintenance/fcrepo-kernel-modeshape/src/main/java/org/fcrepo/kernel/modeshape/spring/DefaultPropertiesLoader.java#L51-L5214:04
Please check this PR and let me know if I'm wrong https://github.com/fcrepo4/fcrepo4/pull/1277/files14:05
<dbernstein>whikloj: I’ll take a look.14:06
* dwilcox joins14:11
<whikloj>dbernstein++14:13
* peichman leaves14:20
<dbernstein>whikloj: would you mind issuing the PR against 4.7.5-RC?
<whikloj>dbernstein: sure
<dbernstein>thansk.14:21
* github-ff joins14:22
[fcrepo4] whikloj opened pull request #1278: Add jbossts-properties.xml file (the release branch remix) (4.7.5-RC...fcrepo-2664) https://git.io/vNuNi
* github-ff leaves
* peichman joins14:23
* peichman leaves14:27
* peichman joins14:37
* dbernstein_ joins15:14
* dbernstein leaves15:17
* github-ff joins16:08
[fcrepo4] dbernstein pushed 1 new commit to 4.7.5-RC: https://git.io/vNzkh
fcrepo4/4.7.5-RC 2fb8f5e Jared Whiklo: Add jbossts-properties.xml file (#1278)
* github-ff leaves
* travis-ci joins16:28
fcrepo4/fcrepo4#5292 (4.7.5-RC - 2fb8f5e : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/8186b971bd8c...2fb8f5e6b43c
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/330990639
* travis-ci leaves
* peichman leaves16:44
* whikloj leaves17:25
* 7JTADC36C joins17:47
fcrepo4-labs/Fedora-API-Test-Suite#55 (master - da7a4a0 : Andrew Woods): The build passed.17:48
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/compare/19bb7d2c22ac...da7a4a0d7483
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/331025984
* 7JTADC36C leaves
* dhlamb leaves18:11
* benpennell leaves18:21
* yamil leaves19:16
* dwilcox leaves21:27