-
Notifications
You must be signed in to change notification settings - Fork 192
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
Update trial pause docs #4636
Update trial pause docs #4636
Conversation
These statements conflict each other.
👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉 You can read more about the versioning within our docs in our documentation guidelines. |
00aa0dc
to
665c363
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving to get the docs out, but just some concerns to check:
- Does this mean paused dev tagged clusters are no longer automatically deleted after 30 paused days? You are removing this so I assume so?
- We no longer need to backport to 8.2 as it is being archived currently, but I can fix that in a follow-up PR once I'm properly back online
The first statement says "No data is lost", the second then "disk space is cleared".
I think we can keep it simple and just keep the latter.
Still needs to be back ported to all versions.
Description
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).