Skip to content

Adds support for <1Gi PVCs and implements signal handling#303

Merged
BeryJu merged 4 commits intoBeryJu:mainfrom
rdmcguire:main
Oct 11, 2024
Merged

Adds support for <1Gi PVCs and implements signal handling#303
BeryJu merged 4 commits intoBeryJu:mainfrom
rdmcguire:main

Conversation

@rdmcguire
Copy link
Contributor

A copy timeout flag is added to set a custom value and support for
volumes <1G is added. Previously if you had a pvc that was, say, 512Mi,
the duration would be calculated as 0 * time.Second.

A parent context is created for the root command that will be done
on os.Kill and os.Interrupt to make the tool more polite and improve
cleanup. It is propagated into BaseStrategy and Migrator to be used with client-go
calls.

A few minor improvements to quiet gopls analyses and warnings, moved
to an opts struct for strategies.NewBaseStrategy, and fixed spurious io.EOF
warning logs.

Upgrades go and pkg deps.

A copy timeout flag is added to set a custom value and support for
volumes <1G is added. Previously if you had a pvc that was, say, 512Mi,
the duration would be calculated as 0 * time.Second.

A parent context is created for the root command that will be done
on os.Kill and os.Interrupt to make the tool more polite and improve
cleanup. It is propagated into BaseStrategy to be used with client-go
calls.

A few minor improvements to quiet gopls analyses and warnings, moved
to an opts struct for strategies.NewBaseStrategy.
Copy link
Owner

@BeryJu BeryJu left a comment

Choose a reason for hiding this comment

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

Thank you!

@BeryJu BeryJu merged commit 4df1102 into BeryJu:main Oct 11, 2024
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