Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

[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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
addon/traefik help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants