Skip to content

feat(base-cluster/monitoring)!: migrate promtail to alloy#1347

Merged
cwrau merged 4 commits intomainfrom
feat/base-cluster/loki-migration
Mar 6, 2025
Merged

feat(base-cluster/monitoring)!: migrate promtail to alloy#1347
cwrau merged 4 commits intomainfrom
feat/base-cluster/loki-migration

Conversation

@cwrau
Copy link
Copy Markdown
Member

@cwrau cwrau commented Feb 20, 2025

  • update loki to an up-to-date version
  • also replace otel collector with alloy, saving resources!

@teutonet-bot
Copy link
Copy Markdown
Contributor

teutonet-bot commented Feb 20, 2025

🤖 I have diffed this beep boop

"/$namespace/$kind/$name.yaml" for normal resources
"/$namespace/HelmRelease/$name/$namespace/$kind/$name.yaml" for HelmReleases <- this is recursive
'null' means it's either cluster-scoped or it's in the default namespace for the HelmRelease

charts/base-cluster/ci/monitoring-ingress-unauthenticated-values.yaml

charts/base-cluster/ci/rbac-values.yaml

charts/base-cluster/ci/pagerduty-values.yaml

charts/base-cluster/ci/velero-backupStorageLocations-gen-values.yaml

charts/base-cluster/ci/flux-gitrepositories-gen-values.yaml

charts/base-cluster/ci/imagepullsecrets-values.yaml

charts/base-cluster/values.yaml

charts/base-cluster/ci/artifacthub-values.yaml

charts/base-cluster/ci/disabled-ingress-values.yaml

charts/base-cluster/ci/priorityclasses-values.yaml

[charts/base-cluster/ci/deadmansswitch-values.yaml](

<title> Error </title> <script src="https://unpkg.com/htmx.org@2.0.2" integrity="sha384-Y7hw+L/jvKeWIRRkqWYfPcvVxHzVzn5REgzbawhxAuQGwX1XWe70vji+VSeHOThJ" crossorigin="anonymous"></script> <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.7.1/jquery.min.js"></script> <script defer src="https://media.ethicalads.io/media/client/ethicalads.min.js"></script> <script> function key(event, letter) { return (event.charCode == letter.charCodeAt() || event.charCode == letter.charCodeAt() + 32) } </script> <script>
    </script>
    <style></style>
    
    
</head>

<body hx-on:keydown="var path=window.location.pathname; if ((event.key=='n' || event.key=='N') && path && path !== '/' && !path.includes('/duplicate/') && !path.includes('/dashboard/') && !path.includes('/accounts/') && !path.includes('/properties/')) { window.location.href = '/'; }">
    <div id="container">
        
        <div class="topbuttons">
            
                <a href="/accounts/login/"
                   title="Personal dashboard, preferences, API token"
                   class="button"><b>Sign in</b></a>
            
            <a href="/" title="Create a new paste (shortcut: 'N')" class=" button">New</a>
            <a href="/api/v2/"
               title="Paste creation API, with code samples"
               class=" activebutton  button">API</a>
            <a href="/help/"
               title="Usage tips, shortcuts"
               class="  button">Help</a>
            <a href="/about/"
               title="Updates, stats, backstory"
               class=" button">About</a>
        </div>
        
<div class="error">
    <h3>Sorry!</h3>
    Sorry, maximum content size is 1,000,000 characters. Your submission is 2,539,997. Your IP will be blocked if you exceed this limit again in the next 60 seconds.
    <hr>
    <p>
        Something unexpected? Please create a <a href="https://dpaste.freshdesk.com/support/tickets/new">support ticket</a>.
    </p>
</div>
<p>
    <a class="button" href="{$ url 'create' %}" onclick="history.back();return false;">Go back</a>
</p>

    </div>
    
    
    <script>
        window.fwSettings={'widget_id':22000000180 };
        !function(){if("function"!=typeof window.FreshworksWidget){var n=function(){n.q.push(arguments)};n.q=[],window.FreshworksWidget=n}}();
    </script>
    <script defer src='https://widget.freshworks.com/widgets/22000000180.js'></script>
</body>
)

charts/base-cluster/ci/monitoring-oidc-values.yaml

charts/base-cluster/ci/basic-values.yaml

charts/base-cluster/ci/monitoring-oidc-ingress-disabled-values.yaml

charts/base-cluster/ci/limitrange-resourcequota-values.yaml

@marvinWolff
Copy link
Copy Markdown
Collaborator

CI fails.

@cwrau cwrau force-pushed the feat/base-cluster/loki-migration branch from 08db455 to 52cba7b Compare February 25, 2025 13:10
Comment thread charts/base-cluster/templates/monitoring/loki/dashboard.yaml
@tasches
Copy link
Copy Markdown
Collaborator

tasches commented Feb 25, 2025

The scaling option for loki is dropped so I guess this is a breaking change right?

@cwrau
Copy link
Copy Markdown
Member Author

cwrau commented Feb 25, 2025

The scaling option for loki is dropped so I guess this is a breaking change right?

That's not really a breaking change. But this PR is a breaking change in any case.

@cwrau cwrau force-pushed the feat/base-cluster/loki-migration branch from 36d0b29 to 464d174 Compare February 25, 2025 15:33
@cwrau cwrau requested a review from tasches February 26, 2025 12:53
@cwrau cwrau force-pushed the feat/base-cluster/loki-migration branch 3 times, most recently from 63c6c82 to 1680c4d Compare March 5, 2025 14:37
cwrau added 3 commits March 6, 2025 14:35
- update loki to an up-to-date version
- also replace otel collector with alloy, saving resources!

this deletes the old setup, beware of lost logs
- disable unneeded features
- trust configmap-reload image for alloy
@cwrau cwrau force-pushed the feat/base-cluster/loki-migration branch from 1680c4d to 0393ed7 Compare March 6, 2025 13:35
@cwrau cwrau changed the title feat(base-cluster/monitoring): migrate promtail to alloy feat(base-cluster/monitoring)!: migrate promtail to alloy Mar 6, 2025
@cwrau cwrau merged commit 24db445 into main Mar 6, 2025
@cwrau cwrau deleted the feat/base-cluster/loki-migration branch March 6, 2025 14:53
github-merge-queue Bot pushed a commit that referenced this pull request May 27, 2025
🤖 I have created a release *beep* *boop*
---


##
[8.0.0](base-cluster-v7.2.1...base-cluster-v8.0.0)
(2025-05-27)


### ⚠ BREAKING CHANGES

* **base-cluster/ingress:** add option traefik for ingress controller
and make it default
([#1420](#1420))
* **base-cluster/monitoring:** migrate promtail to alloy
([#1347](#1347))

### Features

* **base-cluster/ingress-nginx:** use risk-level Critical when
annotations are enabled
([#1417](#1417))
([a9d8ef2](a9d8ef2))
* **base-cluster/ingress:** add option traefik for ingress controller
and make it default
([#1420](#1420))
([f62b197](f62b197))
* **base-cluster/ingress:** rename oauth-proxies to have a clean name
([#1434](#1434))
([27a28d5](27a28d5))
* **base-cluster/monitoring:** migrate promtail to alloy
([#1347](#1347))
([24db445](24db445))
* **base-cluster/monitoring:** rename alloy to be a generic name
([#1433](#1433))
([3f5826a](3f5826a))


### Bug Fixes

* **base-cluster/cert-manager:** metrics collection
([#1397](#1397))
([71e1189](71e1189))
* **base-cluster/rbac:** *RoleBindings should always be prefixed to
avoid collision
([#1484](#1484))
([75de246](75de246))


### Miscellaneous Chores

* **base-cluster/monitoring:** remove deprecated plugin
([#1478](#1478))
([ee22df5](ee22df5))
* **base-cluster:** formatting
([#1424](#1424))
([853f146](853f146))
* **base-cluster:** pin all versions
([#1447](#1447))
([ec8a430](ec8a430))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **New Features**
- Added support for the Traefik ingress controller, now recommended and
available as an alternative to NGINX.
- Updated ingress configuration to use a new provider option, allowing
selection between "nginx", "traefik", or "none".
- Expanded documentation with detailed migration guides and ingress
controller comparisons.

- **Breaking Changes**
- Traefik is now the default ingress controller; configuration for
disabling ingress has changed.
- Monitoring stack migrated from promtail to alloy; Loki deployment
updated and namespace changed.
  - Deprecated properties and configuration options removed or replaced.

- **Bug Fixes**
  - Improved metrics collection in cert-manager.
  - Resolved RBAC role binding name collisions.

- **Chores**
  - Updated container image versions and removed deprecated plugins.
- Reformatted and reorganized chart metadata and documentation for
clarity.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants