Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extract GOLANG_VERSION from versions.mk #559

Merged
merged 4 commits into from
Feb 27, 2024

Conversation

elezar
Copy link
Member

@elezar elezar commented Feb 27, 2024

This PR updates the golang actions to use the GOLANG_VERSION defined in versions.mk

This ensures consistency between what is built for the images and what is tested in CI.

It also adds a vendor and check-vendor make target to ensure that the local go module files are up to date.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the vendor-in-container branch from 840a3f2 to 2dbf357 Compare February 27, 2024 14:55
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@ArangoGutierrez ArangoGutierrez self-requested a review February 27, 2024 15:04
@ArangoGutierrez ArangoGutierrez added maintenance Issue/PR to create or address a team project management need testing issue/PR to fix/edit/create/enhance a project unit/e2e test dependencies Issue/PR Pull about a dependency file labels Feb 27, 2024
@elezar elezar removed the testing issue/PR to fix/edit/create/enhance a project unit/e2e test label Feb 27, 2024
@elezar elezar marked this pull request as ready for review February 27, 2024 15:21
Copy link
Collaborator

@ArangoGutierrez ArangoGutierrez left a comment

Choose a reason for hiding this comment

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

First this and then the NFD one right?

@elezar
Copy link
Member Author

elezar commented Feb 27, 2024

First this and then the NFD one right?

NFD includes these changes, so that's probably better.

- name: Install Go
uses: actions/setup-go@v5
with:
go-version-file: 'go.mod'
go-version: ${{ env.GOLANG_VERSION }}
Copy link
Member Author

Choose a reason for hiding this comment

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

@ArangoGutierrez we should probably look at how we can reuse something like this.

Copy link
Collaborator

Choose a reason for hiding this comment

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

for all other projects, yup

@elezar elezar merged commit a234bda into NVIDIA:main Feb 27, 2024
6 checks passed
@elezar elezar deleted the vendor-in-container branch February 27, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Issue/PR Pull about a dependency file maintenance Issue/PR to create or address a team project management need
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants