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.

Sign up for GitHub

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

Jump to bottom

Remove standalone cert-manager function #301

Closed
eak13 opened this issue Jul 20, 2020 · 5 comments
Closed

Remove standalone cert-manager function #301

eak13 opened this issue Jul 20, 2020 · 5 comments
Assignees
@sirajyasin
Labels
enhancement New feature or request
Milestone
betav1

Comments

@eak13
Copy link

eak13 commented Jul 20, 2020 •
edited
Loading

Proposed change
As a developer, I need manifests/function/cert-manager removed from the codebase so that I no longer need to maintain that code because operators will be using alternative certificate management functions.
@eak13 eak13 added enhancement New feature or request triage Needs evaluation by project members labels Jul 20, 2020
@eak13
Copy link
Author

eak13 commented Jul 20, 2020

capi, capbk, etc. all have cert-manager functionality included so the standalone function can be removed.

@airshipbot
Copy link

airshipbot commented Jul 21, 2020 •
edited
Loading

Related Change #742261

Subject: Remove standalone cert-manager function
Link: https://review.opendev.org/742261
Status: MERGED
Owner: Sirajudeen (sirajudeen.yasin@gmail.com)

This change will close this issue when merged.

Approvals

Code-Review
+1 Stas Egorov
+1 Bijaya Sharma
+2 Kostyantyn Kalynovskyi
+1 Ruslan Aliev
+2 Ian Howell
Verified
+2 Zuul
Workflow
+1 Ian Howell

Last Updated: 2020-07-30 15:15:06 CDT

@airshipbot airshipbot added the ready for review Change related to the issue is ready for review label Jul 21, 2020
@jezogwza jezogwza removed the triage Needs evaluation by project members label Jul 22, 2020
@jezogwza jezogwza added this to the betav1 milestone Jul 22, 2020
@sirajyasin
Copy link
Contributor

I can take up this issue.

@sreejithpunnapuzha
Copy link
Member

All yours @sirajyasin

@airshipbot
Copy link

The Change that closes this issue was merged.

@airshipbot airshipbot removed the ready for review Change related to the issue is ready for review label Jul 30, 2020
@airshipbot airshipbot added the ready for review Change related to the issue is ready for review label Jul 30, 2020
@eak13 eak13 removed the ready for review Change related to the issue is ready for review label Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sirajyasin sirajyasin

Labels
enhancement New feature or request
Projects
None yet
Milestone
betav1
Development

No branches or pull requests
5 participants
@eak13 @jezogwza @sirajyasin @sreejithpunnapuzha @airshipbot