oxidecomputer / oxidecomputer/omicron

Rotated logs should not use timestamps as a marker

Open
#5,718 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
572
Forks
97
Avg merge
2d 12h
Merged PRs (30d)
96

Description

When the rack is coming up, it starts in 1986. This makes timestamps as a suffix for rotated logs meaningless, and it's very difficult to find information. Rotated logs should have a suffix/label that increases relative to the number of resident rotated logs with the same file name, or some other unit of measure that allows us to sort things out after the fact.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No file or test is named in the issue. Start by locating the log-rotation entry point and checking how suffixes are generated when the rack starts in 1986; done means rotated logs with the same file name use an orderable non-timestamp marker, with the behavior covered by a test.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
observability
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.