-
Notifications
You must be signed in to change notification settings - Fork 900
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
configure apache to log to stdout/stderr #7375
Conversation
The following commits need their title changed:
Please format your commit title into the form:
This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here. |
@@ -44,6 +44,10 @@ mv spinnaker.conf /etc/apache2/sites-available | |||
|
|||
a2ensite spinnaker | |||
|
|||
# Disable default site | |||
|
|||
a2dissite 000-default |
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.
we're not using the default, so its good hygiene to disable it.
@@ -1,5 +1,7 @@ | |||
apt-get update | |||
apt-get install apache2 -y | |||
rm -rf /var/lib/apt/lists/* |
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.
there's a bit of space to save by cleaning up the apt cache.
Reports of deck failing with a full /var led me to finding that apache is not configured as per docker best practices to log to stdout/stderr. This addresses that. Signed-off-by: Paul Czarkowski <username.taken@gmail.com>
31e5a5d
to
652bd40
Compare
@Jammy-Louie : I'm not sure I'm the best person to look at this PR...is there someone more familiar with deck you could ask? |
Reports of deck failing with a full /var led me to finding that apache is not configured as per docker best practices to log to stdout/stderr. This addresses that. Signed-off-by: Paul Czarkowski <username.taken@gmail.com>
Apache is configured by default to log to
/var/log...
which fills up real quick with logs from kubernetes health checks and regular traffic, eventually the pod runs out of space. This modifies the config to log to stdout/stderr instead inline with docker/kubernetes best practices and should help protect the filesystem of the containers/pods.Signed-off-by: Paul Czarkowski username.taken@gmail.com