c045335 feat: make log message about patch COB not found clearer — radicle-ci-broker

Radicle CI broker

feat: make log message about patch COB not found clearer

Lars Wirzenius committed 1 year ago

commit c045335b69f0bdfba63bcf324910b8a43974a074
parent d491111726ae697f0a10a1fd75b894dbd9748729

2 files changed +14 -2

modified src/logger.rs

@@ -3,7 +3,7 @@

 use std::{path::Path, process::ExitStatus, time::Duration};
 use clap::ValueEnum;
 use radicle::{git::raw::Oid, identity::RepoId, node::Event};
 use radicle::{git::raw::Oid, identity::RepoId, node::Event, patch::PatchId};
 use serde_json::Value;
 use tracing::{debug, error, info, trace, warn, Level};
 use tracing_subscriber::{fmt, layer::SubscriberExt, util::SubscriberInitExt, EnvFilter};

@@ -179,6 +179,8 @@ enum Id {

     TimeoutWaitStderrReaderEnd,
     TimeoutWaitStdinWriterEnd,
     TimeoutWaitStdoutReaderEnd,
     TriggerCreate,
 }
 #[derive(Debug, thiserror::Error)]

@@ -600,6 +602,16 @@ pub fn broker_end_run(run: &Run) {

     );
 }
 pub fn patch_cob_lookup(repo_id: &RepoId, patch_id: &PatchId) {
     warn!(
         msg_id = ?Id::TriggerCreate,
         kind = %Kind::StartRun,
         ?repo_id,
         ?patch_id,
         "did not find patch COB in repository"
     );
 }
 pub fn adapter_no_first_response() {
     error!(
         msg_id = ?Id::AdapterNoFirstMessage,

modified src/msg.rs

@@ -239,7 +239,7 @@ impl<'a> RequestBuilder<'a> {

             let x = match x {
                 None => {
                     logger::trace("did not find patch COB from repository");
                     logger::patch_cob_lookup(&rad_repo.id, patch_id);
                     return Err(MessageError::PatchCob(*patch_id));
                 }
                 Some(x) => x,