Skip to content

Add JDK path #2069

Merged
Technoboy- merged 3 commits intomasterfrom
bump-1
May 6, 2026
Merged

Add JDK path #2069
Technoboy- merged 3 commits intomasterfrom
bump-1

Conversation

@Technoboy-
Copy link
Copy Markdown
Contributor

@Technoboy- Technoboy- commented May 1, 2026

Motivation

=== RUN   TestSinksOperations
    opeations_test.go:107: last sink status before timeout: {
          "numInstances": 1,
          "numRunning": 0,
          "instances": [
            {
              "instanceId": 0,
              "status": {
                "running": false,
                "error": "Cannot run program \"java\": Exec failed, error: 2 (No such file or directory) ",
                "numRestarts": 5,
                "numReadFromPulsar": 0,
                "numSystemExceptions": 0,
                "latestSystemExceptions": [],
                "numSinkExceptions": 0,
                "latestSinkExceptions": [],
                "numWrittenToSink": 0,
                "lastReceivedTime": 0,
                "workerId": "c-standalone-fw-50f128fd8221-8080"
              }
            }
          ]
        }
    test_help.go:76: /pulsarctl/pkg/ctl/sinks/opeations_test.go 109 task timeout

See here, #2065

the new jdk path has changed, so we need to add the path to ensure the container startup

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@Technoboy- Technoboy- requested review from a team, mattisonchao and zymap as code owners May 1, 2026 07:25
@github-actions github-actions Bot added the no-need-doc This pr does not need any document label May 1, 2026
@Technoboy- Technoboy- closed this May 1, 2026
This reverts commit cbaba7a.
@Technoboy- Technoboy- reopened this May 1, 2026
@Technoboy- Technoboy- changed the title Bump to 4.3.0-SNAPSHOT and ensure JDK is in path Ensure JDK is in path May 1, 2026
@Technoboy- Technoboy- changed the title Ensure JDK is in path Add JDK path May 1, 2026
@Technoboy- Technoboy- merged commit 404e152 into master May 6, 2026
23 checks passed
@Technoboy- Technoboy- deleted the bump-1 branch May 6, 2026 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-need-doc This pr does not need any document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants