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
bug-1878000: reduce spam from make run
#968
Conversation
make run
make run
We should land this after PR #965 because that picks up circleci fixes including updating to a more recent version of Docker. |
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.
Looks good to me. Just wondering – is the Kent upgrade related to that change?
docker/images/fakesentry/Dockerfile
Outdated
ARG groupid=5000 | ||
ARG userid=5000 |
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 these arguments are unused now and can be removed.
The Kent upgrade picks up changes to logging that reduce logging spam. It's not as applicable here because Antenna doesn't have the service healthchecks that Tecken has, but when we implement those, we'll want the fixes in Kent 1.2.0 that quell the logging that triggered with the healthchecks every second. |
160b048
to
6291d05
Compare
This reduces the copious output of backing services when running `make run`.
This upgrades Kent to 1.2.0 to pick up fixes to logging and other things. This also updates the Dockerfile to use the same base image Antenna does.
This moves the `--progress` flag to the global section so docker doesn't complain.
This switches to specifying which services to attach to rather than which ones _not_ to attach to.
6291d05
to
ad6e26d
Compare
Thank you! |
This reduces the spam in the output when running
make run
by explicitly not attaching to backing services and improving Kent logging.While doing that, I saw this output when running
make build
:So I fixed that, too.