Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[mirror] Add mirror pull/push --tmp-dir flag #93

Merged
merged 3 commits into from
Feb 24, 2025
Merged

[mirror] Add mirror pull/push --tmp-dir flag #93

merged 3 commits into from
Feb 24, 2025

Conversation

borg-z
Copy link
Member

@borg-z borg-z commented Feb 21, 2025 •
edited
Loading

Ability to override the path to tmp-dir using --tmp-dir flag

bin/d8 mirror pull v1.67.0  --tmp-dir ~/bob  --license $TOKEN  --no-platform --no-security-db --include-module "stronghold@v1.1.5"  --deckhouse-tag v1.67.0
...
...
...

ls ~/bob/pull/1eff2ae0494b81219cebfb6e33fcb163/modules/stronghold 
blobs      index.json oci-layout release

@borg-z
d8 mirror pull/push --tmp-dir flag
235354e 
Signed-off-by: borg-z <me@nikolay-z.top>
@borg-z borg-z changed the title d8 mirror pull/push --tmp-dir flag [d8] Add mirror pull/push --tmp-dir flag Feb 21, 2025
@borg-z borg-z marked this pull request as ready for review February 21, 2025 10:57
@borg-z borg-z requested review from name212 and mvasl as code owners February 21, 2025 10:57
mvasl added 2 commits February 24, 2025 17:40
@mvasl
Do not change default tmpdir paths
5c17c8e 
Signed-off-by: Maxim Vasilenko <5184586+mvasl@users.noreply.github.com>
@mvasl
Do not change default tmpdir paths
e710aa9 
Signed-off-by: Maxim Vasilenko <5184586+mvasl@users.noreply.github.com>
@mvasl mvasl changed the title [d8] Add mirror pull/push --tmp-dir flag [mirror] Add mirror pull/push --tmp-dir flag Feb 24, 2025
@mvasl mvasl merged commit 4c456ba into main Feb 24, 2025
2 checks passed
@mvasl mvasl deleted the mirror-tmp-dir-flag branch February 24, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvasl mvasl mvasl approved these changes

@name212 name212 Awaiting requested review from name212 name212 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@borg-z @mvasl