Merge pull request #1065 from huanggze/devops-disabled
monitor: no need to produce error when devops is disabled
This commit is contained in:
@@ -679,7 +679,9 @@ func GetClusterStatistics() *Response {
|
||||
go func() {
|
||||
num, err := workspaces.GetAllDevOpsProjectsNums()
|
||||
if err != nil {
|
||||
klog.Errorln(err)
|
||||
if _, notEnabled := err.(cs.ClientSetNotEnabledError); !notEnabled {
|
||||
klog.Errorln(err)
|
||||
}
|
||||
devopsStats.Status = "error"
|
||||
} else {
|
||||
devopsStats.withMetricResult(now, num)
|
||||
@@ -746,7 +748,9 @@ func GetWorkspaceStatistics(workspaceName string) *Response {
|
||||
go func() {
|
||||
num, err := workspaces.GetDevOpsProjectsCount(workspaceName)
|
||||
if err != nil {
|
||||
klog.Errorln(err)
|
||||
if _, notEnabled := err.(cs.ClientSetNotEnabledError); !notEnabled {
|
||||
klog.Errorln(err)
|
||||
}
|
||||
devopsStats.Status = "error"
|
||||
} else {
|
||||
devopsStats.withMetricResult(now, num)
|
||||
|
||||
@@ -169,6 +169,10 @@ func DeleteWorkspaceRoleBinding(workspace, username string, role string) error {
|
||||
}
|
||||
|
||||
func GetDevOpsProjectsCount(workspaceName string) (int, error) {
|
||||
_, err := clientset.ClientSets().Devops()
|
||||
if _, notEnabled := err.(clientset.ClientSetNotEnabledError); notEnabled {
|
||||
return 0, err
|
||||
}
|
||||
|
||||
dbconn, err := clientset.ClientSets().MySQL()
|
||||
if err != nil {
|
||||
@@ -237,6 +241,11 @@ func GetAllProjectNums() (int, error) {
|
||||
}
|
||||
|
||||
func GetAllDevOpsProjectsNums() (int, error) {
|
||||
_, err := clientset.ClientSets().Devops()
|
||||
if _, notEnabled := err.(clientset.ClientSetNotEnabledError); notEnabled {
|
||||
return 0, err
|
||||
}
|
||||
|
||||
dbconn, err := clientset.ClientSets().MySQL()
|
||||
if err != nil {
|
||||
return 0, err
|
||||
|
||||
Reference in New Issue
Block a user