Skip to content

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

Merged
merged 1 commit into from
May 7, 2021
Merged

Updated busybox sha; Added dependabot. #4202

merged 1 commit into from
May 7, 2021

Conversation

bwplotka
Copy link
Member

@bwplotka bwplotka commented May 7, 2021

Signed-off-by: Bartlomiej Plotka [email protected]

schedule:
interval: "weekly"
labels: ["dependencies"]
- package_manager: "docker"
Copy link
Member

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?

Copy link
Member Author

@bwplotka bwplotka May 7, 2021

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 (:

Copy link
Member

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"
Copy link
Member

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?

Copy link
Member

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.

Copy link
Member Author

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

Copy link
Member

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

@wiardvanrij
Copy link
Member

Sweet to use dependabot :D

@daixiang0
Copy link
Member

Need to re-trigger CI

@kakkoyun kakkoyun merged commit 0c765b6 into main May 7, 2021
@bwplotka bwplotka deleted the busybox branch May 7, 2021 12:37
bwplotka added a commit that referenced this pull request May 7, 2021
onprem pushed a commit that referenced this pull request May 19, 2021
kakkoyun pushed a commit that referenced this pull request May 20, 2021
* 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]>
kakkoyun pushed a commit to kakkoyun/thanos that referenced this pull request May 21, 2021
* 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]>
onprem added a commit that referenced this pull request May 21, 2021
* 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]>
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