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

Using timezone: Eastern Standard Time
* jcoyne leaves00:13
* jcoyne joins00:20
* jcoyne leaves00:25
* scossu leaves01:12
* scossu joins01:43
* scossu leaves01:53
* scossu joins03:50
* scossu leaves03:56
* scossu joins05:52
* scossu leaves05:57
* scossu joins07:51
* dwilcox joins08:05
* scossu leaves08:55
* mikeAtUVa joins09:05
* ermadmix joins09:15
* ksclarke joins09:34
<pivotal-bot_>Kevin Clarke started "Enforce @author attribute on classes" https://www.pivotaltracker.com/story/show/6965565209:44
* github-ff joins09:54
[fcrepo4] ksclarke closed pull request #256: Updated throws indent formatting to match Java now that new Checkstyle supports this (master...checkstyle-formatting-66144930) http://git.io/zrjeZw
* github-ff leaves
* github-ff joins09:56
[fcrepo-build-tools] ksclarke closed pull request #1: fixed checkstyle/eclipse throws indent impedance (master...throwsIndent) https://github.com/futures/fcrepo-build-tools/pull/1
* github-ff leaves
<pivotal-bot_>Eric James added "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/7058052410:01
* scossu joins10:10
* gregjansen joins10:14
* scossu1 joins10:17
* scossu leaves10:20
* ermadmix leaves10:36
* ermadmix joins10:43
* scossu1 leaves11:00
* scossu joins
<pivotal-bot_>Andrew Woods added "Address "Critical" Sonar Issues" https://www.pivotaltracker.com/story/show/7058481811:06
Eric James added "configure within the repository the structure and definitions to support PREMIS " https://www.pivotaltracker.com/story/show/7058533811:13
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/7058052411:14
Eric James added "Demonstrate with current REST API the creation of PREMIS persistence nodes and properties on objects" https://www.pivotaltracker.com/story/show/7058562611:18
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/70580524
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/7058052411:19
Eric James added "Integrate into indexer-pluggable functionality that persists PREMIS triples into the triplestore" https://www.pivotaltracker.com/story/show/7058643211:22
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/70580524
Eric James added "Demonstrate that SPARQL queries can used to return PREMIS metadata on a object" https://www.pivotaltracker.com/story/show/7058681811:26
Eric James edited "Integrate into indexer-pluggable functionality that persists PREMIS triples into the triplestore" https://www.pivotaltracker.com/story/show/70586432
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/70580524
Eric James added "Create an endpoint that returns RDF describing all the PREMIS information contained in an object" https://www.pivotaltracker.com/story/show/7058720211:31
* dwilcox leaves
<pivotal-bot_>Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/70580524
* dwilcox joins11:32
<pivotal-bot_>Eric James added "Extend fcrepo observer and/or indexer pluggable to plugin PREMIS event creation functionality" https://www.pivotaltracker.com/story/show/7058742011:35
Eric James edited "PREMIS Implementation master strategy" https://www.pivotaltracker.com/story/show/70580524
* edInCo joins11:39
<pivotal-bot_>Eric James added "Refactor integration tests to support referenced model of fcrepo4 access control" https://www.pivotaltracker.com/story/show/7058798811:44
* nikhiltri joins12:02
* jcoyne joins12:32
* ermadmix leaves12:35
* jcoyne leaves
<pivotal-bot_>Andrew Woods added "Investigate: Rest-Assured testing utility" https://www.pivotaltracker.com/story/show/7059183212:37
* ermadmix joins12:56
* cjcolvar joins13:27
mikeAtUVa: ping
<mikeAtUVa>cjcolvar: pong13:28
<cjcolvar>I'm trying to test the response content types of the lock rest endpoint but running into a problem13:29
should you be able to get the lock properties more than once with the same lock token?
<mikeAtUVa>You should.
<awoods>cjcolvar: are you not seeing that behavior?13:30
<cjcolvar>I have a test which sets up a lock and then attempts to get the lock properties with different accept headers but it appears to be failing on the second one with a 409
<mikeAtUVa>I take it the second GET http://localhost:8080/fedora/lockedobject/fcr:lock fails?
<cjcolvar>I think so
<mikeAtUVa>You're providing the token to both requests?
<cjcolvar>let me make a gist of the test13:31
<awoods>mikeAtUVa: when you create a session (with the lock-token header) the LockManager adds that lock to the workspace...
mikeAtUVa: likewise, when you logout of the session, the LockManager removes the lock from the workspace, no?13:32
<mikeAtUVa>awoods: and when the session.logout() is invoked it removes it.
<awoods>right
<mikeAtUVa>awoods: if we have any sequence of requests that leaks sessions by not calling logout() the behavior would be as Chris describes.
likewise: concurrent sessions that try to both use the lock token would fail in that way.13:33
<awoods>mikeAtUVa: Of if the lock-token header is not added in the request?13:34
<cjcolvar>mikeAtUVa: https://gist.github.com/cjcolvar/5adf1667d86aa1abff00
<mikeAtUVa>awoods: only if attempting to modify a locked resource... viewing the lock can be done without it.
cjcolvar: thanks... looking into it now...13:36
<awoods>mikeAtUVa: It appears that there is no "logout" in this flow.13:38
mikeAtUVa: but it would seem that GETs would be non-conflicting.13:40
<mikeAtUVa>awoods: the logout is required because ModeShape only allows a lock token to be held by one session at a time.
The stack trace in the caught LockException would say as much...13:41
I'm curious how the flow in FedoraLocksIT#testLockMetadataAvalability() doesn't trigger this...
<awoods>mikeAtUVa: testLockMetadataAvalability() only adds the lock-token header in the first request13:43
mikeAtUVa: I suspect cjcolvar's test would pass if the following line were commented out:13:44
https://gist.github.com/cjcolvar/5adf1667d86aa1abff00#file-gistfile1-txt-L14
<cjcolvar>I'll try it
<mikeAtUVa>awoods: assertUnlockWithToken() the last line of testLockMetadataAvailablitly also adds the lock token to the session...13:45
<cjcolvar>mikeAtUVa: tests pass with line commented out13:46
<mikeAtUVa>cjcolvar: does that get you what you need? Should I investigate or document why adding the token to subequent requests fails?13:49
<cjcolvar>mikeAtUVa: that works for me...should I make an issue for the multiple requests with token problem?13:51
<awoods>mikeAtUVa: I wonder why we are not calling Session.logout() in FedoraLocks.getLock()?
mikeAtUVa: that would likely also solve the issue.13:52
<mikeAtUVa>cjcolvar: if you wanted to make an issue that'd be great... sounds like awoods has a solution...13:55
<awoods>mikeAtUVa: sounds good. Do you feel like there is a good reason not to call Session.logout() in FedoraLocks.getLock()?13:56
<cjcolvar>awoods: I see 500s when I put that in a finally block in getLock()
awoods, mikeAtUVa: I have to go...I'll make an issue when I get the chance13:57
<mikeAtUVa>cjcolvar, I can make it... thanks for bringing this to our attention.
<awoods>cjcolvar: Are you saying that if you call Session.logout() in getLock() you get 500s?
cjcolvar: anyways, thanks.13:58
thank you, too, mikeAtUVa.
* cjcolvar leaves13:59
* ermadmix leaves14:00
<pivotal-bot_>Mike Durbin added "Getting lock metadata multiple times in a session throws exception." https://www.pivotaltracker.com/story/show/7059766614:01
* escowles joins14:03
<mikeAtUVa>awoods: any reason I shouldn't just take care of this ticket now, since I've already got the code loaded up and running?14:04
<pivotal-bot_>Andrew Woods added comment: "Apparently adding a simple Session.logout() in FedoraLocks.getLock() produces a 500 response.14:05
It may be nec..." https://www.pivotaltracker.com/story/show/70597666
<awoods>mikeAtUVa: smack it
mikeAtUVa: I added a note to the ticket as well.
* ermadmix joins14:13
* nikhiltri leaves14:15
<pivotal-bot_>Chris Beer added "Content-Type parsing should allow extra parameters (e.g. a charset)" https://www.pivotaltracker.com/story/show/7059901414:19
* scossu leaves
<pivotal-bot_>Chris Beer started "Content-Type parsing should allow extra parameters (e.g. a charset)" https://www.pivotaltracker.com/story/show/7059901414:22
Chris Beer started "Run the LDP test suite against fcrepo4 API and create bugs for any defects" https://www.pivotaltracker.com/story/show/7060044814:35
Chris Beer added "Run the LDP test suite against fcrepo4 API and create bugs for any defects" https://www.pivotaltracker.com/story/show/70600448
Chris Beer added "Integrate the LDP test suite into our HTTP API tsts" https://www.pivotaltracker.com/story/show/7060050414:36
<cbeer>escowles: ping?14:39
<escowles>cbeer: i'm on a call...14:40
* ermadmix leaves14:44
<cbeer>escowles: ok. when you're done, i'm trying to figure out what to do with a PUT request like:14:46
<> a <http://example.com/ns#Bug> ;
should we make a new mixin for example:Bug?
should we store that relation on a separate property?
i'm leaning towards the former.
* gregjansen leaves14:48
<cbeer>i'm also not sure how:
https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/kernel/utils/iterators/PersistingRdfStreamConsumer.java#L155
* ermadmix joins14:49
<cbeer>and https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/kernel/rdf/JcrRdfTools.java#L550
are different.
oh, other than the first takes a Resource and the second takes a Property14:52
<escowles>cbeer: i agree that mapping them to mixins is probably a good idea
and getPropertyNameFromPredicate looks more general -- not tied to mixins/rdf:type/etc.14:53
* dwilcox leaves14:57
* dwilcox joins14:58
* scossu joins15:00
* nikhiltri joins15:02
* gregjansen joins15:04
<cbeer>oh, right. i remember why we punted on this before... registering a new node type programmatically is a pain.15:14
<pivotal-bot_>Mike Durbin started "Getting lock metadata multiple times in a session throws exception." https://www.pivotaltracker.com/story/show/7059766615:24
* gregjansen leaves15:27
<pivotal-bot_>Mike Durbin added comment: "This seems to work, though there are other places where we're leaving sessions open (like FedoraNodes.descri..." https://www.pivotaltracker.com/story/show/7059766615:43
* escowles leaves15:57
<pivotal-bot_>Andrew Woods added comment: "Those other places will only be problematic if the lock-token header is included in the request, no?16:21
In any..." https://www.pivotaltracker.com/story/show/70597666
* mikeAtUVa leaves16:24
* scossu leaves16:25
* dwilcox leaves16:28
<pivotal-bot_>Eric James added "Remove or modify fcrepo:hasAccessRoles" https://www.pivotaltracker.com/story/show/7060977016:45
Eric James estimated "Remove or modify fcrepo:hasAccessRoles" as 2 points https://www.pivotaltracker.com/story/show/7060977016:47
Eric James edited "Remove or modify fcrepo:hasAccessRoles" https://www.pivotaltracker.com/story/show/7060977016:57
Eric James edited "Refactor integration tests to support referenced model of fcrepo4 access control" https://www.pivotaltracker.com/story/show/70587988
Eric James added comment: "additional related work: ""17:01
https://www.pivotaltracker.com/s/projects/684825/stories/70609770 - Remove or modify..." https://www.pivotaltracker.com/story/show/70111766
Eric James started "Event audit design" https://www.pivotaltracker.com/story/show/6981115017:04
Eric James added comment: "tasks ticketed: ""
https://www.pivotaltracker.com/s/projects/684825/stories/70580524" https://www.pivotaltracker.com/story/show/69811150
Eric James finished "Event audit design" https://www.pivotaltracker.com/story/show/69811150
<awoods>ermadmix: What is "RI" in this ticket's description? https://www.pivotaltracker.com/story/show/7060977017:06
<pivotal-bot_>feature: Remove or modify fcrepo:hasAccessRoles (unscheduled) / owner:
Andrew Woods accepted "Event audit design" https://www.pivotaltracker.com/story/show/6981115017:09
<awoods>ksclarke: how far along is your large ingest at present?17:10
<ksclarke>~120k
and it's definitely slowing down
<ermadmix>awoods: Referential Integrity - trying to add a triple like
<> <http://fedora.info/definitions/v4/rest-api#acReference> <http://localhost:8080/rest/testaddrolemissing/fcr:accessroles>
results in that empty POST error.
* github-ff joins17:11
[fcrepo4] cbeer created ldp-compliance (+1 new commit): http://git.io/QHy4LQ
fcrepo4/ldp-compliance 9caddef Chris Beer: Update RdfAdder and RdfRemover to use #getJcrNameForRdfNode machinery for translating a RDF resource into a JCR name
* github-ff leaves
<awoods>ermadmix: RI = Referential Integrity, thanks.
* github-ff joins17:12
[fcrepo4] cbeer created rdf-properties from ldp-compliance (+0 new commits): http://git.io/Bc2Rjg
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer deleted ldp-compliance at 9caddef: http://git.io/TR647A
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer opened pull request #334: Update RdfAdder and RdfRemover to use #getJcrNameForRdfNode machinery for translating a RDF resource into a JCR name (master...rdf-properties) http://git.io/7GJp7A
* github-ff leaves
* travis-ci joins17:13
[travis-ci] futures/fcrepo4#1874 (ldp-compliance - 9caddef : Chris Beer): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/9caddef5bdee
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24302215
* travis-ci leaves
* github-ff joins
[fcrepo4] cbeer created content-types (+1 new commit): http://git.io/RhjH0A
fcrepo4/content-types 624b9b0 Chris Beer: Content types with parameters should be stripped out before translating them into RDF formats...
* github-ff leaves
* github-ff joins17:14
[fcrepo4] cbeer opened pull request #335: Content types with parameters should be stripped out before translating them into RDF formats (master...content-types) http://git.io/epM6rA
* github-ff leaves
<pivotal-bot_>Chris Beer added comment: "https://github.com/futures/fcrepo4/pull/335" https://www.pivotaltracker.com/story/show/70599014
Chris Beer finished "Content-Type parsing should allow extra parameters (e.g. a charset)" https://www.pivotaltracker.com/story/show/70599014
Chris Beer added "RDF type updates with unregistered mixins should work " https://www.pivotaltracker.com/story/show/7061250017:15
Chris Beer started "RDF type updates with unregistered mixins should work " https://www.pivotaltracker.com/story/show/70612500
Chris Beer finished "RDF type updates with unregistered mixins should work " https://www.pivotaltracker.com/story/show/70612500
Chris Beer added comment: "https://github.com/futures/fcrepo4/pull/334" https://www.pivotaltracker.com/story/show/70612500
Chris Beer added comment: "testClientPostToCreate Method Skipped 0 Msec17:17
testRestrictUriReUse..." https://www.pivotaltracker.com/story/show/70600448
Chris Beer added comment: "testRestrictUriReUseSlug Method Failed 25 Msec17:18
testETagHeadersGet ..." https://www.pivotaltracker.com/story/show/70600448
* travis-ci joins17:26
[travis-ci] futures/fcrepo4#1875 (rdf-properties - 9caddef : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/rdf-properties
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24302303
* travis-ci leaves
* travis-ci joins17:30
[travis-ci] futures/fcrepo4#1877 (content-types - 624b9b0 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/624b9b0b77e5
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24302448
* travis-ci leaves
* nikhiltri leaves17:41
* edInCo leaves
* ermadmix leaves17:42
* jcoyne joins17:44
* jcoyne leaves17:46
* martinjd joins17:49
* scossu joins18:27
* ksclarke leaves18:39
* scossu leaves18:44
* scossu joins18:45
* jcoyne joins19:46
* jcoyne leaves19:51
* scossu leaves20:57
* scossu joins21:28
* scossu leaves21:31
* scossu joins
* jcoyne joins21:35
* jcoyne leaves21:44
* jcoyne joins21:50
* jcoyne leaves21:54
* jcoyne1 joins
* github-ff joins22:15
[fcrepo-build-tools] ksclarke opened pull request #3: added check for @author; cleaned up formatting in checkstyle.xml (master...checkstyle-author-69655652) http://git.io/zmtKuw
* github-ff leaves
* github-ff joins22:21
[fcrepo4] ksclarke opened pull request #336: added @author tag to java files missing it; updated checkstyle dep (master...checkstyle-author-69655652) http://git.io/u_YFKQ
* github-ff leaves
<pivotal-bot_>Kevin Clarke added comment: "Added @author tag check in checkstyle config: ""22:23
https://github.com/futures/fcrepo-build-tools/pull/3
Added ..." https://www.pivotaltracker.com/story/show/69655652
Kevin Clarke finished "Enforce @author attribute on classes" https://www.pivotaltracker.com/story/show/6965565222:26
* scossu leaves22:49
* scossu joins22:55
* scossu leaves
* scossu joins
* jcoyne1 leaves23:14
* scossu leaves00:11

Generated by Sualtam