Displaying #apache-syncope/2016-09-07.log:

Wed Sep 7 03:42:41 2016  mansour:Joined the channel
Wed Sep 7 06:23:48 2016  ilgrosso:Joined the channel
Wed Sep 7 06:44:58 2016  fmartelli:Joined the channel
Wed Sep 7 06:53:43 2016  fmartelli1:Joined the channel
Wed Sep 7 08:28:49 2016  fmartelli:Joined the channel
Wed Sep 7 08:33:21 2016  sberyozkin:Joined the channel
Wed Sep 7 08:35:59 2016  _massi_:Joined the channel
Wed Sep 7 08:39:05 2016  coheigea:Joined the channel
Wed Sep 7 08:55:43 2016  ilgrosso:coheigea: I have just added a complete GIT patch to SYNCOPE-931
Wed Sep 7 08:56:00 2016  coheigea:ilgrosso: Thx, I'll test it out
Wed Sep 7 09:03:06 2016  _massi_:Joined the channel
Wed Sep 7 12:31:29 2016  ilgrosso:Joined the channel
Wed Sep 7 12:36:40 2016  ilgrosso:coheigea: pleas try again after applying the further GIT patch attached
Wed Sep 7 12:43:05 2016  coheigea:ilgrosso: It doesn't apply to master?
Wed Sep 7 12:43:11 2016  coheigea:error: patch failed: ext/camel/logic/src/main/java/org/apache/syncope/core/logic/CamelRouteLogic.java:92
Wed Sep 7 12:43:17 2016  ilgrosso:need to apply both
Wed Sep 7 12:43:27 2016  coheigea:ok
Wed Sep 7 12:43:28 2016  ilgrosso:it's a patch to apply after you have applied the first
Wed Sep 7 12:43:33 2016  ilgrosso:two separate commits
Wed Sep 7 12:47:59 2016  fmartelli:Joined the channel
Wed Sep 7 13:04:39 2016  coheigea:ilgrosso: Patch looks good! What was the fix for the second issue?
Wed Sep 7 13:04:53 2016  ilgrosso:coheigea: great to hear!
Wed Sep 7 13:05:26 2016  ilgrosso:coheigea: essentially, a dedicated method for removing the failing route, restoring the original content before failed update and starting the context again
Wed Sep 7 13:05:54 2016  ilgrosso:important: avoid any further database operation, as the transaction is marked as rollback-only
Wed Sep 7 13:07:35 2016  coheigea:ok great
Wed Sep 7 13:11:03 2016  ilgrosso:coheigea: are you going to apply such commits? And mabe adding a test case to CamelRouteITCase?
Wed Sep 7 13:12:11 2016  coheigea:ilgrosso: No please apply them yourself, as they're your patches
Wed Sep 7 13:12:25 2016  coheigea:I'll take a look at adding a test
Wed Sep 7 13:12:29 2016  ilgrosso:cool
Wed Sep 7 13:15:59 2016  ilgrosso:coheigea: squashed into a single commit, and pushed to master
Wed Sep 7 13:28:53 2016  ilgrosso:coheigea: while debugging, I've set the Camel log to info, and got a coule of messsages about disabling streamCaching and AllowUseOriginalMessage
Wed Sep 7 13:29:16 2016  ilgrosso:do you believe there could be problems in setting
Wed Sep 7 13:29:17 2016  ilgrosso:SpringCamelContext.setStreamCaching(false)
Wed Sep 7 13:29:17 2016  ilgrosso:SpringCamelContext.setAllowUseOriginalMessage(false)
Wed Sep 7 13:29:17 2016  ilgrosso:when creating the SpringCamelContext?
Wed Sep 7 13:33:03 2016  coheigea:ilgrosso: No idea tbh
Wed Sep 7 13:36:10 2016  ilgrosso:coheigea: maybe I can try after your commit
Wed Sep 7 13:38:48 2016  coheigea:ilgrosso: I'm expecting this test to fail with the old code, but the user creation is working fine...any ideas? http://pastebin.com/QhcLHHEh
Wed Sep 7 13:41:18 2016  ilgrosso:coheigea: can you print the exception on line 33?
Wed Sep 7 13:42:45 2016  ilgrosso:also, I'd add a catch to the second try, for ensuring nothing is raised there
Wed Sep 7 13:44:39 2016  coheigea:ilgrosso: First exception is as follows: org.apache.syncope.common.lib.SyncopeClientException: Unknown [IllegalArgumentException: No enum constant org.apache.syncope.core.provisioning.camel.component.PropagateType.create123]
Wed Sep 7 13:45:03 2016  ilgrosso:but ths means that the fix is in place
Wed Sep 7 13:45:17 2016  ilgrosso:prior to that, you did not get any error message, no?
Wed Sep 7 13:49:32 2016  coheigea:ilgrosso: Doh, forgot to recompile :-)
Wed Sep 7 13:50:32 2016  ilgrosso:coheigea: LOL :-D
Wed Sep 7 17:11:43 2016  coheigea:Left the channel
Wed Sep 7 20:04:58 2016  coheigea:Joined the channel
Wed Sep 7 20:11:51 2016  coheigea:Left the channel

Comments