Skip to content
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

[WIP] Slip ansible 5 release to account for ansible-core slip. #75470

Closed
wants to merge 2 commits into from

Conversation

abadger
Copy link
Contributor

@abadger abadger commented Aug 11, 2021

SUMMARY

ansible-core-2.12 has slipped (beta1 slips 4 weeks. final slips 2 weeks).

We need to adjust the ansible-5 schedule to account for that and the winter holidays.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@ansibot ansibot added affects_2.12 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docs_only All changes are to files within the docs/docsite/ directory needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Aug 11, 2021
@dmsimard
Copy link
Contributor

LGTM 👍

docs/docsite/rst/roadmap/COLLECTIONS_5.rst Outdated Show resolved Hide resolved
docs/docsite/rst/roadmap/COLLECTIONS_5.rst Outdated Show resolved Hide resolved
docs/docsite/rst/roadmap/COLLECTIONS_5.rst Show resolved Hide resolved
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. core_review In order to be merged, this PR must follow the core review workflow. and removed core_review In order to be merged, this PR must follow the core review workflow. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Aug 12, 2021
@nitzmahone nitzmahone removed the needs_triage Needs a first human triage before being processed. label Aug 12, 2021
@ansibot ansibot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Aug 12, 2021
@samccann
Copy link
Contributor

@abadger sorry for the delay (and now the need for a rebase). Ping me in irc/slack when you rebase and I can merge. Thanks!

@abadger
Copy link
Contributor Author

abadger commented Aug 16, 2021 via email

@ansibot ansibot added the needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI. label Aug 18, 2021
abadger added a commit to abadger/ansible that referenced this pull request Aug 18, 2021
Alternative to ansible#75470

This version keeps the new collection dates from the old schedule which
gives more breathing room between the date on which new collections must
be approved and when ansible-5.0.0-beta1 comes out.  This will allow
collection owners who are also reviewers more time to devote to getting
their collections ready for release.
abadger added a commit to abadger/ansible that referenced this pull request Aug 18, 2021
Alternative to ansible#75470

This version keeps the new collection dates from the old schedule which
gives more breathing room between the date on which new collections must
be approved and when ansible-5.0.0-beta1 comes out.  This will allow
collection owners who are also reviewers more time to devote to getting
their collections ready for release.
@abadger abadger changed the title Slip ansible 5 release to account for ansible-core slip. [WIP] Slip ansible 5 release to account for ansible-core slip. Aug 18, 2021
@ansibot ansibot added the WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. label Aug 18, 2021
Improvements from the 08-11 meeting:
* Don't add a week to final for US Thanksgiving
* Move the date for collection inclusion five days earlier.
@abadger
Copy link
Contributor Author

abadger commented Aug 18, 2021

I think I've rebased this correctly but it could use a once over. There were a few tiny differences (like making sure there was a period at the end of every entry) which would be easy to miss.

At today's meeting, the steering committee will decide between this version of the schedule and this other one: #75529

@ansibot ansibot removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html labels Aug 18, 2021
abadger added a commit to abadger/ansible that referenced this pull request Aug 18, 2021
Alternative to ansible#75470

This version keeps the new collection dates from the old schedule which
gives more breathing room between the date on which new collections must
be approved and when ansible-5.0.0-beta1 comes out.  This will allow
collection owners who are also reviewers more time to devote to getting
their collections ready for release.
@abadger
Copy link
Contributor Author

abadger commented Aug 25, 2021

The steering committee has voted to approve the schedule in #75529 instead of this one. Closing this.

@abadger abadger closed this Aug 25, 2021
@abadger abadger deleted the slip-ansible-5-release branch August 25, 2021 18:10
@ansible ansible locked and limited conversation to collaborators Sep 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.12 docs_only All changes are to files within the docs/docsite/ directory docs This issue/PR relates to or includes documentation. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. support:core This issue/PR relates to code supported by the Ansible Engineering Team. WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants