Project

General

Profile

Actions
Copy link

Bug #7226

closed

service_ensure_started_at_boot does not work on non-systemd systems with upsteam CFEngine stdlib

Bug #7226: service_ensure_started_at_boot does not work on non-systemd systems with upsteam CFEngine stdlib

Added by Tomas Corej over 10 years ago. Updated almost 4 years ago.

Status:
Released
Priority:
N/A
Assignee:
Benoît PECCATTE
Category:
Generic methods
Target version:
ncf-0.x
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

It does not work because there is typo in variable name:

"command_to_register" string => "${paths.path[update_rcd]} ${service_name} defaults";

but paths.path[update_rcd] does not exists, paths.path[update_rc_d] does . There is also same error in linux knowledge bundle so class update_rcd_utility_present is never evaluated .

Updated by Tomas Corej over 10 years ago Actions
Copy link
 #1

Now I see that paths.path[update_rcd] is in cfengine library that is distributed with ncf. I've used original cfengine std lib from masterfiles. But it seems that this typo is in ncf stdlib as well.

Updated by Jonathan CLARKE over 10 years ago Actions
Copy link
 #2

  • Subject changed from service_ensure_started_at_boot does not work on non-systemd systems to service_ensure_started_at_boot does not work on non-systemd systems with upsteam CFEngine stdlib
  • Target version set to 0.x

Thanks for this bug report, Tomas. You're right, something fishy has obviously happened with this key in paths. However, it works OK with ncf's copy of the stdlib, so I've clarified the subject.

Updated by Jonathan CLARKE over 10 years ago Actions
Copy link
 #3

  • Status changed from New to In progress
  • Assignee set to Jonathan CLARKE

Updated by Jonathan CLARKE over 10 years ago Actions
Copy link
 #4

  • Status changed from In progress to Pending technical review
  • Assignee changed from Jonathan CLARKE to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/ncf/pull/279

Updated by Jonathan CLARKE over 10 years ago Actions
Copy link
 #5

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

Updated by Vincent MEMBRÉ over 9 years ago Actions
Copy link
 #7

  • Status changed from Pending release to Released

Updated by Alexis Mousset almost 4 years ago Actions
Copy link
 #8

  • Target version changed from 0.x to ncf-0.x
  • Priority set to 0

Updated by Alexis Mousset almost 4 years ago Actions
Copy link
 #9

  • Project changed from 41 to Rudder
  • Category set to Generic methods
Actions
Copy link

Also available in: PDF Atom