Adding organization_id to ActivationKey entity #990
Conversation
ColeHiggins2
added
fields
CherryPick
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #990 +/- ##
=======================================
Coverage 92.14% 92.14%
=======================================
Files 6 6
Lines 3043 3043
=======================================
Hits 2804 2804
Misses 239 239
☔ View full report in Codecov by Sentry. |
| @@ -226,6 +226,7 @@ def __init__(self, server_config=None, **kwargs): | |||
| Organization, | |||
| required=True, | |||
| ), | |||
| 'organization_id': entity_fields.IntegerField(), | |||
There was a problem hiding this comment.
There's organization field already present, so not sure why organization_id field is required here?
There was a problem hiding this comment.
The API was giving weird responses from what @ColeHiggins2 showed me when using org, but adding org_id helped.
There was a problem hiding this comment.
Let me take a look at it. Nailgun itself should take care of including organization_id in the HTTP request.
There was a problem hiding this comment.
@ogajduse if you look at this response.
Received HTTP 404 response: {"displayMessage":"One of parameters [ organization_id ] required but not specified."
organization_id is not being passed in. Even when I played around with the existing organization entity. I could not get the ID from it.
|
#992 is a more appropriate fix for this issue. |
|
Closing PR, it's been covered by #992 |
Description of changes
Adding
organization_idto the ActivationKey entityUpstream API documentation, plugin, or feature links
Robottelo PR: SatelliteQE/robottelo#12348