-
Notifications
You must be signed in to change notification settings - Fork 83
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 katello-agent related entities #1012
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1012 +/- ##
==========================================
- Coverage 92.18% 92.15% -0.03%
==========================================
Files 6 6
Lines 3057 3047 -10
==========================================
- Hits 2818 2808 -10
Misses 239 239
☔ View full report in Codecov by Sentry. |
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.
ACK
rebased |
@@ -4378,44 +4378,6 @@ def errata_applicability(self, synchronous=True, timeout=None, **kwargs): | |||
response = client.put(self.path('errata/applicability'), **kwargs) | |||
return _handle_response(response, self._server_config, synchronous, timeout) | |||
|
|||
def errata_apply(self, synchronous=True, timeout=None, **kwargs): |
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.
Can we remove errata_applicability method too as it is also removed right?
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.
Negative. Errata applicability recalculation functionality remains present.
[GET /api/hosts/:host_id/errata] List errata available for the content host
[GET /api/hosts/:host_id/errata/:id] Retrieve a single errata for a host
[PUT /api/hosts/:host_id/errata/applicability] Force regenerate applicability.
Katello-agent was deprecated and removed from 6.15 including related API endpoints. This PR removes those supported in nailgun.
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.
Ack pending other comment
Description of changes
Katello-agent was deprecated and removed from 6.15 including related API endpoints. This PR removes those supported in nailgun.
Upstream API documentation, plugin, or feature links
See the 6.15 apidoc for these particular endpoint. They are missing.
Functional demonstration
Additional Information
Robottelo PR for removal of the last used endpoint (
/api/hosts/bulk/install_content
) - SatelliteQE/robottelo#12817