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
Fixes #34063 - Drop deprecated Host#import_facts method #8961
Conversation
Issues: #34063 |
Plugin PRs: There are also a couple of uses in puppetdb and default_hostgroup plugins but they seem to be unmaintained. |
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.
once Katello PR gets in this is ready.
Nice cleanup @tbrisker :)
[test katello] |
1 similar comment
[test katello] |
[test katello] what the heck |
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'll try to open PR against the default_hostgroup before merging, but apart of that 👍
It's already done, not perfectly, but done 🤷 |
Katello test will fail, as will some other plugins that still use this method. Will open PRs to clean up there as well.