-
Notifications
You must be signed in to change notification settings - Fork 348
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
Remove deprecated warnings for inst prefix #5723
base: master
Are you sure you want to change the base?
Remove deprecated warnings for inst prefix #5723
Conversation
/build-image |
/kickstart-tests --testtype dracut |
Images built based on commit 367ac1a:
Download the images from the bottom of the job status page. |
367ac1a
to
5db8a1e
Compare
5db8a1e
to
ca0a458
Compare
There were a couple of places were warnings were left behind to warn the users that using `inst.` was now required for some boot options. This patch removes the warnings as it is mandatory to have the prefix for those options.
ca0a458
to
0a3fcab
Compare
/kickstart-tests --testtype smoke |
/kickstart-tests --testtype dracut |
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.
We need to adjust 'dracut-visible-warnings.sh' test in https://github.com/rhinstaller/kickstart-tests/ repository and merge in one go.
/kickstart-tests --testtype dracut |
bcb7741
to
0a3fcab
Compare
We removed the deprecated warnings for inst.* in anaconda, thus, this test will always assert for something that will not appear anymore Anaconda PR: rhinstaller/anaconda#5723
Kickstart PR: rhinstaller/kickstart-tests#1239 |
@jstodola are you fine with doing this now? I guess it's fine for RHEL-11 to do this? |
Yes, I'm fine with this change. |
There were a couple of places were warnings were left behind to warn the users that using
inst.
was now required for some boot options. This patch removes the warnings as it is mandatory to have the prefix for those options.