Slurm Collector: Handling of .extern jobsteps

[rkleinem]

Depending on the configuration, Slurm will create .extern jobsteps (See https://slurm.schedmd.com/job_launch.html#job_record).
Currently the collector seems to understand these as separate jobs. Since these steps lack some fields, several keys are not found in 240 of

AUDITOR/collectors/slurm/src/sacctcaller.rs

Lines 230 to 254 in 256395c

	sacct_rows
	.keys()
	.filter(|k| !BATCH_REGEX.is_match(k))
	.filter(|k| !SUB_REGEX.is_match(k))
	.map(|id| -> Result<Job> {
	let map1 = sacct_rows.get(id).ok_or(eyre!("Cannot get map1"))?;
	let map2 = sacct_rows.get(&format!("{id}.batch"));
	Ok(keys.iter()
	.cloned()
	.filter_map(|KeyConfig {name: k, key_type: _, allow_empty: _}| {
	let val = match map1.get(&k) {
	Some(Some(v)) => Some(v.clone()),
	_ => {
	if let Some(map2) = map2 {
	match map2.get(&k) {
	Some(Some(v)) => Some(v.clone()),
	_ => {
	tracing::error!("Something went wrong during parsing (map1, id: {id}, key: {k}, value: {:?})", map2.get(&k));
	None
	},
	}
	} else {
	tracing::error!("Something went wrong during parsing (map2, id: {id}, key: {k}, value: {:?})", map1.get(&k));
	None
	}

and an associated .batch step will never be found since the collector would look for a .extern.batch step, resulting in a message from line 252.

I am not yet 100% sure how Slurm calculates these but I think it would be enough to introduce another filter

.filter(|k| !EXTERN_REGEX.is_match(k))

with

static EXTERN_REGEX: Lazy<Regex> = Lazy::new(|| {
    Regex::new(r"^[0-9_]+\.extern$")
        .expect("Could not construct essential Regex for matching job ids.")
});

Sidenote:
There is an underscore missing in

AUDITOR/collectors/slurm/src/sacctcaller.rs

Lines 38 to 41 in 256395c

	static SUB_REGEX: Lazy<Regex> = Lazy::new(|| {
	Regex::new(r"^[0-9]+\.[0-9]*$")
	.expect("Could not construct essential Regex for matching job ids.")
	});