fix: update snapshot policy endpoint (#3391)

* fix: update snapshot policy endpoint * fix: update snapshot policy endpoint * fix: update snapshot policy endpoint
NetApp · Dec 18, 2024 · f3792fc · f3792fc
1 parent 6f5c2ad
commit f3792fc
Show file tree

Hide file tree

Showing 6 changed files with 119 additions and 24 deletions.
diff --git a/conf/rest/9.12.0/snapshotpolicy.yaml b/conf/rest/9.12.0/snapshotpolicy.yaml
@@ -1,11 +1,11 @@
 name:                     SnapshotPolicy
-query:                    api/private/cli/snapshot/policy
+query:                    api/private/cli/volume/snapshot/policy
 object:                   snapshot_policy
 
 counters:
-  - ^^policy          =>  snapshot_policy
+  - ^^policy          => snapshot_policy
   - ^^vserver         => svm
-  - ^comment          =>  comment
+  - ^comment          => comment
   - total_schedules   => total_schedules
 
 export_options:

diff --git a/docs/ontap-metrics.md b/docs/ontap-metrics.md
diff --git a/docs/prepare-cdot-clusters.md b/docs/prepare-cdot-clusters.md
@@ -234,6 +234,7 @@ security login rest-role create -role harvest2-rest-role -access readonly -api /
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/qos/workload
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/qtree
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/snapmirror
+    security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/volume/snapshot/policy
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/storage/failover
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/storage/shelf
     security login rest-role create -role harvest-rest-role -access readonly -api /api/private/cli/system/chassis/fru

diff --git a/integration/go.mod b/integration/go.mod
@@ -8,7 +8,7 @@ replace github.com/netapp/harvest/v2 => ../
 
 require (
 	github.com/carlmjohnson/requests v0.24.3
-	github.com/netapp/harvest/v2 v2.0.0-20241209123857-e401e50ef260
+	github.com/netapp/harvest/v2 v2.0.0-20241217100501-6f5c2ad8ca3a
 	golang.org/x/text v0.21.0
 )
 

diff --git a/integration/test/copy_logs_test.go b/integration/test/copy_logs_test.go
@@ -93,5 +93,5 @@ func checkLogs(t *testing.T, container docker.Container, info containerInfo) {
 
 // pollerIgnore returns a list of regex patterns that will be ignored
 func pollerIgnore() string {
-	return `RPC: Remote system error|connection error|Code: 2426405|failed to fetch data: error making request StatusCode: 403, Error: Permission denied, Message: not authorized for that command, API: (/api/private/cli/snapshot/policy|/api/support/autosupport)`
+	return `RPC: Remote system error|connection error|Code: 2426405|failed to fetch data: error making request StatusCode: 403, Error: Permission denied, Message: not authorized for that command, API: (/api/support/autosupport)`
 }
diff --git a/integration/test/counter_test.go b/integration/test/counter_test.go
@@ -36,7 +36,6 @@ var skipTemplates = map[string]bool{
 }
 
 var skipEndpoints = []string{
-	"api/private/cli/snapshot/policy",
 	"api/support/autosupport",
 	"api/private/cli/export-policy/rule",
 }