-
Notifications
You must be signed in to change notification settings - Fork 904
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
Separate log-level and structured/unstructured cmd options #1977
Comments
I like the idea of decoupling output format from verbosity but I'd prefer to avoid removing the debug flag as that could be considered a breaking/potentially disruptive change. What do you think about instead adding a new flag ala |
I should have mentioned defaults. To match current behavior I had been thinking the default value of |
I'm not seeing why we'd need to deprecate or replace it - it doesn't feel broken to me. That said, I don't feel particularly strongly about this except that I don't want folks who expect to be able to run |
Hi @negz, I would love to work on this issue if no one is working on it. According to the initial look I took at the Also, should I wait for this PR: #2117 to land first? |
Hey @psinghal20! Please do feel free to work on this now that you've moved us over to kong. :) |
Hey, want to contribute, and can pick this up? |
Crossplane does not currently have enough maintainers to address every issue and pull request. This issue has been automatically marked as |
Hi, |
There hasn't been any activity or interest (folks in the community needing this functionality) on this issue for over a year, so I'm not seeing it as a priority to pursue. crossplane/crossplane-runtime#373 was a bigger concern for me personally, and I believe that should be addressed now. |
Any progress on this? Logs management is basic thing |
@Hronom would it be possible for you to describe more deeply what you are wanting to do with the crossplane logs that you cannot currently accomplish? |
@jbw976 simplify scrapping of logs (by using JSON) as same as control level (error, warn, info, debug, trace) Solution described in first message |
What problem are you facing?
Setting the
--debug
flag currently also changes the output formatting of our logs to semi-structured. It can sometimes be helpful to switch to a debug log level in production, or use structured logs in development.How could Crossplane help solve your problem?
Remove the
--debug
flag and add 2 new flags--log-level=(info|debug)
and--output=(json|text)\-o
. My thinking in naming theoutput
flag is to mirror kubectl but I'm not convinced that's the right move since this only pertains to logs (although we don't have any other output?), maybe--log-format
would be more clear.The text was updated successfully, but these errors were encountered: