Skip to content

Commit

Permalink
cleanup: modelmesh monitoring logic (opendatahub-io#703)
Browse files Browse the repository at this point in the history
Signed-off-by: Wen Zhou <wenzhou@redhat.com>
(cherry picked from commit 023d6c4)
  • Loading branch information
zdtsw authored and VaishnaviHire committed Nov 15, 2023
1 parent 7d0323c commit 0ee69d2
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 11 deletions.
12 changes: 3 additions & 9 deletions components/modelmeshserving/modelmeshserving.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,8 @@ import (
)

var (
ComponentName = "model-mesh"
Path = deploy.DefaultManifestPath + "/" + ComponentName + "/overlays/odh"
// monitoringPath = deploy.DefaultManifestPath + "/" + "modelmesh-monitoring/base"
ComponentName = "model-mesh"
Path = deploy.DefaultManifestPath + "/" + ComponentName + "/overlays/odh"
DependentComponentName = "odh-model-controller"
DependentPath = deploy.DefaultManifestPath + "/" + DependentComponentName + "/base"
)
Expand Down Expand Up @@ -144,12 +143,6 @@ func (m *ModelMeshServing) ReconcileComponent(cli client.Client, owner metav1.Ob
if err != nil {
return err
}
// var monitoringNamespace string
// if dscInit.Spec.Monitoring.Namespace != "" {
// monitoringNamespace = dscInit.Spec.Monitoring.Namespace
// } else {
// monitoringNamespace = dscispec.ApplicationsNamespace
// }

// CloudService Monitoring handling
// TODO: cleanup logic in rhods-2.5
Expand All @@ -170,6 +163,7 @@ func (m *ModelMeshServing) ReconcileComponent(cli client.Client, owner metav1.Ob
}
}
return nil

}

func (m *ModelMeshServing) DeepCopyInto(target *ModelMeshServing) {
Expand Down
1 change: 1 addition & 0 deletions controllers/dscinitialization/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,7 @@ func (r *DSCInitializationReconciler) reconcileDefaultNetworkPolicy(ctx context.
Name: name,
Namespace: name,
}, foundNetworkPolicy)

if err != nil {
if apierrs.IsNotFound(err) {
// Set Controller reference
Expand Down
3 changes: 1 addition & 2 deletions get_all_manifests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ MANIFESTS_TARBALL_URL="${GITHUB_URL}/${MANIFEST_ORG}/odh-manifests/tarball/${MAN


# component: dsp, kserve, dashbaord, cf/ray. in the format of "repo-org:repo-name:branch-name:source-folder:target-folder"
# TODO: kserve, mm, trustyai, dashbaord, nbc,odh-mm-monitoring, etc

declare -A COMPONENT_MANIFESTS=(
["codeflare"]="red-hat-data-services:codeflare-operator:rhods-2.4:config:codeflare"
["ray"]="red-hat-data-services:kuberay:rhods-2.4:ray-operator/config:ray"
Expand Down Expand Up @@ -50,7 +50,6 @@ if [ "$#" -ge 1 ]; then
done
fi

<<<<<<< HEAD
# pre-cleanup local env
rm -fr ./odh-manifests/* ./.odh-manifests-tmp/

Expand Down

0 comments on commit 0ee69d2

Please sign in to comment.