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

paddle production docker image default command error #9391

Closed
putcn opened this issue Mar 26, 2018 · 2 comments · Fixed by #9392
Closed

paddle production docker image default command error #9391

putcn opened this issue Mar 26, 2018 · 2 comments · Fixed by #9392
Assignees

Comments

@putcn
Copy link
Contributor

putcn commented Mar 26, 2018

I built my own production image with dockerfile from build directory. When I try to run this image, the default command throws error as follows:

[root@k8s-node1 ~]# docker run --rm -it putcn/paddle:pro
/usr/local/bin/paddle: line 158: [: 0.10.0: binary operator expected
  File "<stdin>", line 3
    if LooseVersion("0.10.0
                          ^
SyntaxError: EOL while scanning string literal

looks like the there are some syntax issues, as well as version detecting issue.
and turnd out this line:

INSTALLED_VERSION=`pip freeze 2>/dev/null | grep "^paddle" | sed 's/.*==//g'`

is hitting both paddlepaddle and paddlepaddle-gpu, which broke the rest of execution.

@putcn
Copy link
Contributor Author

putcn commented Mar 28, 2018

I was. In production image there is a paddle cmd script added to PATH, and it's executed by default as "paddle version" to print paddle version if no training script is provided.
the issue is due to the cmd script's syntax issue and one version detecting logic issue, which are fixed within above PR.

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