Update test and docs

Author: Milla Samuel

controllers/admin_client_test.go

@@ -332,10 +332,26 @@ var _ = Describe("admin_client_test", func() {
 					Expect(err).NotTo(HaveOccurred())
 				})
 
-				It("should mark the backup as stopped", func() {
+				It("should have modified the snapshot time", func() {
 					Expect(status.SnapshotIntervalSeconds).To(Equal(20))
 				})
 			})
+
+			Context("with a modification to the backup url", func() {
+				BeforeEach(func() {
+					err = mockAdminClient.ModifyBackup(
+						20,
+						"blobstore://test@test-service/test-backup-2",
+					)
+					Expect(err).NotTo(HaveOccurred())
+				})
+
+				It("should have modified the snapshot time", func() {
+					Expect(
+						status.DestinationURL,
+					).To(Equal("blobstore://test@test-service/test-backup-2"))
+				})
+			})
 		})
 	})
 

docs/manual/technical_design.md

@@ -387,7 +387,7 @@ The `ToggleBackupPaused` subreconciler is responsible for pausing and unpausing
 
 The `ModifyBackup` command ensures that any properties that can be configured on a live backup are configured to the values in the backup spec. This will run the `modify` command in `fdbbackup` to set the properties from the spec.
 
-Currently, this only supports the `snapshotPeriodSeconds` property.
+Currently, this only supports the `snapshotPeriodSeconds` and `url` property.
 
 ### UpdateBackupStatus (again)