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

Datadog #1384

Closed
wants to merge 6 commits into from
Closed

Datadog #1384

wants to merge 6 commits into from

Conversation

dmitry-spectro
Copy link
Contributor

@dmitry-spectro dmitry-spectro commented Jun 27, 2023

Describe the Change

This PR adds Datatog integration for sending logs

Review Changes

💻 Add Preview URL

🎫 Jira Ticket

@dmitry-spectro dmitry-spectro requested a review from a team as a code owner June 27, 2023 08:56
@dmitry-spectro dmitry-spectro requested review from karl-cardenas-coding, ritawatson and rahulhazra97 and removed request for a team June 27, 2023 08:56
@netlify
Copy link

netlify bot commented Jun 27, 2023

Deploy Preview for docs-spectrocloud ready!

Name Link
🔨 Latest commit a364a95
🔍 Latest deploy log https://app.netlify.com/sites/docs-spectrocloud/deploys/655bad950d59d90008e9932f
😎 Deploy Preview https://deploy-preview-1384--docs-spectrocloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

vale

content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|100 col 19| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|100 col 241| [Google.Exclamation] Don't use exclamation points in text.
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|101 col 43| [Vale.Terms] Use 'admin' instead of 'Admin'.
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|108 col 23| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|156 col 113| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|157 col 34| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|167 col 66| [spectrocloud.ableism] Avoid using ableism terms. Use 'display' instead of 'see'.
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|188 col 50| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|196 col 27| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|196 col 93| [Google.Exclamation] Don't use exclamation points in text.
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|205 col 70| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|219 col 44| [Vale.Terms] Use 'admin' instead of 'Admin'.
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|234 col 18| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|248 col 100| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|255 col 67| [Vale.Spelling] Did you really mean 'Keycloak'?
content/docs/08-user-management/04-saml-sso/05-palette-sso-with-keycloak.md|263 col 77| [Google.Exclamation] Don't use exclamation points in text.


* Docs to Markdown version 1.0β34
* Tue Jun 27 2023 01:55:24 GMT-0700 (PDT)
* Source doc: Palette Datadog Integration
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?



<p style="color: red; font-weight: bold">>>>>> gd2md-html alert: ERRORs: 0; WARNINGs: 0; ALERTS: 8.</p>
<ul style="color: red; font-weight: bold"><li>See top comment block for details on ERRORs and WARNINGs. <li>In the converted Markdown or HTML, search for inline alerts that start with >>>>> gd2md-html alert: for specific instances that need correction.</ul>
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[spectrocloud.ableism] Avoid using ableism terms. Use 'display' instead of 'See'.


1. Access to Palette
2. Kubernetes Cluster
3. Datadog account with API key
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?

## Installation


### Datadog preparation
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?




