You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is somewhat related to #331 . I was working with crossplane and the wordpress gcp example locally, and was wanting more detail in the logs for debugging issues (such as the issue described in #597). But, the information to debug this type of issue seems like it's at an even more verbose level than logging.Debug, which is the only level we support now.
I feel it would make sense to create another logging level, logging.Trace, which could be used for extra-verbose debug statements. An example of a statement at Trace could be "syncing of the cache instance is done because it hasn't been created yet", and other messages about enter and exit conditions.
What do other folks think? Does this seem like a good idea? /cc @jbw976@negz
The text was updated successfully, but these errors were encountered:
My current thinking is to avoid this, somewhat explained by the logging package doc in crossplane/crossplane-runtime#108. I agree debug logging is important, but I think it should be an on or off thing - not at multiple levels. Please comment on that pull if you disagree!
This is somewhat related to #331 . I was working with crossplane and the wordpress gcp example locally, and was wanting more detail in the logs for debugging issues (such as the issue described in #597). But, the information to debug this type of issue seems like it's at an even more verbose level than
logging.Debug
, which is the only level we support now.I feel it would make sense to create another logging level,
logging.Trace
, which could be used for extra-verbose debug statements. An example of a statement atTrace
could be "syncing of the cache instance is done because it hasn't been created yet", and other messages about enter and exit conditions.What do other folks think? Does this seem like a good idea? /cc @jbw976 @negz
The text was updated successfully, but these errors were encountered: