Skip to content

Conversation

Techassi
Copy link
Member

@Techassi Techassi commented Oct 2, 2025

This PR adjusts the end-of-support warning message as discussed in today's meeting.

@Techassi Techassi requested a review from lfrancke October 2, 2025 14:08
@Techassi Techassi self-assigned this Oct 2, 2025
@Techassi Techassi moved this to Development: In Progress in Stackable Engineering Oct 2, 2025
@Techassi Techassi marked this pull request as ready for review October 2, 2025 14:09
@Techassi Techassi moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Oct 2, 2025
"the operator reached end-of-support"
eos.built.datetime = built_datetime,
eos.build.age = build_age,
"This operator version was built on {built_datetime} ({build_age} days ago) and may has reached end-of-support. \
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"This operator version was built on {built_datetime} ({build_age} days ago) and may has reached end-of-support. \
"This operator version was built on {built_datetime} ({build_age} days ago) and may have reached end-of-support. \

@lfrancke
Copy link
Member

lfrancke commented Oct 2, 2025

One typo. I can't really comment on the Rust changes :(

Thank you!

@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Oct 2, 2025

### Changed

- Update the end-of-support warning message ([#1103])
Copy link
Member

Choose a reason for hiding this comment

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

I personally wouldn't add a new changelog entry, but just add this PR number to Add end-of-support checker.
EOS was never released, this change might confuse users, as they will not see the change.
Not blocking this PR though

Comment on lines +156 to +157
eos.built.datetime = built_datetime,
eos.build.age = build_age,
Copy link
Member

Choose a reason for hiding this comment

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

Is built vs build intentional?

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

Successfully merging this pull request may close these issues.

3 participants