Release 5.10.0 #101

IvanKustau · 2023-10-02T13:35:36Z

No description provided.

Update dockerfile and gradle.properties

* EPMRPP-82486 || Update personal data retention policy * EPMRPP-82486 || Add schedulerLock * EPMRPP-82486 || Remove dao * EPMRPP-82486 || Fix typo --------- Co-authored-by: Andrei Piankouski <[email protected]>

…e to retention policy (#85) * EPMRPP-83085 || Implement user notification about account deletion due to retention policy * EPMRPP-83085 || Add user deletion notification * EPMRPP-83085 || Fix checkstyle * EPMRPP-83085 || Fix checkstyle --------- Co-authored-by: Andrei Piankouski <[email protected]>

…mail

…ation_job_fix NotifyUserExpiration || NotifyUserExpiration job fails when sending email

…mail

… the job

EPMRPP-85418 || Email is not sent to user when the user is deleted by the job

… the job

* Update Dockerfile

EPMRPP-84893 || Add event on User Account Deletion by retention job

EPMRPP-85345 || Update the retention job with default value

EPMRPP-84893 || Added new EventObject type

Merge master to develop

Update Dockerfile for Release build

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2023-10-02T13:36:08Z

src/test/java/com/epam/reportportal/jobs/storage/CalculateAllocatedStorageJobTest.java

@@ -52,7 +52,7 @@ void shouldUpdateAllocatedStorageForAllProjects() {
 		when(jdbcTemplate.queryForList(SELECT_PROJECT_IDS_QUERY, Long.class)).thenReturn(projectIds);
 		when(jdbcTemplate.queryForObject(eq(SELECT_FILE_SIZE_SUM_BY_PROJECT_ID_QUERY), eq(Long.class), anyLong())).thenReturn(1000L);

-		calculateAllocatedStorageJob.calculate();
+		calculateAllocatedStorageJob.execute();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:36:08Z

src/test/java/com/epam/reportportal/jobs/storage/CalculateAllocatedStorageJobTest.java

@@ -52,7 +52,7 @@ void shouldUpdateAllocatedStorageForAllProjects() {
 		when(jdbcTemplate.queryForList(SELECT_PROJECT_IDS_QUERY, Long.class)).thenReturn(projectIds);
 		when(jdbcTemplate.queryForObject(eq(SELECT_FILE_SIZE_SUM_BY_PROJECT_ID_QUERY), eq(Long.class), anyLong())).thenReturn(1000L);

-		calculateAllocatedStorageJob.calculate();
+		calculateAllocatedStorageJob.execute();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def' child has incorrect indentation level 16, expected level should be 4.

github-actions · 2023-10-02T13:36:08Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

@@ -15,6 +21,7 @@
 import java.util.Map;
 import java.util.stream.Collectors;

+import static com.epam.reportportal.analyzer.AnalyzerUtils.DOES_SUPPORT_SUGGEST;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> _{reported by reviewdog 🐶}
Import statement for 'com.epam.reportportal.analyzer.AnalyzerUtils.DOES_SUPPORT_SUGGEST' is in the wrong order. Should be in the 'STATIC' group, expecting not assigned imports on this line.

github-actions · 2023-10-02T13:36:08Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

@@ -23,10 +30,14 @@
 @Service
 public class IndexerServiceClientImpl implements IndexerServiceClient {

+	private static final Logger LOGGER = LoggerFactory.getLogger(IndexerServiceClient.class);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:36:08Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

@@ -23,10 +30,14 @@
 @Service
 public class IndexerServiceClientImpl implements IndexerServiceClient {

+	private static final Logger LOGGER = LoggerFactory.getLogger(IndexerServiceClient.class);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2023-10-02T13:36:11Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+						}
+				))
+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:36:11Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+						}
+				))
+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'if' has incorrect indentation level 40, expected level should be 34.

github-actions · 2023-10-02T13:36:11Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				))
+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 102).

github-actions · 2023-10-02T13:36:11Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				))
+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:36:11Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				))
+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'if' child has incorrect indentation level 48, expected level should be 36.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2023-10-02T13:50:45Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);
+					} else {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:50:45Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				.forEach(it -> {
+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);
+					} else {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'if rcurly' has incorrect indentation level 40, expected level should be 34.

github-actions · 2023-10-02T13:50:45Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);
+					} else {
+						LOGGER.error("Error deleting index '{}'", index);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:50:45Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+					if (DELETE_INDEX_SUCCESS_CODE.equals(it)) {
+						LOGGER.info("Successfully deleted index '{}'", index);
+					} else {
+						LOGGER.error("Error deleting index '{}'", index);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'else' child has incorrect indentation level 48, expected level should be 36.

github-actions · 2023-10-02T13:50:46Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+						LOGGER.info("Successfully deleted index '{}'", index);
+					} else {
+						LOGGER.error("Error deleting index '{}'", index);
+					}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:50:48Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				.stream()
+				.filter(supportCondition)
+				.min(Comparator.comparingInt(EXCHANGE_PRIORITY))
+				.map(ExchangeInfo::getName);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:50:48Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				.filter(supportCondition)
+				.min(Comparator.comparingInt(EXCHANGE_PRIORITY))
+				.map(ExchangeInfo::getName);
+	}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2023-10-02T13:50:48Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClientImpl.java

+				.filter(supportCondition)
+				.min(Comparator.comparingInt(EXCHANGE_PRIORITY))
+				.map(ExchangeInfo::getName);
+	}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def rcurly' has incorrect indentation level 8, expected level should be 2.

github-actions · 2023-10-02T13:50:48Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClient.java

