Project

General

Profile

Actions

Bug #18353

closed

Intermitent failure test on certificate inventory

Bug #18353: Intermitent failure test on certificate inventory

Added by François ARMAND about 5 years ago. Updated about 5 years ago.

Status:
Released
Priority:
N/A
Category:
System integration
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

This error happens from time to time since: #18336

19:13:25 Tests run: 8, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 3.73 sec <<< FAILURE!
19:13:25 when a node is not in repository, it is ok to have a signature with cfe-key(com.normation.rudder.inventory.TestCertificate)  Time elapsed: 0.007 sec  <<< FAILURE!
19:13:25 java.lang.Exception: None is not Some
19:13:25     at com.normation.rudder.inventory.TestCertificate.$anonfun$new$13(TestCertificate.scala:143)
19:13:25     at com.normation.rudder.inventory.TestCertificate.$anonfun$new$9(TestCertificate.scala:143)
19:13:25     at com.normation.rudder.inventory.TestCertificate.$anonfun$new$8(TestCertificate.scala:144)
19:13:25     at com.normation.rudder.inventory.TestCertificate.$anonfun$new$5(TestCertificate.scala:145)
19:13:25 

I'm not sure what is the cause.

Updated by François ARMAND about 5 years ago Actions #1

  • Status changed from New to In progress

Updated by François ARMAND about 5 years ago Actions #2

It can be consistantly reproduced by changing the "save" method in ReportSaver to add a delay:

    override def save(report: InventoryReport): IOResult[Seq[LDIFChangeRecord]] = IOResult.effect {
      Thread.sleep(500)  // add a delay
      repository += ((report.node.main.id, FullInventory(report.node, Some(report.machine))))
    }.map(_ => Nil)

Updated by François ARMAND about 5 years ago Actions #3

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Elaad FURREEDAN
  • Pull Request set to https://github.com/Normation/rudder/pull/3271

Updated by François ARMAND about 5 years ago Actions #4

  • Status changed from Pending technical review to Pending release

Updated by Nicolas CHARLES about 5 years ago Actions #5

  • Fix check changed from To do to Checked

Updated by Alexis Mousset about 5 years ago Actions #6

  • Name check changed from To do to Reviewed

Updated by Vincent MEMBRÉ about 5 years ago Actions #7

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.6 and 6.2.0~beta1 which were released today.

Actions

Also available in: PDF Atom