Alon Bar-Lev has posted comments on this change.
Change subject: storage: set block schedule elevator using udev
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File vdsm.spec.in
Line 477:
Line 478: #
Line 479: # Mark the existance of the rules before upgrade
Line 480: #
Line 481: rules="/lib/udev/rules.d/12-vdsm-elevator.rules"
There are two sequences we need to support:
vdsm-without-udev installed
vdsm-with-udev upgrade
result=no rule
vdsm-with-udev installed
vdsm-with-udev upgrade
result=rule of upgrade
So if udev rule exists before upgrade we keep it after upgrade.
If udev rule is missing before upgrade we remove it after upgrade.
I don't really understand your command, but facing the above, where do you see
a problem?
Line 482: if [ "$1" = 2 -a -f "${rules}" ]; then
Line 483: touch "${rules}.lock"
Line 484: fi
Line 485:
--
To view, visit http://gerrit.ovirt.org/8700
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I0a8de1c861bf4570509599b6f47235ed38cc424d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Fabian Deutsch <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
