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

Bump golang from 1.19-alpine to 1.20-alpine #2380

Merged
merged 3 commits into from
Mar 4, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 20, 2023

Bumps golang from 1.19-alpine to 1.20-alpine.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from nvuillam as a code owner February 20, 2023 16:10
@dependabot dependabot bot added dependencies Pull requests that update a dependency file docker Pull requests that update Docker code labels Feb 20, 2023
@bdovaz
Copy link
Collaborator

bdovaz commented Feb 20, 2023

@echoix in PR #2318 why didn't you choose this tag?

https://hub.docker.com/layers/library/golang/1-alpine/images/sha256-18da4399cedd9e383beb6b104d43aa1d48bd41167e312bb5306d72c51bd11548?context=explore

It is less prone to change (only in major versions) by dependabot.

Otherwise we will have a new PR every few days/weeks....

@echoix
Copy link
Collaborator

echoix commented Feb 20, 2023

At that time, I didn't know if a newer Go compiler could be used on older code. It seems that it should work, as the go 1.19 in revive's go.mod file (https://github.com/mgechev/revive/blob/4981346ff97e22e439bbe27dba11524d44c17af5/go.mod#L3) means the minimum go version, as explained in
https://go.dev/doc/modules/gomod-ref#go-syntax

It's only later that I knew about it.
The change that we could do is as you say, pin to go major version 1, and any alpine, since the produced executable doesn't depend on the os it was built on (go produces static binaries that work as long as cpu arch is compatible, and os type is the same).

@echoix
Copy link
Collaborator

echoix commented Feb 20, 2023

I'll do the pr on my cellphone tonight, and use the /build command as a little test :)

@echoix
Copy link
Collaborator

echoix commented Feb 20, 2023

/build

Command run output
Build command workflow started.
Installing dependencies
Running script ./build.sh
Build command workflow completed without updating files.
Command run output
Build command workflow started.
Installing dependencies
Running script ./build.sh
Build command workflow completed without updating files.

@nvuillam
Copy link
Member

nvuillam commented Feb 20, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ BASH bash-exec 6 0 0.01s
✅ BASH shellcheck 6 0 0.16s
✅ BASH shfmt 6 0 0 0.42s
✅ COPYPASTE jscpd yes no 3.57s
✅ DOCKERFILE hadolint 114 0 19.52s
✅ JSON eslint-plugin-jsonc 21 0 0 2.58s
✅ JSON jsonlint 19 0 0.34s
✅ JSON v8r 21 0 14.36s
⚠️ MARKDOWN markdownlint 309 0 230 8.11s
✅ MARKDOWN markdown-link-check 309 0 6.2s
✅ MARKDOWN markdown-table-formatter 309 0 0 21.56s
✅ OPENAPI spectral 1 0 2.24s
⚠️ PYTHON bandit 183 47 2.51s
✅ PYTHON black 183 0 0 4.77s
✅ PYTHON flake8 183 0 2.16s
✅ PYTHON isort 183 0 0 1.62s
✅ PYTHON mypy 183 0 8.99s
✅ PYTHON pylint 183 0 14.12s
⚠️ PYTHON pyright 183 254 20.45s
✅ REPOSITORY checkov yes no 34.42s
✅ REPOSITORY git_diff yes no 0.41s
✅ REPOSITORY secretlint yes no 15.46s
✅ REPOSITORY trivy yes no 33.84s
✅ SPELL cspell 745 0 28.71s
✅ SPELL misspell 566 0 0 1.15s
✅ XML xmllint 3 0 0 0.5s
✅ YAML prettier 81 0 0 3.57s
✅ YAML v8r 23 0 70.91s
✅ YAML yamllint 82 0 1.38s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@echoix
Copy link
Collaborator

echoix commented Feb 21, 2023

/build ref=1d9bec75a7db74169e8940c037a4e4b17ff3cd90

No ref found for: 1d9bec7

@echoix
Copy link
Collaborator

echoix commented Feb 21, 2023

/build ref=echoix:fix/build-sh-changes

No ref found for: echoix:fix/build-sh-changes

@echoix
Copy link
Collaborator

echoix commented Feb 21, 2023

Once #2381 merged, call @dependabot rebase, then /build

@echoix
Copy link
Collaborator

echoix commented Feb 26, 2023

@dependabot rebase

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 26, 2023

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@echoix
Copy link
Collaborator

echoix commented Feb 26, 2023

@dependabot recreate

Bumps golang from 1.19-alpine to 1.20-alpine.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/docker/golang-1.20-alpine branch from cb03cc9 to eb04a10 Compare February 26, 2023 20:35
@echoix
Copy link
Collaborator

echoix commented Feb 26, 2023

/build

Command run output
Build command workflow started.
Installing dependencies
Running script ./build.sh
Build command workflow completed without updating files.

@echoix
Copy link
Collaborator

echoix commented Feb 26, 2023

/build

Command run output
Build command workflow started.
Installing dependencies
Running script ./build.sh
Build command workflow completed without updating files.

@echoix
Copy link
Collaborator

echoix commented Mar 2, 2023

/build ref=pull/2406/merge

No ref found for: pull/2406/merge

@echoix
Copy link
Collaborator

echoix commented Mar 2, 2023

/build ref=dependabot/docker/golang-1.20-alpine

Command run output
Build command workflow started.
Installing dependencies
Running script ./build.sh
Build command workflow completed updating files.

@echoix
Copy link
Collaborator

echoix commented Mar 2, 2023

@dependabot rebase

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 2, 2023

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@nvuillam
Copy link
Member

nvuillam commented Mar 2, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ BASH bash-exec 6 0 0.02s
✅ BASH shellcheck 6 0 0.14s
✅ BASH shfmt 6 0 0 0.05s
✅ COPYPASTE jscpd yes no 2.85s
✅ DOCKERFILE hadolint 114 0 17.14s
✅ JSON eslint-plugin-jsonc 21 0 0 1.81s
✅ JSON jsonlint 19 0 0.29s
✅ JSON npm-package-json-lint yes no 0.64s
✅ JSON v8r 21 0 12.41s
⚠️ MARKDOWN markdownlint 309 2 230 6.33s
✅ MARKDOWN markdown-link-check 309 0 5.53s
✅ MARKDOWN markdown-table-formatter 309 2 0 17.94s
✅ OPENAPI spectral 1 0 1.68s
⚠️ PYTHON bandit 183 47 2.2s
✅ PYTHON black 183 0 0 3.6s
✅ PYTHON flake8 183 0 1.8s
✅ PYTHON isort 183 0 0 0.43s
✅ PYTHON mypy 183 0 7.55s
✅ PYTHON pylint 183 0 12.21s
⚠️ PYTHON pyright 183 251 17.51s
✅ REPOSITORY checkov yes no 30.91s
⚠️ REPOSITORY devskim yes 61 1.41s
✅ REPOSITORY dustilock yes no 1.43s
✅ REPOSITORY git_diff yes no 0.05s
✅ REPOSITORY secretlint yes no 8.55s
✅ REPOSITORY syft yes no 0.98s
✅ REPOSITORY trivy yes no 23.32s
✅ SPELL cspell 745 0 19.53s
✅ SPELL misspell 566 2 0 0.6s
✅ XML xmllint 3 0 0 0.03s
✅ YAML prettier 81 0 0 2.78s
✅ YAML v8r 23 0 60.71s
✅ YAML yamllint 82 0 1.08s

See detailed report in MegaLinter reports

You could have same capabilities but better runtime performances if you request a new MegaLinter flavor.

MegaLinter is graciously provided by OX Security

@nvuillam nvuillam merged commit 9c6f020 into main Mar 4, 2023
@nvuillam nvuillam deleted the dependabot/docker/golang-1.20-alpine branch March 4, 2023 22:56
@echoix
Copy link
Collaborator

echoix commented Mar 4, 2023

Did it need to have a different title for the change log, since it doesn't use the solution proposed by dependabot? @nvuillam

@nvuillam
Copy link
Member

nvuillam commented Mar 4, 2023

It could be another PR, but as it's a dependency update, if we forget it's not the end of the world ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file docker Pull requests that update Docker code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants