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

sync-gateway: Add pod annotations #77

Open
wants to merge 2 commits into
base: 2.2.x
Choose a base branch
from

Conversation

psact
Copy link

@psact psact commented Dec 9, 2021

This is useful for configuring things like Datadog autodiscovery for
monitoring:

syncGateway:
podAnnotations:
ad.datadoghq.com/sync-gateway.check_names: '[ "couchbase" ]'
ad.datadoghq.com/sync-gateway.init_configs: '[{}]'
ad.datadoghq.com/sync-gateway.instances: |
[
{
"server": "http://server:8091",
"sync_gateway_url": "http://%%host%%:4986",
"user":"%%env_DD_COUCHBASE_USERNAME%%",
"password":"%%env_DD_COUCHBASE_PASSWORD%%",
}
]

@tahmmee
Copy link
Contributor

tahmmee commented Dec 10, 2021

Looks good. You will also need to update the App version to 2.2.202 in Chart.yaml for this to publish correctly.

pscamardo added 2 commits June 17, 2022 16:28
This is useful for configuring things like Datadog autodiscovery for
monitoring:

  syncGateway:
    podAnnotations:
      ad.datadoghq.com/sync-gateway.check_names: '[ "couchbase" ]'
      ad.datadoghq.com/sync-gateway.init_configs: '[{}]'
      ad.datadoghq.com/sync-gateway.instances: |
        [
          {
            "server": "http://server:8091",
            "sync_gateway_url": "http://%%host%%:4986",
            "user":"%%env_DD_COUCHBASE_USERNAME%%",
            "password":"%%env_DD_COUCHBASE_PASSWORD%%",
          }
        ]
@psact psact force-pushed the sgw-pod-annotations branch from 8615ade to 6d3d8dc Compare June 17, 2022 23:30
@psact psact changed the base branch from master to 2.2.x June 17, 2022 23:31
@psact
Copy link
Author

psact commented Jun 17, 2022

@tahmmee - I rebased this on the 2.2.x branch, but this can also probably apply to 2.3.x/master as well

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