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

Add ingress, catalogue relations to terraform modules #245

Open
sed-i opened this issue Dec 4, 2024 · 2 comments
Open

Add ingress, catalogue relations to terraform modules #245

sed-i opened this issue Dec 4, 2024 · 2 comments

Comments

@sed-i
Copy link
Contributor

sed-i commented Dec 4, 2024

We need to have mimir, tempo listed on the catalogue page.

Depends on:

This would involve adding ingress relations to mimir, tempo, as they are currently missing from the terraform module.
This may be a bit involved: after manually relating mimir to traefik, I am getting a 404 from nginx for all mimir endpoints.

References:

@sed-i sed-i changed the title Add catalogue relations to terraform modules Add ingress, catalogue relations to terraform modules Dec 4, 2024
@sed-i
Copy link
Contributor Author

sed-i commented Dec 6, 2024

This could have been happening because the /status path is already used by an nginx endpoint mapping.
We shouldn't shadow known workload paths with our custom ones.
The trouble is that /status may be a hard dep for the nginx exporter.

@simskij
Copy link
Member

simskij commented Dec 10, 2024

Why do we want to have tempo? It has no UI. Mimir potentially makes sense, but I'd honestly skip it. I'm sure I've asked for this at some point, but I think we can skip this.

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

No branches or pull requests

2 participants