-
Notifications
You must be signed in to change notification settings - Fork 41
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
Code 9 on scbw.play --install #79
Comments
Tried going back to docker version Version 17.09.0-ce-win33 (13620), with the same result. |
Do you use Docker for Windows or Docker Toolbox?
…On Sat, Jun 2, 2018 at 23:21 RothNath ***@***.***> wrote:
Tried going back to docker version Version 17.09.0-ce-win33 (13620), with
the same result.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#79 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AED1N4ZXkGYghtvQqoRTA-fHez-KHcPbks5t4smCgaJpZM4UVmI7>
.
|
Docker for windows. Version 17.09.0-ce-win33 (13620) is the version directly from the install directions. Thanks again for the help! |
I not sure that this is applicable to your case,
|
I think we are onto something here. I can't extract the archive. Bad DL? What size is starcraft.zip supposed to be? |
@RothNath I have @michalsustr it is probably indication that verification of downloading starcraft.zip file should be added to the source code. That allow us to spot such issues before creation of local docker container. That way error could be recovered. |
I don't have the best connection in the world, but it seems that the DL is extra touchy as far as connection quality. Here's what 3 install / DL attempts look like. It way be worth having it DL in a bit more robust way. scbw.play --install scbw.play --install |
Re-downloading in the end resolved the issue. Many thanks! |
expected behaviour
Successful installation
actual behaviour
scbw.play --install
WARNING Path C:\Users\557\AppData\Roaming/scbw found, re-installing scbw package.
WARNING Re-creating the base game image...
INFO checking docker can run
INFO checking whether docker has network sc_net
INFO docker image starcraft:game-1.0.3 removed.
INFO checking if there is local image starcraft:game-1.0.3
INFO image not found locally, creating...
INFO copying files from c:\python364\lib\site-packages\scbw\local_docker to C:\Users\557\AppData\Roaming/scbw\docker.
INFO pulling image ggaic/starcraft:java-1.0.3, this may take a while...
INFO building local image starcraft:game-1.0.3, this may take a while...
Traceback (most recent call last):
File "c:\python364\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\python364\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Python364\Scripts\scbw.play.exe_main.py", line 9, in
File "c:\python364\lib\site-packages\scbw\cli.py", line 209, in main
install()
File "c:\python364\lib\site-packages\scbw\install.py", line 19, in install
ensure_local_image(SC_IMAGE)
File "c:\python364\lib\site-packages\scbw\docker_utils.py", line 119, in ensure_local_image
path=base_dir, dockerfile='game.dockerfile', tag=local_image)
File "c:\python364\lib\site-packages\docker\models\images.py", line 266, in build
raise BuildError(chunk['error'], result_stream)
docker.errors.BuildError: The command '/bin/sh -c unzip -q /tmp/starcraft.zip -d /tmp/starcraft && rm -rf /tmp/starcraft/characters/* /tmp/starcraft/maps/* && chown starcraft:users -R /tmp/starcraft && cp /tmp/player* /tmp/starcraft/characters && chown starcraft:users -R /tmp/starcraft/characters' returned a non-zero code: 9
steps to reproduce
Install scbw in docker - 1.03 update directly. I can install 1.0.1 and upgrade to 1.0.3, but not go to 1.0.3 directly. At least in this current release of docker.
operating system
docker version
Client:
Version: 18.03.1-ce
API version: 1.37
Go version: go1.9.5
Git commit: 9ee9f40
Built: Thu Apr 26 07:12:48 2018
OS/Arch: windows/amd64
Experimental: false
Orchestrator: swarm
Server:
Engine:
Version: 18.03.1-ce
API version: 1.37 (minimum version 1.12)
Go version: go1.9.5
Git commit: 9ee9f40
Built: Thu Apr 26 07:22:38 2018
OS/Arch: linux/amd64
Experimental: true
The text was updated successfully, but these errors were encountered: