-
Notifications
You must be signed in to change notification settings - Fork 147
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
remove unnecessary docker images for local sources #70
Conversation
The new dockerfiles were added by PR #69 that was approved by @tim-ywliu |
@abousselmi I think it is better to reuse same Dockerfile to build and less maintaining effort. |
nf_upf/Dockerfile
Outdated
@@ -9,7 +9,10 @@ ENV DEBIAN_FRONTEND noninteractive | |||
ARG DEBUG_TOOLS | |||
|
|||
# Install debug tools ~ 100MB (if DEBUG_TOOLS is set to true) | |||
RUN if [ "$DEBUG_TOOLS" = "true" ] ; then apt-get update && apt-get install -y vim strace net-tools iputils-ping curl netcat ; fi | |||
RUN if [ "$DEBUG_TOOLS" = "true" ] ; then apt-get update && apt-get install -y golang-go git vim strace net-tools iputils-ping curl netcat && \ | |||
git clone https://github.com/free5gc/go-gtp5gnl.git && mkdir "go-gtp5gnl/bin" && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move go-gtp5gnl tool build to base Dockerfile and copy it from base
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I move this build action to the base Dockerfile, I think it can't be optional due to we won't use the base image directly (Won't use DEBUG_TOOLS
).
base/Dockerfile.base-slim.nf
Outdated
@@ -16,7 +16,7 @@ ARG F5GC_MODULE | |||
COPY free5gc/ $GOPATH/src/free5gc/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can merge all base Dockerfile into one.
use COPY free5gc as default Dockerfile setting.
always git clone free5gc into base folder first when using local build.
LGTM |
Hi @ianchen0119 , @tim-ywliu , The readme states:
I tried this but Thanks. |
Hi @aviweit
|
Hi @ianchen0119 , I tried it. I looked into I think it would be very good to provide the ability to compile a specific NF which saves a lot of time. I am using #69 to compile only the NF I am working on. What do you think ? |
Hi @aviweit |
No description provided.