Hi,
Alright, looks good! Another question that I have is for the beforeProvision
mapping transformation actions and the MappingItemTransformer class that is
mentioned in the documentation: the link from the documentation gives a 404 for
this item and I cannot find it within the source either? Is it supposed to be
there?
In version 2.1, all my transformation actions based on the ItemTransformer
interface are not visible anymore. Might be an issue with the previous
upgrade too, so I’ll revert back to the latest snapshot and try again from
scratch and see.
Regards,
Mikael
Mikael Ekblom
IT-Service manager
Arcada, IT
Jan-Magnus Janssons plats 1,
FIN-00560 Helsingfors,
Finland
TFn: +358 207 699 467
Mobil: +358 207 699 467
From: Francesco Chicchiriccò [mailto:[email protected]]
Sent: torstai 12. heinäkuuta 2018 9.18
To: [email protected]
Subject: Re: Apache Syncope version 2.1 findings
FYI,
https://cwiki.apache.org/confluence/display/SYNCOPE/Upgrade+from+2.0+Jazz
is now complete.
Regards.
On 11/07/2018 12:21, Francesco Chicchiriccò wrote:
On 10/07/2018 17:08, Mikael Ekblom wrote:
Hi,
I have now tested the 2.1 version or let us say that the installation procedure
was tested. My installation path was from 2.0.9.
Hi Mikael,
thanks for this review.
You can find my replies embedded below.
I am currently working at a better upgrade page:
https://cwiki.apache.org/confluence/display/SYNCOPE/Upgrade+from+2.0+Jazz
Regards.
My findings are the following:
1. I had the same regarding the Junit version error message regarding
syncope console, but I did not delete the junit dependency in the console
pom-file. I just added the version information as <version>4.8.1</version>
dummy value and it went through.
Yep, SYNCOPE-1334 addresses this issue.
I will add this to the upgrade page.
1. The database script generated the required sql-file as expected, but I
had to manually run most of it to double check that i got all the relations and
indexes set. I noted during this process that the table ownership was changed
from syncope to the default postgres DBA user, so I had to change this
manually. This was for the new relations that were created through the sql
scipt file. Easily fixed though.
I understand, but I think this is expected. It is also the reason why we
preferred to generate a SQL script rather than directly executing the generate
upgrade statements.
BTW, I have found some missing statements (see SYNCOPE-1335), that I will also
report in the upgrade page.
1. The core is not so keen on starting...:) The rest is up and running
(enduser and syncope-console) but not the core. The log claims the following:
* 17:31:00.300 ERROR org.flowable.engine.impl.cmd.ValidateV5EntitiesCmd
- Found v5 process definitions that are the latest version. Enable the
'flowable5CompatibilityEnabled' property in the process engine configuration
and make sure the flowable5-compatibility dependency is available on the
classpath
17:31:00.304 ERROR org.flowable.engine.impl.cmd.ValidateV5EntitiesCmd - Found
v5 process definition with id: userWorkflow:61:5012291, and key: userWorkflow
I will publish the complete procedure to avoid this error in the upgrade page.
1. I hope that I do not have to recreate every user due to this error...:)
No, not needed, of course.
1. Regarding number 3: can I somehow affect this during run time within
workflow.properties or must it be done within the source and the corresponding
xml configuration for flowable itself? The flowable engine has not been in use
here before. I just do not find out right now where this configuration change
should go?
Otherwise...all my own customization did compile for version 2.1 also after
some modifications. Optional seems to be in pretty heavy use....:) More has
been packed into interfaces.
I will also include the expected code modifications, mainly due to widespread
adoption of Java 8 language features, in the upgrade page.
--
Francesco Chicchiriccò
Tirasa - Open Source Excellence
http://www.tirasa.net/
Member at The Apache Software Foundation
Syncope, Cocoon, Olingo, CXF, OpenJPA, PonyMail
http://home.apache.org/~ilgrosso/
