stderr from explorers #94
Labels
No labels
bugfix
cleanup
discussion
documentation
doing
done
feature
improvement
packaging
Stale
testing
TODO
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: ungleich-public/cdist#94
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Created by: 4nd3r
in global and type explorers, when
exit 1
withecho error >&2
is emitted, stderr of explorer is not shown, but it is with gencode-remote:also, since type explorer fails, nothing is saved into cache directory, so I can't even use cdist-dump.
usecase: I would like to do preflight checks in type explorer and if something is wrong, type should terminate with exit 1 and error show to admin. workaround right now is collect messages into output of type explorer and do the check in gencode-remote.
mentioned in commit
087066687c
Created by: darko-poljak
@4nd3r I'll review later code for explorers and then I will push it so you can test it, ok?
And then we can include it in 4.11.1.
@asteven @telmich You might also be very interested in this upcoming improvement.
Created by: darko-poljak
@4nd3r For type manifest, current implementation suffices, right?
Created by: darko-poljak
@4nd3r For type explorers:
Created by: darko-poljak
I am already on it...
Created by: 4nd3r
no relevant files will be created locally or remotely if type explorer fails.
i created simple type:
Created by: darko-poljak
@4nd3r I already have this for global explorers :)
Created by: darko-poljak
@4nd3r I see what you mean. I will add support equivalent as for manifest and code execution. In the next release.
Created by: darko-poljak
@4nd3r If you use -vvvv you can see tmp directory and inside in
/tmp/tmp.../<host-dir>/data/stderr/remote
you can see explorer output.Created by: darko-poljak
@4nd3r stderr of global explorers should be in
.../cache/<cache-dir>/stderr/remote
.Can you check?