@@ -45,4 +45,18 @@ public interface IndexerServiceClient {
 	 */
 	void removeFromIndexLessThanLaunchDate(Long index, LocalDateTime lessThanDate);

+	/**


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
First sentence of Javadoc is missing an ending period.

github-actions · 2023-10-02T13:50:48Z

src/main/java/com/epam/reportportal/analyzer/index/IndexerServiceClient.java

@@ -45,4 +45,18 @@ public interface IndexerServiceClient {
 	 */
 	void removeFromIndexLessThanLaunchDate(Long index, LocalDateTime lessThanDate);

+	/**


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

sonarcloud · 2023-10-02T13:52:51Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

raikbitters and others added 30 commits July 13, 2023 15:40

Improve docker image build (#79)

ed97bab

Update dockerfile and gradle.properties

EPMRPP-82486 || Update personal data retention policy (#82)

8d83175

* EPMRPP-82486 || Update personal data retention policy * EPMRPP-82486 || Add schedulerLock * EPMRPP-82486 || Remove dao * EPMRPP-82486 || Fix typo --------- Co-authored-by: Andrei Piankouski <[email protected]>

EPMRPP-85017 || Postfix for bucket names in binary storage

ec72ad1

NotifyUserExpiration || NotifyUserExpiration job fails when sending e…

5cbbce7

…mail

Merge pull request #88 from reportportal/EPMRPP-85381_NotifyUserExpir…

03f5284

…ation_job_fix NotifyUserExpiration || NotifyUserExpiration job fails when sending email

NotifyUserExpiration || NotifyUserExpiration job fails when sending e…

eba6859

…mail

EPMRPP-83085 || Fix date calculation

b85a66b

EPMRPP-83085 || Fix delete-account-notification-template

14f8bef

EPMRPP-83085 || Fix delete-account-notification-template

70fd5ef

EPMRPP-85418 || Email is not sent to user when the user is deleted by…

e2bacaa

… the job

Merge pull request #90 from reportportal/EPMRPP-85418

4df7fbc

EPMRPP-85418 || Email is not sent to user when the user is deleted by the job

EPMRPP-85418 || Email is not sent to user when the user is deleted by…

14e7fe1

… the job

EPMRPP-85418 || Email is not sent to user when the user is deleted by…

4888d2e

… the job

Patch Dockerfile (#92)

d9c30e7

* Update Dockerfile

EPMRPP-84893 || added MessageBus to jobs-service

868f59d

EPMRPP-84893 || added activity events to jobs-service

3ab31c7

Merge pull request #93 from reportportal/EPMRPP-84893-JOBS

4fa065c

EPMRPP-84893 || Add event on User Account Deletion by retention job

EPMRPP-85345 || rewrite email notification

85a00d6

EPMRPP-85345 || updated retention policy for users

ada856c

Merge pull request #94 from reportportal/EPMRPP-85345-JOBS

73cef69

EPMRPP-85345 || Update the retention job with default value

EPMRPP-85482 || Remove creating dockerfile after release

667bd56

EPMRPP-85482 || Remove creating dockerfile after release

3e2a76d

EPMRPP-84893 || added new EventObject type

1b0b390

Merge pull request #96 from reportportal/EPMRPP-84893-JOBS-minor-change

b10a021

EPMRPP-84893 || Added new EventObject type

Merge branch 'develop' into master_to_develop

8d53926

Merge pull request #98 from reportportal/master_to_develop

cb1ca17

Merge master to develop

Update Dockerfile for Release build

d8ade74

Merge branch 'develop' into patch/docker-for-release-build

49dd205

Update Dockerfile

a0306d1

raikbitters and others added 7 commits August 25, 2023 15:43

Update Dockerfile

898eb91

Merge pull request #99 from reportportal/patch/docker-for-release-build

c308714

Update Dockerfile for Release build

added github action rc build images

8e8f557

changed exit-code

7e744f8

Release 5.10.0

4d4034c

Create dockerhub-release.yaml

ffdd7e7

Update app version

bde18cb

IvanKustau requested review from hlebkanonik and APiankouski October 2, 2023 13:35

IvanKustau self-assigned this Oct 2, 2023

github-actions bot reviewed Oct 2, 2023

View reviewed changes

hlebkanonik approved these changes Oct 2, 2023

View reviewed changes

hlebkanonik had a problem deploying to rc October 2, 2023 13:41 — with GitHub Actions Failure

Update rc.yaml

d0f0dcd

hlebkanonik temporarily deployed to rc October 2, 2023 13:50 — with GitHub Actions Inactive

github-actions bot reviewed Oct 2, 2023

View reviewed changes

APiankouski approved these changes Oct 2, 2023

View reviewed changes

APiankouski had a problem deploying to rc October 2, 2023 13:51 — with GitHub Actions Failure

hlebkanonik self-requested a review October 2, 2023 14:02

hlebkanonik approved these changes Oct 2, 2023

View reviewed changes

hlebkanonik temporarily deployed to rc October 2, 2023 14:02 — with GitHub Actions Inactive

IvanKustau merged commit 1149116 into master Oct 2, 2023
10 of 11 checks passed

IvanKustau deleted the rc/5.10.0 branch October 2, 2023 14:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Release 5.10.0 #101

Release 5.10.0 #101

IvanKustau commented Oct 2, 2023

github-actions bot left a comment

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot left a comment

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

github-actions bot Oct 2, 2023

sonarcloud bot commented Oct 2, 2023

Release 5.10.0 #101

Release 5.10.0 #101

Conversation

IvanKustau commented Oct 2, 2023

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

github-actions bot Oct 2, 2023

Choose a reason for hiding this comment

sonarcloud bot commented Oct 2, 2023