[pull] main from kubeflow:main#1700
Merged
Merged
Conversation
Signed-off-by: Yulia Krimerman <juliapiterova@hotmail.com>
* add fix for the searchbar Made-with: Cursor Signed-off-by: Philip Colares Carneiro <philip.colares@gmail.com> * fix the mcp catalog search placeholder Signed-off-by: Philip Colares Carneiro <philip.colares@gmail.com> --------- Signed-off-by: Philip Colares Carneiro <philip.colares@gmail.com>
#2518) * build(deps): bump pygments from 2.19.2 to 2.20.0 in /jobs/async-upload Bumps [pygments](https://github.com/pygments/pygments) from 2.19.2 to 2.20.0. - [Release notes](https://github.com/pygments/pygments/releases) - [Changelog](https://github.com/pygments/pygments/blob/master/CHANGES) - [Commits](pygments/pygments@2.19.2...2.20.0) --- updated-dependencies: - dependency-name: pygments dependency-version: 2.20.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chore(async-upload): regenerate requirements.txt Signed-off-by: Jon Burdo <jon@jonburdo.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Jon Burdo <jon@jonburdo.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jon Burdo <jon@jonburdo.com>
…pload (#2556) * build(deps): bump rfc3161-client in /jobs/async-upload Bumps [rfc3161-client](https://github.com/trailofbits/rfc3161-client) from 1.0.5 to 1.0.6. - [Release notes](https://github.com/trailofbits/rfc3161-client/releases) - [Changelog](https://github.com/trailofbits/rfc3161-client/blob/main/CHANGELOG.md) - [Commits](trailofbits/rfc3161-client@v1.0.5...v1.0.6) --- updated-dependencies: - dependency-name: rfc3161-client dependency-version: 1.0.6 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chore(async-upload): regenerate requirements.txt Signed-off-by: Jon Burdo <jon@jonburdo.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Jon Burdo <jon@jonburdo.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jon Burdo <jon@jonburdo.com>
Signed-off-by: ppadti <ppadti@redhat.com>
…#2574) Bumps [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) from 8.1.0 to 8.1.1. - [Release notes](https://github.com/peter-evans/create-pull-request/releases) - [Commits](peter-evans/create-pull-request@c0f553f...5f6978f) --- updated-dependencies: - dependency-name: peter-evans/create-pull-request dependency-version: 8.1.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…2575) Bumps [pypa/gh-action-pypi-publish](https://github.com/pypa/gh-action-pypi-publish) from 1.13.0 to 1.14.0. - [Release notes](https://github.com/pypa/gh-action-pypi-publish/releases) - [Commits](pypa/gh-action-pypi-publish@ed0c539...cef2210) --- updated-dependencies: - dependency-name: pypa/gh-action-pypi-publish dependency-version: 1.14.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Member
Bumps [actions/github-script](https://github.com/actions/github-script) from 8.0.0 to 9.0.0. - [Release notes](https://github.com/actions/github-script/releases) - [Commits](actions/github-script@ed59741...3a2844b) --- updated-dependencies: - dependency-name: actions/github-script dependency-version: 9.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 7.0.0 to 7.1.0. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](docker/build-push-action@d08e5c3...bcafcac) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-version: 7.1.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…41.0 to 0.42.0 (#2578) Bumps [github.com/testcontainers/testcontainers-go](https://github.com/testcontainers/testcontainers-go) from 0.41.0 to 0.42.0. - [Release notes](https://github.com/testcontainers/testcontainers-go/releases) - [Commits](testcontainers/testcontainers-go@v0.41.0...v0.42.0) --- updated-dependencies: - dependency-name: github.com/testcontainers/testcontainers-go dependency-version: 0.42.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…1 in /clients/ui/bff (#2582) Bumps [github.com/brianvoe/gofakeit/v7](https://github.com/brianvoe/gofakeit) from 7.7.3 to 7.14.1. - [Release notes](https://github.com/brianvoe/gofakeit/releases) - [Commits](brianvoe/gofakeit@v7.7.3...v7.14.1) --- updated-dependencies: - dependency-name: github.com/brianvoe/gofakeit/v7 dependency-version: 7.14.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix(catalog): implement pagination on MCP server tools endpoint
The tools endpoint accepted pageSize but always returned an empty
nextPageToken, making multi-page traversal impossible.
Delegate MCPServerToolRepository.List() to GenericRepository.List(),
which handles ordering, the fetch-N+1 hasMore check, and cursor token
generation. Move the Association join into applyMCPServerToolListFilters
so the generic list path can apply it. Wire the resulting NextPageToken
through to the API response.
Assisted-by: Claude Sonnet 4.6
Signed-off-by: Paul Boyd <paul@pboyd.io>
* fix(catalog): wire toolLimit to GetMCPServer endpoint
The toolLimit query parameter was accepted but silently ignored on
GET /mcp_servers/{server_id}. The parameter was missing from the
OpenAPI spec for that endpoint, so the generated controller never
parsed it and no downstream layer received it.
Add toolLimit to the GetMCPServer spec, regenerate the controller and
client, and propagate the value through the service, provider interface,
and DB implementation — mirroring the pattern already used in
ListMCPServers.
Assisted-by: Claude Sonnet 4.6
Signed-off-by: Paul Boyd <paul@pboyd.io>
* fix(catalog): filter MCP server tool by name at DB level and enforce toolLimit minimum
GetMCPServerTool previously loaded all tools into memory to find one by
name. With paginated List results, tools beyond the first page would be
silently missed if a default page size were ever applied. Move the name
filter to the database layer using a LIKE suffix match on the qualified
name column.
Also adds minimum: 0 to the toolLimit OpenAPI parameter so negative
values are rejected before reaching the service layer.
Assisted-by: Claude Sonnet 4.6
Signed-off-by: Paul Boyd <paul@pboyd.io>
* Update catalog/internal/catalog/mcpcatalog/service/mcp_server_tool.go
Co-authored-by: Chris Hambridge <chambrid@redhat.com>
Signed-off-by: Paul Boyd <paul@pboyd.io>
---------
Signed-off-by: Paul Boyd <paul@pboyd.io>
Co-authored-by: Chris Hambridge <chambrid@redhat.com>
…/mysql from 0.41.0 to 0.42.0 (#2579) Bumps [github.com/testcontainers/testcontainers-go/modules/mysql](https://github.com/testcontainers/testcontainers-go) from 0.41.0 to 0.42.0. - [Release notes](https://github.com/testcontainers/testcontainers-go/releases) - [Commits](testcontainers/testcontainers-go@v0.41.0...v0.42.0) --- updated-dependencies: - dependency-name: github.com/testcontainers/testcontainers-go/modules/mysql dependency-version: 0.42.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
When filterQuery is provided without an explicit orderBy, the API layer
set OrderBy/SortOrder to pointers to empty strings, bypassing the
nil-based defaults ("id"/"ASC"). This caused non-deterministic results
and empty subsequent pages with cursor-based pagination.
Only set the pagination pointers when values are non-empty, so the
defaults apply correctly.
Assisted-by: Claude Code <noreply@anthropic.com>
Signed-off-by: Alessio Pragliola <seth.pro@gmail.com>
openshift-merge-bot
Bot
merged commit ce521cc
into
opendatahub-io:main
33 of 35 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )