Consistant hash reporting #1981

pbortnik · 2024-04-30T08:31:54Z

No description provided.

…th separated exception handlers.

…ring page

…arked as system

…eue (#1822) * EPMRPP-86359 || Implement pattern analyzer handling using rabbitmq queue * EPMRPP-86361 || Add some checkstyle refactoring

* EPMRPP-86742 || extended launch import response

…:false attribute

* EPMRPP-86742 || launch name fix

…erController (#1827) * EPMRPP-84054 || Add DeprecatedUserController and updated route for UserController * EPMRPP-84054 || Update DeprecatedUserController

#1828)

* EPMRPP-86867 || updated logos in email templates

…1835) * EPMRPP-80989 || Add attribute operator in duplicate operation check * EPMRPP-80989 || Fix unit tests

…ty/item/{itemId} (#1838) Co-authored-by: rkukharenka <[email protected]>

…cture (#1834) * EPMRPP-86867 || updated icons || re-organized template directory structure

…aunch finish

github-actions

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

reviewdog

...ain/java/com/epam/ta/reportportal/reporting/async/config/ReportingTopologyConfiguration.java

github-actions · 2024-05-06T08:43:45Z

...m/epam/ta/reportportal/reporting/async/controller/TestItemConsistentHashAsyncController.java

+import org.springframework.web.bind.annotation.ResponseStatus;
+import org.springframework.web.bind.annotation.RestController;
+
+/**


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

...m/epam/ta/reportportal/reporting/async/controller/TestItemConsistentHashAsyncController.java

…into draft/consistant_hash_reporting

src/test/java/com/epam/ta/reportportal/TestConfig.java

src/test/java/com/epam/ta/reportportal/demodata/service/ContentUtilsTest.java

github-actions · 2024-06-17T15:41:13Z

src/test/java/com/epam/ta/reportportal/demodata/service/ContentUtilsTest.java

+    String suiteDescription = ContentUtils.getSuiteDescription();
+    List<String> result = new ArrayList<>();
+    result.add(
+        "**This is demonstration description.** This `root-item` contains automatically generated test cases with logs and attachments.");


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

src/test/java/com/epam/ta/reportportal/demodata/service/ContentUtilsTest.java

github-actions · 2024-06-17T15:41:13Z

src/test/java/com/epam/ta/reportportal/demodata/service/ContentUtilsTest.java

+    String stepDescription = ContentUtils.getStepDescription();
+    List<String> result = new ArrayList<>();
+    result.add(
+        "This is the last **test case** of demo launch. There are only `logs` with `attachments` inside it.");


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

src/main/java/com/epam/ta/reportportal/reporting/async/producer/LaunchFinishProducer.java

github-actions · 2024-06-17T15:41:14Z

src/main/java/com/epam/ta/reportportal/reporting/async/producer/ItemFinishProducer.java

+import org.springframework.beans.factory.annotation.Qualifier;
+import org.springframework.stereotype.Service;
+
+/**


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

src/main/java/com/epam/ta/reportportal/reporting/async/producer/ItemStartProducer.java

github-actions · 2024-06-17T15:41:14Z

src/main/java/com/epam/ta/reportportal/reporting/async/producer/LogProducer.java

+import org.springframework.stereotype.Service;
+import org.springframework.web.multipart.MultipartFile;
+
+/**


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

src/main/java/com/epam/ta/reportportal/reporting/async/producer/LaunchStartProducer.java

...ain/java/com/epam/ta/reportportal/reporting/async/config/ReportingTopologyConfiguration.java

…istent hash

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LaunchAsyncController.java

github-actions · 2024-06-17T18:52:07Z

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LaunchAsyncController.java

@@ -85,23 +86,27 @@
  @ResponseStatus(CREATED)
  @Operation(summary = "Starts launch for specified project")
  public StartLaunchRS startLaunch(@PathVariable String projectName,
-      @Parameter(description = "Start launch request body", required = true) @RequestBody @Validated
-          StartLaunchRQ startLaunchRQ, @AuthenticationPrincipal ReportPortalUser user) {
+      @Parameter(description = "Start launch request body", required = true) @RequestBody @Validated StartLaunchRQ startLaunchRQ,


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

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LaunchAsyncController.java

github-actions · 2024-06-17T18:52:08Z

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LaunchAsyncController.java

-    return finishLaunchHandler.finishLaunch(launchId, finishLaunchRQ,
-        projectExtractor.extractProjectDetails(user, normalizeId(projectName)), user,
+      @PathVariable String launchId,
+      @RequestBody @Validated FinishExecutionRQ finishLaunchRQ,


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'finishLaunchRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2024-06-17T18:52:08Z

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LogAsyncController.java

    this.projectExtractor = projectExtractor;
    this.createLogHandler = createLogHandler;
    this.validator = validator;
  }

  /**
   * @deprecated in favour of
-   * {@link LogAsyncController#createLogEntry(String, SaveLogRQ, ReportPortalUser)} because of
-   * mapping collisions
+   * {@link LogAsyncController#createLogEntry(String, SaveLogRQ, ReportPortalUser)}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTagContinuationIndentationCheck> _{reported by reviewdog 🐶}
Line continuation have incorrect indentation level, expected level should be 4.

github-actions · 2024-06-17T18:52:08Z

src/main/java/com/epam/ta/reportportal/reporting/async/controller/LogAsyncController.java

-   * {@link LogAsyncController#createLogEntry(String, SaveLogRQ, ReportPortalUser)} because of
-   * mapping collisions
+   * {@link LogAsyncController#createLogEntry(String, SaveLogRQ, ReportPortalUser)}
+   * because of mapping collisions


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTagContinuationIndentationCheck> _{reported by reviewdog 🐶}
Line continuation have incorrect indentation level, expected level should be 4.

...ain/java/com/epam/ta/reportportal/reporting/async/config/ReportingTopologyConfiguration.java

src/main/java/com/epam/ta/reportportal/reporting/async/producer/LogProducer.java

github-actions · 2024-06-18T14:44:28Z

src/test/java/com/epam/ta/reportportal/core/log/impl/CreateLogHandlerAsyncImplTest.java

+        () -> createLogHandlerAsync.sendMessage(request, binaryDataMetaInfo, 0L)
+    );
+    assertEquals(
+        "Error in handled Request. Please, check specified parameters: 'Launch UUID should not be null or empty.'",


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

…into draft/consistant_hash_reporting

…reportportal/service-api into draft/consistant_hash_reporting

github-actions · 2024-07-01T19:28:23Z

...ain/java/com/epam/ta/reportportal/reporting/async/config/ReportingTopologyConfiguration.java

+    return containers;
+  }
+
+  @Bean("retryListener")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

…into draft/consistant_hash_reporting

src/main/java/com/epam/ta/reportportal/reporting/async/OutdatedQueuesManagementJob.java

src/main/java/com/epam/ta/reportportal/reporting/async/UnbindQueuesOnShutdown.java

pbortnik and others added 30 commits October 2, 2023 14:03

EPMRPP-85815 || Introduce consistent hash based exchange reporting wi…

1bccae9

…th separated exception handlers.

EPMRPP-86542 || fixed loading resources for email (#1816)

96010d9

EPMRPP-85520 || Improve the filtering option 'User' on Project Monito…

d248fe2

…ring page

EPMRPP-86219 || Implement immediate AutoAnalyze on TestItemFinishing (#…

623b186

…1818)

EPMRPP-86947 || ImmediateAA works when immediateAA attribute is not m…

88542a6

…arked as system

EPMRPP-86775 || Add possibility to have no auth in ES

a6a58be

EPMRPP-86775 || Add possibility to have no auth in ES

609e9f2

EPMRPP-86945 ||Check application name into Swagger base-path (#1820)

2e9a251

EPMRPP-86359 || Implement pattern analyzer handling using rabbitmq qu…

5e9414b

…eue (#1822) * EPMRPP-86359 || Implement pattern analyzer handling using rabbitmq queue * EPMRPP-86361 || Add some checkstyle refactoring

EPMRPP-86359 || Change default single item prop value

7efcd40

EPMRPP-86742 || extended launch import response (#1821)

d9f63ea

* EPMRPP-86742 || extended launch import response

EPMRPP-86969 || ImmediateAA works not only for TI defect type (#1824)

37e34ad

EPMRPP-86967 || ImmediateAA works when Step finishes with immediateAA…

5636a4a

…:false attribute

EPMRPP-86967 || Add null check

a5d0872

EPMRPP-86742 || message fix (#1825)

ca09a3a

EPMRPP-86969 || ImmediateAA works not only for TI defect type

99f68e2

EPMRPP-86742 || launch name fix (#1826)

5f584a9

* EPMRPP-86742 || launch name fix

EPMRPP-86969 || ImmediateAA works not only for TI defect type

af92e48

EPMRPP-84054 || Add DeprecatedUserController and updated route for Us…

5c61941

…erController (#1827) * EPMRPP-84054 || Add DeprecatedUserController and updated route for UserController * EPMRPP-84054 || Update DeprecatedUserController

EPMRPP-86466 || changed activity parameters upon creating invited user (

4719146

#1828)

EPMRPP-80574 || Update dao version. Remove redundant encryptor (#1830)

24ce357

EPMRPP-87028 || Update security configuration for user endpoints (#1831)

a9103cf

EPMRPP-86812 || extended actuator with "jobs" module info (#1832)

b29a8e5

EPMRPP-86867 || updated logos in email templates (#1833)

0700973

* EPMRPP-86867 || updated logos in email templates

EPMRPP-80574 || Update dao version.

fe7c53a

EPMRPP-80989 || Add attribute operator in duplicate operation check (#…

7074fe2

…1835) * EPMRPP-80989 || Add attribute operator in duplicate operation check * EPMRPP-80989 || Fix unit tests

EPMRPP-86109 || changed response model for GET {activeProject}/activi…

5843af7

…ty/item/{itemId} (#1838) Co-authored-by: rkukharenka <[email protected]>

EPMRPP-86867 || updated icons || re-organized template directory stru…

1dbc682

…cture (#1834) * EPMRPP-86867 || updated icons || re-organized template directory structure

EPMRPP-86248 || Auto-Analysis should skip already analyzed items on l…

00dbab8

…aunch finish

EPMRPP-86867 github icon fix (#1840)

727cba9

pbortnik added the Do not merge label May 6, 2024

Merge branch 'develop' into draft/consistant_hash_reporting

a129092

github-actions bot reviewed May 6, 2024

View reviewed changes

Merge branch 'develop' of https://github.com/reportportal/service-api …

b70c244

…into draft/consistant_hash_reporting

github-actions bot reviewed Jun 17, 2024

View reviewed changes

EPMRPP-91154 || Change retry handling using separate ttl queue

0ca4dd2

github-actions bot reviewed Jun 17, 2024

View reviewed changes

...ain/java/com/epam/ta/reportportal/reporting/async/config/ReportingTopologyConfiguration.java Show resolved Hide resolved

pbortnik added 3 commits June 17, 2024 21:12

EPMRPP-91154 || Add swagger doc

472e5ee

EPMRPP-91154 || Change default variables

d30e073

EPMRPP-91154 || Replace old v2 reporting with reporting based on cons…

5a39203

…istent hash

github-actions bot reviewed Jun 17, 2024

View reviewed changes

pbortnik added 2 commits June 18, 2024 17:14

EPMRPP-91154 || Clean up

ac7e2bd

EPMRPP-91154 || Update tests

ac1bd67

github-actions bot reviewed Jun 18, 2024

View reviewed changes

pbortnik added 5 commits June 18, 2024 17:50

EPMRPP-91154 || Update tests

66a094a

Generate own UUID in case if empty UUID provided with rq

4c083aa

Merge branch 'develop' of https://github.com/reportportal/service-api …

8eca998

…into draft/consistant_hash_reporting

Merge branch 'draft/consistant_hash_reporting' of https://github.com/…

86c55a9

…reportportal/service-api into draft/consistant_hash_reporting

EPMRPP-92293 || Fix retry logic

5b35e34

github-actions bot reviewed Jul 1, 2024

View reviewed changes

pbortnik added 3 commits July 4, 2024 01:19

Merge branch 'develop' of https://github.com/reportportal/service-api …

f3c73e2

…into draft/consistant_hash_reporting

EPMRPP-92294 || Introduce queues management job

96d8d0d

EPMRPP-92294 || Change scheduled time

e40b6bd

github-actions bot reviewed Jul 8, 2024

View reviewed changes

pbortnik marked this pull request as ready for review July 9, 2024 10:00

pbortnik requested a review from avarabyeu as a code owner July 9, 2024 10:00

pbortnik removed the Do not merge label Jul 9, 2024

pbortnik changed the title ~~Draft/consistant hash reporting~~ Consistant hash reporting Jul 9, 2024

pbortnik merged commit 148cae5 into develop Jul 10, 2024
6 checks passed

pbortnik deleted the draft/consistant_hash_reporting branch July 10, 2024 07:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Consistant hash reporting #1981

Consistant hash reporting #1981

pbortnik commented Apr 30, 2024

github-actions bot left a comment

github-actions bot May 6, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 17, 2024

github-actions bot Jun 18, 2024

github-actions bot Jul 1, 2024

Consistant hash reporting #1981

Consistant hash reporting #1981

Conversation

pbortnik commented Apr 30, 2024

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot May 6, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 17, 2024

Choose a reason for hiding this comment

github-actions bot Jun 18, 2024

Choose a reason for hiding this comment

github-actions bot Jul 1, 2024

Choose a reason for hiding this comment