Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

[Vote ended on 2022-05-01] Schedule community.kubernetes for removal in Ansible 6 or Ansible 7 #93

Closed
felixfontein opened this issue Apr 24, 2022 · 17 comments
Labels
being_implemented This is currently being implemented

Comments

@felixfontein
Copy link
Contributor

felixfontein commented Apr 24, 2022

Summary

community.kubernetes 2.0.0 is mostly empty, it contains deprecated redirects to kubernetes.core. We announced in Ansible 4.2 (changelog fragment) that community.kubernetes will be removed eventually from Ansible, but we did not provide an explicit version cut-off. (See ansible-community/community-topics#22 for the general discussion.)

Let's vote on three (mutually exclusive) proposals:

  1. Remove community.kubernetes from Ansible 6.

  2. Announce removal of community.kubernetes from Ansible 7 in Ansible 6.0.0.

  3. Do not remove, or announce removal, of community.kubernetes from ansible for now. (ie, no change.)

The main con for 1), and pro for 2), is that we give users a longer deprecation period (~1.5 years instead of ~1 year) to update their FQCNs to kubernetes.core. Still including community.kubernetes does not really hurt, since there is almost no content.

Please vote by writing the option (1, 2 or 3) you like. You can also specify multiple options in order of preference (like: 2, 1, 3 if you prefer 2, but if there is no majority to it, prefer 1, and if there's no majority for it either, 3).

I've set the deadline to in one week, since on May 3rd the second Ansible 6 alpha will be released. If we want to remove something from Ansible 6, it would be good to do that from the alpha releases as early as possible.

CC @ansible-community/steering-committee

@felixfontein felixfontein added the active-vote These are currently active votes label Apr 24, 2022
@felixfontein felixfontein moved this from Fresh to Final Comments in Community Topics TODO Apr 24, 2022
@felixfontein
Copy link
Contributor Author

2, then 1

@mariolenz
Copy link
Contributor

The main con for 1), and pro for 2), is that we give users a longer deprecation period (> one year instead of > 6 months) to update their FQCNs to kubernetes.core.

Ansible 4.2 was released Jun 30, 2021. Even without an explicit version cut-off, that's nearly a year for users to react if we remove the collection from Ansible 6.

Still including community.kubernetes does not really hurt, since there is almost no content.

I slightly prefer 1 over 2. But 2 would be OK for me.

@felixfontein
Copy link
Contributor Author

You're right, the official release date is 2021-06-29 (I guess the release happened later in a timezone so it shows up as 2021-06-30 for us :) ). Since Ansible 6 GA is planned for 2022-06-21, this will be ~1.5 years vs. ~1 year. I'll update the issue.

@briantist
Copy link

I also slightly prefer 1) but am fine with 2)

Main reason for 1) is to not have to think about this again, and it's already been a long time, but Felix said, there is little downside to 2) without much in the way of content to break.

@markuman
Copy link
Contributor

+1 for 1.)

@gravesm
Copy link

gravesm commented Apr 25, 2022

Prefer 1, would be okay with 2.

@jillr
Copy link

jillr commented Apr 25, 2022

Strongly prefer 1, then 2.

@Andersson007
Copy link
Contributor

1, 2

@tima
Copy link

tima commented Apr 26, 2022

Like @gravesm and @jillr, I prefer 1, but would be okay with 2.

@mhalano
Copy link

mhalano commented Apr 29, 2022

I also prefer 1. Less is more sometimes. But we also should respect the users, so 2 is fine too.

@geerlingguy
Copy link

2, then 1, if only because that collection was pretty popular and I think until people see an official deprecation, they might not begin work to move to the new naming (except for those of us who follow development more closely).

1 wouldn't be too outlandish but I like the precedent of an official deprecation announced a release in advance for a removal.

It's not so bad, of course, considering that as a stopgap someone could still install the collection manually if it's gone, but that process should be highlighted in the release notes if #1 is chosen.

@acozine
Copy link
Contributor

acozine commented Apr 29, 2022

1 or 2, no preference

@russoz
Copy link
Contributor

russoz commented Apr 29, 2022

Users had ~1 year to be aware of the change. I think one more year as a deadline is enough. I vote 1. Then 2 but a hard NO on 3.

@felixfontein felixfontein changed the title [Vote ending on 2022-05-01] Schedule community.kubernetes for removal in Ansible 6 or Ansible 7 [Vote ended on 2022-05-01] Schedule community.kubernetes for removal in Ansible 6 or Ansible 7 May 1, 2022
@felixfontein felixfontein removed the active-vote These are currently active votes label May 1, 2022
@felixfontein felixfontein moved this from Final Comments to Resolved in Community Topics TODO May 1, 2022
@felixfontein
Copy link
Contributor Author

Counting votes:

  • 2, then 1:

    • SC: felixfontein
    • community: geerlingguy
  • 1, then 2:

    • SC: mariolenz, briantist, gravesm, jillr, Andersson007, russoz
    • community: tima, mhalano
  • 1:

    • SC: markuman
  • 1 or 2 (no preference):

    • SC: acozine

This means that 1 has a total of 9 SC votes and 3 community votes, 2 has a total of 8 SC votes and 3 community votes, and 7 SC votes and 2 comunity votes prefer 1, while 1 SC vote and 1 community vote prefer 2. I think it's pretty clear that 1 is accepted:

Remove community.kubernetes from Ansible 6.

Once someone from SC can confirm this vote count, this is accepted.

@briantist
Copy link

Confirming the count matches @felixfontein's count, counted a few times (this feels harder than it should be!). Someone suggested we try the new GitHub Discussions poll feature for vote counts, I think that's a good idea.

@felixfontein
Copy link
Contributor Author

Regarding voting: if you want to use something else, please create a discussion topic for that so we can properly discuss it :)

@felixfontein felixfontein added the being_implemented This is currently being implemented label May 2, 2022
@felixfontein
Copy link
Contributor Author

PR to remove community.kubernetes from Ansible 6: ansible-community/ansible-build-data#116

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
being_implemented This is currently being implemented
Projects
None yet
Development

No branches or pull requests