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

[auto-bump][chart] traefik-1.90.0 #1287

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

d2iq-dispatch
Copy link
Contributor

The init container doesn't need this obscurity anymore.

What type of PR is this?

bug

What this PR does/ why we need it:

The certificate init process used to do something weird, reading the temporary and writing the actual (or something like that). kubeaddons-extrasteps was fixed to work on the final certificate so this indirection is no longer needed.

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Checklist

  • If a chart is changed, the chart version is correctly incremented.
  • The commit message explains the changes and why are needed.
  • The code builds and passes lint/style checks locally.
  • The relevant subset of integration tests pass locally.
  • The core changes are covered by tests.
  • The documentation is updated where needed.

@d2iq-dispatch d2iq-dispatch requested a review from a team as a code owner August 17, 2021 14:15
@d2iq-mergebot
Copy link
Contributor

This repo has @mesosphere-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@mesosphere-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.

@mesosphere-mergebot backport  

@d2iq-dispatch d2iq-dispatch added the help wanted Extra attention is needed label Aug 17, 2021
@joejulian joejulian added this to the release/next milestone Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
addon/traefik help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants