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.

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

CP-12200: Files beloging to SM moved from dom0 #262

Closed
wants to merge 1 commit into from

Conversation

martinjorge
Copy link
Contributor

Removed files from dom0 to sm. The changes are as follows:

- dom0.hg/Makefile: Moved lvm.conf and rules file related instructions: Moved to sm/mk/sm.spec.in
- dom0.hg/binarty-overlay.dom0/etc/udev/rules.d/58-xapi.rules: Moved this file to sm/etc/udev along with all the multipath folder.
- dom0.hg/overlay.dom0/etc/modprobe.d/cifs.conf: Moved to /etc/modprobe.d/cifs.conf

Signed-off-by: Jorge Martin jorge.martin@citrix.com

@germanop
Copy link
Contributor

First line of commit message is a like a title: no periods, no multiple sentences, surely no longer than 72 chars (if 50 usually better).
Find a more concise title

Following block starts indented: it is not the continuation of the first line, it is just a more detailed description.
And, again, no lines bigger than 72 chars. You can add a newline and continue if it does not fit in 72 chars.


%config /etc/modprobe.d/cifs.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason why it is detached from the rest?

@martinjorge martinjorge force-pushed the CP-12200 branch 2 times, most recently from 18d742d to 8769d1b Compare June 16, 2015 20:03
@martinjorge martinjorge changed the title CP-12200: Package datapath related dom0 files CP-12200: Files beloging to SM moved from dom0 Jun 16, 2015
Moved files and modified Makefile from dom0 that belong to sm.
-dom0.hg/Makefile: lvm.conf and rules instructions -> sm/mk/spec.in and sm/Makefile
-dom0.hg/.../58-xapi.rules -> sm/etc/udev + rules files from sm/multipath
-dom0.hg/.../cifs.conf -> sm/etc/modprobe.d/cifs.conf

Signed-off-by: Jorge Martin <jorge.martin@citrix.com>
@germanop
Copy link
Contributor

Reviewed-by: Germano Percossi germano.percossi@citrix.com

@germanop germanop closed this in 3a979c5 Sep 25, 2015
andrey-podko pushed a commit to andrey-podko/sm that referenced this pull request Aug 16, 2022
Moved files and modified Makefile from dom0 that belong to sm.
-dom0.hg/Makefile: lvm.conf and rules instructions -> sm/mk/spec.in and sm/Makefile
-dom0.hg/.../58-xapi.rules -> sm/etc/udev + rules files from sm/multipath
-dom0.hg/.../cifs.conf -> sm/etc/modprobe.d/cifs.conf

Reviewed-by: Germano Percossi <germano.percossi@citrix.com>

GitHub: closes xapi-project#262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants