Skip to content

Commit

Permalink
feat(mr): undo separate disableModelRegistryManager flag
Browse files Browse the repository at this point in the history
Signed-off-by: gitdallas <[email protected]>
  • Loading branch information
gitdallas committed May 13, 2024
1 parent ac8ebd8 commit ca003db
Show file tree
Hide file tree
Showing 9 changed files with 1 addition and 14 deletions.
1 change: 0 additions & 1 deletion backend/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ export type DashboardConfig = K8sResourceCommon & {
disableInfo: boolean;
disableSupport: boolean;
disableClusterManager: boolean;
disableModelRegistryManager: boolean;
disableTracking: boolean;
disableBYONImageStream: boolean;
disableISVBadges: boolean;
Expand Down
1 change: 0 additions & 1 deletion backend/src/utils/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ export const blankDashboardCR: DashboardConfig = {
disableInfo: false,
disableSupport: false,
disableClusterManager: false,
disableModelRegistryManager: false,
disableTracking: true,
disableBYONImageStream: false,
disableISVBadges: false,
Expand Down
3 changes: 0 additions & 3 deletions docs/dashboard-config.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ The following are a list of features that are supported, along with there defaul
| disableInfo | false | Removes the information panel in Explore Application section |
| disableSupport | false | Disables components related to support. |
| disableClusterManager | false | Disables cluster management section for admins |
| disableModelRegistryManager | false | Disables model registry management section for admins |
| disableTracking | true | Disables telemetry UI data. Note for this feature to work you need woopra and segement.io configured |
| disableBYONImageStream | false | Disables custom notebook images that are created via image streams |
| disableISVBadges | false | Removes the badge that indicate if a product is ISV or not. |
Expand Down Expand Up @@ -47,7 +46,6 @@ spec:
disableInfo: false
disableSupport: false
disableClusterManager: false
disableModelRegistryManager: false
disableTracking: true
disableBYONImageStream: false
disableISVBadges: false
Expand Down Expand Up @@ -144,7 +142,6 @@ spec:
enablement: true
disableBYONImageStream: false
disableClusterManager: false
disableModelRegistryManager: false
disableISVBadges: false
disableInfo: false
disableSupport: false
Expand Down
3 changes: 0 additions & 3 deletions frontend/src/__mocks__/mockDashboardConfig.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ type MockDashboardConfigType = {
disableInfo?: boolean;
disableSupport?: boolean;
disableClusterManager?: boolean;
disableModelRegistryManager?: boolean;
disableTracking?: boolean;
disableBYONImageStream?: boolean;
disableISVBadges?: boolean;
Expand All @@ -31,7 +30,6 @@ export const mockDashboardConfig = ({
disableInfo = false,
disableSupport = false,
disableClusterManager = false,
disableModelRegistryManager = false,
disableTracking = false,
disableBYONImageStream = false,
disableISVBadges = false,
Expand Down Expand Up @@ -68,7 +66,6 @@ export const mockDashboardConfig = ({
disableInfo,
disableSupport,
disableClusterManager,
disableModelRegistryManager,
disableTracking,
disableBYONImageStream,
disableISVBadges,
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/concepts/areas/const.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export const SupportedAreasStateMap: SupportedAreasState = {
featureFlags: ['disableClusterManager'],
},
[SupportedArea.MODEL_REGISTRY_SETTINGS]: {
featureFlags: ['disableModelRegistryManager'],
featureFlags: ['disableModelRegistry'],
requiredComponents: [StackComponent.MODEL_REGISTRY],
requiredCapabilities: [StackCapability.SERVICE_MESH, StackCapability.SERVICE_MESH_AUTHZ],
},
Expand Down
1 change: 0 additions & 1 deletion frontend/src/k8sTypes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1196,7 +1196,6 @@ export type DashboardCommonConfig = {
disableInfo: boolean;
disableSupport: boolean;
disableClusterManager: boolean;
disableModelRegistryManager: boolean;
disableTracking: boolean;
disableBYONImageStream: boolean;
disableISVBadges: boolean;
Expand Down
2 changes: 0 additions & 2 deletions manifests/crd/odhdashboardconfigs.opendatahub.io.crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@ spec:
type: boolean
disableClusterManager:
type: boolean
disableModelRegistryManager:
type: boolean
disableTracking:
type: boolean
disableBYONImageStream:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ spec:
dashboardConfig:
disableBYONImageStream: false
disableClusterManager: false
disableModelRegistryManager: false
disableISVBadges: false
disableInfo: false
disableSupport: false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ spec:
dashboardConfig:
disableBYONImageStream: false
disableClusterManager: false
disableModelRegistryManager: false
disableISVBadges: false
disableInfo: false
disableSupport: false
Expand Down

0 comments on commit ca003db

Please sign in to comment.