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

Update _alertmanager-config.rst #1062

Merged
merged 1 commit into from
Sep 2, 2023
Merged

Conversation

alifiroozi80
Copy link
Contributor

the send_resolved is going to fail.

Error from server (BadRequest): error when creating "robusta-alert-config.yaml": AlertmanagerConfig in version "v1alpha1" cannot be handled as a AlertmanagerConfig: strict decoding error: unknown field "spec.receivers[0].webhookConfigs[0].send_resolved"

It should be sendResolved


$ k explain alertmanagerconfig.spec.receivers.webhookConfigs
GROUP:      monitoring.coreos.com
KIND:       AlertmanagerConfig
VERSION:    v1alpha1

FIELD: webhookConfigs <[]Object>

DESCRIPTION:
    List of webhook configurations.
    WebhookConfig configures notifications via a generic receiver supporting the
    webhook payload. See
    https://prometheus.io/docs/alerting/latest/configuration/#webhook_config
    
FIELDS:
  httpConfig	<Object>
    HTTP client configuration.

  maxAlerts	<integer>
    Maximum number of alerts to be sent per webhook message. When 0, all alerts
    are included.

  sendResolved	<boolean>
    Whether or not to notify about resolved alerts.

  url	<string>
    The URL to send HTTP POST requests to. `urlSecret` takes precedence over
    `url`. One of `urlSecret` and `url` should be defined.

  urlSecret	<Object>
    The secret's key that contains the webhook URL to send HTTP requests to.
    `urlSecret` takes precedence over `url`. One of `urlSecret` and `url` should
    be defined. The secret needs to be in the same namespace as the
    AlertmanagerConfig object and accessible by the Prometheus Operator.

@CLAassistant
Copy link

CLAassistant commented Aug 29, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@arikalon1 arikalon1 left a comment

Choose a reason for hiding this comment

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

nice catch @alifiroozi80
Thanks for your contribution

@alifiroozi80
Copy link
Contributor Author

My pleasure @arikalon1🍻

@arikalon1 arikalon1 merged commit ba724ad into robusta-dev:master Sep 2, 2023
5 of 6 checks passed
pavangudiwada pushed a commit to pavangudiwada/robusta that referenced this pull request Nov 6, 2023
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