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

fix: sidecar: reloader: fix output config file permission #4308

Merged
merged 1 commit into from
Jun 4, 2021

Conversation

OGKevin
Copy link
Contributor

@OGKevin OGKevin commented Jun 4, 2021

fixes: #4307

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Revert back permission change of reloader made in #4173

Verification

Used https://github.com/users/OGKevin/packages/container/package/thanos and works as before 👌🏾

@onprem
Copy link
Member

onprem commented Jun 4, 2021

Can you point this PR to the release-0.21 branch instead of main? We need this because we want to do a patch release.

@OGKevin OGKevin changed the base branch from main to release-0.21 June 4, 2021 10:09
@OGKevin
Copy link
Contributor Author

OGKevin commented Jun 4, 2021

Ok, gotta do some git magic to fix the diff. gimme a few.

Fixes: 4307

Signed-off-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com>
@OGKevin
Copy link
Contributor Author

OGKevin commented Jun 4, 2021

Done

Copy link
Member

@onprem onprem left a comment

Choose a reason for hiding this comment

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

LGTM!

@onprem onprem merged commit 7375f18 into thanos-io:release-0.21 Jun 4, 2021
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks!

onprem pushed a commit that referenced this pull request Jun 4, 2021
Fixes: 4307

Signed-off-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com>
@OGKevin OGKevin deleted the 4307-fix-permission-bug branch June 4, 2021 12:11
kakkoyun pushed a commit that referenced this pull request Jun 10, 2021
* Cut v0.21.0-rc.0

Signed-off-by: Prem Saraswat <prmsrswt@gmail.com>

* Cut v0.21.0

Signed-off-by: Prem Saraswat <prmsrswt@gmail.com>

* fix: sidecar: reloader: fix output config file permission (#4308)

Fixes: 4307

Signed-off-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com>

* Fixed CHANGELOG for 0.21.1

Signed-off-by: Prem Saraswat <prmsrswt@gmail.com>

* Cut release v0.21.1

Signed-off-by: Prem Saraswat <prmsrswt@gmail.com>

* Revert version change

Signed-off-by: Prem Saraswat <prmsrswt@gmail.com>

Co-authored-by: Kevin Hellemun <17928966+OGKevin@users.noreply.github.com>
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.

sidecar: file permission issue for envsub config file
3 participants