Skip to content

Commit

Permalink
Merge branch 'master' into sy/rel-dcd2
Browse files Browse the repository at this point in the history
  • Loading branch information
steveny91 authored Aug 5, 2024
2 parents 8ae6432 + 4c01f09 commit de69369
Show file tree
Hide file tree
Showing 4 changed files with 43 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pr-quick-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,4 +75,4 @@ jobs:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
const { issue: { number: issue_number }, repo: { owner, repo } } = context;
github.issues.createComment({ issue_number, owner, repo, body: "The changelog type `changed` or `removed` was used in this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the `fixed` or `added` type instead." });
github.rest.issues.createComment({ issue_number, owner, repo, body: "The changelog type `changed` or `removed` was used in this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the `fixed` or `added` type instead." });
2 changes: 1 addition & 1 deletion .github/workflows/run-validations.yml
Original file line number Diff line number Diff line change
Expand Up @@ -255,4 +255,4 @@ jobs:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
const { issue: { number: issue_number }, repo: { owner, repo } } = context;
github.issues.createComment({ issue_number, owner, repo, body: "The `validations` job has failed; please review the `Files changed` tab for possible suggestions to resolve." });
github.rest.issues.createComment({ issue_number, owner, repo, body: "The `validations` job has failed; please review the `Files changed` tab for possible suggestions to resolve." });
1 change: 1 addition & 0 deletions ibm_mq/tests/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ def test_calculate_elapsed_time_valid_tz(datestamp, timestamp, time_zone, valid)
assert calculate_elapsed_time(datestamp, timestamp, time_zone) is not None


@pytest.mark.flaky
@pytest.mark.parametrize(
'datestamp,timestamp,timestamp_dst,time_zone,expected',
[
Expand Down
48 changes: 40 additions & 8 deletions istio/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,9 @@ See [Configure integrations with Autodiscovery on Kubernetes][4] or [Configure i
To monitor the Istio control plane and report the `mixer`, `galley`, `pilot`, and `citadel` metrics, you must configure the Agent to monitor the `istiod` deployment. In Istio v1.5 or later, apply the following pod annotations for the deployment `istiod` in the `istio-system` namespace:

```yaml
ad.datadoghq.com/discovery.check_names: '["istio"]'
ad.datadoghq.com/discovery.init_configs: '[{}]'
ad.datadoghq.com/discovery.instances: |
ad.datadoghq.com/<CONTAINER_IDENTIFIER>.check_names: '["istio"]'
ad.datadoghq.com/<CONTAINER_IDENTIFIER>.init_configs: '[{}]'
ad.datadoghq.com/<CONTAINER_IDENTIFIER>.instances: |
[
{
"istiod_endpoint": "http://%%host%%:15014/metrics",
Expand All @@ -96,9 +96,9 @@ ad.datadoghq.com/discovery.instances: |
]
```

The method for applying these annotations varies depending on the [Istio deployment strategy (Istioctl, Helm, Operator)][22] used. Consult the [Istio documentation][23] for the proper method to apply these pod annotations. See the [sample istio.d/conf.yaml][8] for all available configuration options.
Replace `<CONTAINER_IDENTIFIER>` with the name (`.spec.containers[i].name`) of your container. For example, `discovery`.

These annotations reference `discovery` to match the default container name of the pods for the `istiod` deployment. If your container name is different, adjust accordingly.
The method for applying these annotations varies depending on the [Istio deployment strategy (Istioctl, Helm, Operator)][22] used. Consult the [Istio documentation][23] for the proper method to apply these pod annotations. See the [sample istio.d/conf.yaml][8] for all available configuration options.

#### Disable sidecar injection for Datadog Agent pods

Expand Down Expand Up @@ -146,11 +146,43 @@ kubectl patch daemonset datadog-agent -p '{"spec":{"template":{"metadata":{"anno

#### Log collection

Istio contains two types of logs: Envoy access logs that are collected with the [Envoy integration][11] and [Istio logs][12].

_Available for Agent versions >6.0_

To enable log collection in Kubernetes, see [Kubernetes Log Collection][13].
First, enable the Datadog Agent to perform log collection in Kubernetes. See [Kubernetes Log Collection][13].

#### Istio logs

To collect [Istio logs][12], apply the following annotation to the pod where Istio containers are running:

```yaml
ad.datadoghq.com/<CONTAINER_IDENTIFIER>.logs: |
[
{
"source": "istio",
"service": "<SERVICE_NAME>"
}
]
```

Replace `<CONTAINER_IDENTIFIER>` with the name (`.spec.containers[i].name`) of your container. Replace `<SERVICE_NAME>` with the name of your Istio service.

#### Envoy access logs

To collect Envoy access logs:
1. Ensure the [Envoy integration][11] is installed.
2. Apply the following annotation to the pod where Envoy is running:

```yaml
ad.datadoghq.com/<CONTAINER_IDENTIFIER>.logs: |
[
{
"type": "file",
"path": "/var/log/envoy.log",
"source": "envoy"
}
]
```
Replace `<CONTAINER_IDENTIFIER>` with the name (`.spec.containers[i].name`) of your container.

### Validation

Expand Down

0 comments on commit de69369

Please sign in to comment.