Skip to content

fix: support Loki without object storage#2882

Merged
j-zimnowoda merged 29 commits into
mainfrom
APL-1498
Feb 3, 2026
Merged

fix: support Loki without object storage#2882
j-zimnowoda merged 29 commits into
mainfrom
APL-1498

Conversation

@merll
Copy link
Copy Markdown
Collaborator

@merll merll commented Jan 28, 2026

📌 Summary

Loki can run without object storage by using local volumes. It then needs to run in "SingleBinary" mode with one instance. This PR implements this, when object storage is not set up. It also removes values which were not actually functional, as they assumed distributed mode would also work with local storage. The latter does not seem to be supported.

🔍 Reviewer Notes

With this change, functionality with object storage should not change. In order to test the different deployment mode, set the platform's object storage provider to "Disabled".

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

@linode linode deleted a comment from svcAPLBot Jan 29, 2026
@linode linode deleted a comment from svcAPLBot Jan 29, 2026
@linode linode deleted a comment from svcAPLBot Jan 29, 2026
@svcAPLBot
Copy link
Copy Markdown
Contributor

Comparison of Helm chart templating output:



data.config.yaml
  ± value change in multiline text (one insert, one deletion)
      
      auth_enabled: true
      bloom_build:
        builder:
      
      [five lines unchanged)]
      
        enabled: false
      common:
        compactor_grpc_address: 'loki-compactor.monitoring.svc.cluster.local:9095'
        path_prefix: /var/loki
    -   replication_factor: 3
    +   replication_factor: 1
        storage:
          s3:
            backoff_config:
              max_period: 5s
      
      [101 lines unchanged)]
      
            server_address: dns+loki-index-gateway-headless.monitoring.svc.cluster.local:9095
        use_thanos_objstore: false
      tracing:
        enabled: false
  
  








data.VERSIONS
  ± value change in multiline text (one insert, one deletion)
      
    - {"api":"main","aplCharts":"main","console":"main","consoleLogin":"main","core":"main","specVersion":48,"tasks":"main","tools":"main"}
    + {"api":"main","aplCharts":"main","console":"main","consoleLogin":"main","core":"main","specVersion":50,"tasks":"main","tools":"main"}
  
  




apps.loki
  - one map entry removed:
    persistence:
      compactor:
        size: 10Gi
      ingester:
        size: 20Gi
      querier:
        size: 10Gi

versions.specVersion
  ± value change
    - 48
    + 50

@linode linode deleted a comment from svcAPLBot Jan 30, 2026
Copy link
Copy Markdown
Contributor

@ElderMatt ElderMatt left a comment

Choose a reason for hiding this comment

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

Tested on cluster, deployed Loki without object storage enabled. Loki is working and running in singlebinary

Image

Copy link
Copy Markdown
Contributor

@Ani1357 Ani1357 left a comment

Choose a reason for hiding this comment

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

Tested by deploying a cluster with obj sorage disabled.

@j-zimnowoda j-zimnowoda merged commit 1461593 into main Feb 3, 2026
19 checks passed
@j-zimnowoda j-zimnowoda deleted the APL-1498 branch February 3, 2026 08:54
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.

5 participants