-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat: add Dockerfile and Makefile target to build UBI based sidecar container image #194
Conversation
fc5c3c4
to
82690f5
Compare
sidecar/Makefile
Outdated
docker build --tag $(TAG) --build-arg VERSION=${VERSION} --build-arg RELEASE_NUMBER=${RELEASE_NUMBER} \ | ||
--file ${root_dir}sidecar/Dockerfile.ubi ${root_dir}sidecar/ |
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.
Formatting nitpick for readability
docker build --tag $(TAG) --build-arg VERSION=${VERSION} --build-arg RELEASE_NUMBER=${RELEASE_NUMBER} \ | |
--file ${root_dir}sidecar/Dockerfile.ubi ${root_dir}sidecar/ | |
docker build \ | |
--tag $(TAG) \ | |
--build-arg VERSION=${VERSION} \ | |
--build-arg RELEASE_NUMBER=${RELEASE_NUMBER} \ | |
--file ${root_dir}sidecar/Dockerfile.ubi ${root_dir}sidecar/ |
sidecar/Dockerfile.ubi
Outdated
|
||
LABEL name="Tailing Sidecar" \ | ||
maintainer="collection@sumologic.com" \ | ||
vendor="sumologic" \ |
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.
vendor="sumologic" \ | |
vendor="SumoLogic" \ |
?
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.
I think that we should write company name as "Sumo Logic". What do you think?
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.
🤔 might be. We'd have to change the labels in other images as well
82690f5
to
c7ef90f
Compare
c7ef90f
to
cdff540
Compare
example command to build image: