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 role argument spec for activation_keys role #1602

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Apr 27, 2023

No description provided.

@evgeni
Copy link
Member Author

evgeni commented Apr 27, 2023

#1235 is where this originates from, but I didn't put in a "Fixes" as it's only one role to test out

Comment on lines +138 to +140
for role in $(notdir $(ROLES)); do \
$(PYTHON_COMMAND) ./docs/copy-role-examples.py roles/$${role}/examples.rst docs/plugins/$${role}_role.rst; \
done
Copy link
Member Author

Choose a reason for hiding this comment

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

this is a workaround for ansible/ansible#81734

description:
- This role creates and manages Activation Keys.
options:
foreman_server_url:
Copy link
Member Author

Choose a reason for hiding this comment

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

we have to duplicate these in all roles for now, see ansible/ansible#80657

- List of System Purpose Addons (ELS, EUS) to set on registering hosts.
- This should only be used when it is supported by the OS of registering hosts (RHEL 8 only at the time of writing).

notes:
Copy link
Member Author

Choose a reason for hiding this comment

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

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.

None yet

1 participant