Skip to content
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.

Sign up for GitHub

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

Jump to bottom

doc: added tz section to access log #7178

Merged
merged 4 commits into from
Aug 25, 2020
Merged

doc: added tz section to access log #7178

merged 4 commits into from
Aug 25, 2020

Conversation

kevtainer
Copy link
Contributor

What does this PR do?

Fixes #7177

Motivation

NO-yeast-Homemade-donuts-FINAL

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

@traefiker traefiker added this to the 2.2 milestone Aug 14, 2020
@ldez ldez added this to To review in v2 via automation Aug 14, 2020
rtribotte
rtribotte approved these changes Aug 24, 2020
View reviewed changes
Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

ldez
ldez approved these changes Aug 24, 2020
View reviewed changes
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

juliens
juliens approved these changes Aug 25, 2020
View reviewed changes
Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

kevin crawley and others added 4 commits August 25, 2020 12:32
@traefiker
doc: added tz section to access log
114ae66
@kevtainer @traefiker
Update access-logs.md
f5cdf27 
new line
@kevtainer @rtribotte @traefiker
Update docs/content/observability/access-logs.md
43ddbde 
removed unnecessary volume mount for tz declare

Co-authored-by: Romain <rtribotte@users.noreply.github.com>
@kevtainer @rtribotte @traefiker
condense TZ / file for access log timezone configuration
e2c21eb 
Co-authored-by: Romain <rtribotte@users.noreply.github.com>
@traefiker traefiker merged commit 532b586 into traefik:v2.2 Aug 25, 2020
v2 automation moved this from To review to Done Aug 25, 2020
@traefiker traefiker mentioned this pull request Aug 25, 2020
Closed
@ldez ldez added the area/logs label Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliens juliens juliens approved these changes

@ldez ldez ldez approved these changes

@rtribotte rtribotte rtribotte approved these changes

Assignees
No one assigned
Labels
area/documentation area/logs kind/enhancement a new or improved feature. size/S
Projects
No open projects
v2
Done
Milestone
2.2
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@kevtainer @juliens @ldez @rtribotte @traefiker