Log of the #duraspace-ff channel on chat.freenode.net

Using timezone: Eastern Standard Time
* longshou leaves01:56
* scossu joins04:03
* scossu leaves04:24
* scossu joins
* scossu leaves05:25
<pivotal-bot>Esme Cowles added comment: "Yes, when I create a new resource and get the NTriples or Turtle, I'm seeing lots duplicate rdf:type triples..." https://www.pivotaltracker.com/story/show/7096562007:47
Esme Cowles added "rdf:type triples are duplicated in output" https://www.pivotaltracker.com/story/show/7175153607:50
Esme Cowles added comment: "See #71751536" https://www.pivotaltracker.com/story/show/70965620
* dwilcox joins07:58
* dwilcox_ joins08:28
* dwilcox leaves08:31
<pivotal-bot>Andrew Woods added comment: "reference: #70965620" https://www.pivotaltracker.com/story/show/7175153608:55
Andrew Woods delivered "Update REST API docs with new POST/PUT behavior for creating objects" https://www.pivotaltracker.com/story/show/7096562008:56
* ksclarke joins09:39
* tecoripa joins09:41
* escowles leaves09:53
<ksclarke>awoods: I see that iiif image in my browser but not on my phone... perhaps confluence is not showing large images to small mobile screens?09:54
<awoods>ksclarke: I do not see the image in my browser09:55
<ksclarke>wait, now I don't
must have just been cached for me
<awoods>ksclarke: is the text on your wiki page the most recent?10:01
ksclarke: it would be interesting to know if the server upgrade missed your more recent edits.10:02
<ksclarke>when was the upgrade? the page was just added monday night; I wasn't sure what sort of narrative was needed/wanted (beyond what the image illustrates) so the text is definitely just a rough first draft, for comments10:05
awoods
okay, image added back now
<awoods>ksclarke: sure, I am not questioning the completeness of the draft from a content perspective, but rather from a server-upgrade perspective.10:06
* ksclarke nods
<awoods>ksclarke: the server upgrade was last week and the weekend. It seems fairly certain that the image was lost in that process... and I was wondering if other edits may have been lost a well.10:07
<ksclarke>so the image/page wasn't created until monday night so I don't think the server upgrade was a factor10:08
awoods ^
<awoods>ksclarke: ok
<ksclarke>not sure what would have happened... because it was displaying for me in the page on creation10:09
* longshou joins10:13
* awoods shrugs
<pivotal-bot>Andrew Woods started "Run new Checkstyle rules across code base and fix problems resulting from the new rules" https://www.pivotaltracker.com/story/show/7126717610:16
<mikeAtUVa>awoods: is there a reason the modifications to the FileSystemConnector that detected file changes is no longer with us?10:21
<awoods>mikeAtUVa: when was it with us?10:26
mikeAtUVa: and I was under the impression that it may now be in the ModeShape connector.10:27
<mikeAtUVa>awoods: I thought that's what https://github.com/fcrepo4-labs/fcrepo-filesystem-modeshape-federation-connector/tree/master/src/main/java/org/fcrepo/connector/filesystem did, but I didn't look very closely.10:30
awoods: I'm just trying to come up with the most useful way of handing updates to the underlying file system. I'm not sure I've found a good way to handle a fixity check on a file whose metadata is cached but has been modified since it was cached.10:31
<awoods>mikeAtUVa: Have you investigated whether filesystem notifications are now available from the ModeShape FilesystemConnector?10:32
<mikeAtUVa>awoods: there's this package org.modeshape.jcr.federation.spi.ConnectorChangeSet class (that it looks like Adam had some part in) that would seem to be for that purpose.10:33
<awoods>mikeAtUVa: As a note, fcrepo-filesystem-modeshape-federation-connector was a wreck due to the fact that it copied over ModeShape's code, then reformatted it and then mixed in new functionality.10:34
<mikeAtUVa>awoods: understood. I'm much happier with the new minimal class.
<awoods>mikeAtUVa: I suspect there is a way to get filesystem events out of the ModeShape connector. A quick question to rhauch over on #modeshape may resolve that.10:36
<pivotal-bot>Andrew Woods edited "Run new Checkstyle rules across code base and fix problems resulting from the new rules" https://www.pivotaltracker.com/story/show/7126717610:39
* github-ff joins10:40
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/NdvC2w
fcrepo4/master 7e3f87d Andrew Woods: Address checkstyle issues in preparation for tighter enforcement...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo4/commit/7e3f87d3dd9d20d627f87e098acbeb734b37454a" https://www.pivotaltracker.com/story/show/7126717610:41
Andrew Woods delivered "Run new Checkstyle rules across code base and fix problems resulting from the new rules" https://www.pivotaltracker.com/story/show/71267176
Andrew Woods started "Turn on checkstyle rules for tests for jms indexer project" https://www.pivotaltracker.com/story/show/7132380410:47
Andrew Woods added "Run new Checkstyle rules across jms-indexer codebase" https://www.pivotaltracker.com/story/show/7176400010:49
Andrew Woods started "Run new Checkstyle rules across jms-indexer codebase" https://www.pivotaltracker.com/story/show/71764000
* travis-ci joins11:00
[travis-ci] fcrepo4/fcrepo4#2021 (master - 7e3f87d : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/8af7da1cce5c...7e3f87d3dd9d
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/25700431
* travis-ci leaves
* tecoripa leaves
* tecoripa joins11:10
* longshou leaves11:21
* nikhiltri joins11:22
* github-ff joins11:34
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: http://git.io/98dxJA
fcrepo-jms-indexer-pluggable/master a8ccb20 Andrew Woods: Address checkstyle issues in preparation for tighter enforcement...
* github-ff leaves
* github-ff joins11:35
[fcrepo-jms-indexer-pluggable] awoods force-pushed master from a8ccb20 to b0712ab: http://git.io/yEUS3Q
fcrepo-jms-indexer-pluggable/master b0712ab Andrew Woods: Address checkstyle issues in preparation for tighter enforcement...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/commit/b0712ab1a2aaeab78b834386f91f9..." https://www.pivotaltracker.com/story/show/7176400011:36
Andrew Woods delivered "Run new Checkstyle rules across jms-indexer codebase" https://www.pivotaltracker.com/story/show/71764000
* dwilcox_ leaves
* travis-ci joins11:40
[travis-ci] fcrepo4/fcrepo-jms-indexer-pluggable#119 (master - a8ccb20 : Andrew Woods): The build has errored.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/compare/f89a84118dce...a8ccb209555e
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo-jms-indexer-pluggable/builds/25705639
* travis-ci leaves
* travis-ci joins12:00
[travis-ci] fcrepo4/fcrepo-jms-indexer-pluggable#120 (master - b0712ab : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/compare/a8ccb209555e...b0712ab1a2aa
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo-jms-indexer-pluggable/builds/25705789
* travis-ci leaves
* longshou joins12:15
* edInCo joins12:19
<tecoripa>awoods: pulling and building the latest fcrepo4 first...12:29
awoods: I'll get back to you in a ten, fifteen minutes or so
<awoods>tecoripa: ok
tecoripa: thanks
<tecoripa>awoods: np
* dwilcox joins12:39
* edInCo leaves13:07
<tecoripa>awoods: hmm, so now I'm getting a compilation error when I try to run it from a fresh checkout of my triplefinder branch in my fcrepo-module-auth-xacml forked repo: package javax.servlet.http does not exist13:12
awoods: should that dependency be in the project pom, or come from fcrepo4, or...?13:13
it's odd that it worked in Asheville, and that it compiled for you now
<awoods>tecoripa: I saw the same, you need to add the following to your pom.xml:
+ <dependency>
+ <groupId>javax.servlet</groupId>
+ <artifactId>javax.servlet-api</artifactId>
+ <scope>provided</scope>
+ </dependency>
<tecoripa>the fcrepo-module-auth-xacml pom?13:14
<awoods>tecoripa: yes
<tecoripa>awoods: that did the trick. I can add that dependency to the pom, check it in, have it show up as part of the triplefinder PR13:16
awoods: I also see 150 integration tests failing. :(
<awoods>tecoripa: at least 1 passes13:17
<tecoripa>it looks like some container requests for creating the request: lotsa NPEs
makes me think we're missing another dependency13:18
gregjansen, where are you?
<awoods>tecoripa: I guess we (you?) can use "git bisect" to see with which commit the error was introduced.
<tecoripa>awoods: yeah, it's Grizzly that's unhappy. Our test env isn't getting set up correctly
<awoods>tecoripa: http://git-scm.com/book/en/Git-Tools-Debugging-with-Git13:19
<tecoripa>awoods: I'm not sure that this is a bug introduced with a commit, but rather, some dependency linkage that made it into our development environments somehow, but was never explicitly configured in the POM. This seems to be a maven problem to me... at least, so far13:21
<awoods>tecoripa: I assume the tests used to run? in which case, "git bisect" will reveal with which commit that changed.13:22
<tecoripa>awoods: okay13:23
<awoods>tecoripa: which may help determine which dependencies need to be included/excluded.
<tecoripa>awoods: I'll take a look. it may take me a while... if I don't find the culprit immediately, I'll need to put it aside and look at it this evening.13:25
<awoods>tecoripa: keep me posted, and I will do the same.13:26
<tecoripa>awoods: will do
awoods: looking up the stack, I see a bean creation exception, caused by parse error of one of the config files, I think:13:29
awoods: nested exception is javax.jcr.RepositoryException: org.infinispan.config.ConfigurationException: javax.xml.stream.XMLStreamException: ParseError at [row,col]:[33,13]
Message: Unexpected element '{urn:infinispan:config:leveldb:5.2}leveldbStore'
awoods: I'll try and track that down, see where it's coming from13:30
awoods: have any of the fcrepo-configs changed recently?
<awoods>no13:31
tecoripa: no
tecoripa: but underlying modeshape versions have been updated... hmm.
<tecoripa>hmmmm13:32
<awoods>hmmmmmmmmmmm
<tecoripa>hmmm hmmm hmmmm
<awoods>.
<tecoripa>awoods: I *knew* it was somebody's fault.
awoods: odd that that wouldn't be picked up during the fctrepo4 integration tests.13:33
awoods: let me look at our config.
<awoods>tecoripa: right, we do not yet know what the issue actually is.13:34
<tecoripa>awoods: okay, I think I'm on track...13:37
awoods: we have fcrepo-build-tools hardcoded to version 1.3 in the module pom
<awoods>go tecoripa, go!
<tecoripa>but the fcrepo4 project sets the version to 1.4-SNAPSHOT
<awoods>tecoripa: and changing that fixes things?13:38
<tecoripa>awoods: I'll tell you in a minute :)
awoods: no.13:46
awoods: still investigating...
* github-ff joins14:49
[fcrepo4] awoods pushed 2 new commits to master: http://git.io/law9TA
fcrepo4/master c2b42ce Ben Pennell: Switched @date javadoc tag to @since
fcrepo4/master 6bb1c05 Ben Pennell: Javadoc warnings cleanup
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo4/compare/7e3f87d3dd9d...6bb1c0564edf" https://www.pivotaltracker.com/story/show/69747108
Andrew Woods delivered "Javadoc build warnings" https://www.pivotaltracker.com/story/show/69747108
* travis-ci joins15:19
[travis-ci] fcrepo4/fcrepo4#2022 (master - 6bb1c05 : Ben Pennell): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/7e3f87d3dd9d...6bb1c0564edf
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/25722022
* travis-ci leaves
<pivotal-bot>Andrew Woods added comment: "https://wiki.duraspace.org/display/FF/Versioning+Performance" https://www.pivotaltracker.com/story/show/7028894015:21
* nikhiltri leaves15:30
* nikhiltri joins15:52
* dwilcox leaves16:30
* github-ff joins17:03
[fcrepo4] lsitu opened pull request #374: Hierarchy children (master...hierarchy_children) http://git.io/bm18jQ
* github-ff leaves
<pivotal-bot>Longshou Situ added comment: "https://github.com/fcrepo4/fcrepo4/pull/374" https://www.pivotaltracker.com/story/show/7161129817:04
Longshou Situ finished "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
* tecoripa leaves17:36
* ksclarke1 joins18:29
* ksclarke leaves18:38
* longshou leaves
* ksclarke1 leaves18:41
* github-ff joins18:42
[fcrepo4] lsitu opened pull request #375: Hierarchy children (hierarchy...hierarchy_children) http://git.io/o-4yZw
* github-ff leaves
<pivotal-bot>Longshou Situ added comment: "@awoods Please use the following PR: ""18:44
https://github.com/fcrepo4/fcrepo4/pull/375" https://www.pivotaltracker.com/story/show/71611298
* nikhiltri leaves18:49
* github-ff joins19:04
[fcrepo4] awoods pushed 4 new commits to master: http://git.io/Leb6dA
fcrepo4/master ceb4950 Kevin S. Clarke: configured checkstyle for tests
fcrepo4/master 9d0b234 Kevin S. Clarke: Added @author tags to test classes without it
fcrepo4/master 68910ed Kevin S. Clarke: fixed license-maven-plugin dependency so it wouldn't be a snapshot
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo4/compare/6bb1c0564edf...8b51bd174bb9" https://www.pivotaltracker.com/story/show/69355194
* longshou joins19:05
* github-ff joins
[fcrepo4] awoods closed pull request #355: Checkstyle tests too 69355194 (master...checkstyle-tests-too-69355194) http://git.io/gcCOwg
* github-ff leaves
<pivotal-bot>Andrew Woods delivered "Turn on checkstyle checking for fcrepo4's test classes" https://www.pivotaltracker.com/story/show/6935519419:10
* github-ff joins20:07
[fcrepo-jms-indexer-pluggable] awoods pushed 2 new commits to master: http://git.io/unATOA
fcrepo-jms-indexer-pluggable/master 12fc8c2 Kevin S. Clarke: Turned on checkstyle checks for tests and added @author tag where missing
fcrepo-jms-indexer-pluggable/master 96efe97 Andrew Woods: Address checkstyle issues in preparation for tighter enforcement...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo-jms-indexer-pluggable/compare/b0712ab1a2aa...96efe9731e81" https://www.pivotaltracker.com/story/show/7132380420:08
Andrew Woods delivered "Turn on checkstyle rules for tests for jms indexer project" https://www.pivotaltracker.com/story/show/71323804
* github-ff joins
[fcrepo-jms-indexer-pluggable] awoods closed pull request #40: Turned on checkstyle checks for tests and added @author tag where missing (master...add-tests-to-checkstyle-71323804) http://git.io/OoYlww
* github-ff leaves
* github-ff joins20:09
[fcrepo-build-tools] awoods pushed 1 new commit to master: http://git.io/E_L0Lg
fcrepo-build-tools/master 98d807c Kevin S. Clarke: Tightened code styling rules in Checkstyle config
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo-build-tools/commit/98d807c41834c2514c37b148bd6adb8293e53f49" https://www.pivotaltracker.com/story/show/70648784
Andrew Woods delivered "Tighten code styling" https://www.pivotaltracker.com/story/show/70648784
* travis-ci joins20:11
[travis-ci] fcrepo4/fcrepo-build-tools#9 (master - 98d807c : Kevin S. Clarke): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo-build-tools/compare/a68a39c43050...98d807c41834
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo-build-tools/builds/25745344
* travis-ci leaves
* dwilcox joins20:15
* dwilcox leaves20:59
* longshou leaves21:01
* ksclarke joins21:48
* bljenkins joins22:21
<bljenkins>Yippee, build fixed!23:12
Project fcrepo4 build #1888: FIXED in 42 min: http://ci.fcrepo.org/jenkins/job/fcrepo4/1888/
* longshou joins23:14
<pivotal-bot>Andrew Woods accepted "Turn on checkstyle checking for fcrepo4's test classes" https://www.pivotaltracker.com/story/show/6935519423:15
Andrew Woods accepted "Tighten code styling" https://www.pivotaltracker.com/story/show/70648784
Andrew Woods accepted "Javadoc build warnings" https://www.pivotaltracker.com/story/show/69747108
Andrew Woods accepted "Update REST API docs with new POST/PUT behavior for creating objects" https://www.pivotaltracker.com/story/show/70965620
Andrew Woods accepted "Trying to create a workspace that exists should return 409 Conflict" https://www.pivotaltracker.com/story/show/71669076
Andrew Woods accepted "Posting invalid CND should result in 400 Bad Request" https://www.pivotaltracker.com/story/show/71684002
Andrew Woods accepted "Importing a node with the same UUID as an existing node should return 409 conflict" https://www.pivotaltracker.com/story/show/71693832
Andrew Woods accepted "Adding an invalid namespace declaration should result in 400 Bad Request" https://www.pivotaltracker.com/story/show/71698470
Andrew Woods accepted "Listing versions of an unversioned node should result in 404 Not Found" https://www.pivotaltracker.com/story/show/71705296
Andrew Woods accepted "Run new Checkstyle rules across code base and fix problems resulting from the new rules" https://www.pivotaltracker.com/story/show/71267176
Andrew Woods accepted "Turn on checkstyle rules for tests for jms indexer project" https://www.pivotaltracker.com/story/show/71323804
Andrew Woods accepted "Run new Checkstyle rules across jms-indexer codebase" https://www.pivotaltracker.com/story/show/71764000
<bljenkins>Project fcrepo-jms-indexer-pluggable build #522: NOW UNSTABLE in 10 min: http://ci.fcrepo.org/jenkins/job/fcrepo-jms-indexer-pluggable/522/23:24

Generated by Sualtam