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.

Sign up for GitHub

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

Jump to bottom

CNN serving image build is broken #223

Open
ustiugov opened this issue Apr 30, 2021 · 5 comments
Open

CNN serving image build is broken #223

ustiugov opened this issue Apr 30, 2021 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@ustiugov
Copy link
Member

Describe the bug
CNN serving image is broken due to an update in a dependency.

To Reproduce
the error appears in the nightly testing

Expected behavior
Successful build

Logs
https://github.com/ease-lab/vhive/runs/2436625774?check_suite_focus=true#step:3:133

Workaround
Use pre-built image vhiveease/cnn_serving on DockerHub.
@ustiugov ustiugov added bug Something isn't working help wanted Extra attention is needed labels Apr 30, 2021
@adayaru
Copy link

adayaru commented Aug 22, 2022

I tried rebuilding this recently - since I made changes in my copy of server.py to help with my use-case.
I see errors in Docker build about untrusted signature with these lines:

    echo "http://dl-cdn.alpinelinux.org/alpine/edge/community" >> /etc/apk/repositories && \
    echo "http://dl-cdn.alpinelinux.org/alpine/edge/testing" >> /etc/apk/repositories && \

Eventually this too gives errors:
apk add --allow-untrusted --repository http://dl-3.alpinelinux.org/alpine/edge/testing hdf5 hdf5-dev && \
I tried rebuilding the base image (tatsushid/alpine-py3-tensorflow-jupyter) since that was a suggestion in one of the search results. That fails with this message:

[1mERROR: �[0m/tmp/bazel-0.7.0/src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD:29:1: Building src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jar (9 source files) failed (Exit 1): java failed: error executing command 
  (cd /tmp/bazel_XXMiCOln/out/execroot/io_bazel && \
  exec env - \
    LC_CTYPE=en_US.UTF-8 \
�[0m�[91m  /usr/lib/jvm/java-1.8-openjdk/bin/java -XX:+TieredCompilation '-XX:TieredStopAtLevel=1' -Xbootclasspath/p:third_party/java/jdk/langtools/javac-9-dev-r4023-3.jar -jar bazel-out/host/bin/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/bootstrap_deploy.jar @bazel-out/local-opt/bin/src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jar-2.params).
�[0m�[91mjava.lang.InternalError: Cannot find requested resource bundle for locale en_US

There is a warning message that could be related:
.�[0m�[91m�[35mWARNING: �[0m/tmp/bazel_XXMiCOln/out/external/bazel_tools/WORKSPACE:1: Workspace name in /tmp/bazel_XXMiCOln/out/external/bazel_tools/WORKSPACE (@io_bazel) does not match the name given in the repository's definition (@bazel_tools); this will cause a build error in future versions.

@ustiugov
Copy link
Member Author

@adayaru I suggest re-doing this image atop of python-slim instead of Alpine, as it is much easier to get right and maintain in a long term. We don't plan this work as of now so your contribution is more than welcome.

@niravnshah
Copy link
Contributor

Hi @ustiugov, @adayaru,

I have tried to hack my way to get it working.. The issue is due to rotation of the keys since Alpine 3.15.

I manually downloaded the required packages and have them installed in the docker image.
Working copy is here. 006bdda

However, same problem is present in rnn_serving image as well. it is also not getting built due to the same issue. However, i could not get it to work.

@ustiugov
Copy link
Member Author

@niravnshah could you please open a PR with the fix?

@niravnshah
Copy link
Contributor

@ustiugov, I am not sure if we can call this as a fix :)
I have explicitly downloaded x86_64 packages, meaning, it will not work for aarch64. Also, I am not sure if having a fixed version of apk is a good idea (they may become stale also, that's why we have those package managers). I guess, this needs to be looked from the base image perspective, and if that needs to/should be changed.

HermioneKT added a commit that referenced this issue Jan 31, 2024
@HermioneKT
# This is a combination of 252 commits.
3c6919f 
parent 6674807
author HermioneKT <hermionegrangerkt@gmail.com> 1706694164 +0800
committer HermioneKT <hermionegrangerkt@gmail.com> 1706694164 +0800

test

# This is the commit message #157:

test

# This is the commit message #159:

test

# This is the commit message #160:

test

# This is the commit message #161:

test

# This is the commit message #162:

test

# This is the commit message #163:

test

# This is the commit message #164:

tesT

# This is the commit message #165:

test

# This is the commit message #166:

test

# This is the commit message #167:

test

# This is the commit message #168:

test

# This is the commit message #169:

test

# This is the commit message #170:

test

# This is the commit message #171:

Test

# This is the commit message #172:

test

# This is the commit message #173:

test

# This is the commit message #174:

test

# This is the commit message #175:

test

# This is the commit message #176:

test

# This is the commit message #177:

test

# This is the commit message #178:

test

# This is the commit message #179:

test

# This is the commit message #180:

test

# This is the commit message #181:

test

# This is the commit message #182:

test

# This is the commit message #183:

test

# This is the commit message #184:

test

# This is the commit message #185:

test

# This is the commit message #186:

test

# This is the commit message #187:

test

# This is the commit message #188:

test

# This is the commit message #189:

test

# This is the commit message #190:

Test

# This is the commit message #191:

Test

# This is the commit message #192:

test

# This is the commit message #193:

Test

# This is the commit message #194:

test

# This is the commit message #195:

test

# This is the commit message #196:

test

# This is the commit message #197:

test

# This is the commit message #198:

test

# This is the commit message #199:

Test

# This is the commit message #200:

test

# This is the commit message #201:

test

# This is the commit message #202:

Test

# This is the commit message #203:

test

# This is the commit message #204:

test

# This is the commit message #205:

test

# This is the commit message #206:

test

# This is the commit message #207:

test

# This is the commit message #208:

test

# This is the commit message #209:

test

# This is the commit message #210:

test

# This is the commit message #211:

Test

# This is the commit message #212:

test

# This is the commit message #213:

Test

# This is the commit message #214:

Test

# This is the commit message #215:

Test

# This is the commit message #216:

test

# This is the commit message #217:

Test

# This is the commit message #218:

test

# This is the commit message #219:

test

# This is the commit message #220:

test

# This is the commit message #221:

test

# This is the commit message #222:

test

# This is the commit message #223:

test

# This is the commit message #224:

test

# This is the commit message #225:

test

# This is the commit message #226:

test

# This is the commit message #227:

test

# This is the commit message #228:

test

# This is the commit message #229:

Test

# This is the commit message #230:

test

# This is the commit message #231:

test

# This is the commit message #232:

test

# This is the commit message #233:

test

# This is the commit message #234:

Test

# This is the commit message #235:

test

# This is the commit message #236:

test

# This is the commit message #237:

test

# This is the commit message #238:

test

# This is the commit message #239:

test

# This is the commit message #240:

test

# This is the commit message #241:

test

# This is the commit message #242:

test

# This is the commit message #243:

test

# This is the commit message #244:

test

# This is the commit message #245:

test

# This is the commit message #246:

test

# This is the commit message #247:

test

# This is the commit message #248:

test

# This is the commit message #249:

test

# This is the commit message #250:

test

# This is the commit message #251:

test

# This is the commit message #252:

test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ustiugov @niravnshah @adayaru