Skip to content
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 3654: Remove UUID from public repos #588

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

Andrewgdewar
Copy link
Contributor

Summary

Simply removes the UUID from the public_repos response.

Testing steps

  • Get /public_repositories/ list, check that the UUID is no longer present.

@jlsherrill
Copy link
Member

@xbhouse
Copy link
Contributor

xbhouse commented Feb 27, 2024

lgtm! thanks for the addition to the readme too :)

@swadeley
Copy link
Member

swadeley commented Feb 27, 2024

Hi

lgtm

In [3]: app.content_sources.rest_client.public_repositories_api.list_public_repositories(limit=4)
2024-02-27 18:45:26.209 [    INFO] [iqe.base.rest_client] REST: METHOD=GET, request_id=<>, params=[('limit', 4)]
Out[3]: 
{'data': [{'last_introspection_error': '',
           'last_introspection_time': '',
           'last_success_introspection_time': '',
           'last_update_introspection_time': '',
           'package_count': 0,
           'status': 'Pending',
           'url': 'http://mirror.centos.org/centos/8-stream/AppStream/x86_64/os/'},
          {'last_introspection_error': '',
           'last_introspection_time': '',
           'last_success_introspection_time': '',
           'last_update_introspection_time': '',
           'package_count': 0,
           'status': 'Pending',
           'url': 'http://mirror.centos.org/centos/8-stream/BaseOS/x86_64/os/'},
          {'last_introspection_error': '',
           'last_introspection_time': '',
           'last_success_introspection_time': '',
           'last_update_introspection_time': '',
           'package_count': 0,
           'status': 'Pending',
           'url': 'http://mirror.centos.org/centos/8-stream/extras/x86_64/os/'},
          {'last_introspection_error': '',
           'last_introspection_time': '',
           'last_success_introspection_time': '',
           'last_update_introspection_time': '',
           'package_count': 0,
           'status': 'Pending',
           'url': 'http://mirror.stream.centos.org/9-stream/AppStream/x86_64/os/'}],
 'links': {'first': '/api/content-sources/v1/public_repositories/?limit=4&offset=0',
           'last': '/api/content-sources/v1/public_repositories/?limit=4&offset=52',
           'next': '/api/content-sources/v1/public_repositories/?limit=4&offset=4'},
 'meta': {'count': 56, 'limit': 4, 'offset': 0}}

In [4]: 


@swadeley
Copy link
Member

/retest

@swadeley swadeley merged commit 1de4b54 into content-services:main Feb 27, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants