Skip to content

feat: support previewConfig.target.path#271

Merged
christiansiegel merged 7 commits intobaloise:masterfrom
lintermansjens:feat/previewconfig-target-path-support
Apr 13, 2026
Merged

feat: support previewConfig.target.path#271
christiansiegel merged 7 commits intobaloise:masterfrom
lintermansjens:feat/previewconfig-target-path-support

Conversation

@lintermansjens
Copy link
Copy Markdown
Contributor

@lintermansjens lintermansjens commented Apr 8, 2026

Summary

This PR adds support for storing preview environments under a configurable subpath (previewConfig.target.path) instead of only in the deployment repo root.

Why

Some deployment repos keep manifests in certain folders.
Before this change, preview commands only worked with root-level folders.

What changed

Added previewConfig.target.path parsing and validation in GitOps config.
Added path resolution logic to combine configured target path + generated preview namespace

Updated:

  • create-preview to create/update files at the resolved path
  • delete-preview to delete previews at the resolved path
  • Improved logs to show preview namespace and resolved folder path.

Added tests for:

  • default root behavior
  • path parsing/validation (v1/v2)
  • create/delete preview with configured target path

Updated docs:

  • preview configuration now documents target.path
  • getting-started command list reflects current CLI commands

Compatibility

Default behavior is unchanged: if previewConfig.target.path is not set, previews are still created in the repository root.

Copy link
Copy Markdown
Member

@christiansiegel christiansiegel left a comment

Choose a reason for hiding this comment

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

Thanks for contributing!
Left a few minor comments regarding log and error outputs. Otherwise LGTM

lintermansjens and others added 4 commits April 13, 2026 08:25
Co-authored-by: Christian Siegel <christian.siegel@gmx.net>
Co-authored-by: Christian Siegel <christian.siegel@gmx.net>
Co-authored-by: Christian Siegel <christian.siegel@gmx.net>
Co-authored-by: Christian Siegel <christian.siegel@gmx.net>
@lintermansjens
Copy link
Copy Markdown
Contributor Author

Thank you for reviewing @christiansiegel! I have accepted your suggestions, please verify them when you have time.

@christiansiegel christiansiegel merged commit e8288a4 into baloise:master Apr 13, 2026
1 check passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 5.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants