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
Activate python2 as workaround on SLE15-SP1+ #14790
Activate python2 as workaround on SLE15-SP1+ #14790
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files |
65ce9a5
to
571cc34
Compare
lib/registration.pm
Outdated
@@ -867,7 +867,7 @@ sub scc_deregistration { | |||
quit_packagekit; | |||
wait_for_purge_kernels; | |||
assert_script_run('SUSEConnect --version'); | |||
if (get_var('UPGRADE_TARGET_VERSION') == '15-SP3') { | |||
if ((check_var('UPGRADE_TARGET_VERSION', '15-SP3')) && (is_sle('15+'))) { |
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.
LGTM
Please run a case from SLES15GA in continuous group. On SLES15GA, python2 module is not available too. So maybe need to change (is_sle('15+') -> (is_sle('15.1+')? |
Update condition for workaround for bsc#1189543 to make sure on SLE15-SP1+.
571cc34
to
3496378
Compare
Yes, I think so. I have updated the code and VRs. |
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.
LGTM
Update condition for workaround for bsc#1189543 to make sure on SLE15-SP1+.
http://openqa.nue.suse.com/tests/8644254# HA case
http://openqa.nue.suse.com/tests/8644255#step/patch_sle/103 12SP4
http://openqa.nue.suse.com/tests/8644259#details 15SP1
http://openqa.nue.suse.com/tests/8644387# Continuous migration SLE15GA ->15SP2, no activate python2, pass
https://openqa.nue.suse.com/tests/8644416#step/scc_deregistration/11 Continuous migration SLE15SP1->15SP3, activate python2, pass