Sync from kubeflow/model-registry 6eb25cf

[ppadti]

Description

Sync to pull in changes from:

• fix a11y from model catalog details kubeflow/hub#2627
• hide cat toogle in case only one cat is available kubeflow/hub#2599
• truncate error msg from transfer jobs kubeflow/hub#2625
• chore: rename container images kubeflow/hub#2639
• chore: rename go modules kubeflow/hub#2635
• Update project name and repo links kubeflow/hub#2640
• fix(ui): forward incoming auth headers in webpack dev proxy kubeflow/hub#2544
• build(deps-dev): bump postcss from 8.5.6 to 8.5.10 in /clients/ui/frontend kubeflow/hub#2641

Conflicts Resolved

The following conflicts were resolved during this sync:
#2599 - hide cat toggle in case only one cat is available

• frontend/src/app/pages/modelCatalog/screens/ModelCatalog.tsx
  Nature of conflict: The downstream file has additional imports and logic not present upstream — extension point integration (LazyCodeRefComponent, useExtensions, isModelCatalogBannerExtension) and search param handling for the ?validated=true URL parameter. The upstream patch expected a simpler file without these additions.
  Resolution: Merged both sets of changes: kept the downstream extension point banner rendering and validated search param logic, while incorporating all upstream additions (single-category detection, empty state handling, isSingleCategory / singleCategoryLabel props, and the hasNoCategories empty state).
  #2635 - chore: rename go modules
• bff/go.mod
  Nature of conflict: The downstream go.mod had a slightly different Go version (1.25.0) than what upstream expected (1.25.7), causing context mismatch for the module rename patch.
  Resolution: Applied the module rename from github.com/kubeflow/model-registry/ui/bff to github.com/kubeflow/hub/ui/bff while keeping the downstream dependency versions intact.

Additional Downstream Fix

The Go module rename in #2635 only updated upstream files via the sync script. Downstream-only Go files in internal/redhat/, internal/helpers/, and cmd/ still referenced the old module path github.com/kubeflow/model-registry/ui/bff. Updated all 19 downstream files to use the new github.com/kubeflow/hub/ui/bff module path.

How Has This Been Tested?

Tested by running the federated model-registry package locally, verifying existing behavior in the files this diff touches, and verifying the incoming changes.
Also ran available test scripts in packages/model-registry/upstream/frontend:

• npm run test:unit — 57 suites, 697 tests passed
• npm run test:type-check — clean, no errors
• npm run test:lint — 0 warnings, 0 errors
• go vet ./... — clean after downstream import fix

Test Impact

Upstream changes include their own tests.

Request review criteria:

Self checklist (all need to be checked):

• The developer has manually tested the changes and verified that the changes work
• Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
• The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)
• The code follows our Best Practices (React coding standards, PatternFly usage, performance considerations)

If you have UI changes:

• Included any necessary screenshots or gifs if it was a UI change.
• Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

• The developer has tested their solution on a cluster by using the image produced by the PR to main

Summary by CodeRabbit

• Chores
  • Updated internal dependency configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: e417526a-94c7-444d-837a-a8a3553ee13a

📥 Commits

Reviewing files that changed from the base of the PR and between d73dabc and 11adb5a.

⛔ Files ignored due to path filters (146)

