fix some related incorrect comments while we're here

Author: hawkw

nexus/db-queries/src/db/datastore/instance.rs

@@ -1756,8 +1756,8 @@ impl DataStore {
 
         match result {
             // If we updated the record, the lock has been successfully
-            // inherited! Return `Ok(true)` to indicate that we have acquired
-            // the lock successfully.
+            // inherited! Return the new lock to indicate that we have acquired
+            // it successfully.
             UpdateAndQueryResult { status: UpdateStatus::Updated, .. } => {
                 slog::debug!(
                     &opctx.log,
@@ -1775,7 +1775,7 @@ impl DataStore {
             }
             // The generation has advanced past the generation at which the
             // lock was held. This means that we have already inherited the
-            // lock. Return `Ok(false)` here for idempotency.
+            // lock. Return `Ok` here for idempotency.
             UpdateAndQueryResult {
                 status: UpdateStatus::NotUpdatedButExists,
                 ref found,

nexus/src/app/sagas/instance_update/start.rs

@@ -125,7 +125,7 @@ async fn siu_lock_instance(
     // This is necessary in order to satisfy the distributed saga requirement
     // that executing an action must be idempotent. Retrying transient database
     // errors indefinitely is necessary to ensure idempotency because it is
-    // possible that a previous execution of this action *did* succesfully
+    // possible that a previous execution of this action *did* successfully
     // acquire the lock but crashed before it completed.
     //
     // As an example of why this is important, consider a particularly