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

fix: use ISO 8601 year number #2621

Merged
merged 3 commits into from
Jan 2, 2025
Merged

fix: use ISO 8601 year number #2621

merged 3 commits into from
Jan 2, 2025

Conversation

tedim52
Copy link
Contributor

@tedim52 tedim52 commented Jan 2, 2025

Description

The Vector logs aggregator output %Y specifier for year which is the proleptic Gregorian year instead of %G specifier the ISO 8601. Logs are read with the ISO format so this created a mismatch in logs being read / written at the end of 2024 (eg. Y output the year 2024 on Dec. 30th 2024 whereas the ISO 8601 year on that date is 2025)

Is this change user facing?

NO

This reverts commit 7007fb4.
@tedim52 tedim52 added this pull request to the merge queue Jan 2, 2025
Merged via the queue into main with commit 97324e8 Jan 2, 2025
55 checks passed
@tedim52 tedim52 deleted the tedi/logsfix branch January 2, 2025 22:30
github-merge-queue bot pushed a commit that referenced this pull request Jan 7, 2025
🤖 I have created a release *beep* *boop*
---


##
[2.0.0](1.4.3...2.0.0)
(2025-01-02)


### ⚠ BREAKING CHANGES

* impl dependency list flag
([#2543](#2543))

### Features

* impl dependency list flag
([#2543](#2543))
([a6c1113](a6c1113))


### Bug Fixes

* use ISO 8601 year number
([#2621](#2621))
([97324e8](97324e8))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants