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

Using timezone: Eastern Standard Time
* thomz joins02:51
* dwilcox joins08:34
* coblej joins08:41
* dwilcox leaves08:42
* dwilcox joins08:43
* dwilcox leaves
* youn joins08:49
* coblej leaves09:04
* acoburn joins09:15
* ajs6f joins09:17
* mikeAtUVa joins09:35
* coblej joins09:50
* kestlund joins09:59
* coblej leaves10:07
* thomz leaves10:09
* peichman joins10:10
* github-ff joins10:21
[fcrepo4] acoburn opened pull request #1138: Upgrade to Jena 3.1.1 (master...fcrepo-2295) https://git.io/vXgos
* github-ff leaves
<acoburn>ajs6f: ^^^10:22
* coblej joins
<ajs6f>acoburn: Quick off the mark, you are.
* dwilcox joins10:40
* coblej leaves10:41
* coblej joins10:57
* dbernstein joins11:02
<awoods>dbernstein: Would you mind creating the fcrepo-audit-4.6.1-RC-1 tag? https://github.com/fcrepo4-exts/fcrepo-audit/tags11:06
dbernstein: see: https://github.com/fcrepo4-exts/fcrepo-mint/tags11:07
dbernstein: followed by an update to: https://docs.google.com/spreadsheets/d/1I_zTMxh2l2rf2wpafoTwhSTR5GZuEoaTcZmTKCI3xT4/edit?usp=sharing
* dbernstein leaves11:14
* dbernstein joins
* kefo joins11:17
* github-ff joins11:21
[fcrepo-audit] dbernstein tagged fcrepo-audit-4.6.1-RC-1 at 86c1aba: https://git.io/vXgy0
* github-ff leaves
* awoods leaves11:23
* awoods joins11:24
* awoods leaves11:27
* awoods joins11:28
* travis-ci joins11:29
fcrepo4-exts/fcrepo-audit#136 (fcrepo-audit-4.6.1-RC-1 - e35b7b4 : ajs6f): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-audit/compare/fcrepo-audit-4.6.1-RC-1
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-audit/builds/174533334
* travis-ci leaves
* dwilcox leaves11:46
<awoods> 11:49
* awoods leaves11:55
* awoods joins11:56
* coblej leaves11:57
* coblej joins11:58
* coblej leaves12:02
* awoods leaves12:04
* awoods joins12:12
* jonathangee leaves12:32
* ajs6f leaves12:43
* coblej joins12:50
* tolloid joins12:58
<dbernstein>awoods : I just updated the 4.6.1 release testing page - I was mistaken: there was a unit test failure.13:01
I’ve updated the release testing page: https://wiki.duraspace.org/display/FF/Release+Testing+-+4.6.1
<awoods>mikeAtUVa/acoburn/ruebot: could you please pull down the 4.6.1-RC of fcrepo4 and try a simple "mvn clean install"? https://github.com/fcrepo4/fcrepo4/tree/4.6.1-RC13:03
dbernstein: could you please describe you OSX version and java version in the wiki?13:04
* barmintor_meetin leaves13:08
<dbernstein>will do. such a sad day.13:13
<ajwagner>awoods: Just built fine for me on both Ubuntu 14.04 and 16.0413:21
<awoods>ajwagner: thanks, could you please update the wiki: https://wiki.duraspace.org/display/FF/Release+Testing+-+4.6.1#ReleaseTesting-4.6.1-SanityBuilds
* jonathangee joins13:22
<mikeAtUVa>awoods: it built for me too, and I updated the wiki... the error dbernstein has reported to that page I've seen in the past and will run fcrepo-http-api tests a few more times to see if it pops up.13:29
<acoburn>awoods: I tried the branch and I got errors in the DefaultPathLockManagerTest (after building twice)
<mikeAtUVa>awoods:... yup saw that error by doing an mvn clean install -rf :fcrepo-http-api13:30
<awoods>mikeAtUVa: I am a little surprised... I though this was addressed with: https://github.com/fcrepo4/fcrepo4/commit/21f202fd998cf99ae8443702dcd1f99d861e33b013:31
<mikeAtUVa>awoods: that was the unit tests.13:32
<acoburn>awoods: that's the error I'm seeing
<awoods>acoburn: It sounded like mikeAtUVa "saw that error by doing..." as well.13:33
<mikeAtUVa>awoods: the integration tests were supposed to get intense scrutiny and rewritten using awatility which would have determined the exact cause of this intermittent test error.
<acoburn>awoods: yes
<awoods>mikeAtUVa: and we agreed not to hold the patch up on the integration test... I believe.13:34
<mikeAtUVa>awoods: agreed.
<awoods>mikeAtUVa: that does not, however, address the error with: DefaultPathLockManagerTest
<acoburn>awoods: it's fine by me, I'm just reporting what I saw
<mikeAtUVa>awoods: It's my understanding that we've closed a vast majority of the exposure to concurrency issues but a small window still remains. One that's infrequently exercised by the integration tests.13:35
<awoods>mikeAtUVa: yes... and we are ok with that, for now. The question is about the unit test.
<mikeAtUVa>acoburn: it was relatively easy for me to debug the error with trace level logging before, if you can get me that output I'll look into those errors.13:37
<awoods>mikeAtUVa/acoburn: I am not sure it makes sense to "Won't Fix" this issue, thoughts? https://jira.duraspace.org/browse/FCREPO-227413:38
<mikeAtUVa>awoods, acoburn: hang on a sec... that commit (https://github.com/fcrepo4/fcrepo4/commit/21f202fd998cf99ae8443702dcd1f99d861e33b0) doesn't look like the fix I recall.13:39
awoods, acoburn: This was the critical part of that fix.. https://github.com/fcrepo4/fcrepo4/pull/1124/commits/c6b0bc99053a376e969a724c34900055e588196b13:40
<acoburn>mikeAtUVa: yes, that is exactly the line we need13:41
<mikeAtUVa>awoods, acoburn: why was only one of the two commits from that PR merged in?
* th5 joins13:42
<awoods>mikeAtUVa: I have no idea... probably a mistake.
<mikeAtUVa>awoods, acoburn: er... how... not why. Yeah... either way, it can be fixed easily enough.
* coblej leaves
<awoods>mikeAtUVa: that is why we are doing the testing.13:43
mikeAtUVa: I will pull the second commit into both master and the 4.6.1-RC branch
<mikeAtUVa>awoods: :) And only acoburn's machine is fast enough to catch that bug...
<awoods>acoburn: while I do the commits, can you respond to: https://wiki.duraspace.org/display/FEDORA451/RESTful+HTTP+API+-+Transform?focusedCommentId=78161391#comment-7816139113:44
* coblej joins13:49
* bseeger joins
<awoods>mikeAtUVa: it looks like your fix is in master... because it apparently was included in this PR/commit: https://github.com/fcrepo4/fcrepo4/pull/112213:56
mikeAtUVa: however, that same PR is not in the 4.6.0-maintenance branch... which I will address now.13:57
* github-ff joins14:01
[fcrepo4] awoods opened pull request #1139: Fixed a bug that cause intermittent test failures. (4.6.0-maintenance...4.6.0-maintenance) https://git.io/vX2qT
* github-ff leaves
* github-ff joins
[fcrepo4] awoods opened pull request #1140: Fixed a bug that cause intermittent test failures. (4.6.1-RC...4.6.0-maintenance) https://git.io/vX2qs
* github-ff leaves
<awoods>acoburn: would you mind pushing the two new PRs in this ticket?14:04
<acoburn>awoods: they look good — do you want me to wait for travis?14:06
<awoods>acoburn: sure, thanks.
dbernstein: ping14:08
<dbernstein>awoods : yes?14:09
<awoods>dbernstein: I saw you -Pconnector error in the wiki
dbernstein: that seems very very odd
dbernstein: are you sure you are on the 4.6.1-RC branch?
dbernstein: git status14:10
Perhaps it occurred on your side but you didn’t notice it? The build succeeded.14:12
<awoods>dbernstein: good point... running again
dbernstein: right you are...14:14
<acoburn>awoods: you know, perhaps this page shouldn't be part of the 4.7 release docs: https://wiki.duraspace.org/display/FEDORA47/RESTful+HTTP+API+-+Transform
awoods: or in the 4.x docs
<awoods>dbernstein: the -Pconnector test should not be on this wiki page, because in 4.6 the file-connector was still a part of the core codebase.14:15
<dbernstein>okay cool.14:16
<awoods>acoburn: correct... and the transform page is referenced by three other pages:14:17
acoburn: shall we drop this page? or update it with the new approach? https://wiki.duraspace.org/display/FEDORA47/Indexing+Transformations14:20
<acoburn>awoods: the question, to me, is whether the Fedora documentation is responsible for documenting extension modules. I would say it is not14:22
<awoods>acoburn: at this point, I agree
<acoburn>awoods: though there are other places on the wiki where such extension modules are documented
awoods: I think that's a good separation14:23
<awoods>acoburn: on the wiki, we should note that the various extensions exist, but then point to their respective githubs
<acoburn>awoods: yes, I agree
<awoods>acoburn: there is some good stuff mixed in here: https://wiki.duraspace.org/display/FEDORA47/Setup+Camel+Message+Integrations14:24
acoburn: but also some stuff that could-be/is documented in github
<acoburn>awoods: somehow I thought that was separate from the Fedora docs
awoods: sounds like there could be some wiki gardening14:25
<awoods>acoburn: interested?
<acoburn>awoods: not really, but you might be able to twist my arm14:26
<awoods>acoburn: that would be nice, unless someone else is more interested... bseeger?14:27
* tolloid leaves14:29
* github-ff joins14:30
[fcrepo4] acoburn pushed 1 new commit to 4.6.0-maintenance: https://git.io/vX23O
fcrepo4/4.6.0-maintenance 0f38471 Andrew Woods: Fixed a bug that cause intermitted test failures. (#1139)...
* github-ff leaves
* github-ff joins14:31
[fcrepo4] acoburn closed pull request #1140: Fixed a bug that cause intermittent test failures. (4.6.1-RC...4.6.0-maintenance) https://git.io/vX2qs
* github-ff leaves
* tolloid joins
<bseeger>awoods: not exactly sure what's involved here - essentially clean up extension module info from the main fedora wiki?14:33
<awoods>bseeger: yes, the same updates need to happen in two wiki spaces: 4.7 and 4.x
bseeger: These four pages:14:34
<awoods> https://wiki.duraspace.org/display/FEDORA47/Integration+Services
<awoods> https://wiki.duraspace.org/display/FEDORA47/RESTful+HTTP+API
<acoburn>awoods: it's not a big issue, but I think the maintenance branches would be better named if they didn't have the trailing 0, i.e: 4.6-maintenance or 4.6.x-maintenance14:38
<awoods>acoburn: agreed, whikloj and I talked about that yesterday.
acoburn: that will be the case going forward.
<acoburn>awoods: great, thanks
<bseeger>awoods: sure, I can take a shot at it, though I'll probably have some questions when I get to it (w/i next few days).14:43
<f4jenkins>Project fcrepo-module-auth-rbacl build #1198: UNSTABLE in 3 min 23 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/1198/14:45
* dwilcox joins14:50
* travis-ci joins14:52
fcrepo4/fcrepo4#4842 (4.6.0-maintenance - 0f38471 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/2984e2136936...0f38471d9925
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/174580455
* travis-ci leaves
* travis-ci joins14:53
fcrepo4/fcrepo4#4843 (4.6.1-RC - 268b484 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/2984e2136936...268b4847fa5e
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/174580542
* travis-ci leaves
<f4jenkins>Yippee, build fixed!14:54
Project fcrepo-module-auth-rbacl build #1199: FIXED in 3 min 19 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/1199/
* coblej leaves15:07
* coblej joins15:09
* ajs6f joins15:10
* coblej leaves15:14
* kestlund leaves15:15
* youn leaves15:22
* coblej joins15:25
* dwilcox leaves15:28
* dwilcox joins15:29
<awoods>dbernstein: re:your binary version issue...15:42
dbernstein: could it just be a browser caching situation?
* dwilcox leaves16:09
* dhlamb_ leaves16:20
* th5 leaves16:32
* mikeAtUVa leaves16:57
* bseeger leaves17:04
* coblej leaves17:05
* acoburn leaves17:13
* github-ff joins17:14
[fcrepo4] awoods tagged list at c5a9b37: https://git.io/vX2iC
* github-ff leaves
* github-ff joins
[fcrepo-module-auth-rbacl] awoods tagged fcrepo-module-auth-rbacl-4.6.1-RC-2 at 6a34efe: https://git.io/vX2i4
* github-ff leaves
* github-ff joins
[fcrepo-module-auth-xacml] awoods tagged fcrepo-module-auth-xacml-4.6.1-RC-2 at 95f045a: https://git.io/vX2iR
* github-ff leaves
* github-ff joins
[fcrepo-module-auth-webac] awoods tagged fcrepo-module-auth-webac-4.6.1-RC-2 at 87176e8: https://git.io/vX2ig
* github-ff leaves
* github-ff joins17:15
[fcrepo-mint] awoods tagged fcrepo-mint-4.6.1-RC-2 at 5a322a0: https://git.io/vX2i2
* github-ff leaves
* github-ff joins
[fcrepo-audit] awoods tagged fcrepo-audit-4.6.1-RC-2 at cb3d62d: https://git.io/vX2iV
* github-ff leaves
<dbernstein>awooods: it could be. I’ll double check it.
<awoods>dbernstein: thanks17:16
* travis-ci joins17:17
fcrepo4-exts/fcrepo-mint#44 (fcrepo-mint-4.6.1-RC-2 - 4d82f50 : ajs6f): The build failed.
Change view : https://github.com/fcrepo4-exts/fcrepo-mint/compare/fcrepo-mint-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-mint/builds/174620538
* travis-ci leaves
<awoods>dbernstein: also, I just pushed a new tag for the modules due to the missing commit on fcrepo4
<f4jenkins>Project fcrepo-module-auth-rbacl build #1200: UNSTABLE in 3 min 22 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/1200/
* travis-ci joins
fcrepo4/fcrepo-module-auth-xacml#146 (fcrepo-module-auth-xacml-4.6.1-RC-2 - 338af10 : ajs6f): The build passed.
Change view : https://github.com/fcrepo4/fcrepo-module-auth-xacml/compare/fcrepo-module-auth-xacml-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4/fcrepo-module-auth-xacml/builds/174620403
* travis-ci leaves
* travis-ci joins17:18
fcrepo4-exts/fcrepo-audit#137 (fcrepo-audit-4.6.1-RC-2 - e35b7b4 : ajs6f): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-audit/compare/fcrepo-audit-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-audit/builds/174620556
* travis-ci leaves
* travis-ci joins17:21
fcrepo4/fcrepo-module-auth-rbacl#103 (fcrepo-module-auth-rbacl-4.6.1-RC-2 - 50a9ab2 : ajs6f): The build passed.
Change view : https://github.com/fcrepo4/fcrepo-module-auth-rbacl/compare/fcrepo-module-auth-rbacl-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4/fcrepo-module-auth-rbacl/builds/174620375
* travis-ci leaves
* travis-ci joins17:22
fcrepo4-exts/fcrepo-webapp-plus#177 (fcrepo-webapp-plus-4.6.1-RC-2 - fa6274f : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-webapp-plus/compare/fcrepo-webapp-plus-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-webapp-plus/builds/174620580
* travis-ci leaves
* tolloid leaves
* peichman leaves17:25
* travis-ci joins17:28
fcrepo4/fcrepo4#4845 (list - c5a9b37 : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/list
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/174620260
* travis-ci leaves
<f4jenkins>Yippee, build fixed!17:29
Project fcrepo-module-auth-rbacl build #1201: FIXED in 3 min 19 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/1201/
* ajs6f leaves17:35
* whikloj joins17:38
awoods: ping
<whikloj>awoods: I missed changing the tag in vagrant, do you want me to push a change onto 4.6.1-RC or open a quick PR?17:39
awoods: It's still pulling 4.6.0 artifacts
* travis-ci joins
fcrepo4/fcrepo4#4844 (fcrepo-4.6.1-RC-2 - 268b484 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/fcrepo-4.6.1-RC-2
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/174620258
* travis-ci leaves
<whikloj>awoods: or did I miss something ^^17:40
<awoods>whikloj: sure, a PR would be good
<awoods>whikloj: half of mikeAtUVa's PR did not make it into 4.6.0-maintenance
whikloj: we worked through it this morning...17:41
whikloj: ...adding the commit to fcrepo4...
whikloj: now I have updated the tags and artifacts.
<whikloj>awoods: ok seeing as we haven't released the initial RC, do you want to add tags to all the rest? Or just re-tag for RC-1?
<awoods>whikloj: I retagged all of the modules with RC-217:42
<whikloj>awoods: ok
<awoods>whikloj: tags are cheap
<whikloj>awoods: So then should I wait and put a PR for 4.6.1-RC-2 ?17:43
<awoods>whikloj: you can do that now.17:44
whikloj: https://github.com/fcrepo4-exts/fcrepo-webapp-plus/releases/tag/fcrepo-webapp-plus-4.6.1-RC-217:45
<whikloj>awoods++ Doing a test run now17:46
* whikloj leaves17:54
* whikloj joins17:55
<awoods>whikloj: let me know if your test run is successful... then I will send the RC email
<whikloj>awoods: ok
awoods: Should we clean out the release testing page?17:56
<awoods>whikloj: because of RC-2?
* awoods lunch17:57
<whikloj>awoods: yes, all my tests were on RC-1
<awoods>whikloj: I think it is ok17:58
<whikloj>awoods: ok, I'll tag you on my vagrant PR.
awoods: enjoy your lunch, next time try it in the middle of the day. My favourite time for for lunch17:59
<awoods>whikloj: if you could test dbernstein's issue in the "Database Tests" section, that would be good.18:00
whikloj: I was not able to reproduce it
<whikloj>awoods: ok
<awoods>kefo: Will you be able to test the "concurrent creation" issue?18:01
whikloj: please work with kefo, if/when he responds... lunch.
* kefo leaves18:09
* github-ff joins18:13
[fcrepo4-vagrant] whikloj opened pull request #70: Update tag and version (4.6.1-RC...update-tag) https://git.io/vX277
* github-ff leaves
<dbernstein>awoods: it was a caching issue.18:23
<awoods>dbernstein: good... please update the testing page when you have a moment.18:24
* github-ff joins18:25
[fcrepo4-vagrant] awoods closed pull request #70: Update tag and version (4.6.1-RC...update-tag) https://git.io/vX277
* github-ff leaves
<whikloj>awoods/dbernstein: So I don't need to test MySQL?18:50
<awoods>whikloj: you are welcome to...18:51
whikloj: but if you could do postgres, that is missing18:52
* whikloj leaves19:33
* whikloj joins
* github-ff joins19:34
[fcrepo4-vagrant] awoods tagged fcrepo4-vagrant-4.6.1-RC-2 at be888bc: https://git.io/vXaej
* github-ff leaves
* whikloj leaves19:37
* whikloj joins19:56
* whikloj leaves20:00
* tolloid joins20:09
* dwilcox joins20:13
* tolloid leaves20:31
* dbernstein leaves22:04
* dbernstein joins22:09
* dwilcox leaves23:34

Generated by Sualtam