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

small certificate related cleanups#984

Merged
rbradford merged 2 commits into
ciao-project:masterfrom
tpepper:misc
Jan 5, 2017
Merged

small certificate related cleanups#984
rbradford merged 2 commits into
ciao-project:masterfrom
tpepper:misc

Conversation

@tpepper

@tpepper tpepper commented Jan 4, 2017

Copy link
Copy Markdown

Over the past weeks I've bumped into a few little oddities around certs and am just doing some cleanup in response. This PR simplifies an old and very verbose error message and incrementally extends documentation.

Tim Pepper added 2 commits January 4, 2017 10:35
This message is stale.  The RBAC scheme is described in the ciao-cert
and other documentation, but I add an additional footnote to the
ciao-cert readme to help an operator.  And then simplify the error
message.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>
There's a lot of variation between distros, golang runtime builds, and
possible operator configuration choices around certificate authorities.
How to interpret the "x509: certificate signed by unknown authority"
error message is becoming a FAQ.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>
@bryteise

bryteise commented Jan 4, 2017

Copy link
Copy Markdown
Contributor

lgtm +1

@coveralls

coveralls commented Jan 4, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.02%) to 66.432% when pulling 6aaf9cb on tpepper:misc into 0ffc205 on 01org:master.

@rbradford rbradford merged commit 10e7261 into ciao-project:master Jan 5, 2017
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