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

Allow helm --debug to be conditionally rendered #26

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

timchepeleff
Copy link

@timchepeleff timchepeleff commented Jul 8, 2024

Allow for debug logs to be rendered when using this tool.

Ex.:

./mani-diffy -root=bootstrap/child-applications/us-east-1/apps  -output=zz.auto-generated/root-use1-apps-nonprod/ -hash-store=json -ignore-suffix=-console -skip-render-key=appTag -ignore-value-file=overrides/apptags -debug

@timchepeleff timchepeleff requested a review from a team as a code owner July 8, 2024 16:09
@@ -226,6 +226,7 @@ func main() {
skipRenderKey := flag.String("skip-render-key", "do-not-render", "Key to not render")
ignoreValueFile := flag.String("ignore-value-file", "overrides-to-ignore", "Override file to ignore based on filename")
postRenderer := flag.String("post-renderer", "", "When provided, binary will be called after an application is rendered.")
debug := flag.Bool("debug", false, "When true provides additional logs.")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we could add a way (this or other PR) to pass this and any other additional arguments to Helm, something like additionalArgs="--debug --other=x" 🤷

BTW, you could also set HELM_DEBUG to enable debugging 🤷

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, two great points!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@d1egoaz - I'll work on getting additionalHelmArgs added 👍

@timchepeleff timchepeleff marked this pull request as draft July 8, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants