Skip to content

Conversation

@sitole
Copy link
Member

@sitole sitole commented Jan 7, 2026

Disabled option that was trying to scrape and map all services.
This was failing because we did not configure them to be exposed, and it was spamming the ingress logs.


Note

Disables auto-exposure of Nomad services in Traefik to prevent unintended service discovery and log noise.

  • Adds --providers.nomad.exposedByDefault=false to Traefik args in iac/provider-gcp/nomad/jobs/ingress.hcl, aligning Nomad behavior with the Consul provider setting

Written by Cursor Bugbot for commit b9166b1. This will update automatically on new commits. Configure here.

Disabled option that was trying to scrape and map all services. This was
failing as we did not configured them in way they should be exposed and
it was spamming ingress logs.
@sitole sitole marked this pull request as ready for review January 7, 2026 11:56
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b9166b1b7e

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@sitole sitole enabled auto-merge (squash) January 7, 2026 11:59
@sitole sitole merged commit 6b5daf3 into main Jan 7, 2026
30 checks passed
@sitole sitole deleted the fix/ingress-disable-auto-config branch January 7, 2026 12:07
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.

4 participants