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

Update to Roles Documentation main.yml Minimum Requirements #1262

Open
chynasan opened this issue Apr 10, 2024 · 4 comments
Open

Update to Roles Documentation main.yml Minimum Requirements #1262

chynasan opened this issue Apr 10, 2024 · 4 comments
Labels
needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member.

Comments

@chynasan
Copy link
Contributor

Galaxy and AutomationHub use roles/meta/main.yml to generate the description field for roles.

It's recently surfaced that the minimum required parameters for a meta/main.yml file seem to be:

galaxy_info:
  description: Generates Output folders for arista.avd collection.
  standalone: false

Using these parameters allows ansible-lint to still pass, and seems to fulfill the requirements for role metadata population in Galaxy/AutomationHub.

While many of our partners do use additional roles and fields successfully, the documentation at ansible-documentation/docs/docsite/rst/playbook_guide/playbooks_reuse_roles.rst does not specify what the actual minimum required parameters are for meta/main.yml. Is this something worth considering adding to the documentation to make it easier for collections developers to know what to configure?

@ansible-documentation-bot ansible-documentation-bot bot added needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. labels Apr 10, 2024
@ansible-documentation-bot
Copy link
Contributor

Thanks for your Ansible docs contribution! We talk about Ansible documentation on matrix at #docs:ansible.im and on libera IRC at #ansible-docs if you ever want to join us and chat about the docs! We meet there on Tuesdays (see the Ansible calendar) and welcome additions to our weekly agenda items - scroll down to find the upcoming agenda and add a comment to put something new on that agenda.

@oraNod
Copy link
Contributor

oraNod commented Apr 10, 2024

Thanks for the issue @chynasan This seems related to #1241 and, to answer your question, worthy of consideration in the docs imo. Feel free to go ahead and submit a PR with the doc changes if you're so inclined. You can also join us on Matrix in the docs channel if you have more questions or want to chat it out. Cheers.

@samccann
Copy link
Contributor

Discussed in the DaWGS meeting:

  • We need to discover what Galaxy requires. AH requirements should be covered by Red Hat official docs (tho if folks know the details, add to this issue and we can pass on to the correct team)
  • Need to decide where to put this info in our docs, since roles don't require this for local use, only for being shared in Galaxy.

@samccann
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member.
Projects
Status: 🆕 Triage
Development

No branches or pull requests

3 participants