I’m not sure how returning UUIDs in our responses helps Katello. In our previous conversation, it was concluded that Katello should use the hrefs[0]. Why expose UUIDs if Katello is not going to store them?
Katello could store/use UUIDs but then it's going to run into problems when dealing with parameters that are hrefs (such as repository_version for publishing[1]). [0] https://www.redhat.com/archives/pulp-dev/2018-January/msg00004.html [1] https://github.com/pulp/pulp_file/blob/5ffb33d8c70ffbb247aba8bf5b4563 3eba414b79/pulp_file/app/viewsets.py#L54 David On Fri, Apr 27, 2018 at 4:29 PM, Dennis Kliban <[email protected]> wrote: > I can't remember why we decided to remove UUID from the responses. It > sounds like we should add them back. > > On Fri, Apr 27, 2018 at 12:26 PM, Justin Sherrill <[email protected]> > wrote: > >> Hi All! >> >> I started playing around with pulp 3 and generated bindings via >> https://pulp.plan.io/issues/3580 and it results somewhat in what you >> would expect. Here's an example: >> >> # @param id A UUID string identifying this repository. >> # @param [Hash] opts the optional parameters >> # @return [Repository] >> def repositories_read(id, opts = {}) >> data, _status_code, _headers = repositories_read_with_http_info(id, >> opts) >> return data >> end >> >> >> Notice that the UUID is to be passed in. When creating a repository, i >> only get the _href: >> >> { >> "_href": "http://localhost:8000/pulp/api/v3/repositories/bfc61565-89b >> 1-4b7b-9c4a-2ec91f299aca/", >> "_latest_version_href": null, >> "_versions_href": "http://localhost:8000/pulp/ap >> i/v3/repositories/bfc61565-89b1-4b7b-9c4a-2ec91f299aca/versions/", >> "created": "2018-04-27T15:26:03.546956Z", >> "description": "", >> "name": "test", >> "notes": {} >> } >> >> Meaning, there's really no way to use this specific binding with the >> return format for pulp. I imagine most binding generation would be >> expecting the user to know the ID of the objects and not work off of >> _hrefs. Any reason to not include the IDs in the response? >> >> Justin >> >> _______________________________________________ >> Pulp-dev mailing list >> [email protected] >> https://www.redhat.com/mailman/listinfo/pulp-dev >> > > > _______________________________________________ > Pulp-dev mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-dev > >
_______________________________________________ Pulp-dev mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-dev
