Skip to content

docs(nx-cloud): document record expiration and align file server default with hash window#189

Merged
rarmatei merged 1 commit into
mainfrom
rares/document-record-expiration
Apr 29, 2026
Merged

docs(nx-cloud): document record expiration and align file server default with hash window#189
rarmatei merged 1 commit into
mainfrom
rares/document-record-expiration

Conversation

@rarmatei
Copy link
Copy Markdown
Contributor

@rarmatei rarmatei commented Apr 29, 2026

The issue

  • Legacy clearRecordsOlderThanDays removed in v1
  • No replacement documented in MIGRATION.md
  • File server default: 1 day
  • Aggregator hash default: 28 days
  • Files deleted before their hashes
  • Cache reads fail for ~27 days

The fix

  • File server default: 129
  • One day past the 28-day hash window
  • Added expiration section to MIGRATION.md
  • Documents all four env vars
  • Explains the safe-ordering rule
  • Patch bump to 1.2.4

…ver default to 29 days

The legacy `clearRecordsOlderThanDays` value kept the file server's
`NX_CACHE_EXPIRATION_PERIOD_IN_DAYS` and the aggregator's hash
expiration env vars in sync. It was removed in v1 with no replacement
documented in MIGRATION.md.

The v1 chart shipped `NX_CACHE_EXPIRATION_PERIOD_IN_DAYS: "1"` while
leaving the aggregator env unset — so by default Mongo retained hashes
for 28 days but files were deleted after 1, producing "artifact not
found" errors on cache reads for ~27 days after install.

This change:

- bumps the chart's file server default to 29 (one day past the
  aggregator's 28-day hash default) and explains why in a comment
- adds a "Configuring record and artifact expiration" section to
  MIGRATION.md covering defaults, the safe-ordering rule, and an
  override example
- patch-bumps the chart to 1.2.4
@rarmatei rarmatei merged commit ad3934f into main Apr 29, 2026
1 of 2 checks passed
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.

3 participants