-
Notifications
You must be signed in to change notification settings - Fork 173
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
IPU:add stream installation way for leapp tool #4113
base: master
Are you sure you want to change the base?
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.
Please make the code flexible with less hard code, and don't provide internal info. cc @lijinlijin
Thanks @PaulYuuu will move the parameter to the internal cfg. |
8c0f57e
to
8862821
Compare
Test results |
8862821
to
5eaf2ad
Compare
@fbq815 could you also help to review this patch ? |
@@ -53,6 +53,8 @@ | |||
- compose_install: | |||
com_install = "yes" | |||
com_ins_leapp = "yum install leapp-upgrade -y" | |||
- stream_install: | |||
# Please modify the build in the internal cfg |
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.
I think this should add an example rather than point to internal.
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.
updated
48f9979
to
5e23f7b
Compare
Updated. Can you help to review it ? Thank you ! |
ID:2634 Signed-off-by: Miriam Deng <[email protected]>
5e23f7b
to
f7f55c3
Compare
seems there's some issue need to be handle, log is paste in the jira ticket |
The issue mentioned by Boqiao, it's a known leapp product issue and I've upgrade related information to ID:2641 |
Hi @fbq815 , |
I shared the test results from s390x |
ID:2634
Provide installing particular packages for leapp tool