From 0f93277fcb2691b3d3097d1f07c139d0f337750a Mon Sep 17 00:00:00 2001 From: Amit_Noyberg1 <219515702+amit-noy@users.noreply.github.com> Date: Tue, 30 Jun 2026 13:24:21 +0300 Subject: [PATCH 1/2] fix: add missing owner domain in Applications -> API Gateways --- .../ApiGateways/apiGatewaysColumns.jsx | 10 +++------- .../ApiGateways/applicationApiGateways.util.js | 7 ++----- .../applicationDetails.constants.js | 2 -- src/utils/nuclioEnrichment.util.js | 13 ++++++++++--- 4 files changed, 15 insertions(+), 17 deletions(-) mode change 100644 => 100755 src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx mode change 100644 => 100755 src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/applicationDetails.constants.js diff --git a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx old mode 100644 new mode 100755 index f0b2885f0..4535ed80b --- a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx +++ b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx @@ -26,8 +26,8 @@ import { API_GATEWAY_STATE_CLASS, API_GATEWAY_STATE_LABEL, FORCE_SSL_REDIRECT_ANNOTATION, - NUCLIO_OWNER_LABEL } from '../applicationDetails.constants' +import { buildNuclioOwner } from '../../../../../utils/nuclioEnrichment.util' export const apiGatewaysColumns = [ { @@ -136,11 +136,7 @@ export const apiGatewaysColumns = [ id: 'owner', header: 'Owner', size: 10, - accessorFn: row => row.metadata?.labels?.[NUCLIO_OWNER_LABEL] ?? '', - cell: ({ row }) => ( - - {row.original.metadata?.labels?.[NUCLIO_OWNER_LABEL] || ''} - - ) + accessorFn: ({ metadata }) => buildNuclioOwner(metadata?.labels), + cell: ({ getValue }) => {getValue()} } ] diff --git a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js index 2b68b53bf..742fb4555 100644 --- a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js +++ b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js @@ -17,7 +17,7 @@ illegal under applicable law, and the grant of the foregoing license under the Apache 2.0 license is conditioned upon your compliance with such restriction. */ -import { NUCLIO_OWNER_LABEL } from '../applicationDetails.constants' +import { buildNuclioOwner } from '../../../../../utils/nuclioEnrichment.util' export { buildGatewayEndpoint, @@ -37,10 +37,7 @@ export const filterApiGatewaysBySearchFields = (gateways, filters) => { return false } - if ( - ownerFilter && - !gateway.metadata?.labels?.[NUCLIO_OWNER_LABEL]?.toLowerCase().includes(ownerFilter) - ) { + if (ownerFilter && !buildNuclioOwner(gateway.metadata?.labels).toLowerCase().includes(ownerFilter)) { return false } diff --git a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/applicationDetails.constants.js b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/applicationDetails.constants.js old mode 100644 new mode 100755 index c8c1fd58b..a19e94822 --- a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/applicationDetails.constants.js +++ b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/applicationDetails.constants.js @@ -94,8 +94,6 @@ export const API_GATEWAY_STATE_CLASS = { export { GATEWAY_RELATIONSHIP } from '../../../../utils/apiGateway.util' -export const NUCLIO_OWNER_LABEL = 'iguazio.com/username' - export const FORCE_SSL_REDIRECT_ANNOTATION = 'nginx.ingress.kubernetes.io/force-ssl-redirect' export const API_GATEWAYS_FILTER_CONFIG = { diff --git a/src/utils/nuclioEnrichment.util.js b/src/utils/nuclioEnrichment.util.js index c5df47d0b..34f52d947 100644 --- a/src/utils/nuclioEnrichment.util.js +++ b/src/utils/nuclioEnrichment.util.js @@ -30,9 +30,16 @@ import { FUNCTIONS_PAGE, UNHEALTHY_STATE } from '../constants' - const NUCLIO_FUNCTIONS_STATE_KIND = 'nuclioFunctions' -const NUCLIO_OWNER_LABEL = 'iguazio.com/username' +const NUCLIO_USERNAME_LABEL = 'iguazio.com/username' +const NUCLIO_DOMAIN_LABEL = 'iguazio.com/domain' + +export const buildNuclioOwner = labels => { + const username = labels?.[NUCLIO_USERNAME_LABEL] + const domain = labels?.[NUCLIO_DOMAIN_LABEL] + + return domain ? `${username}@${domain}` : (username ?? '') +} const buildEndpointsCountMap = modelEndpoints => { const countMap = {} @@ -67,7 +74,7 @@ export const enrichFunctionsWithNuclio = ( : '' const state = nuclioFuncState || func.state?.value || '' - const owner = nuclioFunc?.metadata?.labels?.[NUCLIO_OWNER_LABEL] ?? '' + const owner = buildNuclioOwner(nuclioFunc?.metadata?.labels) const applicationGateways = filterGatewaysByFunction( projectApiGateways, From 577f9ca2962b8261c99668af250d247af8aecd3e Mon Sep 17 00:00:00 2001 From: Amit_Noyberg1 <219515702+amit-noy@users.noreply.github.com> Date: Wed, 1 Jul 2026 09:29:52 +0300 Subject: [PATCH 2/2] formatting --- .../ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx | 2 +- .../ApiGateways/applicationApiGateways.util.js | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx index 4535ed80b..29b49060d 100755 --- a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx +++ b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/apiGatewaysColumns.jsx @@ -25,7 +25,7 @@ import { buildGatewayEndpoint } from './applicationApiGateways.util' import { API_GATEWAY_STATE_CLASS, API_GATEWAY_STATE_LABEL, - FORCE_SSL_REDIRECT_ANNOTATION, + FORCE_SSL_REDIRECT_ANNOTATION } from '../applicationDetails.constants' import { buildNuclioOwner } from '../../../../../utils/nuclioEnrichment.util' diff --git a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js index 742fb4555..3417e12da 100644 --- a/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js +++ b/src/nextGenComponents/pages/ApplicationsPage/ApplicationDetails/ApiGateways/applicationApiGateways.util.js @@ -37,7 +37,10 @@ export const filterApiGatewaysBySearchFields = (gateways, filters) => { return false } - if (ownerFilter && !buildNuclioOwner(gateway.metadata?.labels).toLowerCase().includes(ownerFilter)) { + if ( + ownerFilter && + !buildNuclioOwner(gateway.metadata?.labels).toLowerCase().includes(ownerFilter) + ) { return false }