[CDAP-21062] Propagate internal credentials in Router · cdapio/cdap@d54a051

Triggered via pull request August 28, 2024 07:17

adrikagupta

labeled #15691

internal-router-cdf

Status Failure

Total duration 1h 13m 10s

Artifacts 2

e2e.yml

on: pull_request

Matrix: build

Annotations

6 errors and 12 warnings

build (cdap-e2e-tests)

Process completed with exit code 1.

build (cdap-e2e-tests)

The v0 series of google-github-actions/upload-cloud-storage is no longer maintained. It will not receive updates, improvements, or security patches. Please upgrade to the latest supported versions: https://github.com/google-github-actions/upload-cloud-storage

ProgramRunStatusMonitorServiceTest.testStoppingRemoteTetheredProgramsBeyondTerminateTimeAreKilled: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/ProgramRunStatusMonitorServiceTest.java#L242

expected:<1> but was:<0>

ProgramRunStatusMonitorServiceTest.testStoppingRemoteTetheredProgramsBeyondTerminateTimeAreKilled: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/ProgramRunStatusMonitorServiceTest.java#L242

expected:<1> but was:<0>

SqlOperationRunsStoreTest.: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/operation/SqlOperationRunsStoreTest.java#L53

Gave up waiting for server to start after 10000ms

SqlOperationRunsStoreTest.: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/operation/SqlOperationRunsStoreTest.java#L53

Gave up waiting for server to start after 10000ms

build (cdap-e2e-tests)

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, dorny/paths-filter@4512585405083f25c027a35db413c2b3b9006d50, actions/cache@v3, actions/upload-artifact@v3, google-github-actions/upload-cloud-storage@v0. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

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: "Cucumber report - cdap-e2e-tests", "Debug files - cdap-e2e-tests". 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/

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L49

Empty line should be followed by <p> tag on the next line.

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L59

Distance between variable 'now' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

'if' is not followed by whitespace.

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

WhitespaceAround: '{' is not preceded with whitespace.

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L49

Empty line should be followed by <p> tag on the next line.

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L59

Distance between variable 'now' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

'if' is not followed by whitespace.

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/auth/ProxyUserIdentityExtractor.java#L92

WhitespaceAround: '{' is not preceded with whitespace.