Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LPD-37739 move protobuf to share-dependencies and upgrade to 3.25.5 #7478

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jeffwu0724
Copy link
Collaborator

@jeffwu0724 jeffwu0724 commented Sep 30, 2024

Hi Tina,

This is for ticket https://liferay.atlassian.net/browse/LPD-37739.
As talked, I create a shared-dependencies module for protobuf, and then upgrade it to higher version.
Thanks for checking.

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:sf

@jeffwu0724 jeffwu0724 changed the title LPD-37739 LPD-37739 move protobuf to share-dependencies and upgrade to 3.25.5 Sep 30, 2024
@jeffwu0724
Copy link
Collaborator Author

ci:test:sf

@jeffwu0724
Copy link
Collaborator Author

ci:test:relevant

@jeffwu0724
Copy link
Collaborator Author

ci:test:core-functional

@jeffwu0724
Copy link
Collaborator Author

ci:test:acceptance

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 25 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 02d74feaa1a4be90f3209096c1c66ca3f659a0b1

Sender Branch:

Branch Name: LPD-37739_upgrade_protobuf_to_3.25.5_new
Branch GIT ID: 3490f819deb1d3039f513b9e1d38c575b825a581

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 25 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 02d74feaa1a4be90f3209096c1c66ca3f659a0b1

Sender Branch:

Branch Name: LPD-37739_upgrade_protobuf_to_3.25.5_new
Branch GIT ID: 3490f819deb1d3039f513b9e1d38c575b825a581

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:stable - 28 out of 28 jobs passed

✔️ ci:test:relevant - 34 out of 35 jobs passed in 1 hour 11 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 02d74feaa1a4be90f3209096c1c66ca3f659a0b1

Upstream Comparison:

Branch GIT ID: 613bcc9f840318d1789c3689ae1f300090956ee4
Jenkins Build URL: EE Development Acceptance (master) - 873 - 2024-09-30[12:38:31]

