-
Notifications
You must be signed in to change notification settings - Fork 4
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
Added a volume for Portalbackend logs. #168
Conversation
4b089ba
to
39814d3
Compare
52d1164
to
492bf5a
Compare
de484ae
to
b58f3cd
Compare
b58f3cd
to
b208969
Compare
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.
@KFilippopolitis if my changes are ok, this can be merged
volumeMode: Filesystem | ||
accessModes: | ||
- ReadWriteOnce | ||
persistentVolumeReclaimPolicy: Delete |
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.
Let's go with Retain
here. Since it's logs, we need to keep them unless we explicitely delete them.
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.
@KFilippopolitis still same question
kubernetes/values.yaml
Outdated
logs: | ||
storage0: | ||
data_path: /opt/mip-deployment/logs | ||
data_size: 100Mi |
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.
In terms of size, this needs to target 1 year of logs for now, until we have a better archival procedure in place.
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.
@KFilippopolitis here too
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.
@jdaln there is the default mechanism of cscs that will back it up each day.
@ThanKarab would you know what is wrong here? |
cb6882b
to
6f108fc
Compare
No description provided.