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-125851 & LPS-128383 Using system settings only to completely disable the feature #1272

Closed

Conversation

4lejandrito
Copy link

@4lejandrito 4lejandrito commented Jun 10, 2021

New behaviour:

  • Api Key and Strategy will only be configurable at instance level. Default values will be set via System Settings -> Virtual Instance Scope.
  • System settings will only have the Enabled check.
  • If the feature is disabled in system settings the instance settings configurations disappear.
  • The Configure Google Page speed button will take you to instance settings.

@liferay-continuous-integration

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@4lejandrito 4lejandrito changed the title LPS 125851 LPS 128383 2 LPS-125851 & LPS-128383 Using system settings only to completely disable the feature Jun 10, 2021
@4lejandrito
Copy link
Author

ci:forward

@liferay-continuous-integration

CI is automatically triggering the following test suites:

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

The pull request will automatically be forwarded to the user brianchandotcom if the following test suites pass:

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

@liferay-continuous-integration

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: cbb73af51883b8ddb6925f2c6e6a1af4013dcfd2

Sender Branch:

Branch Name: LPS-125851-LPS-128383-2
Branch GIT ID: d74de31f8c894a3b4f59c3bc23c6d0d4cd06b494

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

@liferay-continuous-integration

@liferay-continuous-integration

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

❌ ci:test:relevant - 23 out of 25 jobs passed in 4 hours 46 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: cbb73af51883b8ddb6925f2c6e6a1af4013dcfd2

Upstream Comparison:

Branch GIT ID: cbb73af51883b8ddb6925f2c6e6a1af4013dcfd2
Jenkins Build URL: Acceptance Upstream DXP (master) #1973

ci:test:stable - 10 out of 10 jobs PASSED
10 Successful Jobs:
ci:test:relevant - 23 out of 25 jobs PASSED
23 Successful Jobs:
For more details click here.

