Skip to content
This repository was archived by the owner on Jan 27, 2021. It is now read-only.

Conversation

@vbehar
Copy link
Contributor

@vbehar vbehar commented Oct 8, 2019

Document the chart values that affect the global configuration, and all supported annotations.

Depending on which PRs got merged first between this one, and #38 or #39 I'll update the latest ones with the missing documentation.

Document the chart values that affect the global configuration, and all supported annotations.
@codecov-io
Copy link

codecov-io commented Oct 8, 2019

Codecov Report

Merging #42 into master will decrease coverage by 0.8%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   57.78%   56.98%   -0.81%     
==========================================
  Files          11       11              
  Lines         623      623              
==========================================
- Hits          360      355       -5     
- Misses        234      239       +5     
  Partials       29       29
Impacted Files Coverage Δ
pkg/net/http/httputil/reverseproxy.go 70.23% <0%> (-1.99%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 17f760c...cbfb427. Read the comment docs.

@krancour
Copy link
Contributor

@vbehar I left a couple notes about how the sections and tables are organized and formatted, but your descriptions of each option look accurate and informative. I really appreciate this contribution. It closes a sizable gap in the docs.

@krancour
Copy link
Contributor

@vbehar this looks spectacular. Thank you!

@krancour krancour self-requested a review October 11, 2019 19:40
Copy link
Contributor

@krancour krancour left a comment

Choose a reason for hiding this comment

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

LGTM

@krancour krancour merged commit 2639c55 into deislabs:master Oct 11, 2019
vbehar added a commit to vbehar/osiris that referenced this pull request Oct 21, 2019
* Document global config and supported annotations

Document the chart values that affect the global configuration, and all supported annotations.

* improve documentation
vbehar added a commit to dailymotion-oss/osiris that referenced this pull request Sep 14, 2020
backport deislabs#42

Document the chart values that affect the global configuration, and all supported annotations.
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.

3 participants