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.

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

Update file timestamps to reflect real systemtime #1041

Merged
merged 3 commits into from
Oct 28, 2024

Conversation

blt
Copy link
Collaborator

@blt blt commented Oct 16, 2024

What does this PR do?

This commit retains the use of a stateless tick in the model but
allows for the filesystem to report times in systemtime. This
resolves a problem where all times were previously reported by the
filesystem as being UNIX_EPOCH even though the model kept track of
time.

@blt blt added the no-changelog label Oct 16, 2024 — with Graphite App
@blt blt marked this pull request as ready for review October 16, 2024 23:43
@blt blt requested a review from a team as a code owner October 16, 2024 23:43
@blt blt force-pushed the blt/begin_work_on_a_logrotate_filesystem branch from 7fb272d to 7c41181 Compare October 22, 2024 23:50
@blt blt force-pushed the blt/update_file_timestamps_to_reflect_real_systemtime branch from 0595449 to 0a6e9c0 Compare October 22, 2024 23:50
This was referenced Oct 22, 2024
@blt blt force-pushed the blt/begin_work_on_a_logrotate_filesystem branch from 7c41181 to bf824bb Compare October 24, 2024 00:10
@blt blt force-pushed the blt/update_file_timestamps_to_reflect_real_systemtime branch 2 times, most recently from ac43e43 to b66b30d Compare October 24, 2024 00:25
This was referenced Oct 25, 2024
@blt blt force-pushed the blt/begin_work_on_a_logrotate_filesystem branch 2 times, most recently from cded9df to 73825cc Compare October 28, 2024 14:41
@blt blt force-pushed the blt/update_file_timestamps_to_reflect_real_systemtime branch from b66b30d to ce08cc3 Compare October 28, 2024 14:41
@blt blt changed the base branch from blt/begin_work_on_a_logrotate_filesystem to graphite-base/1041 October 28, 2024 15:46
This commit retains the use of a stateless tick in the model but
allows for the filesystem to report times in systemtime. This
resolves a problem where all times were previously reported by the
filesystem as being UNIX_EPOCH even though the model kept track of
time.

Signed-off-by: Brian L. Troutwine <[email protected]>
Signed-off-by: Brian L. Troutwine <[email protected]>
@blt blt force-pushed the blt/update_file_timestamps_to_reflect_real_systemtime branch from ce08cc3 to 71727dc Compare October 28, 2024 15:46
@blt blt changed the base branch from graphite-base/1041 to main October 28, 2024 15:47
Signed-off-by: Brian L. Troutwine <[email protected]>
@blt blt force-pushed the blt/update_file_timestamps_to_reflect_real_systemtime branch from 71727dc to 79cde02 Compare October 28, 2024 15:47
@blt blt merged commit 382b730 into main Oct 28, 2024
15 checks passed
Copy link
Collaborator Author

blt commented Oct 28, 2024

Merge activity

  • Oct 28, 12:25 PM EDT: A user merged this pull request with Graphite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants