Updates for CentOS 7 #249
Closed
Updates for CentOS 7 #249
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
Newer versions of udev do not allow block devices to be renamed so move the naming into the kernel, and correspondingly update some paths. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Add some required parameters to allow reinitializing PVs on CentOS 7. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
force-start doesn't exist with systemd. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Fix scsi_id path. Fix pid file path. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
…ultipath Instead of patching dm-multipath, parse its verbose output. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
This is not needed since newer versions of multipath-tools are integrated differently with udev. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Disable CentOS 7's default rules because SM provides its own rules. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
The /dev/root symlink doesn't exist on CentOS 7 so just get the device number from /. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
The /dev/root symlink doesn't exist on CentOS 7 so do it like dracut. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
udev on CentOS 7 kills all processes started from a RUN rule after it finishes so don't start cdrommon from local-device-change because it is called from a udev rule. Instead, an instantiated systemd cdrommon service is started from the udev rule. Signed-off-by: Ross Lagerwall <ross.lagerwall@citrix.com>
|
Hi Ross, Are these patches exactly the same we have been testing internally? |
|
Yes. |
|
Ross, everything is ok but I have only one concern about The rules names 40-multipath was named this way to override the rule with the same name |
|
Acked-by: Germano Percossi germano.percossi@citrix.com |
andrey-podko
pushed a commit
to andrey-podko/sm
that referenced
this pull request
Aug 16, 2022
GitHub: closes xapi-project#249 on xapi-project/sm
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.
No description provided.