SonarQube Analysis

Fix flaky sigv4 test #284

Sign in to view logs

Triggered via pull request January 3, 2025 21:11

AndreKurait

opened #1222

AndreKurait:fixSigV4TestTime

Status Success

Total duration 5m 1s

Artifacts 1

sonar-qube.yml

on: pull_request

Run SonarQube Analysis

Annotations

10 errors and 2 warnings

java:S1874: RFS/src/main/java/org/opensearch/migrations/bulkload/common/LuceneDocumentsReader.java#L167

Remove this use of "document"; it is deprecated.

java:S3038: RFS/src/main/java/org/opensearch/migrations/bulkload/models/IndexMetadata.java#L33

"getName" is defined in the "Index" interface and can be removed from this class.

java:S1141: RFS/src/main/java/org/opensearch/migrations/bulkload/version_os_2_11/IndexCreator_OS_2_11.java#L47

Extract this nested try block into a separate method.

java:S1168: RFS/src/main/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java#L385

Return an empty collection instead of null.

java:S135: RFS/src/main/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java#L1010

Reduce the total number of break and continue statements in this loop to use at most one.

java:S1192: RFS/src/main/java/org/opensearch/migrations/bulkload/worker/IndexAndShardCursor.java#L20

Define a constant instead of duplicating this literal "Illegal work item name: '" 3 times.

java:S7158: TrafficCapture/captureOffloader/src/main/java/org/opensearch/migrations/trafficcapture/StreamChannelConnectionCaptureSerializer.java#L280

Use isEmpty() to check whether a string is empty or not.

java:S7158: TrafficCapture/captureOffloader/src/main/java/org/opensearch/migrations/trafficcapture/StreamChannelConnectionCaptureSerializer.java#L368

Use isEmpty() to check whether a string is empty or not.

python:S112: TrafficCapture/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/base/main.py#L72

Replace this generic exception class with a more specific one.

python:S2772: TrafficCapture/dockerSolution/src/main/docker/migrationConsole/cluster_tools/src/cluster_tools/tools/disable_compatibility_mode.py#L12

Remove this unneeded "pass".

Run SonarQube Analysis

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Deprecation notice: v1, v2, and v3 of the artifact actions

The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "sonar-reports". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Artifacts

Produced during runtime

Name	Size
sonar-reports	56 KB