Skip to content

[QE] Default SUT for upgrade tests shouldn't wait to be ready, but rely on WaitForAllTestResourcesReadyOrFail #5441

@cardil

Description

@cardil

Currently, there is a code that waits for triggers to report ready:

err := duck.WaitForResourceReady(ctx.Client.Dynamic, meta)
if err != nil {
ctx.T.Fatal(err)
}

And that code is invoked before those trigger's subscribers are even deployed:

p.deployConfiguration()
p.deployReceiver()
if p.config.Serving.Use {
p.deployForwarder()
}

We should check for readiness state, once, only at:

p.client.WaitForAllTestResourcesReadyOrFail(p.config.Ctx)

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions