Skip to content

Add extra sentence about testing core first #2619

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

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

samccann
Copy link
Contributor

@samccann samccann commented May 8, 2025

Based on idea from forum post.

@samccann samccann requested a review from mattclay May 8, 2025 19:43
@samccann samccann requested a review from nitzmahone May 8, 2025 19:43
@samccann samccann added techreview needs technical review backport-2.19 Automatically create a backport for the stable-2.19 branch labels May 8, 2025
@@ -29,6 +29,7 @@ with wide-ranging positive effects on security, performance, and user experience
Backward compatibility has been preserved where practical, but some breaking changes were necessary.
This guide describes some common problem scenarios with example content, error messsages, and suggested solutions.

We recommend you test your playbooks and roles in a staging environment with stable-2.19 to determine where you may need to make changes.
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be a good idea to show an example of how people can "install" stable-2.19?

@@ -29,6 +29,7 @@ with wide-ranging positive effects on security, performance, and user experience
Backward compatibility has been preserved where practical, but some breaking changes were necessary.
This guide describes some common problem scenarios with example content, error messsages, and suggested solutions.

We recommend you test your playbooks and roles in a staging environment with stable-2.19 to determine where you may need to make changes.
Copy link
Member

Choose a reason for hiding this comment

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

We should probably leave it up to users how they're going to acquire 2.19 for testing, rather than mentioning the stable branch, or even repeating the version number. How about something like this?

We recommend you test your playbooks and roles in a staging environment with this release to determine where you may need to make changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2.19 Automatically create a backport for the stable-2.19 branch techreview needs technical review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants