Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Multicasskop annotations #313

Merged
merged 2 commits into from
Apr 20, 2021
Merged

Multicasskop annotations #313

merged 2 commits into from
Apr 20, 2021

Conversation

cscetbon
Copy link
Contributor

@cscetbon cscetbon commented Apr 8, 2021

Q A
New feature? [X]
License Apache 2.0

What's in this PR?

Allow annotations to be passed when installing multicasskop chart

@fdehay
Copy link
Member

fdehay commented Apr 9, 2021

helm seems ok to me.
But why removing the 2 crds? Some cleanup?

@cscetbon
Copy link
Contributor Author

cscetbon commented Apr 9, 2021

helm seems ok to me.
But why removing the 2 crds? Some cleanup?

Those are not CRDs but just objects. They shouldn’t be there

@cscetbon cscetbon merged commit 5156d2b into master Apr 20, 2021
@cscetbon cscetbon deleted the multicasskop-annotations branch April 20, 2021 01:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants