Skip to content

swh/webhooks Limit the useless fallback resolution

Vincent Sellier requested to merge webhooks-dns into production

the default ndots: 5 for a failing resolution to all the configured domains before resolving svix.i.s.o generating ~10 dns request per svix event

Related to swh/infra/sysadm-environment#5326 (closed)

helm diff 
[swh] Comparing changes between branches production and webhooks-dns (per environment)...
Switched to branch 'production'
Your branch is up to date with 'origin/production'.
[swh] Generate config in production branch for environment staging, namespace swh...
[swh] Generate config in production branch for environment staging, namespace swh-cassandra...
[swh] Generate config in production branch for environment staging, namespace swh-cassandra-next-version...
Switched to branch 'webhooks-dns'
[swh] Generate config in webhooks-dns branch for environment staging...
[swh] Generate config in webhooks-dns branch for environment staging...
[swh] Generate config in webhooks-dns branch for environment staging...
Switched to branch 'production'
Your branch is up to date with 'origin/production'.
[swh] Generate config in production branch for environment production, namespace swh...
[swh] Generate config in production branch for environment production, namespace swh-cassandra...
[swh] Generate config in production branch for environment production, namespace swh-cassandra-next-version...
Switched to branch 'webhooks-dns'
[swh] Generate config in webhooks-dns branch for environment production...
[swh] Generate config in webhooks-dns branch for environment production...
[swh] Generate config in webhooks-dns branch for environment production...


------------- diff for environment staging namespace swh -------------

     _        __  __
   _| |_   _ / _|/ _|  between /tmp/swh-chart.swh.v2tMC6Q6/staging-swh.before, 111 documents
 / _' | | | | |_| |_       and /tmp/swh-chart.swh.v2tMC6Q6/staging-swh.after, 111 documents
| (_| | |_| |  _|  _|
 \__,_|\__, |_| |_|   returned no differences
        |___/



------------- diff for environment staging namespace swh-cassandra -------------

     _        __  __
   _| |_   _ / _|/ _|  between /tmp/swh-chart.swh.v2tMC6Q6/staging-swh-cassandra.before, 377 documents
 / _' | | | | |_| |_       and /tmp/swh-chart.swh.v2tMC6Q6/staging-swh-cassandra.after, 377 documents
| (_| | |_| |  _|  _|
 \__,_|\__, |_| |_|   returned one difference
        |___/

spec.template.spec  (apps/v1/Deployment/swh-cassandra/webhooks-origin-visit-status)
  + one map entry added:
    dnsConfig:
    │ options:
    │ - name: ndots
    │ │ value: 2
    │ searches:
    │ - cluster.local
    │ - svc.cluster.local
    │ - swh-cassandra.svc.cluster.local



------------- diff for environment staging namespace swh-cassandra-next-version -------------

     _        __  __
   _| |_   _ / _|/ _|  between /tmp/swh-chart.swh.v2tMC6Q6/staging-swh-cassandra-next-version.before, 146 documents
 / _' | | | | |_| |_       and /tmp/swh-chart.swh.v2tMC6Q6/staging-swh-cassandra-next-version.after, 146 documents
| (_| | |_| |  _|  _|
 \__,_|\__, |_| |_|   returned no differences
        |___/



------------- diff for environment production namespace swh -------------

     _        __  __
   _| |_   _ / _|/ _|  between /tmp/swh-chart.swh.v2tMC6Q6/production-swh.before, 410 documents
 / _' | | | | |_| |_       and /tmp/swh-chart.swh.v2tMC6Q6/production-swh.after, 410 documents
| (_| | |_| |  _|  _|
 \__,_|\__, |_| |_|   returned one difference
        |___/

spec.template.spec  (apps/v1/Deployment/swh/webhooks-origin-visit-status)
  + one map entry added:
    dnsConfig:
    │ options:
    │ - name: ndots
    │ │ value: 2
    │ searches:
    │ - cluster.local
    │ - svc.cluster.local
    │ - swh.svc.cluster.local



------------- diff for environment production namespace swh-cassandra -------------

     _        __  __
   _| |_   _ / _|/ _|  between /tmp/swh-chart.swh.v2tMC6Q6/production-swh-cassandra.before, 94 documents
 / _' | | | | |_| |_       and /tmp/swh-chart.swh.v2tMC6Q6/production-swh-cassandra.after, 94 documents
| (_| | |_| |  _|  _|
 \__,_|\__, |_| |_|   returned no differences
        |___/

Merge request reports