diff options
author | Matthias Beyer <matthias.beyer@atos.net> | 2021-05-03 16:27:02 +0200 |
---|---|---|
committer | Matthias Beyer <mail@beyermatthias.de> | 2021-05-11 09:58:59 +0200 |
commit | 2c0c0eb515aa4d3d3de1ab47351733ba59c983f1 (patch) | |
tree | 0793ea4971511efb561a19c2fe924582c4aa248a /src/endpoint | |
parent | ced09dfe2335c99c3234fa98941a252a1bc94845 (diff) |
Add endpoint name and container id in progress message
This patch adds the endpoint name and a (7 character) appreviated container id
in the progress message, so that the user can see where the package is built.
This reduces the compile+debug loop by one step in the case where the user does
not want to inspect the logs anyways, because they can jump to the right
container right away and do not have to `butido db job -sL <job id>` to find out
where the job was running.
Signed-off-by: Matthias Beyer <matthias.beyer@atos.net>
Tested-by: Matthias Beyer <matthias.beyer@atos.net>
Diffstat (limited to 'src/endpoint')
-rw-r--r-- | src/endpoint/scheduler.rs | 28 |
1 files changed, 16 insertions, 12 deletions
diff --git a/src/endpoint/scheduler.rs b/src/endpoint/scheduler.rs index 38e705d..8b65bb2 100644 --- a/src/endpoint/scheduler.rs +++ b/src/endpoint/scheduler.rs @@ -159,6 +159,8 @@ impl JobHandle { .execute_script(log_sender); let logres = LogReceiver { + endpoint_name: endpoint_name.as_ref(), + container_id_chrs: container_id.chars().take(7).collect(), package_name: &package.name, package_version: &package.version, log_dir: self.log_dir.as_ref(), @@ -317,6 +319,8 @@ impl JobHandle { } struct LogReceiver<'a> { + endpoint_name: &'a str, + container_id_chrs: String, package_name: &'a str, package_version: &'a str, log_dir: Option<&'a PathBuf>, @@ -371,23 +375,23 @@ impl<'a> LogReceiver<'a> { LogItem::CurrentPhase(ref phasename) => { trace!("Setting bar phase to {}", phasename); self.bar.set_message(format!( - "[{} {} {}]: Phase: {}", - self.job_id, self.package_name, self.package_version, phasename + "[{}/{} {} {} {}]: Phase: {}", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version, phasename )); } LogItem::State(Ok(())) => { trace!("Setting bar state to Ok"); self.bar.set_message(format!( - "[{} {} {}]: State Ok", - self.job_id, self.package_name, self.package_version + "[{}/{} {} {} {}]: State Ok", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version )); success = Some(true); } LogItem::State(Err(ref e)) => { trace!("Setting bar state to Err: {}", e); self.bar.set_message(format!( - "[{} {} {}]: State Err: {}", - self.job_id, self.package_name, self.package_version, e + "[{}/{} {} {} {}]: State Err: {}", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version, e )); success = Some(false); } @@ -398,16 +402,16 @@ impl<'a> LogReceiver<'a> { trace!("Finishing bar = {:?}", success); let finish_msg = match success { Some(true) => format!( - "[{} {} {}]: finished successfully", - self.job_id, self.package_name, self.package_version + "[{}/{} {} {} {}]: finished successfully", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version ), Some(false) => format!( - "[{} {} {}]: finished with error", - self.job_id, self.package_name, self.package_version + "[{}/{} {} {} {}]: finished with error", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version ), None => format!( - "[{} {} {}]: finished", - self.job_id, self.package_name, self.package_version + "[{}/{} {} {} {}]: finished", + self.endpoint_name, self.container_id_chrs, self.job_id, self.package_name, self.package_version ), }; self.bar.finish_with_message(finish_msg); |