ci:test:stable - 28 out of 28 jobs PASSED
28 Successful Jobs:
    ci:test:relevant - 33 out of 35 jobs PASSED

    2 Failed Jobs:

    33 Successful Jobs:
      For more details click here.

      Failures unique to this pull:

      1. semantic-versioning-jdk8/0/0

        Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

             [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
             [exec] = ================================================== ========== ========== ========== ========== ==========
             [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
             [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
             [exec]     -   method     isForceable()
             [exec]       -   return     boolean
             [exec]   +   version    2.0.0
             [exec]   -   version    2.1.0
             [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
             [exec] [Baseline Report] Mode: diff (persisted)
             [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

      For upstream results, click here.

      Test bundle downloads:

      @liferay-continuous-integration
      Copy link
      Collaborator

      ✔️ ci:test:acceptance - 33 out of 33 jobs passed in 1 hour 11 minutes

      Click here for more details.

      Base Branch:

      Branch Name: master
      Branch GIT ID: 02d74feaa1a4be90f3209096c1c66ca3f659a0b1

      Upstream Comparison:

      Branch GIT ID: 613bcc9f840318d1789c3689ae1f300090956ee4
      Jenkins Build URL: EE Development Acceptance (master) - 873 - 2024-09-30[12:38:31]

      ci:test:acceptance - 33 out of 33 jobs PASSED
      33 Successful Jobs:
      For more details click here.
      Test bundle downloads:

      @liferay-continuous-integration
      Copy link
      Collaborator

      @liferay-continuous-integration
      Copy link
      Collaborator

      @liferay-continuous-integration
      Copy link
      Collaborator

      ❌ ci:test:core-functional - 192 out of 233 jobs passed in 1 hour 23 minutes

      Click here for more details.

      This pull is eligible for reevaluation. When this upstream build has completed, using the following CI command will compare this pull request result against a more recent upstream result:

      ci:reevaluate:1327401_5128

      Base Branch:

      Branch Name: master
      Branch GIT ID: 02d74feaa1a4be90f3209096c1c66ca3f659a0b1

      Upstream Comparison:

      Branch GIT ID: 613bcc9f840318d1789c3689ae1f300090956ee4
      Jenkins Build URL: EE Development Acceptance (master) - 873 - 2024-09-30[12:38:31]

      ci:test:core-functional - 192 out of 233 jobs PASSED

      41 Failed Jobs:

      192 Successful Jobs:
        For more details click here.

        Failures unique to this pull:

        1. functional-tomcat90-mysql57-jdk8/0/4

          Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

               [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
               [exec] = ================================================== ========== ========== ========== ========== ==========
               [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
               [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
               [exec]     -   method     isForceable()
               [exec]       -   return     boolean
               [exec]   +   version    2.0.0
               [exec]   -   version    2.1.0
               [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
               [exec] [Baseline Report] Mode: diff (persisted)
               [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

        2. functional-tomcat90-mysql57-jdk8/0/6

          Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

               [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
               [exec] = ================================================== ========== ========== ========== ========== ==========
               [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
               [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
               [exec]     -   method     isForceable()
               [exec]       -   return     boolean
               [exec]   +   version    2.0.0
               [exec]   -   version    2.1.0
               [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
               [exec] [Baseline Report] Mode: diff (persisted)
               [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

        3. functional-tomcat90-mysql57-jdk8/0/10

          Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

               [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
               [exec] = ================================================== ========== ========== ========== ========== ==========
               [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
               [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
               [exec]     -   method     isForceable()
               [exec]       -   return     boolean
               [exec]   +   version    2.0.0
               [exec]   -   version    2.1.0
               [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
               [exec] [Baseline Report] Mode: diff (persisted)
               [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

        4. functional-tomcat90-mysql57-jdk8/2/4

          Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

               [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
               [exec] = ================================================== ========== ========== ========== ========== ==========
               [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
               [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
               [exec]     -   method     isForceable()
               [exec]       -   return     boolean
               [exec]   +   version    2.0.0
               [exec]   -   version    2.1.0
               [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
               [exec] [Baseline Report] Mode: diff (persisted)
               [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

        5. functional-tomcat90-mysql57-jdk8/2/11

          Please fix semantic versioning on jeffwu0724/LPD-37739_upgrade_protobuf_to_3.25.5_new

               [exec]   PACKAGE_NAME                                       DELTA      CUR_VER    BASE_VER   REC_VER    WARNINGS  
               [exec] = ================================================== ========== ========== ========== ========== ==========
               [exec] * com.liferay.portal.kernel.security.auth.verifier   MAJOR      2.0.0      2.1.0      3.0.0      VERSION INCREASE REQUIRED
               [exec]   >   interface  com.liferay.portal.kernel.security.auth.verifier.AuthVerifier
               [exec]     -   method     isForceable()
               [exec]       -   return     boolean
               [exec]   +   version    2.0.0
               [exec]   -   version    2.1.0
               [exec] [Baseline Warning] Bundle Version Change Recommended: 152.0.0
               [exec] [Baseline Report] Mode: diff (persisted)
               [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/portal-kernel/portal-kernel.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay.portal/com.liferay.portal.kernel/151.0.0/bab22099c71430308c9bbc164c76c5ddc6cd71a6/com.liferay.portal.kernel-151.0.0.jar

          1. ...
        6. ...

        For upstream results, click here.

        Test bundle downloads:

        @liferay-continuous-integration
        Copy link
        Collaborator

        @jeffwu0724
        Copy link
        Collaborator Author

        ci:report:27533274

        @liferay-continuous-integration
        Copy link
        Collaborator

        Build completed.

        Testray CSV has been generated successfully for testrayBuildID: 27533274.

        Job Link: generate-testray-csv

        Testray CSV Link: testray-results-27533274.csv

        @jeffwu0724
        Copy link
        Collaborator Author

        Hi @vicnate5 ,
        This PR is to upgrade protobuf, and its usages are

        ./modules/apps/portal-store/portal-store-gcs/build.gradle:     compileInclude group: "com.google.protobuf", name: "protobuf-java", version: "3.21.7" 
        ./modules/apps/portal-store/portal-store-gcs/build.gradle:     compileInclude group: "com.google.protobuf", name: "protobuf-java-util", version: "3.21.7" 
        ./modules/apps/site-initializer/site-initializer-testray/site-initializer-testray-dispatch-task-executor/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java", version: "3.21.7" 
        ./modules/apps/site-initializer/site-initializer-testray/site-initializer-testray-dispatch-task-executor/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java-util", version: "3.21.7" 
        ./modules/dxp/apps/portal-rules-engine-drools/portal-rules-engine-drools/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java", version: "3.21.7" 
        ./modules/util/sass-compiler-dart/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java", version: "3.21.7" 
        ./workspaces/liferay-testray-workspace/modules/testray-rest-impl/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java", version: "3.21.7" 
        ./workspaces/liferay-testray-workspace/modules/testray-rest-impl/build.gradle: compileInclude group: "com.google.protobuf", name: "protobuf-java-util", version: "3.21.7"
        

        May I ask do you think we should trigger more test here, please?

        Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
        Projects
        None yet
        Development

        Successfully merging this pull request may close these issues.

        2 participants