-
Notifications
You must be signed in to change notification settings - Fork 16.8k
[stable/jaeger-operator] make compatible with helm3 #19212
[stable/jaeger-operator] make compatible with helm3 #19212
Conversation
/assign @cpanato |
Hi @kimxogus. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
3d1b701
to
d1ee52c
Compare
@scottrigby Added |
355d6c8
to
6221888
Compare
/ok-to-test |
Is ci bot having issue? |
6221888
to
8d9d457
Compare
Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com>
Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com>
8d9d457
to
98e5044
Compare
@davidkarlsen CI seems to be broken in this pr. Can you re-label |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: batazor, davidkarlsen, kimxogus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I don't know if this is the right place to mention, but this is not working for us with helm v3.
The helm reference (https://helm.sh/docs/topics/chart_best_practices/custom_resource_definitions/) also mentions the root level crds folder to be used to store the CRD manifest files. |
* make compatible with helm3 Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com> * add crds.yaml Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com>
* make compatible with helm3 Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com> * add crds.yaml Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com>
* make compatible with helm3 Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com> * add crds.yaml Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com>
* make compatible with helm3 Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com> * add crds.yaml Signed-off-by: Taehyun Kim <kgyoo8232@gmail.com> Signed-off-by: Artur <artur@upbound.io>
Signed-off-by: Taehyun Kim kgyoo8232@gmail.com
Is this a new chart
No
What this PR does / why we need it:
.Capabilities.TillerVersion
. This pr fixes itcrd.install
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/mychartname]
)