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

Update Engine and top level makefiles #1522

Merged
merged 2 commits into from
Mar 9, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 0 additions & 38 deletions Jenkinsfile

This file was deleted.

6 changes: 3 additions & 3 deletions Makefile.ci
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
.PHONY: build
build:
@set -x && cd seldon-controller && make -f Makefile $@
@set -x && cd engine && make -f Makefile.ci clean $@
@set -x && cd engine && make -f Makefile clean $@

.PHONY: push_images_private_repo
push_images_private_repo:
@set -x && cd seldon-controller && make -f Makefile push_image_private_repo
@set -x && cd engine && make -f Makefile.ci push_image_private_repo
@set -x && cd engine && make -f Makefile push_image_private_repo

.PHONY: clean
clean:
cd engine && make -f Makefile.ci clean $@
cd engine && make -f Makefile clean $@
cd seldon-controller && make -f Makefile clean $@

37 changes: 0 additions & 37 deletions build-all-in-minikube

This file was deleted.

16 changes: 0 additions & 16 deletions build-all-private-repo

This file was deleted.

16 changes: 12 additions & 4 deletions engine/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,14 +1,22 @@
FROM openjdk:11.0.5-jre-slim
FROM maven:3.6.3-jdk-11 as builder

WORKDIR /workspace

COPY pom.xml pom.xml
COPY src/ src/

ARG APP_VERSION=UNKOWN_VERSION
# Build
RUN mvn clean verify -Dlicense.useMissingFile -B

FROM openjdk:11.0.5-jre-slim

RUN \
apt-get update -y && \
apt-get remove -y --auto-remove && \
apt-get clean -y && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*

COPY /target/seldon-engine-${APP_VERSION}.jar app.jar
COPY /target/generated-resources /licenses/
COPY --from=builder /workspace/target/seldon-engine-*.jar app.jar
COPY --from=builder /workspace/target/generated-resources /licenses/

ENTRYPOINT [ "sh", "-c", "java -Djava.security.egd=file:/dev/./urandom $JAVA_OPTS -jar app.jar" ]

22 changes: 0 additions & 22 deletions engine/Dockerfile.mvn

This file was deleted.

13 changes: 9 additions & 4 deletions engine/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ build_jar: update_proto update_swagger
mvn clean verify -Dlicense.useMissingFile -B

build_image: update_proto update_swagger
docker build -f Dockerfile.mvn -t seldonio/$(IMAGE_NAME):$(VERSION) .
docker build -f Dockerfile -t seldonio/$(IMAGE_NAME):$(VERSION) .

push_to_registry:
docker push seldonio/$(IMAGE_NAME):$(VERSION)
Expand All @@ -31,10 +31,15 @@ start_engine: engine.json
stop_engine: engine.json
kubectl delete --ignore-not-found=true -f engine.json

download_protos_k8s:
cd ../proto/k8s ; make create_protos

update_proto:
@cp -v ../proto/seldon_deployment.proto src/main/proto/
@cp -v ../proto/prediction.proto src/main/proto/
download_protos_tensorflow:
cd ../proto/tensorflow ; make create_protos

update_proto: download_protos_k8s download_protos_tensorflow
cp -v ../proto/seldon_deployment.proto src/main/proto/
cp -v ../proto/prediction.proto src/main/proto/
cp -vr ../proto/k8s/k8s.io src/main/proto
cp -vr ../proto/tensorflow/tensorflow src/main/proto

Expand Down
51 changes: 0 additions & 51 deletions engine/Makefile.ci

This file was deleted.

2 changes: 1 addition & 1 deletion jenkins-x.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ pipelineConfig:
command: make
args:
- -C engine
- -f Makefile.ci
- -f Makefile
- build_jar
- name: seldon-operator
steps:
Expand Down
17 changes: 0 additions & 17 deletions push-all-private-repo

This file was deleted.