-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Updated busybox sha; Added dependabot. #4202
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
Conversation
schedule: | ||
interval: "weekly" | ||
labels: ["dependencies"] | ||
- package_manager: "docker" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not see any rule about busybox SHA update rule, is it a magic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It actually works with digest, but not sure how it can detect ARM/AMD tbh, so that will not for now. Still I want to try this path (:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's try it :)
ifeq ($(arch), x86_64) | ||
# amd64 | ||
SHA="14d68ca3d69fceaa6224250c83d81d935c053fb13594c811038c461194599973" | ||
BASE_DOCKER_SHA="de4af55df1f648a334e16437c550a2907e0aed4f0b0edf454b0b215a9349bdbb" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sha is different than the default sha's for amd64 in the dockerfiles? Wouldn't these be the same or do we use some fallback?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it is updated, notice the date in comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We just always set it, maybe I can remove the one in dockefiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't it easier that all 'tools' use the same version no? I made this mistake by not understanding/seeing it :p See this PR: #4171
Sweet to use dependabot :D |
Need to re-trigger CI |
Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]>
* UI: Fix infinite redirection loop on / Signed-off-by: Prem Saraswat <[email protected]> * Updated busybox sha; Added dependabot. (#4202) Signed-off-by: Bartlomiej Plotka <[email protected]> * Fixed dependabot yml. (#4205) Signed-off-by: Bartlomiej Plotka <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]>
* UI: Fix infinite redirection loop on / Signed-off-by: Prem Saraswat <[email protected]> * Updated busybox sha; Added dependabot. (thanos-io#4202) Signed-off-by: Bartlomiej Plotka <[email protected]> * Fixed dependabot yml. (thanos-io#4205) Signed-off-by: Bartlomiej Plotka <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]>
* UI: Fix infinite redirection loop on / (#4208) * UI: Fix infinite redirection loop on / Signed-off-by: Prem Saraswat <[email protected]> * Updated busybox sha; Added dependabot. (#4202) Signed-off-by: Bartlomiej Plotka <[email protected]> * Fixed dependabot yml. (#4205) Signed-off-by: Bartlomiej Plotka <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]> * Cut release v0.20.2 (#4262) Signed-off-by: Kemal Akkoyun <[email protected]> Co-authored-by: Prem Saraswat <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka [email protected]