Skip to content

deps: update go version from 1.22.0 to 1.23.0#2985

Merged
mdelapenya merged 1 commit intotestcontainers:mainfrom
mmorel-35:go-1.23
Feb 20, 2025
Merged

deps: update go version from 1.22.0 to 1.23.0#2985
mdelapenya merged 1 commit intotestcontainers:mainfrom
mmorel-35:go-1.23

Conversation

@mmorel-35
Copy link
Copy Markdown
Contributor

What does this PR do?

Go 1.24.0 has been released on since 11 February 2025

This update Go version from 1.22.0 to 1.23.0

@mmorel-35 mmorel-35 requested a review from a team as a code owner February 16, 2025 19:00
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 16, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit d5b4b4d
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/67b62e5610b23200080671eb
😎 Deploy Preview https://deploy-preview-2985--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mmorel-35 mmorel-35 force-pushed the go-1.23 branch 2 times, most recently from a15ab57 to 12b5e47 Compare February 16, 2025 19:33
@mdelapenya
Copy link
Copy Markdown
Member

I think the GH worker is caching 1.x as 1.23, and it could take days/week until the new version is cached 🤔 I think it may be time to be explicit in the upper version and use 1.24.x instead of 1.x, as in the end, we are going to update both values altogether. Thoughts?

GH Link: https://github.com/testcontainers/testcontainers-go/actions/runs/13358160262/job/37319912281?pr=2985#step:4:13

Setup go version spec 1.x
Found in cache @ /opt/hostedtoolcache/go/1.23.6/x64
Added go to the path
Successfully set up Go version 1.x
/opt/hostedtoolcache/go/1.23.6/x64/bin/go env GOMODCACHE
/opt/hostedtoolcache/go/1.23.6/x64/bin/go env GOCACHE
/home/runner/go/pkg/mod
/home/runner/.cache/go-build
Cache hit for: setup-go-Linux-x64-ubuntu24-go-1.23.6-8fbfdf5cbb69770c0eeed237134966b320001c5e2591164a6de504e69371f767
Received 4194304 of 376341404 (1.1%), 4.0 MBs/sec
Received 121634816 of 376341404 (32.3%), 57.9 MBs/sec
Received 243269632 of 376341404 (64.6%), 77.3 MBs/sec
Received 359564188 of 376341404 (95.5%), 85.6 MBs/sec
Received 376341404 of 376341404 (100.0%), 85.1 MBs/sec
Cache Size: ~359 MB (376341404 B)
/usr/bin/tar -xf /home/runner/work/_temp/2097d012-2321-432f-8222-d4160499db85/cache.tzst -P -C /home/runner/work/testcontainers-go/testcontainers-go --use-compress-program unzstd
Cache restored successfully
Cache restored from key: setup-go-Linux-x64-ubuntu24-go-1.23.6-8fbfdf5cbb69770c0eeed237134966b320001c5e2591164a6de504e69371f767
go version go1.23.6 linux/amd64

@mmorel-35
Copy link
Copy Markdown
Contributor Author

Would purging the Github cache in testcountainers-go settings help with that ?

@mdelapenya
Copy link
Copy Markdown
Member

Mmm there are cases where it's not even cached: https://github.com/testcontainers/testcontainers-go/actions/runs/13368415985/job/37331548975#step:4:21

Setup go version spec 1.x
Found in cache @ /opt/hostedtoolcache/go/1.23.6/x64
Added go to the path
Successfully set up Go version 1.x
/opt/hostedtoolcache/go/1.23.6/x64/bin/go env GOMODCACHE
/opt/hostedtoolcache/go/1.23.6/x64/bin/go env GOCACHE
/home/runner/go/pkg/mod
/home/runner/.cache/go-build
Warning: Cache not found for keys: setup-go-Linux-x64-ubuntu24-go-1.23.6-ca467bad5f57cb6c4bddf7c09380c60d79248026037be686d06e0b0763691eb3
Cache is not found
go version go1.23.6 linux/amd64

@mmorel-35 mmorel-35 force-pushed the go-1.23 branch 3 times, most recently from 5e42aa4 to b82098e Compare February 18, 2025 05:53
@mmorel-35
Copy link
Copy Markdown
Contributor Author

Why merge instead of rebase ?

@mdelapenya
Copy link
Copy Markdown
Member

If the final state of the PR is going to be squashed, what's the difference? I personally don't like push-forces in shared branches, probably that's why I think merges are better, but I'd like to understand your point of view to learn from it too

@mmorel-35
Copy link
Copy Markdown
Contributor Author

If the final state of the PR is going to be squashed, what's the difference? I personally don't like push-forces in shared branches, probably that's why I think merges are better, but I'd like to understand your point of view to learn from it too

I like having a clean git history and once a merge commit happen on a branch I'm working on I can't squash them anymore but the maintainer of the original project can throught github. Several project don't necessary do by choice or by mistake. If you then need to cherry-pick a functionality (one expected commit) from a branch to another, that can become a real nightmare.

