Project

General

Profile

Actions
Copy link

Bug #2620

closed

Error message is unclear when a nonexistant file is included in a Policy Template

Added by Nicolas CHARLES over 12 years ago. Updated over 9 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
François ARMAND
Category:
Web - Config management
Target version:
2.3.8
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

When trying to deploy promises, error arises.
The logs say :

13:08:40.074 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new distributePolicy/1.0/initCheck.cf
13:08:40.089 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new distributePolicy/rsyslog.conf/rudder.conf
13:08:40.100 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new distributePolicy/1.0/postgresCheck.cf
13:08:40.117 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new distributePolicy/logrotate.conf/rudder
13:08:40.130 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new clientlist/clientlist.cf
13:08:40.136 [pool-3-thread-1] DEBUG com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Create promises file /var/rudder/cfengine-community/inputs.new distributePolicy/apache.conf/rudder.conf

There's probably a missing / there

Actions
Copy link
 #1

Updated by François ARMAND over 12 years ago

  • Status changed from 2 to In progress
Actions
Copy link
 #2

Updated by Nicolas CHARLES over 12 years ago

  • Target version changed from 47 to 2.4.0~beta2
Actions
Copy link
 #3

Updated by Nicolas CHARLES over 12 years ago

  • Status changed from In progress to Discussion

This bug is caused by bad promises, pointing to non-existant files
The error message should be probably a bit more helpful

Actions
Copy link
 #4

Updated by Nicolas CHARLES over 12 years ago

This was in fact due to #2622

Actions
Copy link
 #5

Updated by François ARMAND over 12 years ago

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

Applied in changeset commit:44f69d3efe3d0b0e861153f509438c4ff747ec2f.

Actions
Copy link
 #6

Updated by Nicolas CHARLES over 12 years ago

  • Status changed from Pending technical review to Released

thank you Francois for improving the error message.

Actions
Copy link
 #7

Updated by François ARMAND over 12 years ago

Now the error message is:
"""
Exception caught during deployment process: Error when trying to open template 'common/1.0/cf-servedXX.st'. Check that the file exists and is correctly commited in Git, or that the metadata for the technique are corrects.
"""

Actions
Copy link
 #8

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Released to 2
  • Target version changed from 2.4.0~beta2 to 2.3.8

This bug also applies to the 2.3 branch. Can you please apply your fix there too?

Actions
Copy link
 #9

Updated by François ARMAND over 12 years ago

  • Status changed from 2 to Pending technical review

Applied in changeset commit:d81b899db7f69c1fc3308db8caf4fe9a109fc7cd.

Actions
Copy link
 #10

Updated by Jonathan CLARKE over 12 years ago

  • Status changed from Pending technical review to Discussion

This was closed by a merge but still needs applying to 2.3

Actions
Copy link
 #11

Updated by François ARMAND over 12 years ago

  • Status changed from Discussion to Pending technical review

Applied in changeset commit:81f7a890394c7e8ec97cfd8d95a606c92fef75d2.

Actions
Copy link
 #12

Updated by Nicolas CHARLES over 12 years ago

  • Status changed from Pending technical review to Released

This seems valid, thank you Francois

Actions
Copy link
 #13

Updated by Jonathan CLARKE over 12 years ago

  • Subject changed from The path of the generated promises is invalid to Error message is unclear when a nonexistant file is included in a Policy Template
Actions
Copy link
 #14

Updated by Benoît PECCATTE over 9 years ago

  • Category changed from 14 to Web - Config management
Actions
Copy link

Also available in: Atom PDF