Skip to content

Commit

Permalink
fix: Remove double status update
Browse files Browse the repository at this point in the history
  • Loading branch information
gonzalezzfelipe committed Oct 31, 2024
1 parent dadeb69 commit fd6ea53
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions src/controller.rs
Original file line number Diff line number Diff line change
Expand Up @@ -162,41 +162,16 @@ impl K8sContext {
let api: Api<HydraDoomNode> =
Api::namespaced(self.client.clone(), &crd.namespace().unwrap());

// Create or patch the deployment
let status = serde_json::to_value(HydraDoomNodeStatus {
state: "Online".to_string(),
transactions: 2423514,
local_url: format!("ws://{}:{}", crd.internal_host(), self.constants.port),
external_url: format!(
"ws://{}:{}",
crd.external_host(&self.config, &self.constants),
self.config.external_port
),
})
.unwrap();

api.patch(
&crd.name_any(),
&PatchParams::default(),
&Patch::Merge(json!({
"status": status,
"metadata": {
"finalizers": [HYDRA_DOOM_NODE_FINALIZER]
}
})),
)
.await
.map_err(|err| {
error!(err = err.to_string(), "Failed to patch CRD.");
anyhow::Error::from(err)
})?;

api.patch_status(
&crd.name_any(),
&PatchParams::default(),
&Patch::Merge(json!({ "status": status })),
)
.await
.map_err(|err| {
error!(err = err.to_string(), "Failed to patch CRD.");
anyhow::Error::from(err)
Expand Down

0 comments on commit fd6ea53

Please sign in to comment.