• packages/model-registry/upstream/.env is excluded by !**/upstream/**
• packages/model-registry/upstream/CONTRIBUTING.md is excluded by !**/upstream/**
• packages/model-registry/upstream/README.md is excluded by !**/upstream/**
• packages/model-registry/upstream/api/openapi/mod-arch.yaml is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/README.md is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/cmd/main.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/docs/extensions.md is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/go.mod is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/app.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/app_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/artifacts_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/artifacts_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_filters_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_filters_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_models_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_models_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_source_preview_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_source_preview_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_sources_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/catalog_sources_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/check_namespace_registry_access_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/check_namespace_registry_access_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/errors.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/healthcheck__handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/mcp_servers_catalog_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/mcp_servers_catalog_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/middleware.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_artifacts_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_artifacts_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_catalog_settings_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_catalog_settings_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_settings_groups_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_settings_groups_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_settings_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_settings_rbac_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_registry_settings_rbac_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_transfer_job_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_transfer_job_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_versions_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/model_versions_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/namespaces_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/namespaces_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/public_helpers.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/registered_models_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/registered_models_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/suite_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/test_app.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/test_utils.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/user_handler.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/api/user_handler_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/helpers/logging.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/helpers/mcpserver_converter.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/helpers/mcpserver_converter_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/httpclient/http.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/factory.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/internal_k8s_client.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/k8mocks/base_testenv.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/k8mocks/internal_k8s_client_mock.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/k8mocks/mock_factory.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/k8mocks/token_k8s_client_mock.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/shared_k8s_client.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/shared_k8s_client_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/tests/factory_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/tests/internal_k8s_client_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/tests/namespace_registry_access_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/tests/suite_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/tests/token_k8s_client_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/integrations/kubernetes/token_k8s_client.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/mocks/model_catalog_client_mock.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/mocks/model_registry_client_mock.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/mocks/static_data_mock.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/kubernetes_services.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_deployment_auth.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_deployments.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_deployments_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_server_availability.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_server_availability_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/mcp_server_converter.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/model_registry_settings.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/model_registry_settings_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/model_transfer_jobs.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/handlers/model_transfer_jobs_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/repositories/mcp_deployment_repository.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/repositories/mcp_deployment_repository_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/repositories/mcp_server_availability_repository.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/repositories/model_registry_conversion.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/redhat/repositories/model_registry_settings_repository.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/artifacts.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/catalog_models.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/catalog_source_preview.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/catalog_source_preview_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/catalog_sources.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/health_check.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/helpers.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/mcp_server_catalog.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/mcp_server_catalog_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_artifacts.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_catalog.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_catalog_settings.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_catalog_settings_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_registry.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_registry_settings.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_registry_settings_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_registry_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_transfer_jobs.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_transfer_jobs_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_version.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/model_version_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/namespace.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/registered_model.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/registered_model_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/suite_test.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/repositories/user.go is excluded by !**/upstream/**
• packages/model-registry/upstream/bff/internal/validation/validation.go is excluded by !**/upstream/**
• packages/model-registry/upstream/docs/README.md is excluded by !**/upstream/**
• packages/model-registry/upstream/docs/kubeflow-development-guide.md is excluded by !**/upstream/**
• packages/model-registry/upstream/docs/local-deployment-guide-ui.md is excluded by !**/upstream/**
• packages/model-registry/upstream/docs/local-deployment-guide.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/README.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/config/webpack.dev.js is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/docs/README.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/docs/architecture.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/docs/dev-setup.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/docs/env-variables.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/docs/testing.md is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/package-lock.json is excluded by !**/package-lock.json, !**/upstream/**
• packages/model-registry/upstream/frontend/package.json is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__mocks__/mockCatalogSourceList.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__tests__/cypress/cypress/pages/modelCatalog.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__tests__/cypress/cypress/pages/modelRegistryView/modelTransferJobs.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__tests__/cypress/cypress/tests/mocked/modelCatalog/modelCatalog.cy.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__tests__/cypress/cypress/tests/mocked/modelCatalog/modelCatalogDetails.cy.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/__tests__/cypress/cypress/tests/mocked/modelRegistry/modelTransferJobs.cy.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/mcpCatalog/screens/McpCatalog.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/mcpCatalog/screens/McpCatalogGalleryView.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/mcpCatalog/screens/McpCatalogSourceLabelBlocks.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/screens/ModelCatalog.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/screens/ModelCatalogGalleryView.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/screens/ModelCatalogSourceLabelBlocks.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/screens/ModelDetailsView.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/utils/__tests__/modelCatalogUtils.spec.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelCatalog/utils/modelCatalogUtils.ts is excluded by !**/upstream/**
• packages/model-registry/upstream/frontend/src/app/pages/modelRegistry/screens/ModelTransferJobs/ModelTransferJobTableRow.tsx is excluded by !**/upstream/**
• packages/model-registry/upstream/scripts/deploy_kind_cluster.sh is excluded by !**/upstream/**

📒 Files selected for processing (1)

• packages/model-registry/package.json

---

📝 Walkthrough

Walkthrough

The subtree.commit field in packages/model-registry/package.json is updated to reference a new upstream commit hash (6eb25cf...) from the previous hash (6e18d5a...). No package metadata, dependencies, or exported interfaces are affected by this configuration change.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the primary change: syncing to a specific upstream commit (6eb25cf) from kubeflow/model-registry.
Description check	✅ Passed	The description comprehensively documents upstream PRs, conflict resolutions with specific file paths and strategies, downstream fixes, and testing performed including test results.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

❌ Patch coverage is 4.25532% with 45 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.06%. Comparing base (d73dabc) to head (11adb5a).
⚠️ Report is 8 commits behind head on main.

Files with missing lines	Patch %	Lines
...nd/src/app/pages/mcpCatalog/screens/McpCatalog.tsx	0.00%	15 Missing ⚠️
...pages/mcpCatalog/screens/McpCatalogGalleryView.tsx	6.66%	14 Missing ⚠️
.../app/pages/modelCatalog/utils/modelCatalogUtils.ts	12.50%	7 Missing ⚠️
...ens/ModelTransferJobs/ModelTransferJobTableRow.tsx	0.00%	6 Missing ⚠️
...mcpCatalog/screens/McpCatalogSourceLabelBlocks.tsx	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7394      +/-   ##
==========================================
- Coverage   65.10%   65.06%   -0.05%     
==========================================
  Files        2466     2466              
  Lines       76565    76604      +39     
  Branches    19338    19358      +20     
==========================================
- Hits        49848    49841       -7     
- Misses      26717    26763      +46     

Files with missing lines	Coverage Δ
...mcpCatalog/screens/McpCatalogSourceLabelBlocks.tsx	7.14% <0.00%> (-0.86%)	⬇️
...ens/ModelTransferJobs/ModelTransferJobTableRow.tsx	3.77% <0.00%> (-0.23%)	⬇️
.../app/pages/modelCatalog/utils/modelCatalogUtils.ts	16.77% <12.50%> (-0.12%)	⬇️
...pages/mcpCatalog/screens/McpCatalogGalleryView.tsx	6.06% <6.66%> (-3.94%)	⬇️
...nd/src/app/pages/mcpCatalog/screens/McpCatalog.tsx	3.84% <0.00%> (-3.30%)	⬇️

... and 7 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d73dabc...11adb5a. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[manaswinidas]

/lgtm
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: manaswinidas

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [manaswinidas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment