Update test and docs

Author: Milla Samuel

controllers/admin_client_test.go

@@ -341,12 +341,28 @@ var _ = Describe("admin_client_test", func() {
 
 				})
 
-				It("should mark the backup as stopped", func() {
+				It("should have modified the snapshot time", func() {
 					Expect(
 						status.SnapshotIntervalSeconds,
 					).To(BeNumerically("==", backup.SnapshotPeriodSeconds()))
 				})
 			})
+
+			Context("with a modification to the backup url", func() {
+				BeforeEach(func() {
+					backup.Spec.BlobStoreConfiguration = &fdbv1beta2.BlobStoreConfiguration{
+						BackupName:  "test-backup-2",
+						AccountName: "test",
+					}
+					Expect(mockAdminClient.ModifyBackup(backup)).NotTo(HaveOccurred())
+				})
+
+				It("should have modified the backup url", 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)