Skip to content

imagecustomizer: Optimize MIC Overlay supporting multiple volumes.#8332

Merged
liulanze merged 2 commits into3.0-devfrom
user/lanzeliu/dracut-overlay
Mar 12, 2024
Merged

imagecustomizer: Optimize MIC Overlay supporting multiple volumes.#8332
liulanze merged 2 commits into3.0-devfrom
user/lanzeliu/dracut-overlay

Conversation

@liulanze
Copy link
Copy Markdown
Contributor

@liulanze liulanze commented Mar 11, 2024

Merge Checklist
  • Ready to merge

Summary

What does the PR accomplish, why was it needed?
Optimize the existing MIC Overlay API to support multiple volumes. Users can assign different persistent volumes to distinct Overlays. If no volumes are set, a volatile Overlay will be created.

Does this affect the toolchain?

NO

Associated issues
Test Methodology
  • Local build

@liulanze liulanze requested review from a team as code owners March 11, 2024 07:13
@liulanze liulanze self-assigned this Mar 11, 2024
Copy link
Copy Markdown
Contributor

@Adub17030MS Adub17030MS left a comment

Choose a reason for hiding this comment

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

Can you provide some more information in the PR description about how this was validated and tested?

Comment thread toolkit/tools/pkg/imagecustomizerlib/customizeoverlay.go Outdated
Copy link
Copy Markdown
Contributor

@cwize1 cwize1 left a comment

Choose a reason for hiding this comment

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

(See other comments.)

# Check if /etc/mdadm.conf exists.
if [ -f "/etc/mdadm.conf" ]; then
mdadm --assemble ${partition} || \
mdadm --assemble ${_volume} || \
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.

Not needed to fix here, but dracut should already be able to assemble mdadm raid on its own. If you add an entry to the init queue to wait for it I think that should work?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Mark here, thanks.

@liulanze liulanze merged commit 07227f0 into 3.0-dev Mar 12, 2024
@liulanze liulanze deleted the user/lanzeliu/dracut-overlay branch March 12, 2024 21:52
BettyRain pushed a commit that referenced this pull request Mar 13, 2024
…8332)

Co-authored-by: lanzeliu <lanzeliu@microsoft.com>
liulanze added a commit that referenced this pull request Mar 21, 2024
…8332)

Co-authored-by: lanzeliu <lanzeliu@microsoft.com>
liulanze added a commit that referenced this pull request Apr 2, 2024
…8332)

Co-authored-by: lanzeliu <lanzeliu@microsoft.com>
Xiaohong-Deng pushed a commit to Xiaohong-Deng/azurelinux that referenced this pull request Nov 23, 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.

4 participants