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

build failure for remote target #913

Closed
Liujingfang1 opened this issue Mar 24, 2019 · 3 comments
Closed

build failure for remote target #913

Liujingfang1 opened this issue Mar 24, 2019 · 3 comments
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug.

Comments

@Liujingfang1
Copy link
Contributor

This is a regression recently added to head

./kustomize build https://github.com/kubernetes/kubectl/docs/book/examples/nginx
Error: evalsymlink failure on '/tmp/kustomize-332098206/docs/book/examples/nginx' : lstat /tmp/kustomize-332098206/docs: no such file or directory

Using 2.0.3, we have

./kustomize build https://github.com/kubernetes/kubectl/docs/book/examples/nginx
apiVersion: v1
kind: Service
metadata:
  name: nginx
spec:
  ports:
  - port: 80
    protocol: TCP
    targetPort: 9376
  selector:
    app: nginx
---
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: nginx
  name: nginx-deployment
spec:
  replicas: 3
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - image: nginx:1.7.9
        name: nginx
        ports:
        - containerPort: 80
@Liujingfang1 Liujingfang1 added the kind/bug Categorizes issue or PR as related to a bug. label Mar 24, 2019
@Liujingfang1
Copy link
Contributor Author

We should also add a test for remote target to make sure such regression is detected.

@Liujingfang1 Liujingfang1 added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Mar 26, 2019
@Liujingfang1
Copy link
Contributor Author

This regress is caused by #857

@Liujingfang1
Copy link
Contributor Author

@pst Can you take a look at this regression and see if you can fix it? Otherwise, we'll need to revert that change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

1 participant