Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#4039 closed defect (fixed)

unit test fail on NetBSD

Reported by: wlodekwencel Owned by: marcin
Priority: high Milestone: Kea1.0-beta
Component: Unclassified Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

unit test TimerMgrTest?.scheduleTimers is failing on NetBSD 6 since #3970 merge which actually brings that tests to master branch

https://jenkins.isc.org/job/NetBSD_6_64/283/testReport/junit/(root)/TimerMgrTest/scheduleTimers/

Subtickets

Change History (8)

comment:1 Changed 4 years ago by hschempf

  • Milestone changed from Kea-proposed to Kea1.0

should be solved w/ 3971, accepted for 1.0 on 16 sept

comment:2 Changed 4 years ago by wlodekwencel

  • Milestone changed from Kea1.0 to Kea-proposed

comment:3 Changed 4 years ago by hschempf

  • Milestone changed from Kea-proposed to Kea1.0
  • Priority changed from medium to high

per team meeting 23 sept, move back to 1.0, high, .25d est

comment:4 Changed 4 years ago by marcin

  • Owner set to marcin
  • Status changed from new to assigned

comment:5 Changed 4 years ago by marcin

  • Owner changed from marcin to tmark
  • Status changed from assigned to reviewing

This is now ready for review. It is a minor change so I don't think it requires a changelog entry.

comment:6 Changed 4 years ago by tmark

  • Owner changed from tmark to marcin

The changes seem ok. If comparing the counts to 10 and 5 respectively prove unreliable I think you could remove those checks completely. We do expect timer1 to run more times than timer2 and we're checking that. Perhaps that's enough.

I think you can merge this as is. If it proves to be a problem we'll take another look.

comment:7 Changed 4 years ago by marcin

  • Resolution set to fixed
  • Status changed from reviewing to closed

Merged with commit ee52a28f7d4fc01ba6ce37e32403832a5d1e7147

comment:8 Changed 4 years ago by tomek

  • Milestone changed from Kea1.0 to Kea1.0-beta

Milestone renamed

Note: See TracTickets for help on using tickets.