-
Type:
Task
-
Status: Closed
-
Priority:
Minor
-
Resolution: Completed
-
Affects Version/s: None
-
Fix Version/s: Master
-
Component/s: Dev Tools, Dev Tools > Source Formatter
-
Labels:None
-
Git Pull Request:
In app.bnd, allow "false" for Liferay-Releng-Restart-Required if a POSHI test exists.
The POSHI test case must have the property hot.deploy.osgi.app.includes set to the name of the app you want to hot deploy.
Notes for the development of the SF check:
For {module_name}/app.bnd where Liferay-Releng-Restart-Required: false
- Search .testcase files in portal-web/test/functional/com/liferay/portalweb/tests
- Look for "hot.deploy.osgi.app.includes"
- The value of this property will be the name of the module being tested
- List all values wherever this property is set
- Check if
{module_name}
is on the list
- If {module_name} is on the list, success
- If {module_name} is not on the list, SF changes the value back to true and displays a message. Suggested:
Liferay-Releng-Restart-Required can only be set to false if a POSHI tests exists. See LPS-110344.
Examples:
https://github.com/liferay/liferay-portal/blob/master/portal-web/test/functional/com/liferay/portalweb/tests/coreinfrastructureee/search/elasticsearch/Elasticsearch6EE.testcase#L685
https://github.com/liferay/liferay-portal/blob/master/portal-web/test/functional/com/liferay/portalweb/tests/coreinfrastructureee/search/elasticsearch/Elasticsearch7EE.testcase#L818
https://github.com/liferay/liferay-portal/blob/master/portal-web/test/functional/com/liferay/portalweb/tests/coreinfrastructure/search/elasticsearch/Elasticsearch7.testcase#L219
Additional related tasks:
- Set Liferay-Releng-Restart-Required: false for modules/dxp/apps/portal-search-learning-to-rank
- Remove suppression for modules/apps/portal-search-elasticsearch7 (revert 6cc4197f146daa5cfcc0f57c7f36a27407c2fba0)
- is a dependency of
-
LPS-110730 Set restart-required to "false" for Learning to Rank and Elasticsearch 7 (restore LPS-103843 that has been reverted by an Auto SF)
-
- Closed
-
- is related to
-
LPS-103843 Remove restart-required for Learning to Rank and Elasticsearch 7
-
- Closed
-