@mdelapenya mdelapenya added chore Changes that do not impact the existing functionality dependencies Dependencies or external services labels Feb 20, 2025
@mdelapenya mdelapenya self-assigned this Feb 20, 2025
@mdelapenya
Copy link
Copy Markdown
Member

I like having a clean git history and once a merge commit happen on a branch I'm working on I can't squash them anymore but the maintainer of the original project can throught github. Several project don't necessary do by choice or by mistake. If you then need to cherry-pick a functionality (one expected commit) from a branch to another, that can become a real nightmare.

For similar reasons on a clean git history, but for reviews, anytime a push-force is done, we maintainers have to create a new mental model for the state of the code, checking if something already reviewed changed. That's why I prefer updating the branch from the UI with a merge (or locally with git merge main --commit), because then new changes will come after the state all participants in the review already agreed, so there is no need to revisit the previous commits/state.

Of course, this is a process that depends on each project. I do like this way because it allows me to focus on the incoming deltas.

Copy link
Copy Markdown
Member

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@mdelapenya mdelapenya removed the chore Changes that do not impact the existing functionality label Feb 20, 2025
@mdelapenya mdelapenya merged commit 79e21bb into testcontainers:main Feb 20, 2025
178 checks passed
@mmorel-35 mmorel-35 deleted the go-1.23 branch February 20, 2025 09:08
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Feb 20, 2025
* main: (54 commits)
  deps: update go version from 1.22.0 to 1.23.0 (testcontainers#2985)
  feat(redpanda): add bootstrap user account option (testcontainers#2975)
  chore(ollama): bump default version to 0.5.7 (testcontainers#2966)
  feat!: log package for consistent output (testcontainers#2979)
  docs: remove duplicated options in the customisers lists (testcontainers#2989)
  chore: exclude "modules/k6" from the build (testcontainers#2987)
  chore: enable var-naming from revive (private vars only) (testcontainers#2978)
  chore(deps): bump actions/checkout from 4.1.7 to 4.2.2 (testcontainers#2971)
  chore(deps): bump release-drafter/release-drafter from 6.0.0 to 6.1.0 (testcontainers#2970)
  chore!: remove variadic arguments from nats ConnectionString (testcontainers#2967)
  fix(ci): use same condition for sonar steps (testcontainers#2974)
  fix: return unique modified modules (testcontainers#2973)
  chore(deps): bump golangci/golangci-lint-action from 6.2.0 to 6.3.0 (testcontainers#2969)
  chore(ci): run lint in a separate build before running the tests (testcontainers#2876)
  fix(deps): update to github.com/shirou/gopsutil/v4 (testcontainers#2964)
  fix(valkey): fix port race (testcontainers#2962)
  chore(deps): bump golang.org/x/net in /modules/pinecone (testcontainers#2963)
  chore(deps): bump golang.org/x/net from 0.26.0 to 0.33.0 (testcontainers#2961)
  deps(fix): include modulegen templates dir in dependabot updates (testcontainers#2956)
  chore(deps): bump docker/setup-docker-action from 4.0.0 to 4.1.0 (testcontainers#2959)
  ...
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Feb 20, 2025
* main: (34 commits)
  deps: update go version from 1.22.0 to 1.23.0 (testcontainers#2985)
  feat(redpanda): add bootstrap user account option (testcontainers#2975)
  chore(ollama): bump default version to 0.5.7 (testcontainers#2966)
  feat!: log package for consistent output (testcontainers#2979)
  docs: remove duplicated options in the customisers lists (testcontainers#2989)
  chore: exclude "modules/k6" from the build (testcontainers#2987)
  chore: enable var-naming from revive (private vars only) (testcontainers#2978)
  chore(deps): bump actions/checkout from 4.1.7 to 4.2.2 (testcontainers#2971)
  chore(deps): bump release-drafter/release-drafter from 6.0.0 to 6.1.0 (testcontainers#2970)
  chore!: remove variadic arguments from nats ConnectionString (testcontainers#2967)
  fix(ci): use same condition for sonar steps (testcontainers#2974)
  fix: return unique modified modules (testcontainers#2973)
  chore(deps): bump golangci/golangci-lint-action from 6.2.0 to 6.3.0 (testcontainers#2969)
  chore(ci): run lint in a separate build before running the tests (testcontainers#2876)
  fix(deps): update to github.com/shirou/gopsutil/v4 (testcontainers#2964)
  fix(valkey): fix port race (testcontainers#2962)
  chore(deps): bump golang.org/x/net in /modules/pinecone (testcontainers#2963)
  chore(deps): bump golang.org/x/net from 0.26.0 to 0.33.0 (testcontainers#2961)
  deps(fix): include modulegen templates dir in dependabot updates (testcontainers#2956)
  chore(deps): bump docker/setup-docker-action from 4.0.0 to 4.1.0 (testcontainers#2959)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependencies or external services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants