Skip to content

Conversation

synarete
Copy link
Collaborator

Fixes for ubuntu22.04, docker 27.5.1

@synarete synarete requested a review from anoopcs9 July 20, 2025 11:25
Copy link
Contributor

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

The commit build: use 'go 1.23.0' in go.sum mentioned go.sum but the change itself is made inside go.mod.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering why it didn't fail with our recent PRs. AFAICT, the runner is already on 24.04. Is that due to some major docker version upgrade these days? If so, can you please paste the error (just out of curiosity)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tried to reproduce this (as error), but now I only just get it as warn:

go: download go1.23 for linux/amd64: toolchain not available

Originally, it happened on a newly installed ubuntu 24.04 vm, but afterwards I upgrade the vm so maybe it fixed the problem.

Copy link
Contributor

Choose a reason for hiding this comment

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

I tried to reproduce this (as error), but now I only just get it as warn:

go: download go1.23 for linux/amd64: toolchain not available

Ah..I think I know why.

Go Toolchains where introduced with 1.21 and they follow a particular naming format. Go toolchain names has some details.

Ok, in that case I think it is wise to mention 1.23.0 to get rid of the warning.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Still, I don't know why is was an error on a newly installed ubuntu vm, but just a warn now. Sigh...

@anoopcs9
Copy link
Contributor

The commit build: use 'go 1.23.0' in go.sum mentioned go.sum but the change itself is made inside go.mod.

I'm good to go with the change once the commit message is fixed.

synarete added 2 commits July 21, 2025 14:12
When running 'docker build' over ubuntu 24.04 (docker version 27.5.1) it
fails to build code due to missing release-number in version string of
'go.mod'. Add explicit zero.

Signed-off-by: Shachar Sharon <[email protected]>
When building with docker over ubuntu 22.04 it emits warning: require
'AS' keyword instead of lowercase 'as'.

Signed-off-by: Shachar Sharon <[email protected]>
@synarete synarete force-pushed the ss-build-image-on-ubuntu-with-docker branch from 33a638b to 77fb4d4 Compare July 21, 2025 11:12
@synarete
Copy link
Collaborator Author

The commit build: use 'go 1.23.0' in go.sum mentioned go.sum but the change itself is made inside go.mod.

I'm good to go with the change once the commit message is fixed.

Done

Copy link
Contributor

@anoopcs9 anoopcs9 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.

@synarete synarete merged commit e83ca96 into samba-in-kubernetes:main Jul 21, 2025
8 checks passed
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.

2 participants