Core 3.70 compatibility #3876
Core 3.70 compatibility #3876
Conversation
github-actions
bot
added
multi-commit
pedro-psb
force-pushed
the
core-3.70-compatibility
branch
from
94104a1 to
f0bf4ba
Compare
| @@ -88,7 +88,7 @@ class ModulemdDefaultsSerializer(NoArtifactContentSerializer): | |||
|
|
|||
| module = serializers.CharField(help_text=_("Modulemd name.")) | |||
| stream = serializers.CharField(help_text=_("Modulemd default stream.")) | |||
| profiles = CustomJSONField(help_text=_("Default profiles for modulemd streams.")) | |||
| profiles = serializers.JSONField(help_text=_("Default profiles for modulemd streams.")) | |||
There was a problem hiding this comment.
Can you explain the distinction between serializers.JSONField, JSONDictField, and JSONListField ?
There was a problem hiding this comment.
serializers.JSONField(drf) accept any json type (bool, str, object, array, ...) and has openapi typeany.JSONDictField(custom) validates it's a json object (aka dict) and has openapi typeobject.JSONListField(custom) validates it's a json array (aka list) and has openapi typearray.
2 and 3 subclasses from 1 and then adds the layer of validation.
But actually I found that my implementation of 2 and 3 is wrong.
I should either:
a) add the correct version of these fields here to get the types and validation right
b) use the serializers.JSONField which will change the types we use in openapi to any (which will probably not break ruby bindings this time because the new bindindg generator image probably supports any type)
There was a problem hiding this comment.
B) is what the python plugin just did I believe
There was a problem hiding this comment.
yes, I'll do that here as there is already a lot going on
pulp_rpm/tests/conftest.py
Outdated
| @pytest.fixture | ||
| def pulp_requests(bindings_cfg): | ||
| """Uses requests lib to issue an http request to pulp server using pulp_href. | ||
|
|
||
| Example: | ||
| >>> response = pulp_requests("get", "/pulp/api/v3/.../?repository_version=...") | ||
| >>> type(response) | ||
| requests.Response | ||
| """ | ||
| ALLOWED_METHODS = ("get", "update", "delete", "post") | ||
| auth = (bindings_cfg.username, bindings_cfg.password) | ||
| host = bindings_cfg.host | ||
|
|
||
| def _pulp_requests(method: str, pulp_href: str, body=None): | ||
| if method not in ALLOWED_METHODS: | ||
| raise ValueError(f"Method should be in: {ALLOWED_METHODS}") | ||
| url = urljoin(host, pulp_href) | ||
| request_fn = getattr(requests, method) | ||
| return request_fn(url, auth=auth) | ||
|
|
||
| return _pulp_requests | ||
|
|
||
|
|
There was a problem hiding this comment.
I think I have incidentally created a similar thing here:
https://github.com/pulp/pulp_ansible/blob/84a32718cbe18216b4067a50eea222d713c818ed/pulp_ansible/tests/functional/api/collection/v3/test_collection.py#L29
In any case, you can use requests.request(method, *) instead of the getattr dance.
There was a problem hiding this comment.
Nice. I guess I'll addopt the session approach.
#3865 declares compatibility with pulpcore 3.70, but the PR is getting a lot of errors not related to the changes.
For example, I can see a lot of binding validation errors which were somehow expected because of our bump of binding generator image version (see here).
I'm creating this PR so other errors related to the compatibility can be addressed independently.