Bug #4125

Probes cleaned by systemd-tmpfiles

Added by Daniel Kelleher over 5 years ago. Updated almost 4 years ago.

Status:ClosedStart date:11/02/2015
Priority:LowDue date:
Assignee:Javi Fontan% Done:

100%

Category:Packaging
Target version:Release 5.0
Resolution:fixed Pull request:
Affected Versions:OpenNebula 4.12, OpenNebula 4.14

Description

In Centos 7 the probes are cleaned by systemd-tmpfiles if they are not updated every 30 days. run_probes and scripts_common.sh still exist so oned logs a successful probe but the onehost list shows blank available resources. This seams like it could be fixed by adding a exclusion for /var/tmp/one . I'm currently testing the exclusion by reducing the timer on /usr/lib/tmpfiles.d/tmp.conf and adding an exclusion to /usr/lib/tmpfiles.d/opennebula.conf .

centos7.spec.patch Magnifier (821 Bytes) Anton Todorov, 07/05/2017 08:00 AM

Associated revisions

Revision 6546928d
Added by Daniel Kelleher over 5 years ago

Bug #4125: Probes cleaned by systemd-tmpfiles

Add systemd-tmpfiles exemption for /var/tmp/one

Revision a79c1fbd
Added by Daniel Kelleher over 5 years ago

Bug #4125: Probes cleaned by systemd-tmpfiles

Add systemd-tmpfiles exemption for /var/tmp/one

(cherry picked from commit 6546928dfe2072ed3731bc00bca653087b397ec0)

History

#1 Updated by Ruben S. Montero over 5 years ago

  • Category set to Packaging
  • Target version set to Release 4.14.2

Good catch :)

#2 Updated by Daniel Kelleher over 5 years ago

Using the above method I recreated the issue as well as confirmed that a exemption solved the issue. Pull request located at https://github.com/OpenNebula/one/pull/73

#3 Updated by Ruben S. Montero over 5 years ago

  • Assignee set to Javi Fontan

#4 Updated by Javi Fontan over 5 years ago

  • Status changed from Pending to Closed
  • Resolution set to fixed

Added the change to one-4.14 and master branches.

#5 Updated by Javi Fontan over 5 years ago

  • Status changed from Closed to Pending
  • Target version changed from Release 4.14.2 to 82
  • Resolution deleted (fixed)

The exclusion configuration file was added to the source repository but the CentOS 7 rpm does not install it.

#6 Updated by Ruben S. Montero over 5 years ago

  • Target version changed from 82 to Release 5.0

#7 Updated by Javi Fontan about 5 years ago

  • % Done changed from 0 to 50

Changed package repository. Needs testing.

#8 Updated by Javi Fontan about 5 years ago

  • Status changed from Pending to Closed
  • % Done changed from 50 to 100
  • Resolution set to fixed

Tested

#9 Updated by Anton Todorov about 4 years ago

Hi,

I think there is a bug in the packaging of OpenNebula for CentOS7.

The source file share/pkgs/CentOS7/opennebula.conf is placed in opennebula-node-kvm as /lib/tmpfiles.d/opennebula-node.conf
And most probably this fix is overwritten in the RPM instead of appended.

I think that both files must be merged in a single opennebula-node.conf

Also the resulting file in the RPM is with executable bits set.

Best Regards,
Anton Todorov

#10 Updated by Anton Todorov about 4 years ago

I can't find the centos7.spec file in the git repository so here is a patch that should fix the issue.

--- centos7.spec.orig   2017-07-05 10:54:11.957638550 +0300
+++ centos7.spec    2017-07-05 10:54:53.301808408 +0300
@@ -291,7 +291,7 @@
 install -p -D -m 644 share/pkgs/CentOS7/opennebula.conf %{buildroot}/lib/tmpfiles.d/opennebula-sunstone.conf
 install -p -D -m 644 share/pkgs/CentOS7/opennebula.conf %{buildroot}/lib/tmpfiles.d/opennebula-gate.conf
 install -p -D -m 644 share/pkgs/CentOS7/opennebula.conf %{buildroot}/lib/tmpfiles.d/opennebula-flow.conf
-install -p -D -m 644 share/pkgs/CentOS7/opennebula.conf %{buildroot}/lib/tmpfiles.d/opennebula-node.conf
+install -p -D -m 644 share/pkgs/CentOS7/opennebula-node.conf %{buildroot}/lib/tmpfiles.d/opennebula-node.conf

 install -p -D -m 644 %{SOURCE1} \
         %{buildroot}%{_sysconfdir}/polkit-1/localauthority/50-local.d/50-org.libvirt.unix.manage-opennebula.pkla

#11 Updated by Javi Fontan almost 4 years ago

I've applied your patch in our packages repo. Thanks!

Also available in: Atom PDF