Skip to content

Path rewrites - #475

Merged
zackbradys merged 19 commits into
mainfrom
path-rewrites
Jan 6, 2026
Merged

Path rewrites#475
zackbradys merged 19 commits into
mainfrom
path-rewrites

Conversation

@CamrynCarter

@CamrynCarter CamrynCarter commented Nov 7, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Please check below, if the PR fulfills these requirements:

  • Commit(s) and code follow the repositories guidelines.
  • Test(s) have been added or updated to support these change(s).
  • Doc(s) have been added or updated to support these change(s).

Associated Links:

Types of Changes:

Proposed Changes:

Verification/Testing of Changes:

Additional Context:

@americancode

americancode commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

@CamrynCarter @amartin120 Still looking for some form of path/name rewrite capability. #436

all that to say I'm excited to see this PR go in.

@CamrynCarter
CamrynCarter marked this pull request as ready for review December 10, 2025 16:50
@CamrynCarter CamrynCarter changed the title WIP - Path rewrites Path rewrites Dec 10, 2025
@CamrynCarter CamrynCarter added this to the Hauler v1.4.0 milestone Dec 10, 2025
@zackbradys
zackbradys requested a review from a team January 6, 2026 03:44
@zackbradys zackbradys added this to Hauler Jan 6, 2026
@github-project-automation github-project-automation Bot moved this to To Triage in Hauler Jan 6, 2026
@zackbradys zackbradys moved this from To Triage to Testing in Hauler Jan 6, 2026
@zackbradys zackbradys added enhancement New feature or request size/L Denotes an issue/PR requiring a relatively large amount of work labels Jan 6, 2026
@zackbradys zackbradys added the go Pull requests that update go code label Jan 6, 2026
amartin120
amartin120 approved these changes Jan 6, 2026
View reviewed changes

@amartin120 amartin120 left a comment

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.

This looks good. Thanks again for this.

@zackbradys
zackbradys merged commit 96bab7b into main Jan 6, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Testing to Resolved in Hauler Jan 6, 2026
@zackbradys
zackbradys deleted the path-rewrites branch January 6, 2026 16:49
This was referenced Jan 8, 2026
Closed
Closed
@chenrui333 chenrui333 mentioned this pull request Jan 13, 2026
Closed
@Shoelace Shoelace mentioned this pull request Jan 14, 2026
Open
atoy3731 pushed a commit to atoy3731/hauler that referenced this pull request Aug 4, 2026
@CamrynCarter @amartin120 @zackbradys
path rewrites (hauler-dev#475)
5a4ad72 
* image reference rewrite
* remove need for rewrite-provider
* handle charts
* handle leading slash and missing tags
* tests
* tool cleanup
* removed intermediate store cleanup
* fix?
* test cleanup
* clean up tools folder again
* debug test
* clear tempdir after each filename provided by load

Signed-off-by: Adam Martin <adam.martin@ranchergovernment.com>

* update tar command in load integration test

Signed-off-by: Adam Martin <adam.martin@ranchergovernment.com>

* clean up debug prints
* clean up debug prints

* fix typo

---------

Signed-off-by: Adam Martin <adam.martin@ranchergovernment.com>
Signed-off-by: Zack Brady <zackbrady123@gmail.com>
Co-authored-by: Adam Martin <adam.martin@ranchergovernment.com>
Co-authored-by: Zack Brady <zackbrady123@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amartin120 amartin120 amartin120 approved these changes

Assignees

@CamrynCarter CamrynCarter

Labels

enhancement New feature or request go Pull requests that update go code size/L Denotes an issue/PR requiring a relatively large amount of work

Projects

Hauler
Status: Resolved

Milestone

Hauler v1.4.0

Development

Successfully merging this pull request may close these issues.

4 participants

@CamrynCarter @americancode @amartin120 @zackbradys