Skip to content

Conversation

@elezar
Copy link
Member

@elezar elezar commented Dec 3, 2025

This change enables golang checks on PRs (e.g. by dependabot). This provides basic checks for minimal dependency bumps without requiring an /ok-to-test by a maintainer.

@coveralls
Copy link

coveralls commented Dec 3, 2025

Pull Request Test Coverage Report for Build 19888361331

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 37.657%

Totals Coverage Status
Change from base Build 19887236158: 0.0%
Covered Lines: 5179
Relevant Lines: 13753

💛 - Coveralls

@ArangoGutierrez
Copy link
Collaborator

We need to set all golang tests to github managed runners as part of this PR

@elezar elezar force-pushed the run-golang-checks-on-prs branch from 0cec30d to 4fd5177 Compare December 3, 2025 08:54
@elezar
Copy link
Member Author

elezar commented Dec 3, 2025

We need to set all golang tests to github managed runners as part of this PR

Could you elaborate? The tests are already using ubuntu-latest, but the build step uses internal runners. I have added logic to only select the internal runners if the job is not triggered by a PR. (this was after your comment though).

@elezar elezar force-pushed the run-golang-checks-on-prs branch 2 times, most recently from b2ff7cc to 56c84fe Compare December 3, 2025 09:06
This change enables golang checks on PRs (e.g. by dependabot).
This provides basic checks for minimal dependency bumps without
requiring an /ok-to-test by a maintainer.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the run-golang-checks-on-prs branch from 56c84fe to d541925 Compare December 3, 2025 09:08
build:
name: Build
runs-on: linux-amd64-cpu4
runs-on: ${{ ( github.event_name == 'pull_request' ) && 'ubuntu-latest' || 'linux-amd64-cpu4' }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍🏽

@elezar elezar merged commit 34882b2 into NVIDIA:main Dec 3, 2025
16 checks passed
@elezar elezar deleted the run-golang-checks-on-prs branch December 3, 2025 10:50
@ArangoGutierrez
Copy link
Collaborator

Is it a good idea to back port this to 1.18?

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.

3 participants