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

kctrl package install … --dry-run logs "Succeeded" to stdout #1387

Open
mamachanko opened this issue Oct 31, 2023 · 2 comments
Open

kctrl package install … --dry-run logs "Succeeded" to stdout #1387

mamachanko opened this issue Oct 31, 2023 · 2 comments
Labels
bug This issue describes a defect or unexpected behavior carvel-accepted This issue should be considered for future work and that the triage process has been completed

Comments

@mamachanko
Copy link
Contributor

mamachanko commented Oct 31, 2023

What steps did you take:

I am piping the output of kctrl package install … --dry-run into ytt (or kapp):

kctrl package install \
  … \
  --dry-run \
  | ytt -f-

What happened:

ytt: Error: Unmarshaling YAML template 'stdin.yml': yaml: line 71: could not find expected ':'

That's because Succeeded is printed to stdout.

What did you expect:

I expected to be able to receive valid YAML on stdout.

Anything else you would like to add:

The issue probably is somewhere around here, but I admit that I was unable to figure out where output directions are configured. It's not obvious.

Environment:

❯ kctrl --version
kctrl version 0.48.1

Succeeded

❯ bash --version
GNU bash, version 5.2.15(1)-release (x86_64-apple-darwin21.6.0)
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

❯ uname -a
Darwin ***** 22.6.0 Darwin Kernel Version 22.6.0: Fri Sep 15 13:39:52 PDT 2023; root:xnu-8796.141.3.700.8~1/RELEASE_X86_64 x86_64


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@mamachanko mamachanko added bug This issue describes a defect or unexpected behavior carvel-triage This issue has not yet been reviewed for validity labels Oct 31, 2023
@praveenrewar
Copy link
Member

@mamachanko You cat use --tty=false to get rid of the "Succeeded" message. In future, we might have to be a little smarter to not print this by default when using dry-run, but I hope this should at least unblock you for now.

Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Dec 11, 2023
@praveenrewar praveenrewar added carvel-accepted This issue should be considered for future work and that the triage process has been completed and removed carvel-triage This issue has not yet been reviewed for validity stale This issue has had no activity for a while and will be closed soon labels Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior carvel-accepted This issue should be considered for future work and that the triage process has been completed
Projects
Status: No status
Development

No branches or pull requests

2 participants