7421619587
53%

Ran 05 Jan 2024 12:31PM UTC

Jobs 1

Files 216

Run time 14s

Badge

Embed ▾

Committed 04 Jan 2024 09:36AM UTC coverage: 50.663% (-0.005%) from 50.668%

Build # 7421619587

Build Type

push

github

Committed by

jcaamano

Commit Message

Fix informer_test: adds existing pod and processes an update event

When starting the informer factory and the worker thread, a delay
may occur between the informer being deemed synced and the worker
completely starting. Due to this, the test could fail as add and
update pod events might coalesce into a single event for the
worker. This change proposes a fix that ensures the worker is fully
initialized before updating the pod.

Step used to reproduce this issue. Should fail after a few iterations.

$ cd go-controller ; \
  while : ; do \
  go test -mod=vendor -test.v \
  -race github.com/ovn-org/ovn-kubernetes/go-controller/pkg/informer \
  -ginkgo.v \
  -ginkgo.focus='adds existing pod and processes an update event' \
  -ginkgo.reportFile /dev/null && echo ok || break ; \
  sleep 1 ; done

Fixes: https://github.com/ovn-org/ovn-kubernetes/issues/3946
Reported-at: https://issues.redhat.com/browse/SDN-4181
Signed-off-by: Flavio Fernandes <flaviof@redhat.com>

Coverage Stats

26222 of 51758 relevant lines covered (50.66%)

87.11 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	7421619587.1	05 Jan 2024 12:31PM UTC	0	50.66	GitHub Action Run

Source Files on build 7421619587

Detailed source file information is not available for this build.

ovn-org / ovn-kubernetes / 7421619587
53%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7421619587

ovn-org / ovn-kubernetes / 7421619587 53%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 7421619587

ovn-org / ovn-kubernetes / 7421619587
53%

README BADGES
x