Skip to content

How to add config/secrets mounts to preview env - #335

Open
liron-sel wants to merge 3 commits into
mainfrom
preview-env-config-secrets-mounts
Open

liron-sel wants to merge 3 commits into
mainfrom
preview-env-config-secrets-mounts

Conversation

@liron-sel

Copy link
Copy Markdown
Contributor

No description provided.

@liron-sel

Copy link
Copy Markdown
Contributor Author

@greptileai

@greptile-apps

greptile-apps Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR documents how preview environments can inject configuration and secret files without rebuilding an image, addressing the file-based override workflows described in the automatically discovered INT-496 and COR-1399 tickets.

  • Adds config_mounts examples for inline and local-file sources.
  • Adds a secret_mounts example and explains Kubernetes Secret storage.
  • Documents mount refresh and payload-size behavior.

Important Files Changed

Filename Overview
docs/use-cases/preview-environments.md Adds accurate preview mount guidance and examples, but the new full-config-reference link points to a nonexistent path.

Fix all with Greploop Fix All in Cursor Fix All in Codex Fix All in Claude Code

Reviews (1): Last reviewed commit: "small edit" | Re-trigger Greptile

Comment thread docs/use-cases/preview-environments.md Outdated
Files are read once at session creation and never refreshed. Re-run `mirrord preview start` with the same key to pick up changes.
{% endhint %}

Combined payload size across all mounts in a session is bound by Kubernetes' ~1 MiB per-object limit. See the [full config reference](../config/options md#feature-preview-config_mounts) for the complete field list.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Broken config reference link

The new ../config/options md#feature-preview-config_mounts target contains a space and points to a page that does not exist in the documentation tree, so readers cannot reach the promised configuration reference.

Suggested change
Combined payload size across all mounts in a session is bound by Kubernetes' ~1 MiB per-object limit. See the [full config reference](../config/options md#feature-preview-config_mounts) for the complete field list.
Combined payload size across all mounts in a session is bound by Kubernetes' ~1 MiB per-object limit. See the [full config reference](https://metalbear.com/mirrord/docs/config/options#feature-preview-config_mounts) for the complete field list.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Fix in Cursor Fix in Codex Fix in Claude Code


### Overriding Config and Secret Files

Testing a config change usually means rebuilding and pushing a new image just to change one file. `config_mounts` and `secret_mounts` override or inject individual files into the preview pod without touching the image, by adding them to the same mirrord configuration file used to start the preview.

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.

Nope. You use config mounts/secret mounts to override the spec/yaml, not the image. You might be able to override image things but that's not hte main use case.
"If you'd like to test your image with a change to the configmap or secrets, you can use the.."

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.

2 participants