-
Notifications
You must be signed in to change notification settings - Fork 5
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
lxd-imagebuilder: Improve win repack (from Incus) #86
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
And make the usage of drivers, windows-version, windows-arch understandable Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit f86cad1afeb501f8294e6ed8a3bec058e60b5c4a) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
The option `--exit-on-error' exists only on debian run-parts version. On centos, redhat or federa, run-parts has no such option. The fix checks version and adds options accordingly. Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit b7240053d8f4db44c94e455f895302524ccb5411) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
And make genisoimage progress output better Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit 5851fd9a1e1ea4070c30f466e5859b45503fc256) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
And refactor find windows directories logic, and fixes the potenial bug if parent directory not found Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit 1c98c9207061dea202706807860a460678a4e2b8) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
And detect windows version via install wim info, and inject drivers to winpe image. So if booting winpe (say, via iPXE and wimboot), the network, input and iSCSI can work correctly. Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit 571596cc456c5bc829860152fe398b86d9ef6ff9) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
… logic Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit b237c6c69d607373365531d06e6a44d842529a39)
Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit f6722e3de6b25deb494b6bf62eaf91c61cb755aa) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit 14721a319c7b04b89335624a039cc08458fc061f) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit b632188dcc95207431cdbf369e602eb669fb16fe) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
Driver like vioinput has multiple sys files `viohidkmdf.sys` and `vioinput.sys`, without `vioinput.sys` copied, the vioinput driver not work correctly. The fix add the multiple files copied back. Signed-off-by: JUN JIE NAN <[email protected]> (cherry picked from commit 44f0a4f35072aeada023fa5bd3a0094d51fce055) Signed-off-by: Din Music <[email protected]> License: Apache-2.0
Signed-off-by: Din Music <[email protected]>
I've tested with Win 11 ISO and this works. But should be rebased after #87 |
simondeziel
reviewed
Aug 7, 2024
Signed-off-by: Din Music <[email protected]>
…iption Signed-off-by: Din Music <[email protected]>
Signed-off-by: Din Music <[email protected]>
simondeziel
approved these changes
Aug 7, 2024
tomponline
changed the title
lxd-imagebuilder: Improve win repack
lxd-imagebuilder: Improve win repack (from Incus)
Aug 8, 2024
tomponline
approved these changes
Aug 8, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cherry-picked from: