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
Support for the online installation medium (jsc#SLE-7214) #815
Conversation
require "fileutils" | ||
require "pp" |
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.
why it is needed?
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.
To better format the logged data:
log.info "Found base products in the control.xml: #{@available_base_products.pretty_inspect}"
@@ -79,6 +86,10 @@ def registered? | |||
rescue LoadError | |||
false | |||
end | |||
|
|||
def product_id(prod) |
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.
hmm, can we extend prod to provide this method? For me it does not belong here, but to prod object itself.
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's actually a widget ID, it not related to the products at all, it's used for identifying the RadioButton widgets. So I'd leave it here. I just renamed it to widget_id
to make it clear.
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.
ah, with "widget_id" it fails, that's already used by CWM, renaming to "item_id"...
✔️ Public Jenkins job #69 successfully finished |
✔️ Internal Jenkins job #10 successfully finished |
control.xml
file