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.

Sign up for GitHub

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

Jump to bottom

[SP3] Use the "norecovery" mount option (bsc#1195894) #179

Merged
merged 2 commits into from
Aug 4, 2022
Merged

[SP3] Use the "norecovery" mount option (bsc#1195894) #179

merged 2 commits into from
Aug 4, 2022

Conversation

lslezak
Copy link
Member

@lslezak lslezak commented Feb 15, 2022

Testing

  • Tested manually with Btrfs root partition

@lslezak
Use the "norecovery" mount option (bsc#1195894)
30a3209 
... when searching the root partitions

- 4.3.4
@coveralls
Copy link

coveralls commented Feb 15, 2022 •
edited
Loading

Coverage Status

Coverage increased (+1.0%) to 29.758% when pulling c016294 on norecovery_mount_SP3 into 4814c0e on SLE-15-SP3.

@lslezak lslezak mentioned this pull request Apr 28, 2022
Merged
@dgdavid
Copy link
Member

dgdavid commented Jul 25, 2022

@lslezak which is the status of this PR?

@lslezak
Copy link
Member Author

lslezak commented Aug 4, 2022

Finally got feedback from the customer, the fix works fine!

dgdavid
dgdavid approved these changes Aug 4, 2022
View reviewed changes
Copy link
Member

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@lslezak lslezak merged commit 9750991 into SLE-15-SP3 Aug 4, 2022
@lslezak lslezak deleted the norecovery_mount_SP3 branch August 4, 2022 08:47
@yast-bot
Copy link
Contributor

yast-bot commented Aug 4, 2022

✔️ Internal Jenkins job #3 successfully finished
✔️ Created IBS submit request #277017

@lslezak lslezak mentioned this pull request Aug 4, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgdavid dgdavid dgdavid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lslezak @coveralls @dgdavid @yast-bot