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

Fix execution environment #6

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Fix execution environment #6

merged 2 commits into from
Nov 25, 2024

Conversation

justinc1
Copy link
Contributor

Just fixing small errors.

Experience should ask for credentials, we cannot guess correct one - same as for inventory.

Execution environment apd-ee-25-windows needs to be used by created job templates.

In argument spec for windows_manage_iis_port, the required: true clashed with default value. We need to remove one.

Error was
fatal: [localhost]: FAILED! => {"argument_errors": ["internal error: required and default are mutually exclusive for windows_manage_iis_port"],

Signed-off-by: Justin Cinkelj <[email protected]>
@p3ck
Copy link
Contributor

p3ck commented Nov 25, 2024

ignoring the changelog sanity failure since this module has never been released yet.

@p3ck p3ck merged commit fffcf5d into redhat-cop:main Nov 25, 2024
20 of 21 checks passed
@justinc1 justinc1 deleted the fix-ee branch November 26, 2024 12:20
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.

2 participants