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

Replaced community.kubernetes with kubernetes.core in the openshift4_aws workshop #183

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

tarexveff
Copy link
Contributor

@tarexveff tarexveff commented Apr 28, 2022

The community.kubernetes Ansible collection has been hard-deprecated in favor of kubernetes.core. This pull request replaces all occurrences of community.kubernetes with the recommended kubernetes.core replacement within the openshift4_aws workshop directory path.

The community.kubernetes Ansible collection has been deprecated in favor of kubernetes.core.
@ajacocks
Copy link
Contributor

ajacocks commented May 3, 2022

Thanks! We're working on this across all the module names, in all the workshops.

@dpullman-emergent
Copy link

Hello! Is there anything we can do to get the two PRs, this one #183 and #181 merged? It may be there is some overlap? We've been running with local fixes in our fork waiting for this to merge, in particular the kubernetes.core FQCN changes.

Thanks!

@ajacocks
Copy link
Contributor

Sorry, there aren't as many folks using these, since the original support team was blown up, 2 years ago. We can try to see if we can get it done. But, we could use someone who wants to be a new maintainer.

Copy link
Contributor

@ajacocks ajacocks left a comment

Choose a reason for hiding this comment

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

Let me know if this breaks anything

@ajacocks ajacocks merged commit c946ce0 into RedHatGov:master Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants