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

Setup logging on CAPZ installations #2942

Closed
4 of 5 tasks
Tracked by #2951 ...
TheoBrigitte opened this issue Nov 6, 2023 · 12 comments
Closed
4 of 5 tasks
Tracked by #2951 ...

Setup logging on CAPZ installations #2942

TheoBrigitte opened this issue Nov 6, 2023 · 12 comments
Assignees
Labels
epic/logging provider/cluster-api-azure Cluster API based running on Azure team/atlas Team Atlas

Comments

@TheoBrigitte
Copy link
Member

TheoBrigitte commented Nov 6, 2023

Towards: #311

In order to expand our logging infrastructure coverage we should extend support to CAPZ installations.

As for object storage Loki does support Azure blob storage.

Tasks

Preview Give feedback
  1. epic/logging provider/cluster-api-azure team/atlas
    marieroque
@TheoBrigitte TheoBrigitte added team/atlas Team Atlas epic/logging provider/cluster-api-azure Cluster API based running on Azure labels Nov 6, 2023
@marieroque marieroque self-assigned this Nov 15, 2023
@TheoBrigitte TheoBrigitte added needs/refinement Needs refinement in order to be actionable and removed needs/refinement Needs refinement in order to be actionable labels Nov 21, 2023
@hervenicol hervenicol self-assigned this Dec 4, 2023
@hervenicol
Copy link

🟢 Loki config should be ready (https://github.com/giantswarm/config/pull/2114)
🟢 it has been tested with a dev version of object-storage-operator (giantswarm/object-storage-operator#46)
🟠 unfortunately we don't receive promtail logs, looks like it can't resolve the ingress address

@marieroque
Copy link

marieroque commented Dec 20, 2023

My tests on glippy:

Manual steps to do first:

  • enable logging on loki-app (loki-user-values configmap)
  • enable logging on logging-operator (logging-operator-user-values configmap)

Then I encountered an issue with loki ingress. Loki DNS is not created.
Current issue on CAPZ:
https://github.com/giantswarm/giantswarm/issues/28848

After annotating loki-gateway ingress with giantswarm.io/external-dns: managed, loki DNS has been created and promtail on WC could sent data to loki:

Image

@marieroque
Copy link

marieroque commented Dec 22, 2023

Logging enabled on CAPZ giantswarm testing installations only:

@marieroque
Copy link

marieroque commented Dec 22, 2023

To have a grafana-agent working well on CAPZ WC, we need to edit the daemon set and change the listen-port from 80 to 8080.
That issue is fixed here https://github.com/giantswarm/giantswarm/issues/29368

=> It will be fixed in the new observability-bundle release.

@QuentinBisson
Copy link

Is it now fixed @marieroque ? Do we have logging on capz?

@QuentinBisson
Copy link

@marieroque
Copy link

✅ Tested on a WC on glippy

Image

@marieroque
Copy link

marieroque commented Jan 12, 2024

🚀 Logging is working on CAPZ with:

  • object-storage-operator 0.4.3
  • logging-operator 0.3.1
  • observability-bundle 1.0.0

⚠ Before enable logging on customer side (Continental/Wepa), wait for customer WC with o11y-bundle 1.0.0 as it contains a fix on grafana-agent

@marieroque
Copy link

Discussion about CAPZ upgrade: https://gigantic.slack.com/archives/C02HLSDH3DZ/p1705400551622879

@QuentinBisson
Copy link

Putting it back to backlog

@QuentinBisson
Copy link

QuentinBisson commented Feb 27, 2024

All capz MCs run 1.25, let's en able logging on all capz mcs

@QuentinBisson
Copy link

Logging is enabled on capz

@github-project-automation github-project-automation bot moved this from Inbox 📥 to Done ✅ in Roadmap Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic/logging provider/cluster-api-azure Cluster API based running on Azure team/atlas Team Atlas
Projects
Archived in project
Development

No branches or pull requests

4 participants