Skip to content

fix: include count of objects added/removed to lifecycle logs - #1695

Open
itslenny wants to merge 1 commit into
masterfrom
lenny/lifecycle-logs-object-change-counter
Open

itslenny wants to merge 1 commit into
masterfrom
lenny/lifecycle-logs-object-change-counter

Conversation

@itslenny

@itslenny itslenny commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

What is the current behavior?

The change in object count is not included in the lifecycle logs.

Prior to object versioning this was trivial to infer from the event type, but with object versioning enabled it's not currently possible to reliably tell when underlying objects are added / removed.

What is the new behavior?

Add objectsChanged to lifecycle log data to allow tracking incremental change in total object count.

Additional context

Follow up to #1555

@itslenny
itslenny requested a review from a team as a code owner November 11, 2026 23:37

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code review found no issues

No high-confidence issues detected in this change.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 43121448393

Coverage increased (+0.002%) to 87.648%

Details

  • Coverage increased (+0.002%) from the base build.
  • Patch coverage: 4 of 4 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 16097
Covered Lines: 14550
Line Coverage: 90.39%
Relevant Branches: 10230
Covered Branches: 8525
Branch Coverage: 83.33%
Branches in Coverage %: Yes
Coverage Strength: 1357.44 hits per line

💛 - Coveralls

This branch has not been deployed

No deployments
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