Skip to content

Commit

Permalink
Add charmhub required fields and files
Browse files Browse the repository at this point in the history
Adds new charmcraft.yaml that is required by charmhub.io. Also adds name
field to bundle.yaml, as required by charmhub.io.
  • Loading branch information
knkski committed Jan 20, 2022
1 parent d3c3f08 commit 68e2761
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 6 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/test-charmed-katib.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,8 @@ jobs:
juju bundle deploy --build --destructive-mode --serial
juju wait -wvt 300
kubectl label namespace kubeflow katib-metricscollector-injection=enabled
- name: Test Katib
run: kubectl apply -f examples/v1beta1/hp-tuning/random.yaml

Expand Down
3 changes: 0 additions & 3 deletions manifests/v1beta1/components/webhook/webhooks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ metadata:
webhooks:
- name: validator.experiment.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
admissionReviewVersions:
- v1
clientConfig:
Expand All @@ -32,7 +31,6 @@ metadata:
webhooks:
- name: defaulter.experiment.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
admissionReviewVersions:
- v1
clientConfig:
Expand All @@ -53,7 +51,6 @@ webhooks:
- experiments
- name: mutator.pod.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
admissionReviewVersions:
- v1
clientConfig:
Expand Down
1 change: 1 addition & 0 deletions operators/bundle.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
name: katib
bundle: kubernetes
applications:
katib-controller: { charm: katib-controller, scale: 1 }
Expand Down
1 change: 1 addition & 0 deletions operators/charmcraft.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
type: bundle
3 changes: 0 additions & 3 deletions operators/katib-controller/src/webhooks.yaml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ metadata:
webhooks:
- name: validator.experiment.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
# TODO (andreyvelich): Migrate to v1 ?
admissionReviewVersions:
- v1beta1
Expand Down Expand Up @@ -34,7 +33,6 @@ metadata:
webhooks:
- name: defaulter.experiment.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
admissionReviewVersions:
- v1beta1
clientConfig:
Expand All @@ -56,7 +54,6 @@ webhooks:
- experiments
- name: mutator.pod.katib.kubeflow.org
sideEffects: None
failurePolicy: Fail
admissionReviewVersions:
- v1beta1
clientConfig:
Expand Down

0 comments on commit 68e2761

Please sign in to comment.