Failures unique to this pull:

  1. test-portal-acceptance-pullrequest-batch(master)/modules-integration-mysql57-jdk8/0
    Job Results:

    1205 Tests Passed.
    2 Tests Failed.

    1. AXIS_VARIABLE=2,label_exp=!master #387940
      1. com.liferay.layout.reports.web.internal.portlet.action.test.LayoutReportsDataMVCResourceCommandTest.testGetData
        java.lang.AssertionError
        	at org.junit.Assert.fail(Assert.java:87)
        	at org.junit.Assert.assertTrue(Assert.java:42)
        	at org.junit.Assert.assertTrue(Assert.java:53)
        	at com.liferay.layout.reports.web.internal.portlet.action.test.LayoutReportsDataMVCResourceCommandTest.testGetData(LayoutReportsDataMVCResourceCommandTest.java:159)
        	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.arquillian.extension.junit.bridge.server.TestExecutorRunnable$3.evaluate(TestExecutorRunnable.java:353)
        	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at org.junit.rules...
      2. com.liferay.layout.reports.web.internal.portlet.action.test.LayoutReportsDataMVCResourceCommandTest.testGetDataWithLayoutTypeAssetDisplay
        java.lang.AssertionError
        	at org.junit.Assert.fail(Assert.java:87)
        	at org.junit.Assert.assertTrue(Assert.java:42)
        	at org.junit.Assert.assertTrue(Assert.java:53)
        	at com.liferay.layout.reports.web.internal.portlet.action.test.LayoutReportsDataMVCResourceCommandTest.testGetDataWithLayoutTypeAssetDisplay(LayoutReportsDataMVCResourceCommandTest.java:296)
        	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.arquillian.extension.junit.bridge.server.TestExecutorRunnable$3.evaluate(TestExecutorRunnable.java:353)
        	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.java:99)
        	at com.liferay.portal.kernel.test.rule.AbstractTestRule$2.evaluate(AbstractTestRule.ja...

Failures in common with acceptance upstream results at cbb73af:
  1. test-portal-acceptance-pullrequest-batch(master)/modules-integration-mysql57-jdk8/0
    Job Results:

    1205 Tests Passed.
    2 Tests Failed.

    1. AXIS_VARIABLE=3,label_exp=!master #387940
      1. com.liferay.layout.set.prototype.exportimport.test.LayoutSetPrototypeExportImportTest.classMethod
        org.springframework.orm.hibernate3.HibernateOptimisticLockingFailureException: Object of class [com.liferay.portal.model.impl.LayoutSetPrototypeImpl] with identifier [41980]: optimistic locking failed; nested exception is org.hibernate.StaleObjectStateException: Row was updated or deleted by another transaction (or unsaved-value mapping was incorrect): [com.liferay.portal.model.impl.LayoutSetPrototypeImpl#41980]
        	at org.springframework.orm.hibernate3.SessionFactoryUtils.convertHibernateAccessException(SessionFactoryUtils.java:699)
        	at org.springframework.orm.hibernate3.SpringSessionSynchronization.translateException(SpringSessionSynchronization.java:165)
        	at org.springframework.orm.hibernate3.SpringSessionSynchronization.beforeCommit(SpringSessionSynchronization.java:153)
        	at org.springframework.transaction.support.TransactionSynchronizationUtils.triggerBeforeCommit(TransactionSynchronizationUtils.java:96)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.triggerBeforeCommit(AbstractPlatformTransactionManager.java:919)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.processCommit(AbstractPlatformTransactionManager.java:727)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.commit(AbstractPlatformTransactionManager.java:711)
        	at com.liferay.portal.spring.transaction.DefaultTransactionExecutor.commit(DefaultTransactionExecutor.java:41)
        	at com.liferay.portal.spring.transaction.BaseTransa...
      2. com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog
        junit.framework.AssertionFailedError: 
        Unable to connect to a valid mail server. Please make sure one is properly configured: Couldn't connect to host, port: localhost, 25; timeout -1
        	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)
        

@liferay-continuous-integration

Instance settings default values will be configured from the system
settings virtual instance section. They will be automatically inherited.
@4lejandrito
Copy link
Author

ci:forward

@liferay-continuous-integration

CI is automatically triggering the following test suites:

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

The pull request will automatically be forwarded to the user brianchandotcom if the following test suites pass:

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

@liferay-continuous-integration

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: e6cd70daf20d8ea4f7a09b7f713c608a85a70bcd

Sender Branch:

Branch Name: LPS-125851-LPS-128383-2
Branch GIT ID: e78d9b0c40602644f112ec52e08ea470d4a76c9d

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

@liferay-continuous-integration

@liferay-continuous-integration

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

✔️ ci:test:relevant - 23 out of 25 jobs passed in 2 hours 35 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: e6cd70daf20d8ea4f7a09b7f713c608a85a70bcd

Upstream Comparison:

Branch GIT ID: e6cd70daf20d8ea4f7a09b7f713c608a85a70bcd
Jenkins Build URL: Acceptance Upstream DXP (master) #1976

ci:test:stable - 10 out of 10 jobs PASSED
10 Successful Jobs:
ci:test:relevant - 22 out of 25 jobs PASSED
22 Successful Jobs:
For more details click here.

This pull contains no unique failures.


Failures in common with acceptance upstream results at e6cd70d:
  1. test-portal-acceptance-pullrequest-batch(master)/semantic-versioning-jdk8/0
    Job Results:

    0 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=0,label_exp=!master #12318
      clean-up-java-processes:
         [delete] Deleting: /opt/dev/projects/github/liferay-portal/clean_up_java_processes.sh
      [stopwatch] [run.batch.test.setup: 35:16.392 sec]
           [echo] run.batch.test.action.start.timestamp: 06-10-2021 19:09:13:143 PDT
           [echo] Checking for baseline log files.
           [echo] 
           [echo] ##
           [echo] ## /opt/dev/projects/github/liferay-portal/baseline-reports/portal-impl-7.log
           [echo] ##
           [echo] 
      [beanshell] [Baseline Warning] Bundle Version Change Recommended: 7.7.8
      [beanshell] 
           [echo] 
           [echo] ##
           [echo] ## /opt/dev/projects/github/liferay-portal/baseline-reports/portal-kernel-11.log
           [echo] ##
           [echo] 
      [beanshell] [Baseline Warning] Bundle Version Change Recommended: 11.8.1
      [beanshell] 
      [stopwatch] [run.batch.test.action: 16.170 sec]
           [echo] Semantic versioning is incorrect.
            [get] Getting: http://test-1-2/job/test-portal-acceptance-pullrequest-batch(master)/AXIS_VARIABLE=0,label_exp=!master/12318//consoleText
            [get] To: /opt/dev/projects/github/liferay-portal/20210610190929315.txt
         [delete] Deleting: /opt/dev/projects/github/liferay-portal/20210610190929315.txt
         [delete] Deleting: /opt/dev/projects/github/liferay-portal/null358157078.properties
  2. test-portal-acceptance-pullrequest-batch(master)/modules-integration-mysql57-jdk8/0
    Job Results:

    1207 Tests Passed.
    2 Tests Failed.

    1. AXIS_VARIABLE=3,label_exp=!master #464377
      1. com.liferay.layout.set.prototype.exportimport.test.LayoutSetPrototypeExportImportTest.classMethod
        org.springframework.orm.hibernate3.HibernateOptimisticLockingFailureException: Object of class [com.liferay.portal.model.impl.LayoutSetPrototypeImpl] with identifier [41980]: optimistic locking failed; nested exception is org.hibernate.StaleObjectStateException: Row was updated or deleted by another transaction (or unsaved-value mapping was incorrect): [com.liferay.portal.model.impl.LayoutSetPrototypeImpl#41980]
        	at org.springframework.orm.hibernate3.SessionFactoryUtils.convertHibernateAccessException(SessionFactoryUtils.java:699)
        	at org.springframework.orm.hibernate3.SpringSessionSynchronization.translateException(SpringSessionSynchronization.java:165)
        	at org.springframework.orm.hibernate3.SpringSessionSynchronization.beforeCommit(SpringSessionSynchronization.java:153)
        	at org.springframework.transaction.support.TransactionSynchronizationUtils.triggerBeforeCommit(TransactionSynchronizationUtils.java:96)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.triggerBeforeCommit(AbstractPlatformTransactionManager.java:919)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.processCommit(AbstractPlatformTransactionManager.java:727)
        	at org.springframework.transaction.support.AbstractPlatformTransactionManager.commit(AbstractPlatformTransactionManager.java:711)
        	at com.liferay.portal.spring.transaction.DefaultTransactionExecutor.commit(DefaultTransactionExecutor.java:41)
        	at com.liferay.portal.spring.transaction.BaseTransa...
      2. com.liferay.portal.log.assertor.PortalLogAssertorTest.testScanXMLLog
        junit.framework.AssertionFailedError: 
        Unable to connect to a valid mail server. Please make sure one is properly configured: Couldn't connect to host, port: localhost, 25; timeout -1
        	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)
        

@liferay-continuous-integration

All required test suite(s) passed.
Forwarding pull request to brianchandotcom.
Console

@liferay-continuous-integration

Pull request has been successfully forwarded to brianchandotcom#103058
Console

@liferay-continuous-integration

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