summaryrefslogtreecommitdiffstats
path: root/src/endpoint
diff options
context:
space:
mode:
authorMatthias Beyer <matthias.beyer@atos.net>2021-05-03 16:27:02 +0200
committerMatthias Beyer <mail@beyermatthias.de>2021-05-11 09:58:59 +0200
commit2c0c0eb515aa4d3d3de1ab47351733ba59c983f1 (patch)
tree0793ea4971511efb561a19c2fe924582c4aa248a /src/endpoint
parentced09dfe2335c99c3234fa98941a252a1bc94845 (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.rs28
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);