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 plugins #512

Open
edvgui opened this issue Mar 28, 2024 · 1 comment
Open

Add logging plugins #512

edvgui opened this issue Mar 28, 2024 · 1 comment
Labels
enhancement New feature or request usability This ticket is related to improved usability

Comments

@edvgui
Copy link
Contributor

edvgui commented Mar 28, 2024

This module has the std::print module, which works well when compiling a model by hand, but very poorly when using the project fixture of pytest_inmanta.

The idea is to offer another interface for the user to "print out" information from the model, based on python logging.

Open questions:

  1. It can either be a single plugin, similar to logging.Logger.log, or multiple plugins, for the different log levels?
  2. Keep the python logging interface, or the agent logging interface (keyword parameters are better there)?
@bartv
Copy link
Contributor

bartv commented Mar 28, 2024

I think it makes sense in general to be able to add logging to the model. I am going to create another ticket to actually handle the project fixture not showing stdout on failure

@sanderr sanderr added enhancement New feature or request usability This ticket is related to improved usability labels Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request usability This ticket is related to improved usability
Projects
None yet
Development

No branches or pull requests

3 participants