1. If you don’t already have access to Datadog, open [https://app.datadoghq.eu/signup](https://app.datadoghq.eu/signup) choose your region (EU1 in my case), give your name, email address and company name, and press “Sign up”
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


1. Provision a k8s cluster from Palette and download its kubeconfig
2. Let’s install keycloak on our Kubernetes cluster with CLI:
3. After a couple of minutes, verify which address Keycloak got:
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?


4. Let’s create an Ingress from an official doc, replacing KEYCLOAK_HOST with your hostname for LoadBalance:

This creates an ingress and we can now see keycloak available at services in Palette:
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[spectrocloud.ableism] Avoid using ableism terms. Use 'display' instead of 'see'.


4. Let’s create an Ingress from an official doc, replacing KEYCLOAK_HOST with your hostname for LoadBalance:

This creates an ingress and we can now see keycloak available at services in Palette:
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'keycloak'?

<p id="gdcalert1" ><span style="color: red; font-weight: bold">>>>>> gd2md-html alert: inline image link here (to images/image1.png). Store image on your image server and adjust path/filename/extension if necessary. </span><br>(<a href="#">Back to top</a>)(<a href="#gdcalert2">Next alert</a>)<br><span style="color: red; font-weight: bold">>>>>> </span></p>


![alt_text](images/image1.png "image_tooltip")
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'alt_text'?




5. I’ll also create a CNAME _keycloak.dmitry.sa.spectrodemos.com_ pointing to _a5f3a06f0063143fea26b6a086cb2c89-1900572074.eu-west-3.elb.amazonaws.com_ so that it’d be more convenient to type and to get access to our Keycloak
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?


* Docs to Markdown version 1.0β34
* Tue Jun 27 2023 01:32:57 GMT-0700 (PDT)
* Source doc: Palette Keycloak Integration
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?



5. I’ll also create a CNAME _keycloak.dmitry.sa.spectrodemos.com_ pointing to _a5f3a06f0063143fea26b6a086cb2c89-1900572074.eu-west-3.elb.amazonaws.com_ so that it’d be more convenient to type and to get access to our Keycloak
6. Let’s login to Keycloak with default _admin:admin_ credentials at _[http://keycloak.dmitry.sa.spectrodemos.com:8080/admin](http://keycloak.dmitry.sa.spectrodemos.com:8080/admin)_, First thing is to change the admin password to something sensible!
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?



5. I’ll also create a CNAME _keycloak.dmitry.sa.spectrodemos.com_ pointing to _a5f3a06f0063143fea26b6a086cb2c89-1900572074.eu-west-3.elb.amazonaws.com_ so that it’d be more convenient to type and to get access to our Keycloak
6. Let’s login to Keycloak with default _admin:admin_ credentials at _[http://keycloak.dmitry.sa.spectrodemos.com:8080/admin](http://keycloak.dmitry.sa.spectrodemos.com:8080/admin)_, First thing is to change the admin password to something sensible!
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Google.Exclamation] Don't use exclamation points in text.


5. I’ll also create a CNAME _keycloak.dmitry.sa.spectrodemos.com_ pointing to _a5f3a06f0063143fea26b6a086cb2c89-1900572074.eu-west-3.elb.amazonaws.com_ so that it’d be more convenient to type and to get access to our Keycloak
6. Let’s login to Keycloak with default _admin:admin_ credentials at _[http://keycloak.dmitry.sa.spectrodemos.com:8080/admin](http://keycloak.dmitry.sa.spectrodemos.com:8080/admin)_, First thing is to change the admin password to something sensible!
7. Let’s login to Palette, choose “Tenant Admin” at the top, click on “Tenant Settings” on the left hand side, then click on “SSO” in the center and “OIDC” on the right hand side. We’ll need to copy “Callback URL” and “Logout URL”
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'admin' instead of 'Admin'.


![alt_text](images/image2.png "image_tooltip")

8. Let’s now login to Keycloak and create a client for Palette: I click on “Create client”
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?

![alt_text](images/image14.png "image_tooltip")


23. I add a team “admins” and add “Project Admin” role to my project “Default”
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Terms] Use 'admin' instead of 'Admin'.


![alt_text](images/image16.png "image_tooltip")

26. I go back to Keycloak, click on “Users” on the left hand side and create a user, making sure “Username”, “Email”, “First Name” and “Last Name” are all filled in.
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?


![alt_text](images/image18.png "image_tooltip")

28. If I now open another incognito window, go to Palette and log in with my newly created user in Keycloak, I should be automatically logged in Palette as admin!
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?


![alt_text](images/image19.png "image_tooltip")

29. Voilà, our example is done,we’re logged in with the user from Keycloak without touching anything in Palette
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Keycloak'?

![alt_text](images/image20.png "image_tooltip")


Please contact us if you have any further questions on how to set up proper RBAC!
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Google.Exclamation] Don't use exclamation points in text.

@karl-cardenas-coding
Copy link
Contributor

@dmitry-spectro We took a peek at this guide. Dmitry, the voice and style does not follow our docs style. Can you update the content to follow our styleguide and voice please? Here is a link to a good how-to example.

@karl-cardenas-coding karl-cardenas-coding removed the request for review from rahulhazra97 November 8, 2023 22:59
@dmitry-spectro
Copy link
Contributor Author

@dmitry-spectro We took a peek at this guide. Dmitry, the voice and style does not follow our docs style. Can you update the content to follow our styleguide and voice please? Here is a link to a good how-to example.

Hey @karl-cardenas-coding Karl, happy to review and to improve, but can you be a little bit more specific in what you want me to change please?

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

vale

content/docs/10-audit-logs/06-integrating-with-datadog.md|20 col 98| [Vale.Spelling] Did you really mean 'Datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|26 col 29| [Vale.Spelling] Did you really mean 'datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|26 col 42| [Vale.Terms] Use 'Repo' instead of 'repo'.
content/docs/10-audit-logs/06-integrating-with-datadog.md|26 col 183| [Vale.Spelling] Did you really mean 'Datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|30 col 67| [Vale.Spelling] Did you really mean 'Datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|30 col 171| [Vale.Spelling] Did you really mean 'datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|30 col 308| [Vale.Spelling] Did you really mean 'datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|34 col 65| [Vale.Spelling] Did you really mean 'changesi'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|34 col 103| [Vale.Spelling] Did you really mean 'Datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|38 col 8| [Vale.Spelling] Did you really mean 'api'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|38 col 29| [Vale.Spelling] Did you really mean 'your_api_key_here'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|74 col 12| [Vale.Spelling] Did you really mean 'mountPath'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|78 col 118| [Vale.Spelling] Did you really mean 'datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|82 col 25| [Vale.Spelling] Did you really mean 'Datadog'?
content/docs/10-audit-logs/06-integrating-with-datadog.md|111 col 6| [Vale.Spelling] Did you really mean 'Datadog'?

@@ -0,0 +1,118 @@

# Palette Datadog Integration
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


1. Access to Palette
2. Kubernetes Cluster
3. Datadog account with API key
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


## Installation

This documents permits to integrate Palette and Datadog: export logs from Kubernetes clusters managed by Palette to Datadog instance
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


## Installation

This documents permits to integrate Palette and Datadog: export logs from Kubernetes clusters managed by Palette to Datadog instance
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


This documents permits to integrate Palette and Datadog: export logs from Kubernetes clusters managed by Palette to Datadog instance

### Datadog preparation
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


## Installation

This documents permits to integrate Palette and Datadog: export logs from Kubernetes clusters managed by Palette to Datadog instance
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


## Installation

This documents permits to integrate Palette and Datadog: export logs from Kubernetes clusters managed by Palette to Datadog instance
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


### Datadog preparation

1. If you don’t already have access to Datadog, open [https://app.datadoghq.eu/signup](https://app.datadoghq.eu/signup) choose appropriate region (for example, EU1), give your name, email address and company name, and press “Sign up”
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?


1. If you don’t already have access to Datadog, open [https://app.datadoghq.eu/signup](https://app.datadoghq.eu/signup) choose appropriate region (for example, EU1), give your name, email address and company name, and press “Sign up”

![alt_text](/audit-logs_datadog-01_datadog_signup.png "Datadog Signup" )
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'alt_text'?


1. If you don’t already have access to Datadog, open [https://app.datadoghq.eu/signup](https://app.datadoghq.eu/signup) choose appropriate region (for example, EU1), give your name, email address and company name, and press “Sign up”

![alt_text](/audit-logs_datadog-01_datadog_signup.png "Datadog Signup" )
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Datadog'?

Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 8.3%, saving 238.45 KB.

Filename Before After Improvement Visual comparison
static/assets/docs/images/audit-logs_datadog-01_datadog_signup.png 337.56 KB 332.31 KB -1.6% View diff
static/assets/docs/images/audit-logs_datadog-02_datadog_apikey.png 395.51 KB 388.24 KB -1.8% View diff
static/assets/docs/images/audit-logs_datadog-03_helm_repository.png 183.38 KB 179.73 KB -2.0% View diff
static/assets/docs/images/audit-logs_datadog-04_helm_chart.png 360.00 KB 189.73 KB -47.3% View diff
static/assets/docs/images/audit-logs_datadog-05_helm_modifications.png 514.55 KB 497.84 KB -3.2% View diff
static/assets/docs/images/audit-logs_datadog-06_profile_confirm.png 100.47 KB 97.73 KB -2.7% View diff
static/assets/docs/images/audit-logs_datadog-07_datadog_agent.png 395.96 KB 385.25 KB -2.7% View diff
static/assets/docs/images/audit-logs_datadog-08_datadog_example.png 578.79 KB 556.93 KB -3.8% View diff

507 images did not require optimisation.

@karl-cardenas-coding
Copy link
Contributor

Closed as #1961 carries over the work.

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.

3 participants