Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Remove deprecated certificate-service extension #518

Merged

Conversation

rfranzke
Copy link
Contributor

What this PR does / why we need it:
This PR deletes all code related to the deprecated certificate-service extension. It was replaced some months back by the new shoot-cert-service extension.

Special notes for your reviewer:
/assign @timuthy

Release note:

The deprecated certificate-service extension has been removed (it was replaced some months back by the `shoot-cert-service` extension).

@rfranzke rfranzke requested a review from timuthy December 27, 2019 15:25
@rfranzke rfranzke requested a review from a team as a code owner December 27, 2019 15:25
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 27, 2019
Copy link
Contributor

@timuthy timuthy left a comment

Choose a reason for hiding this comment

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

/lgtm - thanks for removing the extension.

@rfranzke rfranzke merged commit 8fc565d into gardener-attic:master Dec 30, 2019
@rfranzke rfranzke deleted the cleanup/remove-certificate-service branch December 30, 2019 08:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants