Project

General

Profile

Actions

Bug #7240

closed

The rudder_var path on Solaris is incorrect

Added by Matthieu CERDA over 8 years ago. Updated over 8 years ago.

Status:
Released
Priority:
1
Category:
Techniques
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

Running cf-agent on Solaris yields:

bash-3.2# /var/rudder/cfengine-community/bin/cf-agent -KI
   error: Can't stat file '/var/rudder/ncf/common/10_ncf_internals/list-compatible-inputs: /var/rudder/ncf/common/10_ncf_internals/list-compatible-inputs: cannot open' for parsing. (stat: No such file or directory)
   error: Policy failed validation with command '"/var/rudder/cfengine-community/bin/cf-promises" -c "/var/rudder/cfengine-community/inputs/promises.cf"'

The error comes from the fact that in failsafe.cf, rudder_var is:

    solaris::
      "rudder_var"               string => "/opt/rudder/var";

But in promises.cf, the condition is:

    !android.!windows::
      "rudder_var"     string => "/var/rudder";

Rudder tries to find list-compatible-inputs in the wrong place, and fails miserably.

Actions #1

Updated by Matthieu CERDA over 8 years ago

  • Status changed from New to In progress
Actions #2

Updated by Matthieu CERDA over 8 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/748
Actions #3

Updated by Matthieu CERDA over 8 years ago

  • Assignee changed from Nicolas CHARLES to Benoît PECCATTE

Giving this to BPE instead

Actions #4

Updated by Benoît PECCATTE over 8 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Benoît PECCATTE to Matthieu CERDA

I think we should put everything ins /var/rudder and make exceptions only for a strong reason.

Actions #5

Updated by Nicolas CHARLES over 8 years ago

yeah, this is all broken

Actions #6

Updated by Jonathan CLARKE over 8 years ago

Benoît PECCATTE wrote:

I think we should put everything ins /var/rudder and make exceptions only for a strong reason.

Agreed.

Actions #7

Updated by Nicolas CHARLES over 8 years ago

  • Status changed from Discussion to In progress
  • Assignee changed from Matthieu CERDA to Nicolas CHARLES
Actions #8

Updated by Nicolas CHARLES over 8 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Benoît PECCATTE
  • Pull Request changed from https://github.com/Normation/rudder-techniques/pull/748 to https://github.com/Normation/rudder-techniques/pull/788/files
Actions #9

Updated by Nicolas CHARLES over 8 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions #11

Updated by Vincent MEMBRÉ over 8 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.2.0~beta1 which was released today.

Actions

Also available in: Atom PDF