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

docs: migration guide from equinix.metal to equinix.cloud #130

Merged
merged 5 commits into from
Nov 15, 2023

Conversation

t0mk
Copy link
Contributor

@t0mk t0mk commented Nov 6, 2023

I gpted the migration guide. Fixes #122

Let's discuss the form and content.

Rendered guide at
https://github.com/equinix-labs/ansible-collection-equinix/blob/migration_guide/MIGRATION.md

@displague What do you think?

@displague
Copy link
Member

I think I see some problems in the table of resources. It would be good to double check that all of the resources from the Metal collection are listed with their new equivalent. Perhaps we can link to the open issue for any modules that don't have an equivalent.

It would also be helpful to show a small example that takes advantage of the most common modules. How would this example be translated to the new collection.

@t0mk
Copy link
Contributor Author

t0mk commented Nov 9, 2023

@displague I looked through the table, sorted it and removed one duplicate.

I am not keen to do an example of old and new style. It's a lot of work to create working example of the old collection, and it would be a random setup, not guaranteed to be actually useful. If we have an issue where a user will struggle with migration, I'm willing to do it. At this point, I would just leave it as it is, the table itself is a good reference for changing playbook from old to new style.

@displague displague changed the title Migration guide from equinix.metal to equinix.cloud docs: migration guide from equinix.metal to equinix.cloud Nov 15, 2023
@displague displague merged commit 76ed5fc into main Nov 15, 2023
@displague displague deleted the migration_guide branch November 15, 2023 00:04
Copy link

This PR is included in version 0.4.0 🎉

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.

Docs: Provide a migration guide on how to convert from the Packet and Metal providers
2 participants