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

Remove onyx from old collection for merging with new one #74

Closed
wants to merge 6 commits into from

Conversation

samerd
Copy link
Contributor

@samerd samerd commented Jun 19, 2020

SUMMARY

Remove all references to onyx from collection towards merging to a new collection

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@felixfontein felixfontein requested a review from gundalow June 20, 2020 07:40
@felixfontein
Copy link
Collaborator

@samerd please update https://github.com/ansible/ansible/blob/devel/lib/ansible/config/ansible_builtin_runtime.yml so that it points the modules to the correct location.

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

You need to add redirect statements to meta/runtime.yml for every removed module, so that the modules can still be called via this collection:

    onyx_<xxx>:
      deprecation:
        removal_version: 3.0.0
        warning_text: The onyx_<xxx> module has been moved to the <namespace>.<name> collection
      redirect: <namespace>.<name>.onyx_<xxx>

(You need similar entries for the plugins in new entries in plugin_routing.)

@felixfontein
Copy link
Collaborator

I'm not really sure how this works, but I think there also need to be redirects for:

  1. the plugins (terminal and cliconf);
  2. the docs fragment;
  3. Python import redirects for module_utils.

I have really no idea whether a redirect is needed for the action plugin, or whether that's covered by the module...

@felixfontein
Copy link
Collaborator

@samerd could you please change the removal version to 2.0.0? We discussed this and decided to remove the leftovers earlier, since this should affect only pre-1.0.0 users.

@samerd
Copy link
Contributor Author

samerd commented Jun 23, 2020

@samerd could you please change the removal version to 2.0.0? We discussed this and decided to remove the leftovers earlier, since this should affect only pre-1.0.0 users.

sure will do

felixfontein and others added 4 commits June 24, 2020 22:30
* Remove F5 leftovers.

* Update sanity ignores.
* Comment out dead documentation link.

* Move changelog into root.

* Link to generated changelog in README.

* Add release summary.

* Require Ansible(-base) 2.9.10 or newer.

Needed for deprecation syntax support.

* Generate changelog.
Signed-off-by: Samer Deeb <[email protected]>
@samerd
Copy link
Contributor Author

samerd commented Jun 24, 2020

@felixfontein : please review
I changed the removal version

@felixfontein
Copy link
Collaborator

There's something strange going on here. GitHub shows that the PR contains a lot of changes that are already in master. Could you try rebasing? (Or is this a GitHub bug?)

@samerd
Copy link
Contributor Author

samerd commented Jun 29, 2020

Closing and opening a new clean PR

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