-
Notifications
You must be signed in to change notification settings - Fork 159
Open
Labels
plugin | mockRelated to mock provision pluginRelated to mock provision pluginstep | provisionStuff related to the provision stepStuff related to the provision step
Description
Follow-up improvements suggested in #4034.
- I would start mirroring the functionality found in
Command.run
. There are a few questions related todry
run for example. Whether we should actually enter mock and runecho $command
or just print the cmd.- I agree that the logging as written by me was quite ad-hoc and more for development. It does need to be unified with the rest of the sources.
- I noticed that I had to write:
stdout = MockShell.Stream(lambda text: output_logger("out", text, 'yellow', level = 0)) stderr = MockShell.Stream(lambda text: output_logger("err", text, 'yellow', level = 0))
to get any output, I know that for example the
container
provision useslevel=2
, but I could not see the logs even when runningtmt -vvv
.
Originally posted by @happz in #4034 (comment)
Metadata
Metadata
Assignees
Labels
plugin | mockRelated to mock provision pluginRelated to mock provision pluginstep | provisionStuff related to the provision stepStuff related to the provision step
Type
Projects
Status
backlog