Skip to content

[CP-142] Add ListGlobalPackages service definition#97

Merged
mikheillomidzeq merged 12 commits into
mainfrom
feature/mikheil/publicly-accessible-packages-api
Jun 4, 2025
Merged

[CP-142] Add ListGlobalPackages service definition#97
mikheillomidzeq merged 12 commits into
mainfrom
feature/mikheil/publicly-accessible-packages-api

Conversation

@mikheillomidzeq
Copy link
Copy Markdown
Contributor

@mikheillomidzeq mikheillomidzeq commented May 30, 2025

  • Add new service for listing publicly(no auth) available packages - ListGlobalPackages
  • Extend message Package to include information about optional AvailableAdditionalResources

Relates to: https://qdrant.atlassian.net/browse/CP-142

@mikheillomidzeq mikheillomidzeq self-assigned this May 30, 2025
@mikheillomidzeq mikheillomidzeq requested a review from a team as a code owner May 30, 2025 05:15
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 30, 2025

The latest Buf updates on your PR. Results from workflow Pull Request / linting (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJun 4, 2025, 6:10 AM

Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
// permissions
option (common.v1.permissions) = "";
// gRPC Gateway REST call
option (google.api.http) = {get: "/api/public/v1/packages"};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't the intended way of creating URLs, but this is related to the fact a new Service got introduced

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed the url, please take a look if it looks okay now.

Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Copy link
Copy Markdown
Contributor

@Robert-Stam Robert-Stam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments / questions
PS: Name if the PR is a bit off.

@github-actions

This comment has been minimized.

Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the PR doesn't include updates for the generated code.

Please make sure you've run the following commands from the root directory:

make generate

Once you've added them, you can dismiss this review.

Comment thread proto/qdrant/cloud/booking/v1/booking.proto Outdated
Copy link
Copy Markdown
Contributor

@Robert-Stam Robert-Stam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See 2 nits, further LGTM
PS: Name if the PR is a bit off.

@mikheillomidzeq mikheillomidzeq changed the title [CP-142] Add PublicBookingService to fetch public packages [CP-142] Add ListGlobalPackages service definition Jun 3, 2025
…-api

# Conflicts:
#	.github/actions/spelling/allow.txt
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the PR doesn't include updates for the generated code.

Please make sure you've run the following commands from the root directory:

make generate

Once you've added them, you can dismiss this review.

@mikheillomidzeq mikheillomidzeq merged commit 1311a10 into main Jun 4, 2025
14 checks passed
@mikheillomidzeq mikheillomidzeq deleted the feature/mikheil/publicly-accessible-packages-api branch June 4, 2025 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants