-
Notifications
You must be signed in to change notification settings - Fork 608
Specify absolute path to rootlesskit-docker-proxy #231
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
Specify absolute path to rootlesskit-docker-proxy #231
Conversation
😢 Seems like this flag should allow items from LGTM |
BTW the latest version should work without this flag to be explicitly specified |
Yeah, I agree, this sounds like a Docker bug -- I think those flags are supposed to do a If we can just update |
The latest Docker has been already updated not to require this flag |
I have the following error when running a testcase using dockerjava Does this fix apply in this case as well? More context:
|
Yes it should, but the right fix is to just remove |
Changes: - docker-library/docker@7b00742: Merge pull request docker-library/docker#231 from angustrau/fix/rootlesskit-docker-proxy-absolute-path - docker-library/docker@a73d96e: Remove --userland-proxy-path (no longer necessary) - docker-library/docker@db82a88: Merge pull request docker-library/docker#232 from docker-library/github-actions - docker-library/docker@66fafcf: Add initial GitHub Actions CI
Changes: - docker-library/docker@7b00742: Merge pull request docker-library/docker#231 from angustrau/fix/rootlesskit-docker-proxy-absolute-path - docker-library/docker@a73d96e: Remove --userland-proxy-path (no longer necessary) - docker-library/docker@db82a88: Merge pull request docker-library/docker#232 from docker-library/github-actions - docker-library/docker@66fafcf: Add initial GitHub Actions CI
This fixes #230