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

[Makefile:132: hydrate-kubeflow] Error 1 #2166

Closed
ryo-matsuzaka opened this issue Aug 31, 2020 · 11 comments
Closed

[Makefile:132: hydrate-kubeflow] Error 1 #2166

ryo-matsuzaka opened this issue Aug 31, 2020 · 11 comments

Comments

@ryo-matsuzaka
Copy link

At Deploy using kubectl and kpt,
when I run make apply, then I got the bellow error.

./kustomize build --load_restrictor none -o .build/kubeflow-apps ./instance/kustomize/kubeflow-apps
Error: accumulating resources: accumulateFile "accumulating resources from '../../../upstream/manifests/stacks/gcp': read /home/uyorikazatsuma/kubeflow/upstream/manifests/stacks/gcp: is a directory", accumulateDirector: "recursed accumulation of path '/home/uyorikazatsuma/kubeflow/upstream/manifests/stacks/gcp': accumulating resources: accumulateFile \"accumulating resources from '../../pipeline/installs/multi-user': read /home/uyorikazatsuma/kubeflow/upstream/manifests/pipeline/installs/multi-user: is a directory\", accumulateDirector: \"recursed accumulation of path '/home/uyorikazatsuma/kubeflow/upstream/manifests/pipeline/installs/multi-user': accumulating resources: accumulateFile \\\"accumulating resources from 'api-service': read /home/uyorikazatsuma/kubeflow/upstream/manifests/pipeline/installs/multi-user/api-service: is a directory\\\", accumulateDirector: \\\"couldn't make target for path '/home/uyorikazatsuma/kubeflow/upstream/manifests/pipeline/installs/multi-user/api-service': json: unknown field \\\\\\\"env\\\\\\\"\\\"\""
make: *** [Makefile:132: hydrate-kubeflow] Error 1

How can I solve this error?
Maybe it is related to this.

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the labels:

Label Probability
platform/gcp 0.56
area/docs 0.59
kind/bug 0.88

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@angoothachap
Copy link

angoothachap commented Oct 7, 2020

I tried to run the command without the --load_restrictor none and I get the below error:

teDirector: "recursed accumulation of path './istio-1.4.10-asm.18/kfdir/upstream/manifests/stacks/gcp': accumulating resources: accumulateFile \"accumulating resources from '../../admission-webhook/webhook/v3':
 './istio-1.4.10-asm.18/kfdir/upstream/manifests/admission-webhook/webhook/v3' must resolve to a file\", accumulateDirector: \"recursed accumulation of path './istio-1.4.10-asm.18/kfdir/ups
tream/manifests/admission-webhook/webhook/v3': accumulating resources: accumulateFile \\\"accumulating resources from '../overlays/application/application.yaml': security; file './istio-1.4.10-asm.18/kfdir/upst
ream/manifests/admission-webhook/webhook/overlays/application/application.yaml' is not in or below './istio-1.4.10-asm.18/kfdir/upstream/manifests/admission-webhook/webhook/v3'\\\", loader.New \\\"error loading
 ../overlays/application/application.yaml with git: url lacks host: ../overlays/application/application.yaml, dir: got file 'application.yaml', but './istio-1.4.10-asm.18/kfdir/upstream/manifests/admission-webh
ook/webhook/overlays/application/application.yaml' must be a directory to be a root, get: invalid source string: ../overlays/application/application.yaml\\\"\""

PS: I replaced my home directory with . in the above error stack

@angoothachap
Copy link

/assign @Bobgy @joeliedtke @rmgogogo

@Bobgy
Copy link
Contributor

Bobgy commented Oct 14, 2020

@ryo-matsuzaka Did you install kustomize 3.2.1?
It seems the kustomize breaking change after that --- "removal of env field support"

@Bobgy
Copy link
Contributor

Bobgy commented Oct 14, 2020

Upstream issue in kubeflow: kubeflow/manifests#538

@angoothachap
Copy link

@Bobgy Yes, I did use 3.2.1 as in the documentation. Do you suggest using a lower version?

@Bobgy
Copy link
Contributor

Bobgy commented Oct 14, 2020

No, that's correct, let me try to reproduce

@rcarmstrong
Copy link

rcarmstrong commented Oct 27, 2020

Experiencing the same issue when running make apply for a GCP deployment, specifically when it hits kustomize build --load_restrictor none -o .build/kubeflow-apps ./instance/kustomize/kubeflow-apps, I get the following error:

Error: accumulating resources: accumulateFile "accumulating resources from '../../../upstream/manifests/stacks/gcp': read /home/rarmstrong/kubeflow/deployment/upstream/manifests/stacks/gcp: is a directory", accumulateDirector: "recursed accumulation of path '/home/rarmstrong/kubeflow/deployment/upstream/manifests/stacks/gcp': accumulating resources: accumulateFile \"accumulating resources from '../../pipeline/installs/multi-user': read /home/rarmstrong/kubeflow/deployment/upstream/manifests/pipeline/installs/multi-user: is a directory\", accumulateDirector: \"recursed accumulation of path '/home/rarmstrong/kubeflow/deployment/upstream/manifests/pipeline/installs/multi-user': accumulating resources: accumulateFile \\\"accumulating resources from 'api-service': read /home/rarmstrong/kubeflow/deployment/upstream/manifests/pipeline/installs/multi-user/api-service: is a directory\\\", accumulateDirector: \\\"couldn't make target for path '/home/rarmstrong/kubeflow/deployment/upstream/manifests/pipeline/installs/multi-user/api-service': json: unknown field \\\\\\\"env\\\\\\\"\\\"\""

Is there a known work around? Thank you.

@Bobgy
Copy link
Contributor

Bobgy commented Nov 15, 2020

This looks exactly like kubeflow/metadata#202

Can you confirm kustomize version by

kustomize version

Or edit the file in the error message to replace the env field with envs as suggested in above issue

@thesuperzapper
Copy link
Member

This has been resolved by the new manifests refactor kubeflow/manifests#1777

But google does need to update their distribution docs on the website.

/close

@google-oss-robot
Copy link

@thesuperzapper: Closing this issue.

In response to this:

This has been resolved by the new manifests refactor kubeflow/manifests#1777

But google does need to update their distribution docs on the website.

/close

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.

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

9 participants