[ovs-dev] [PATCH] rhel, xenserver: Create /var/log/openvswitch directory.

Ben Pfaff blp at nicira.com
Sat Jul 13 17:53:54 UTC 2013


That's even better then, thank you!
On Jul 13, 2013 9:32 AM, "Gurucharan Shetty" <shettyg at nicira.com> wrote:

> On Fri, Jul 12, 2013 at 9:35 PM, Ben Pfaff <blp at nicira.com> wrote:
>
>> Thanks, this looks good. But I am surprised that "make install" does not
>> create the log directory. Perhaps that is an underlying problem that we
>> should fix instead of adding a directory creation call to every packager.
>>
> make install does create a "/var/log/openvswitch" directory. The directory
> creation call is redundant and was not needed. I should just include it in
> the files section.
>
>
>> On Jul 12, 2013 9:29 PM, "Gurucharan Shetty" <shettyg at nicira.com> wrote:
>>
>>> During installation create the /var/log/openvswitch directory
>>> so that openvswitch startup script is able to write the ovs-ctl.log
>>>
>>> Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
>>> ---
>>>  rhel/openvswitch-fedora.spec.in   |    2 ++
>>>  rhel/openvswitch.spec.in          |    2 ++
>>>  xenserver/openvswitch-xen.spec.in |    2 ++
>>>  3 files changed, 6 insertions(+)
>>>
>>> diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/
>>> openvswitch-fedora.spec.in
>>> index dc40403..cc0f9db 100644
>>> --- a/rhel/openvswitch-fedora.spec.in
>>> +++ b/rhel/openvswitch-fedora.spec.in
>>> @@ -68,6 +68,7 @@ install python/compat/uuid.py
>>> $RPM_BUILD_ROOT/usr/share/openvswitch/python
>>>  install python/compat/argparse.py
>>> $RPM_BUILD_ROOT/usr/share/openvswitch/python
>>>
>>>  install -d -m 755 $RPM_BUILD_ROOT/var/lib/openvswitch
>>> +install -d -m 755 $RPM_BUILD_ROOT/var/log/openvswitch
>>>
>>>  %clean
>>>  rm -rf $RPM_BUILD_ROOT
>>> @@ -174,6 +175,7 @@ systemctl start openvswitch.service
>>>  %doc /usr/share/man/man8/ovs-test.8.gz
>>>  %doc /usr/share/man/man8/ovs-l3ping.8.gz
>>>  /var/lib/openvswitch
>>> +/var/log/openvswitch
>>>  /usr/share/openvswitch/scripts/ovs-ctl
>>>  %exclude /etc/openvswitch
>>>  %exclude /usr/bin/ovs-benchmark
>>> diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
>>> index 53512bc..37ec271 100644
>>> --- a/rhel/openvswitch.spec.in
>>> +++ b/rhel/openvswitch.spec.in
>>> @@ -66,6 +66,7 @@ rm \
>>>      $RPM_BUILD_ROOT/usr/share/man/man8/ovs-vlan-bug-workaround.8
>>>
>>>  install -d -m 755 $RPM_BUILD_ROOT/var/lib/openvswitch
>>> +install -d -m 755 $RPM_BUILD_ROOT/var/log/openvswitch
>>>
>>>  %clean
>>>  rm -rf $RPM_BUILD_ROOT
>>> @@ -155,3 +156,4 @@ exit 0
>>>  /usr/share/doc/openvswitch-%{version}/FAQ
>>>  /usr/share/doc/openvswitch-%{version}/README.RHEL
>>>  /var/lib/openvswitch
>>> +/var/log/openvswitch
>>> diff --git a/xenserver/openvswitch-xen.spec.in b/xenserver/
>>> openvswitch-xen.spec.in
>>> index dff18d0..a8c6395 100644
>>> --- a/xenserver/openvswitch-xen.spec.in
>>> +++ b/xenserver/openvswitch-xen.spec.in
>>> @@ -134,6 +134,7 @@ rm \
>>>      $RPM_BUILD_ROOT/usr/share/man/man8/ovs-test.8
>>>
>>>  install -d -m 755 $RPM_BUILD_ROOT/var/lib/openvswitch
>>> +install -d -m 755 $RPM_BUILD_ROOT/var/log/openvswitch
>>>
>>>  %clean
>>>  rm -rf $RPM_BUILD_ROOT
>>> @@ -452,6 +453,7 @@ exit 0
>>>  /usr/share/man/man8/ovs-vsctl.8.gz
>>>  /usr/share/man/man8/ovs-vswitchd.8.gz
>>>  /var/lib/openvswitch
>>> +/var/log/openvswitch
>>>  %exclude /usr/lib/xsconsole/plugins-base/*.py[co]
>>>  %exclude /usr/share/openvswitch/scripts/*.py[co]
>>>  %exclude /usr/share/openvswitch/python/*.py[co]
>>> --
>>> 1.7.9.5
>>>
>>> _______________________________________________
>>> dev mailing list
>>> dev at openvswitch.org
>>> http://openvswitch.org/mailman/listinfo/dev
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20130713/c8a4256d/attachment-0003.html>


More information about the dev mailing list