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

LPS-147076 #1682

Closed
wants to merge 6 commits into from
Closed

LPS-147076 #1682

wants to merge 6 commits into from

Conversation

mbowerman
Copy link

Ticket: LPS-147076

Patched third-party org.apache.velocity .jar file with changes from apache/velocity-engine#16 to resolve CVE-2020-13936.

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:relevant
  •     ci:test:sf

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 823a5121de53e7851e7017e037ce8a8890b1cfe5

Sender Branch:

Branch Name: LPS-147076
Branch GIT ID: f0b95880c7f635c30bba5796637bbb677d18d6ca

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

@liferay-continuous-integration
Copy link
Collaborator

@dantewang
Copy link
Collaborator

@catarineaguiar Could you review the test results when they finish? Thanks!

Copy link
Collaborator

@dantewang dantewang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tinatian Code looks good. Waiting for test results and QA check.

@dantewang dantewang assigned tinatian and unassigned dantewang Sep 13, 2022
@catarineaguiar
Copy link

ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

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

❌ ci:test:relevant - 123 out of 180 jobs passed in 2 hours 53 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_5793

Base Branch:

Branch Name: master
Branch GIT ID: 840c9a5d114cc58f36c3baf695c13d3c68f92d01

Upstream Comparison:

Branch GIT ID: 7fe3fcc7dd7578866b6e758c90769c47003b2a06
Jenkins Build URL: EE Development Acceptance (master) - 3798 - 2022-09-13[20:29:27]

ci:test:stable - 27 out of 27 jobs PASSED
27 Successful Jobs:
    ci:test:relevant - 123 out of 180 jobs PASSED

    57 Failed Jobs:

    123 Successful Jobs:
      For more details click here.

      Failures unique to this pull:

      1. semantic-versioning-jdk8/0/0
        clean-up-java-processes:
           [delete] Deleting: /opt/dev/projects/github/liferay-portal/clean_up_java_processes.sh
        [stopwatch] [run.batch.test.setup: 34:01.933 sec]
             [echo] run.batch.test.action.start.timestamp: 09-14-2022 07:52:18:917 PDT
             [echo] Checking for baseline log files.
             [echo] 
             [echo] ##
             [echo] ## /opt/dev/projects/github/liferay-portal/baseline-reports/portal-kernel.log
             [echo] ##
             [echo] 
        [beanshell] [Baseline Warning] Bundle Version Change Recommended: 74.0.0
        [beanshell] 
        [stopwatch] [run.batch.test.action: 8.557 sec]
             [echo] Semantic versioning is incorrect.
              [get] Getting: ${dependencies.url}/archive/test-1-21/test-portal-acceptance-pullrequest-downstream(master)/84052consoleText
              [get] To: /opt/dev/projects/github/liferay-portal/20220914075227475.txt
           [delete] Deleting: /opt/dev/projects/github/liferay-portal/20220914075227475.txt
           [delete] Deleting: /opt/dev/projects/github/liferay-portal/null2041044067.properties
      2. modules-unit-jdk8/0/3
        1. com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtilTest.classMethod
          java.lang.AssertionError: com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtil is not fully covered
            at com.liferay.whip.agent.InstrumentationAgent._assertClassDataCoverage(InstrumentationAgent.java:343)
            at com.liferay.whip.agent.InstrumentationAgent.assertCoverage(InstrumentationAgent.java:66)
            at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
            at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
            at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
            at java.lang.reflect.Method.invoke(Method.java:498)
            at com.liferay.portal.kernel.test.rule.CodeCoverageAssertor.afterClass(CodeCoverageAssertor.java:119)
            at com.liferay.portal.kernel.test.rule.CodeCoverageAssertor$1.evaluate(CodeCoverageAssertor.java:84)
            at org.junit.rules.RunRules.evaluate(RunRules.java:20)
            at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.runTestClass(JUnitTestClassExecutor.java:110)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:58)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:38)
            at org.gradle.api.internal.tasks.testing.junit.AbstractJUnitTestClassProcessor.processTestClass(AbstractJUnitTestClassProcessor.java:62)
            at org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClas...
        2. com.liferay.portal.proxy.internal.ASMDelegateProxyFactoryTest.classMethod
          java.lang.AssertionError: com.liferay.portal.proxy.internal.ASMDelegateProxyFactory is not fully covered
            at com.liferay.whip.agent.InstrumentationAgent._assertClassDataCoverage(InstrumentationAgent.java:343)
            at com.liferay.whip.agent.InstrumentationAgent.assertCoverage(InstrumentationAgent.java:66)
            at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
            at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
            at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
            at java.lang.reflect.Method.invoke(Method.java:498)
            at com.liferay.portal.kernel.test.rule.CodeCoverageAssertor.afterClass(CodeCoverageAssertor.java:119)
            at com.liferay.portal.kernel.test.rule.CodeCoverageAssertor$1.evaluate(CodeCoverageAssertor.java:84)
            at org.junit.rules.RunRules.evaluate(RunRules.java:20)
            at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.runTestClass(JUnitTestClassExecutor.java:110)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:58)
            at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:38)
            at org.gradle.api.internal.tasks.testing.junit.AbstractJUnitTestClassProcessor.processTestClass(AbstractJUnitTestClassProcessor.java:62)
            at org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor...

      Failures in common with acceptance upstream results at 7fe3fcc:
      1. modules-integration-mysql57-jdk8/0/0
        1. com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog
          junit.framework.AssertionFailedError: 
          Table 'lportal.User_x_51805' doesn't exist
            at com.liferay.portal.log.assertor.PortalLogAssertorTest.scanXMLLogFile(PortalLogAssertorTest.java:171)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest$1.visitFile(PortalLogAssertorTest.java:98)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest$1.visitFile(PortalLogAssertorTest.java:88)
            at java.nio.file.Files.walkFileTree(Files.java:2670)
            at java.nio.file.Files.walkFileTree(Files.java:2742)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog(PortalLogAssertorTest.java:86)
          
      2. modules-integration-mysql57-jdk8/0/1
        1. com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog
          junit.framework.AssertionFailedError: 
          Table 'lportal.User_x_45328' doesn't exist
            at com.liferay.portal.log.assertor.PortalLogAssertorTest.scanXMLLogFile(PortalLogAssertorTest.java:171)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest$1.visitFile(PortalLogAssertorTest.java:98)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest$1.visitFile(PortalLogAssertorTest.java:88)
            at java.nio.file.Files.walkFileTree(Files.java:2670)
            at java.nio.file.Files.walkFileTree(Files.java:2742)
            at com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog(PortalLogAssertorTest.java:86)
          
      3. ...
      Test bundle downloads:

      @liferay-continuous-integration
      Copy link
      Collaborator

      @catarineaguiar
      Copy link

      @mbowerman @dantewang no regressions found.

      @tinatian
      Copy link
      Collaborator

      @shuyangzhou This pull is good to go.

      Tina

      @tinatian
      Copy link
      Collaborator

      See brianchandotcom#123354

      @tinatian tinatian closed this Sep 19, 2022
      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.

      6 participants