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

Images on quay.io can't run #127

Closed
jsafrane opened this issue Oct 23, 2018 · 6 comments · Fixed by #128
Closed

Images on quay.io can't run #127

jsafrane opened this issue Oct 23, 2018 · 6 comments · Fixed by #128

Comments

@jsafrane
Copy link
Contributor

$ docker run -ti quay.io/k8scsi/mock-driver:canary
standard_init_linux.go:190: exec user process caused "no such file or directory"

The reason is that mock binary inside the container can't be dynamically linked:

$  docker run -ti --entrypoint /bin/sh quay.io/k8scsi/mock-driver:canary
(inside the container)#  ldd mock
        /lib64/ld-linux-x86-64.so.2 (0x7fa22001d000)
        libpthread.so.0 => /lib64/ld-linux-x86-64.so.2 (0x7fa22001d000)
        libc.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7fa22001d000)

(inside the container)# ls /lib64/ld-linux-x86-64.so.2 
ls: /lib64/ld-linux-x86-64.so.2: No such file or directory

I've no clue how the image was built, perhaps some hiccup on travis.

@jsafrane
Copy link
Contributor Author

/reopen

It's still broken and now I know why. Travis does:

  1. make test
  2. make
  3. make container

Problem is that make test compiles its own bin/mock:

go build -o bin/mock ./mock || exit 1

Subsequent make and make container won't rebuild it, because it's fresh enough.

@k8s-ci-robot
Copy link
Contributor

@jsafrane: Reopening this issue.

In response to this:

/reopen

It's still broken and now I know why. Travis does:

  1. make test
  2. make
  3. make container

Problem is that make test compiles its own bin/mock:

go build -o bin/mock ./mock || exit 1

Subsequent make and make container won't rebuild it, because it's fresh enough.

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.

@jsafrane
Copy link
Contributor Author

/reopen

It's still broken in v0.3.0-2:

$ docker run -ti quay.io/k8scsi/mock-driver:v0.3.0-2
standard_init_linux.go:178: exec user process caused "no such file or directory"

today's canary is the same.

@k8s-ci-robot
Copy link
Contributor

@jsafrane: Reopening this issue.

In response to this:

/reopen

It's still broken in v0.3.0-2:

$ docker run -ti quay.io/k8scsi/mock-driver:v0.3.0-2
standard_init_linux.go:178: exec user process caused "no such file or directory"

today's canary is the same.

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.

@jsafrane
Copy link
Contributor Author

jsafrane commented Nov 2, 2018

/close

v0.3.0-3 works well, thanks to @lpabon!

@k8s-ci-robot
Copy link
Contributor

@jsafrane: Closing this issue.

In response to this:

/close

v0.3.0-3 works well, thanks to @lpabon!

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.

pohly added a commit to pohly/csi-test that referenced this issue Jan 26, 2021
1d60e77 Merge pull request kubernetes-csi#131 from pohly/kubernetes-1.20-tag
9f10459 prow.sh: support building Kubernetes for a specific version
fe1f284 Merge pull request kubernetes-csi#121 from kvaps/namespace-check
8fdf0f7 Merge pull request kubernetes-csi#128 from fengzixu/master
1c94220 fix: fix a bug of csi-sanity
a4c41e6 Merge pull request kubernetes-csi#127 from pohly/fix-boilerplate
ece0f50 check namespace for snapshot-controller
dbd8967 verify-boilerplate.sh: fix path to script
9289fd1 Merge pull request kubernetes-csi#125 from sachinkumarsingh092/optional-spelling-boilerplate-checks
ad29307 Make the spelling and boilerplate checks optional
5f06d02 Merge pull request kubernetes-csi#124 from sachinkumarsingh092/fix-spellcheck-boilerplate-tests
48186eb Fix spelling and boilerplate errors
71690af Merge pull request kubernetes-csi#122 from sachinkumarsingh092/include-spellcheck-boilerplate-tests
981be3f Adding spelling and boilerplate checks.
2bb7525 Merge pull request kubernetes-csi#117 from fengzixu/master
3b6d17b Merge pull request kubernetes-csi#118 from pohly/cloud-build-timeout
9318c6c cloud build: double the timeout, now 1 hour
4ab8b15 use the tag to replace commit of csi-test
5d74e45 change the csi-test import path to v4
7dcd0a9 upgrade csi-test to v4.0.2
86ff580 Merge pull request kubernetes-csi#116 from andyzhangx/export-image-name
c3a9662 allow export image name and registry name

git-subtree-dir: release-tools
git-subtree-split: 1d60e7792624a9938c0bd1b045211fbb89e513d6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants