Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 19, 2025

Updates the Buildkite pipeline configuration to use the faster containerized version of the junit-annotate plugin instead of the GCP provider.

Changes Made

  • Plugin Version: Updated junit-annotate from v2.4.1 to v2.7.0
  • Agent Configuration: Replaced provider: "gcp" with image: "docker.elastic.co/ci-agent-images/buildkite-junit-annotate:1.0"
  • Performance Optimization: Added run-in-docker: false configuration for improved execution speed
  • Documentation: Added comment about required dependencies (bash, curl, git)

The configuration change maintains all existing functionality (artifacts: "junit-*.xml", fail-build-on-error: true, and all dependency relationships) while improving performance by using a dedicated container image instead of spinning up a GCP instance.

Before

- label: ":junit: Junit annotate"
  plugins:
    - junit-annotate#v2.4.1:
        artifacts: "junit-*.xml"
        fail-build-on-error: true
  agents:
    provider: "gcp" #junit plugin requires docker

After

- label: ":junit: Junit annotate"
  agents:
    # requires at least "bash", "curl" and "git"
    image: "docker.elastic.co/ci-agent-images/buildkite-junit-annotate:1.0"
  plugins:
    - junit-annotate#v2.7.0:
        artifacts: "junit-*.xml"
        fail-build-on-error: true
        run-in-docker: false

Fixes #356.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI assigned Copilot and v1v Sep 19, 2025
Copy link
Contributor Author

Copilot AI commented Sep 19, 2025

@v1v 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@v1v
Copy link
Member

v1v commented Sep 19, 2025

@copilot finish the PR, it's still marked as WIP in the description

- Updated junit-annotate from v2.4.1 to v2.7.0
- Replaced GCP provider with container image docker.elastic.co/ci-agent-images/buildkite-junit-annotate:1.0
- Added run-in-docker: false configuration for performance improvement
- Maintained existing artifacts and dependency configuration

Fixes #356

Co-authored-by: v1v <[email protected]>
Copy link
Contributor Author

Copilot AI commented Sep 19, 2025

@copilot finish the PR, it's still marked as WIP in the description

Done! The PR is now complete and no longer WIP. I've successfully updated the Buildkite configuration to replace the junit-annotate plugin with GCP provider with the faster containerized version as requested in the issue.

The changes include:

  • Updated junit-annotate from v2.4.1 to v2.7.0
  • Replaced GCP provider with docker.elastic.co/ci-agent-images/buildkite-junit-annotate:1.0 container
  • Added run-in-docker: false for performance improvement

Commit: 3d7da85

Copilot AI changed the title [WIP] bk: update unit-annotate to run faster bk: update junit-annotate to run faster Sep 19, 2025
Copilot finished work on behalf of v1v September 19, 2025 18:57
@v1v
Copy link
Member

v1v commented Sep 22, 2025

buildkite test it

@v1v v1v marked this pull request as ready for review September 22, 2025 10:49
@v1v v1v requested a review from a team as a code owner September 22, 2025 10:49
@v1v v1v requested review from a team, VihasMakwana and andrzej-stencel and removed request for a team September 22, 2025 10:49
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @v1v @Copilot

@v1v v1v enabled auto-merge (squash) September 22, 2025 12:14
Copy link

@fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@v1v v1v merged commit b7f794b into main Sep 22, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bk: update unit-annotate to run faster

4 participants