Skip to content

Fix failing GitHub Actions job Bazel build & test - #4924

Merged
kubernetes-prow[bot] merged 1 commit into
masterfrom
copilot/fix-github-actions-job
Oct 16, 2026
Merged

kubernetes-prow[bot] merged 1 commit into
masterfrom
copilot/fix-github-actions-job

Conversation

Copilot AI commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Fix the failing GitHub Actions job "Bazel build & test (Java 17.0.x on windows-latest)"
Analyze the Actions logs, identify the root cause of the failure, and implement a fix.
Check run ID: 107714620399
Job URL: https://github.com/kubernetes-client/java/actions/runs/35485163949/job/107714620399

@kubernetes-prow kubernetes-prow Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 28, 2026
@linux-foundation-easycla

linux-foundation-easycla Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: brendandburns / name: brendandburns (030c666)

@kubernetes-prow kubernetes-prow Bot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 28, 2026
@kubernetes-prow kubernetes-prow Bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 28, 2026
Copilot AI requested a review from brendandburns September 28, 2026 19:49
@brendandburns

Copy link
Copy Markdown
Contributor

Please switch this to use a synchronization primitive like a countdown latch instead of a sleep.

@brendandburns

Copy link
Copy Markdown
Contributor

@copilot please address comments.

Copilot AI commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

Please switch this to use a synchronization primitive like a countdown latch instead of a sleep.

Done in 3d7ff8e. The three thread-based tests now wait on a CountDownLatch. A WireMock post-serve action counts it down when the exec request arrives, the same pattern ExecTest uses. The wait has a 30s limit (assertTrue(latch.await(30, TimeUnit.SECONDS))) so a missing request fails the test instead of hanging it.

@kubernetes-prow kubernetes-prow Bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 30, 2026
@brendandburns
brendandburns force-pushed the copilot/fix-github-actions-job branch from 3d7ff8e to 030c666 Compare October 16, 2026 20:53
@kubernetes-prow kubernetes-prow Bot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Oct 16, 2026
@brendandburns brendandburns changed the title [WIP] Fix failing GitHub Actions job Bazel build & test Fix failing GitHub Actions job Bazel build & test Oct 16, 2026
@brendandburns
brendandburns marked this pull request as ready for review October 16, 2026 20:59
@kubernetes-prow kubernetes-prow Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 16, 2026
@kubernetes-prow
kubernetes-prow Bot requested a review from yue9944882 October 16, 2026 21:00
@brendandburns

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@kubernetes-prow kubernetes-prow Bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 16, 2026
@kubernetes-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brendandburns, Copilot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 16, 2026
@kubernetes-prow
kubernetes-prow Bot merged commit e583d5d into master Oct 16, 2026
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants