Skip to content

Conversation

oliverbeck-dev
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Feb 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ansible-build ❌ Failed (Inspect) May 7, 2025 0:49am

Copy link
Contributor

Choose a reason for hiding this comment

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

I would like to keep the pattern with the exporter attribute, even though it looks very complicated for defining one hostname.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had real issues with that, the cert_bot failed while deploying

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@janikvonrotz I've fixed the other issues above by moving everything in vars/inventory

I still would like to recommend the change below, because the certbot failed with that. maybe you can reproduce it

Copy link
Contributor

Choose a reason for hiding this comment

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

@oliverbeck-dev Not sure how certbot is involved here.

The exporter: uptime-kuma proxyies entry does not have any functionality. It is just a label for the proxy entry.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok I wasn't able to reproduce it with ansible-play prometheus.yml. I just remember that when I used the ansible-play certbot (for the new subdomain), it crashed because of that.

let's remove it for now

@oliverbeck-dev oliverbeck-dev changed the title add dedicated nginx proxy for loki if you want to use it with a dedicated domain add nginx proxy for loki if you want to use it with a dedicated domain Feb 12, 2025
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