Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging capabilities #4

Open
NucciTheBoss opened this issue Oct 6, 2022 · 2 comments
Open

Add logging capabilities #4

NucciTheBoss opened this issue Oct 6, 2022 · 2 comments

Comments

@NucciTheBoss
Copy link
Owner

Self explanatory - enable automatic logging for tests. This could involve "injectables" that copy stdout and stderr from the subprocess and save to a log file.

The question is whether or not this logging should be enabled by default? Personally, I would rather have the test be verbose and then allow end-users to pass a silent option through the test provider decorator. Another question is what should be captured in the log file? Information strictly about the testlet? What data is being loaded by cleantest? What arguments where passed to the hooks? The possibilities.

For now, let's keep it pretty simple.

@NucciTheBoss NucciTheBoss added this to the 0.1.1 milestone Oct 6, 2022
@NucciTheBoss NucciTheBoss removed this from the 0.2.0 milestone Oct 27, 2022
@NucciTheBoss
Copy link
Owner Author

Removed from 0.2.0 milestone due to time constraints and presentations.

@NucciTheBoss NucciTheBoss added this to the 0.4.0 milestone Jan 20, 2023
@NucciTheBoss NucciTheBoss self-assigned this Jan 20, 2023
@NucciTheBoss
Copy link
Owner Author

I added logging capabilities to the LXDArchon class in the 0.5.0-dev branch so once those commits land I can finally close this issue after all these months 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

1 participant