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:
    1.
        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/

Reply via email to