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

chore: bump go version #448

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

csatib02
Copy link
Member

@csatib02 csatib02 commented Apr 22, 2024

Overview

@csatib02 csatib02 self-assigned this Apr 22, 2024
@csatib02 csatib02 requested a review from a team as a code owner April 22, 2024 08:10
@github-actions github-actions bot added the size/M Denotes a PR that changes 100-499 lines label Apr 22, 2024
@csatib02 csatib02 added the triage/on-hold Indicates an issue or PR is on hold for some reason. label Apr 22, 2024
Signed-off-by: Bence Csati <bcsati@cisco.com>
@csatib02 csatib02 force-pushed the chore/bump-go-version-update-dependencies branch from 1ab167d to 3b10cca Compare May 6, 2024 06:44
@csatib02 csatib02 removed the request for review from a team May 6, 2024 06:59
@csatib02 csatib02 changed the title chore: bump go version, update dependencies chore: bump go version, update dependencies, address breaking changes May 6, 2024
@csatib02 csatib02 removed the triage/on-hold Indicates an issue or PR is on hold for some reason. label May 6, 2024
Signed-off-by: Bence Csati <bcsati@cisco.com>
@github-actions github-actions bot added size/XL Denotes a PR that changes 1000+ lines and removed size/M Denotes a PR that changes 100-499 lines labels May 6, 2024
@ramizpolic ramizpolic added the triage/on-hold Indicates an issue or PR is on hold for some reason. label May 10, 2024
@ramizpolic ramizpolic changed the title chore: bump go version, update dependencies, address breaking changes chore: bump go version May 10, 2024
@ramizpolic
Copy link
Member

The pipeline (maybe NIX too) also need to be updated to use proper Go version

Signed-off-by: Bence Csati <bcsati@cisco.com>
Signed-off-by: Bence Csati <bcsati@cisco.com>
@csatib02 csatib02 force-pushed the chore/bump-go-version-update-dependencies branch from 055cfae to 546662c Compare May 13, 2024 11:54
Signed-off-by: Bence Csati <bcsati@cisco.com>
Comment on lines +104 to +106
vault = pkgs.buildGoModule rec {
pname = "vault";
version = "1.14.8";
Copy link
Member

Choose a reason for hiding this comment

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

any specific reason why we are building vault package here?

Copy link
Member Author

@csatib02 csatib02 May 15, 2024

Choose a reason for hiding this comment

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

Forgot to add it to the services, now it works:

image

Signed-off-by: Bence Csati <bcsati@cisco.com>
@csatib02 csatib02 requested a review from ramizpolic May 15, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies size/XL Denotes a PR that changes 1000+ lines triage/on-hold Indicates an issue or PR is on hold for some reason.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants