Project

General

Profile

Actions

Bug #18353

closed

Intermitent failure test on certificate inventory

Added by François ARMAND over 3 years ago. Updated over 3 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.

Actions #1

Updated by François ARMAND over 3 years ago

  • Status changed from New to In progress
Actions #2

Updated by François ARMAND over 3 years ago

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)
Actions #3

Updated by François ARMAND over 3 years ago

  • 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
Actions #4

Updated by François ARMAND over 3 years ago

  • Status changed from Pending technical review to Pending release
Actions #5

Updated by Nicolas CHARLES over 3 years ago

  • Fix check changed from To do to Checked
Actions #6

Updated by Alexis Mousset over 3 years ago

  • Name check changed from To do to Reviewed
Actions #7

Updated by Vincent MEMBRÉ over 3 years ago

  • 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: Atom PDF