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

add requirements keyword to meta/main.yml #65809

Closed
wants to merge 2 commits into from

Conversation

pieteradams
Copy link

SUMMARY

Add requirements list to meta/main.yml for a role.
Ansible Galaxy will download the roles in the list, but playbook run won't execute them automatically (like it does for the roles specified in the dependencies list).

Fixes #35905

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ansible-galaxy

ADDITIONAL INFORMATION

meta/main.yml:

dependencies: []

requirements:
  - role1
  - role2

@ansibot ansibot added affects_2.10 This issue/PR affects Ansible v2.10 core_review In order to be merged, this PR must follow the core review workflow. feature This issue/PR relates to a feature request. needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Dec 13, 2019
@sivel
Copy link
Member

sivel commented Dec 13, 2019

fwiw, we have previously stated that we would accept a feature that included a requirements.yml file in the meta directory, as opposed to a change to main.yml. This would have to be discussed to determine if such a change is acceptable.

@pieteradams
Copy link
Author

A requirements.yml in meta directory would work for me, is there more information available for this feature request? Is it on the roadmap?

@sivel
Copy link
Member

sivel commented Dec 17, 2019

is there more information available for this feature request? Is it on the roadmap?

No, and no.

It has been discussed, and we have mentioned in several discussions that we would be open to this new file location. No one has begun development on it, and as such, there is no real info.

This is not something we are actively working on.

@bcoca
Copy link
Member

bcoca commented Dec 17, 2019

for context ansible/proposals#7 ansible/proposals#57

@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Dec 25, 2019
@samdoran
Copy link
Contributor

samdoran commented Jan 7, 2020

Closing per above.

@samdoran samdoran removed the needs_triage Needs a first human triage before being processed. label Jan 7, 2020
@samdoran samdoran closed this Jan 7, 2020
@ansible ansible locked and limited conversation to collaborators Feb 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.10 This issue/PR affects Ansible v2.10 feature This issue/PR relates to a feature request. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. new_contributor This PR is the first contribution by a new community member. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow role dependencies to not be executed
5 participants