Skip to content

object store integration in plural up - #4576

Merged
zreigz merged 2 commits into
masterfrom
lukasz/prod-5293-object-store-integration-in-plural-up
Oct 26, 2026
Merged

zreigz merged 2 commits into
masterfrom
lukasz/prod-5293-object-store-integration-in-plural-up

Conversation

@zreigz

@zreigz zreigz commented Oct 23, 2026

Copy link
Copy Markdown
Member

Test Plan

Test environment: https://console.your-env.onplural.sh/

Checklist

  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have deployed the agent to a test environment and verified that it works as expected (required only when changing agent code).

Plural Flow: console

@zreigz zreigz added the enhancement New feature or request label Oct 23, 2026
@linear

linear Bot commented Oct 23, 2026

Copy link
Copy Markdown

PROD-5293

@soffi-ai

soffi-ai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Soffi AI Summary

This PR adds object store configuration support to the plural up deployment templates, enabling the Console to be configured with an external object storage backend (S3, Azure Blob Storage, or GCS) during initial setup. Previously, the Helm values templates generated by plural up had no mechanism to wire in object store credentials and bucket references. The changes add conditional blocks to both the Liquid template (used for Plural-managed environments) and the Go template (used for self-hosted/Terraform flows) so that whichever cloud provider's object store is detected via imported values, the correct console.config.objectStore Helm values are emitted automatically.

Changes

Object store integration in plural up

  • Added conditional S3 and initial object store configuration blocks to both the Liquid and Go Helm values templates, so that plural up can automatically inject the correct console.config.objectStore provider settings when an S3 bucket is detected in the imported management cluster values. (8381fe1)
  • Extended the object store template blocks to also handle Azure Blob Storage and GCS as provider options, completing multi-cloud coverage. Each provider's block conditionally includes optional fields (Azure access key, GCS credentials JSON) only when those values are present. (61cc752)

Updated: 2026-10-23 14:18 UTC

Deploy in Soffi

@greptile-apps

greptile-apps Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable defects were found.

Summary

Adds optional S3 settings to both console setup templates.

  • Console deployments use configured S3 storage for large files.

No actionable issues were found.

Reviews (1) · Last reviewed commit: "object store integration in plural up" · Reviewed by Greptile

@michaeljguarino michaeljguarino left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we will need this to support other object stores/providers eventually too

@zreigz
zreigz force-pushed the lukasz/prod-5293-object-store-integration-in-plural-up branch 2 times, most recently from 29ecf2f to 4cbbbba Compare October 23, 2026 14:16
Allow gitops liquid and day-0 values to select azure or gcs when
mgmt stack outputs (or control-plane-values flags) provide them,
matching the chart's multi-provider objectStore shape.
@zreigz
zreigz force-pushed the lukasz/prod-5293-object-store-integration-in-plural-up branch from 4cbbbba to 61cc752 Compare October 23, 2026 14:17
@zreigz
zreigz merged commit fecd7db into master Oct 26, 2026
18 of 19 checks passed
@zreigz
zreigz deleted the lukasz/prod-5293-object-store-integration-in-plural-up branch October 26, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants