fix: address review comments

Author: conorsch

crates/bin/pd/src/migrate/reset_halt_bit.rs

@@ -14,6 +14,7 @@ pub async fn migrate(
     let mut delta = StateDelta::new(export_state);
     delta.ready_to_start();
     let _ = storage.commit_in_place(delta).await?;
+    storage.release().await;
     tracing::info!("migration completed: halt bit is turned off, chain is ready to start");
     Ok(())
 }

crates/bin/pd/src/migrate/testnet77.rs

@@ -2,14 +2,15 @@
 //! The Testnet 77 release included several consensus-breaking changes,
 //! but no state-breaking changes, so the migration is essentially a no-op,
 //! other than resetting the halt bit.
-use anyhow;
+use anyhow::Context;
 use cnidarium::Storage;
 use jmt::RootHash;
 use penumbra_app::app::StateReadExt as _;
 use penumbra_sct::component::clock::EpochRead;
 use std::path::PathBuf;
 use tracing::instrument;
 
+use crate::migrate::reset_halt_bit;
 use crate::testnet::generate::TestnetConfig;
 
 /// Run the full migration, given an export path and a start time for genesis.
@@ -35,7 +36,9 @@ pub async fn migrate(
         .await
         .expect("chain state has a block height");
     let post_upgrade_height = pre_upgrade_height.wrapping_add(1);
-    storage.release().await;
+    reset_halt_bit::migrate(storage, pd_home.clone(), genesis_start.clone())
+        .await
+        .context("failed to reset halt bit")?;
 
     // The migration is complete, now we need to generate a genesis file. To do this, we need
     // to lookup a validator view from the chain, and specify the post-upgrade app hash and