-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: Cloud Deployment Azure #15397
docs: Cloud Deployment Azure #15397
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Docs team] One thing I noticed, there's a lot of bolding in this PR. Try to use bold sparingly, because if you use it too much, it starts to lose its emphasis.
Hey @JStickler that is the changes in and ready for your review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[docs team] Looking good!
💻 Deploy preview deleted. |
Co-authored-by: J Stickler <[email protected]> Signed-off-by: Jay Clifford <[email protected]>
Happy new year @JStickler! Ready for another review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[docs team] A couple more suggestions, but LGTM.
Co-authored-by: J Stickler <[email protected]> Signed-off-by: Jay Clifford <[email protected]>
Thanks @DylanGuedes! @JStickler I think we will need a backport on this :) |
**What this PR does / why we need it**: Continuation of the cloud deployment guides. This teaches the user how to deploy Loki on Azure using the helm. Including: * Azure Setup * Helm values config * Testing (cherry picked from commit 672f91c)
**What this PR does / why we need it**: Continuation of the cloud deployment guides. This teaches the user how to deploy Loki on Azure using the helm. Including: * Azure Setup * Helm values config * Testing (cherry picked from commit 672f91c)
**What this PR does / why we need it**: Continuation of the cloud deployment guides. This teaches the user how to deploy Loki on Azure using the helm. Including: * Azure Setup * Helm values config * Testing (cherry picked from commit 672f91c)
What this PR does / why we need it:
Continuation of the cloud deployment guides. This teaches the user how to deploy Loki on Azure using the helm. Including:
I have also added some opinionated recommendations to the helm index page for review aswell.
Checklist
CONTRIBUTING.md
guide (required)feat
PRs are unlikely to be accepted unless a case can be made for the feature actually being a bug fix to existing behavior.docs/sources/setup/upgrade/_index.md
deprecated-config.yaml
anddeleted-config.yaml
files respectively in thetools/deprecated-config-checker
directory. Example PR