-
Notifications
You must be signed in to change notification settings - Fork 95
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
Fixes #38094 - Support --els switch in convert2rhel #933
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apt
Thank you @stejskalleos ! |
template_inputs: | ||
- name: ELS | ||
required: true | ||
input_type: user | ||
description: Use an Extended Lifecycle Support (ELS) add-on subscription | ||
advanced: false | ||
value_type: plain | ||
options: "true\nfalse" | ||
default: "true" | ||
hidden_value: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After applying this patch, this option is not appearing in the inputs section on the Run Job page in UI for this job template, so just curious to know where do we define this template_inputs for UI?
advanced: false | ||
value_type: plain | ||
options: "true\nfalse" | ||
default: "true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be default no/false, right?
@@ -5,7 +5,20 @@ model: JobTemplate | |||
job_category: Convert 2 RHEL | |||
provider_type: script | |||
kind: job_template | |||
template_inputs: | |||
- name: ELS | |||
required: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be required=false, right?
No description provided.