-
Notifications
You must be signed in to change notification settings - Fork 44
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 ProductFeatures#InitIfNeeded #907
Conversation
14d10d3
to
2fe30e8
Compare
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.
Just some notes to the test.
2fe30e8
to
8f4da9d
Compare
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.
Just a pair of comments about the unit tests.
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
d4b39af
to
55fda39
Compare
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.
Still LGTM 😉
✔️ Public Jenkins job #89 successfully finished |
✔️ Internal Jenkins job #94 successfully finished |
Problem
Thanks to a change made in the licenses location (FATE#324053 / SLE-4173), we realized that the
ProductFeatures#InitIfNeeded
method is not working as expected, since it is not restoring the available features values in the running system.This could be easily checked fixing by hand the wrong value for
base_product_license_directory
in the/etc/YaST2/ProductFeatures
file in a SLE-115-SP1, changing it from /etc/YaST2/licenses/base to /usr/share/licenses/product/base and running the yast2-firstboot module: it skips the License agreement step.Solution
Add tests and fix the method.
Related PR and discussions,