Re: [PATCH datacenter-manager 09/15] task cache: include archive file path in error log messages

["Lukas Wagner" <l.wagner@proxmox.com>]

On Fri Jul 10, 2026 at 1:36 PM CEST, Dominik Csapak wrote:
>> @@ -1088,9 +1091,22 @@ impl Iterator for InnerTaskArchiveIterator {
>>                   Some(current) => {
>>                       let next = current.next();
>>                       if next.is_some() {
>> -                        return next;
>> +                        return next.map(|res| {
>> +                            res.with_context(|| {
>> +                                format!(
>> +                                    "failed to read from {file}",
>> +                                    file = self
>> +                                        .current_file
>> +                                        .as_ref()
>> +                                        .expect("self.current and self.current_file are both Some")
>
> i guess we could prevent this expect here if `current` is changed to an
> `Option<(ArchiveIterator, ArchiveFile)>`
>
> so that both must be either Some or None?

Good call, will change it in v2!