Skip to content

Conversation

godrei
Copy link
Contributor

@godrei godrei commented Apr 10, 2025

Checklist

Version

Requires a PATCH version update

Context

In version 2.29.2 the project's package was updated to github.com/bitrise-io/bitrise/v2, but in the .goreleaser.yaml config, the version package's name was not updated. Because of this, no proper version information was built into the binaries, e.g.:

$ bitrise version --full

version: 2.30.5
format version: 23
os: darwin (arm64)
go: go1.22.7
build number: 
commit: (devel)

This PR fixes the issue by updating the version package name in .goreleaser.yaml.

@godrei godrei marked this pull request as ready for review April 10, 2025 15:19
ofalvai
ofalvai previously approved these changes Apr 10, 2025
gaborszakacs
gaborszakacs previously approved these changes Apr 11, 2025
tothszabi
tothszabi previously approved these changes Apr 11, 2025
@godrei godrei dismissed stale reviews from tothszabi, gaborszakacs, and ofalvai via 4c186ce April 11, 2025 07:44
@@ -63,7 +63,7 @@ func Test_Docker(t *testing.T) {
requireErr: true,
requireLogs: []string{
"failed to start containers:",
"bind: address already in use",
"address already in use",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems the error message we expected here has changed, now it looks like:

Error response from daemon: failed to set up container networking: driver failed programming external connectivity on endpoint bitrise-workflow-7ff29188-f7e6-4327-83ac-3cc0ae7296de (b2740f9aa549ac5bfb952008f343aedd1cca1979e65d8711efb8580f4e3af269): failed to bind host port for 0.0.0.0:22:172.18.0.2:22/tcp: address already in use
Error: failed to start containers: bitrise-workflow-7ff29188-f7e6-4327-83ac-3cc0ae7296de

The end of the error message has changed from bind: address already in use to failed to bind host port for <PORT>/tcp: address already in use

@godrei godrei merged commit c7bc712 into master Apr 11, 2025
8 checks passed
@godrei godrei deleted the fix-goreleaser-config branch April 11, 2025 08:10
@godrei godrei mentioned this pull request Apr 11, 2025
2 tasks
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.

4 participants