-
Notifications
You must be signed in to change notification settings - Fork 329
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
Deprecate role_name
in meta/main.yml
#1042
Comments
Back port this to 3.0. |
See corresponding Mazer issue #115 |
Will there be any way to have the Galaxy-hosted role name be different than the GitHub-hosted repo name? |
Marking as blocked, until we re-confirm this change with @tima and the product team. |
Deprecating Also, why did you remove the hyphens? Only noticed this change today and this seems like another, quite frankly, poorly justified change that adds little benefit. |
A datapoint for your consideration: As users of Galaxy our setup is broken by this proposal. The repos that back our Galaxy roles are in some cases widely used, forked and cloned. They all use the implicit naming convention of an We have just been hit by having all our roles unexpectedly renamed to use underscores. What do you suggest users like us (and users of our roles) should do to maintain consistent developer and user experiences? |
we were told that "use web interface to change role name". then, "use role_name". and now role_name is deprecated. the underscore change broke my deployments. |
@chouseknecht i appreciate your efforts to work on the issue. but i think a clear statement from the project should be given to users. Role Names in "Creating Role" does not say seriously, the galaxy project team needs to learn how to communicate with users from ansible project. i rarely read release notes from ansible project (sorry, my bad) unless my CI processes reports errors because ansible always tell us that something will be deprecated in version $X.$Y. ansible module documentations even says "this module is not yet stable", or "this module is stable enough and you may expect it will not break". I am quite faithful to ansible project. |
for those of you who decided to rename repository name to |
Yeah... just now noticed this issue, but I've spent waaaaay too much time this year managing role names with the UI changes (used to be able to adjust in Galaxy UI, then needed to manage via field in These kinds of changes may seem a bit trivial in isolation, but I think I've spent more time for some of my simpler roles just managing all the changes to Galaxy (and soon Mazer) than on maintaining the roles themselves. |
Based on more recent conversations, |
See also: ansible/galaxy#1042
https://galaxy.ansible.com/docs/contributing/creating_role.html#role-names Unsure about the future of role-name in ansible galaxy, though: ansible/galaxy#1042
https://galaxy.ansible.com/docs/contributing/creating_role.html#role-names Unsure about the future of role-name in ansible galaxy, though: ansible/galaxy#1042
During the Galaxy CLI Developer Workflow meeting we decided that Galaxy and Mazer should no longer synthesize the role (or repository) name, and therefore
role_name
needs to be deprecated.For reference see: https://docs.google.com/document/d/1nx65quXCyYORNel72rRCovyQo_qO5fhHlqrnKyYNlYU/edit?usp=sharing
To let users know about this change, we need to start issuing a deprecation warning in the import output whenever
role_name
is found inmeta/main.yml
.The text was updated successfully, but these errors were encountered: