Skip to content

[FIX] Add missing owner domain in Applications → API Gateways#3714

Open
amit-noy wants to merge 2 commits into
mlrun:developmentfrom
amit-noy:ML-12822
Open

[FIX] Add missing owner domain in Applications → API Gateways#3714
amit-noy wants to merge 2 commits into
mlrun:developmentfrom
amit-noy:ML-12822

Conversation

@amit-noy

@amit-noy amit-noy commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

📝 Description

The owner field in the Applications → API Gateways tab was displaying only the username instead of the full username@domain format. The domain is stored in a separate Nuclio label (iguazio.com/domain) that was not being read.

The fix also extends to enrichFunctionsWithNuclio, which feeds the owner field on the Applications list and Overview tab, and to the owner filter so it matches against the full composed string.


🛠️ Changes Made

  • Added buildNuclioOwner(labels) to nuclioEnrichment.util.js — composes username@domain from iguazio.com/username and iguazio.com/domain Nuclio labels
  • Updated apiGatewaysColumns.jsx owner column to use buildNuclioOwner
  • Fixed filterApiGatewaysBySearchFields to filter against the full composed owner string (supports searching by domain)
  • Updated enrichFunctionsWithNuclio to use buildNuclioOwner so the Applications list and Overview tab also display the correct owner

✅ Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I added or updated unit / integration tests (if needed)
  • I checked that this change doesn’t introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

🔗 References


🚨 Potentially Breaking Changes

  • Yes
  • No

Includes DRC change

  • Yes
  • No

If yes -> requires bump NPM version


🔍 Additional Notes


📸 Screenshots / Demos


@amit-noy amit-noy requested a review from muli-cohen July 1, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant