Skip to content

Commit

Permalink
OCM-7801 | ci: Remove redundant cleaner to fix resource clean issue
Browse files Browse the repository at this point in the history
  • Loading branch information
xueli181114 committed May 9, 2024
1 parent 0f7d409 commit 6aaef6a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
5 changes: 5 additions & 0 deletions tests/e2e/test_rosacli_cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,10 +172,15 @@ var _ = Describe("Edit cluster",
clusterDetail, err := clusterService.ReflectClusterDescription(output)
Expect(err).ToNot(HaveOccurred())
expectedUWMValue := "Enabled"
recoverUWMStatus := false
if clusterConfig.DisableWorkloadMonitoring {
expectedUWMValue = "Disabled"
recoverUWMStatus = true
}
Expect(clusterDetail.UserWorkloadMonitoring).To(Equal(expectedUWMValue))
defer clusterService.EditCluster(clusterID,
fmt.Sprintf("--disable-workload-monitoring=%v", recoverUWMStatus),
"-y")

By("Disable the UWM")
expectedUWMValue = "Disabled"
Expand Down
1 change: 0 additions & 1 deletion tests/e2e/test_rosacli_idp.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ var _ = Describe("Edit IDP",
By("Clean remaining resources")
var errorList []error
errorList = append(errorList, rosaClient.CleanResources(clusterID))
errorList = append(errorList, rosaClient.CleanResources(clusterID))
Expect(errors.Join(errorList...)).ToNot(HaveOccurred())

})
Expand Down

0 comments on commit 6aaef6a

Please sign in to comment.