Skip to content

Conversation

@deitch
Copy link
Contributor

@deitch deitch commented Sep 26, 2025

Supersedes #141

@deitch deitch requested a review from milan-zededa September 26, 2025 09:17
milan-zededa
milan-zededa previously approved these changes Sep 26, 2025
@milan-zededa
Copy link
Contributor

Build is failing:

0.080 go: errors parsing go.mod:
0.080 /adam/src/go.mod:3: invalid go version '1.24.0': must match format 1.23
0.080 /adam/src/go.mod:5: unknown directive: toolchain

@deitch
Copy link
Contributor Author

deitch commented Sep 26, 2025

Yes, looks like it. Adam in dockerfile is based on an old eve-alpine, which has an older go 1.20. I am seeing what the most recent has, and will bump that, too, if it solves. Should know in about 2 mins.

@deitch
Copy link
Contributor Author

deitch commented Sep 26, 2025

Yup, eve-alpine:15.4.0 does it. Pushed and updated.

@deitch deitch changed the title bump golang.org/x deps bump golang.org/x deps; eve-alpine base version; go version in dockerfile Sep 26, 2025
@deitch
Copy link
Contributor Author

deitch commented Sep 26, 2025

OK, cool, this is a different error!

Signed-off-by: Avi Deitcher <avi@deitcher.net>
@deitch
Copy link
Contributor Author

deitch commented Sep 26, 2025

swagger needed to be updated as well, older version crashed. Was able to replicate, newer version works.

@milan-zededa mind another approval?

@milan-zededa milan-zededa merged commit e95f763 into lf-edge:master Sep 26, 2025
2 checks passed
@deitch deitch deleted the bump-go-x branch September 26, 2025 11:44
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