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.

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

mount: merge mounting code #1163

Merged
merged 1 commit into from
Jan 22, 2025
Merged

mount: merge mounting code #1163

merged 1 commit into from
Jan 22, 2025

Conversation

jmxnzo
Copy link
Contributor

@jmxnzo jmxnzo commented Jan 17, 2025

This PR merges the shared code of mounting logic between the coordinator and initializer after #1153 to internal/mount. In addition it adds some more documentation to mount.go and unit testing for the blkid parsing logic.

@jmxnzo jmxnzo added the changelog PRs that should be part of the release notes label Jan 17, 2025
@jmxnzo jmxnzo force-pushed the initializer/cryptsetup-integration-jla branch from f28e6c8 to 957cc7f Compare January 20, 2025 09:17
@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch from 38e81f2 to c0ba201 Compare January 20, 2025 09:19
@jmxnzo jmxnzo force-pushed the initializer/cryptsetup-integration-jla branch 3 times, most recently from 1bbc7e2 to a6c11ec Compare January 21, 2025 12:17
@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch from c0ba201 to fce3c71 Compare January 21, 2025 12:36
@jmxnzo jmxnzo changed the base branch from initializer/cryptsetup-integration-jla to main January 21, 2025 12:36
Copy link

PR Preview Action v1.6.0

🚀 View preview at
https://edgelesssys.github.io/contrast/pr-preview/pr-1163/

Built to branch gh-pages at 2025-01-21 12:38 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@jmxnzo jmxnzo marked this pull request as ready for review January 21, 2025 12:47
@jmxnzo jmxnzo requested a review from burgerdev as a code owner January 21, 2025 12:47
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch 3 times, most recently from 134dcd7 to c728ff7 Compare January 21, 2025 15:49
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount.go Outdated Show resolved Hide resolved
internal/mount/mount_test.go Outdated Show resolved Hide resolved
internal/mount/mount_test.go Outdated Show resolved Hide resolved
@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch from c607234 to 1fd8541 Compare January 22, 2025 08:09
@jmxnzo jmxnzo requested a review from burgerdev January 22, 2025 08:14
@burgerdev
Copy link
Contributor

Remember to squash, and please drop the leading internal/ from the commit message.

@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch 2 times, most recently from 0ca6bdf to bf41aa8 Compare January 22, 2025 09:51
@jmxnzo jmxnzo changed the title internal/mount: merge mounting code mount: merge mounting code Jan 22, 2025
@jmxnzo jmxnzo force-pushed the internal/mount-refactoring-jla branch from bf41aa8 to f752f49 Compare January 22, 2025 10:22
@jmxnzo jmxnzo merged commit 78d3219 into main Jan 22, 2025
10 checks passed
@jmxnzo jmxnzo deleted the internal/mount-refactoring-jla branch January 22, 2025 10:42
@jmxnzo jmxnzo added no changelog PRs not listed in the release notes and removed changelog PRs that should be part of the release notes labels Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants