Skip to content

yupdate - allow patching the D-Installer#1072

Merged
lslezak merged 5 commits intomasterfrom
d-installer_support
Jan 9, 2023
Merged

yupdate - allow patching the D-Installer#1072
lslezak merged 5 commits intomasterfrom
d-installer_support

Conversation

@lslezak
Copy link
Member

@lslezak lslezak commented Jan 5, 2023

Problem

The current yupdate script can be only used from inst-sys for patching the standard YaST modules. It would be nice to extend it so it could be used the same way with the D-Installer. This would make testing and development of new features easier.

Solution

Adapt the script so it can work in a different environment (live medium).

  • Check for the live medium environment (the root uses the overlay file system)
  • Do not try to modify /etc/install.inf if it does not exist (it is not present on the live medium)
  • Add pre and post hooks so some actions can be done before patching the system (installing some RPM packages) or after (restarting the DBus service). It runs the .yupdate.pre and .yupdate.post scripts when present in the source code.

Testing

  • Updated unit test
  • Tested manually

@coveralls
Copy link

coveralls commented Jan 5, 2023

Coverage Status

Coverage: 40.77% (-0.02%) from 40.786% when pulling a9e06fb on d-installer_support into 88bc3ba on master.

@lslezak lslezak marked this pull request as ready for review January 6, 2023 13:04
@lslezak lslezak merged commit ece31be into master Jan 9, 2023
@lslezak lslezak deleted the d-installer_support branch January 9, 2023 09:53
@yast-bot
Copy link
Contributor

yast-bot commented Jan 9, 2023

❌ Public Jenkins job #244 failed

@yast-bot
Copy link
Contributor

yast-bot commented Jan 9, 2023

✔️ Internal Jenkins job #142 successfully finished
✔️ Created IBS submit request #287788

@yast-bot
Copy link
Contributor

yast-bot commented Jan 9, 2023

❌ Public Jenkins job #245 failed

@yast-bot
Copy link
Contributor

yast-bot commented Jan 9, 2023

✔️ Public Jenkins job #246 successfully finished
✔️ Created OBS submit request #1057021

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