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.

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

kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl #2635

Closed
kkasravi opened this issue Mar 6, 2019 · 3 comments

Comments

@kkasravi
Copy link
Contributor

kkasravi commented Mar 6, 2019

Currently both package managers dynamically load their .so (ksonnet.so, kustomize.so).
We should go back to a build option where these are statically linked in.

@kkasravi
Copy link
Contributor Author

kkasravi commented Mar 6, 2019

/area kfctl

@kkasravi
Copy link
Contributor Author

kkasravi commented Mar 6, 2019

/priority p1

kkasravi pushed a commit to kkasravi/kubeflow that referenced this issue Mar 11, 2019
kkasravi pushed a commit to kkasravi/kubeflow that referenced this issue Mar 11, 2019
kkasravi pushed a commit to kkasravi/kubeflow that referenced this issue Mar 11, 2019
k8s-ci-robot pushed a commit that referenced this issue Mar 12, 2019
* fixes 'add kustomize to kfctl'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl #2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl #2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl #2635'

* document what forms the version cli can have

* cleanup for static kfctl, fix for k8sSpecs, fix for --version pull/id/head
@kkasravi
Copy link
Contributor Author

fixed in #2673

saffaalvi pushed a commit to StatCan/kubeflow that referenced this issue Feb 11, 2021
* fixes 'add kustomize to kfctl'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* document what forms the version cli can have

* cleanup for static kfctl, fix for k8sSpecs, fix for --version pull/id/head
saffaalvi pushed a commit to StatCan/kubeflow that referenced this issue Feb 12, 2021
* fixes 'add kustomize to kfctl'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* fixes 'kfctl - provide option (default) to statically link ksonnet and kustomize into kfctl kubeflow#2635'

* document what forms the version cli can have

* cleanup for static kfctl, fix for k8sSpecs, fix for --version pull/id/head
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants