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

[docs] Add links to Sonobuoy Scanner #87

Merged
merged 1 commit into from
Oct 2, 2017
Merged

[docs] Add links to Sonobuoy Scanner #87

merged 1 commit into from
Oct 2, 2017

Conversation

jesscodez
Copy link
Contributor

@timothysc PTAL

See https://github.com/abiogenesis-now/sonobuoy/blob/39285b8ec5c78a0aba07a867476f2c3f5fd2c9b9/README.md to see what it looks like.

For better visibility, I decided to put the Scanner link in its own section before "Getting Started" (rather than under "Run"). Let me know if you think this is appropriate, or if I should change it. Thanks!

Signed-off-by: Jessica Yao <jessica@heptio.com>
@jesscodez
Copy link
Contributor Author

Addresses #86 (and possibly #19)?

@jesscodez
Copy link
Contributor Author

jesscodez commented Sep 29, 2017

And also maybe addresses #29, since it looks like (in Sonobuoy Scanner) we get the relevant system logs for failed tests?

Copy link
Contributor

@timothysc timothysc left a comment

Choose a reason for hiding this comment

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

/lgtm

@timothysc timothysc merged commit 5b0ce60 into vmware-tanzu:master Oct 2, 2017
jhamilton1 pushed a commit to jhamilton1/sonobuoy that referenced this pull request Jun 27, 2018
…ner-links

[docs] Add links to Sonobuoy Scanner
Signed-off-by: Jesse Hamilton jesse.hamilton@heptio.com
jhamilton1 pushed a commit to jhamilton1/sonobuoy that referenced this pull request Jun 27, 2018
…ner-links

[docs] Add links to Sonobuoy Scanner
Signed-off-by: Jesse Hamilton jesse.hamilton@heptio.com

Signed-off-by: Jesse Hamilton jesse.hamilton@heptio.com
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.

2 participants