Skip to content
This repository was archived by the owner on Oct 10, 2020. It is now read-only.

trust, sign: prevent gpg warnings to be printed#1126

Closed
giuseppe wants to merge 1 commit intoprojectatomic:masterfrom
giuseppe:fix-gpg-warnings
Closed

trust, sign: prevent gpg warnings to be printed#1126
giuseppe wants to merge 1 commit intoprojectatomic:masterfrom
giuseppe:fix-gpg-warnings

Conversation

@giuseppe
Copy link
Copy Markdown
Collaborator

@giuseppe giuseppe commented Nov 8, 2017

I've noticed this warning while using "atomic trust show":

gpg: WARNING: no command supplied. Trying to guess what you mean ...

Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com

@giuseppe giuseppe requested a review from baude November 8, 2017 08:28
@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Nov 8, 2017

Will this eat real errors? Should we not do this when setting the --debug flag?

I've noticed this warning while using "atomic trust show":

gpg: WARNING: no command supplied.  Trying to guess what you mean ...

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe
Copy link
Copy Markdown
Collaborator Author

giuseppe commented Nov 8, 2017

I've changed it to redirect stderr to /dev/null only when --debug is not used

@TomSweeneyRedHat
Copy link
Copy Markdown

LGTM

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Nov 8, 2017

@giuseppe did you verify that the message comes out when you pass in the debug flag? If so then
LGTM

@giuseppe
Copy link
Copy Markdown
Collaborator Author

giuseppe commented Nov 9, 2017

yes, I've verified it:

# ./atomic trust show
* (default)                         accept                               
registry.access.redhat.com          signed security@redhat.com,security@redhat.com https://access.redhat.com/webassets/docker/content/sigstore

# ./atomic --debug trust show
* (default)                         accept                               
gpg: WARNING: no command supplied.  Trying to guess what you mean ...
registry.access.redhat.com          signed security@redhat.com,security@redhat.com https://access.redhat.com/webassets/docker/content/sigstore

@rh-atomic-bot r=rhatdan

@rh-atomic-bot
Copy link
Copy Markdown

📌 Commit 2062958 has been approved by rhatdan

@rh-atomic-bot
Copy link
Copy Markdown

⚡ Test exempted: merge already tested.

eyusupov pushed a commit to eyusupov/atomic that referenced this pull request Mar 10, 2018
I've noticed this warning while using "atomic trust show":

gpg: WARNING: no command supplied.  Trying to guess what you mean ...

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>

Closes: projectatomic#1126
Approved by: rhatdan
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants