Merge ~ltrager/curtin:lp1895067 into curtin:master
Proposed by
Lee Trager
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Chad Smith | ||||
Approved revision: | 840b8ae4e3743c8b0f652f5bb207d9e9082d43e8 | ||||
Merge reported by: | Server Team CI bot | ||||
Merged at revision: | not available | ||||
Proposed branch: | ~ltrager/curtin:lp1895067 | ||||
Merge into: | curtin:master | ||||
Diff against target: |
81 lines (+54/-6) 2 files modified
curtin/commands/install_grub.py (+13/-6) tests/unittests/test_commands_install_grub.py (+41/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chad Smith | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Ryan Harper (community) | Approve | ||
Review via email: mp+390517@code.launchpad.net |
Commit message
Don't install grub if it is already found on CentOS/RHEL
LP: #1895067
To post a comment you must log in.
PASSED: Continuous integration, rev:a375fbde51a f4069b312c46a9d 28dfc1708bbc0d /jenkins. ubuntu. com/server/ job/curtin- ci/183/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 183/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 183/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 183/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 183/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/183/ /rebuild
https:/