Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add OIDC e2e test mt-broker #7458

Closed
creydr opened this issue Nov 17, 2023 · 1 comment · Fixed by #7481
Closed

Add OIDC e2e test mt-broker #7458

creydr opened this issue Nov 17, 2023 · 1 comment · Fixed by #7481
Assignees
Labels
kind/TBD Parked issue that required triaging/revisit in a near future.

Comments

@creydr
Copy link
Member

creydr commented Nov 17, 2023

We should add e2e tests, which test the flows through the mt-broker:

Simple flow:

Simply check, that a event goes through the broker to the sink.

flowchart LR
    SOURCE[Source] --> BROKER[Broker with Audience]
    BROKER --> TRIGGER[Trigger]
    TRIGGER --> SINK[Sink with Audience]
Loading

With Reply:

Check that a reply from the sink is received well by the Broker (which has an audience).

flowchart LR
    SOURCE[Source] -->|eventtype-1| BROKER[Broker with Audience]
    BROKER --> TRIGGER[Trigger with filter for eventtype-1]
    TRIGGER --> SINK[Sink with Audience]
    SINK -->|respond with eventtype-2| TRIGGER
    BROKER --> TRIGGER2[Trigger with filter for eventtype-2] --> SINK2[Sink 2]
Loading

The 2nd Trigger (with filter for eventtype-2) is only to ensure, that the broker received a reply and thus the sink 2 does not need an Audience configured.

With Dead Letter Sink:

Check that a dead letter sink request gets delivered correctly, if the DLS has an Audience.

flowchart LR
    SOURCE[Source] --> BROKER[Broker with Audience]
    BROKER --> TRIGGER[Trigger with Dead-Letter-Sink]
    TRIGGER --> SINK[Sink with Audience]
    SINK -->|respond with non retryable Statuscode| TRIGGER
    TRIGGER --> DLS[Dead-Letter-Sink with Audience]
Loading

Requires the following issues to be done:

@creydr creydr added the kind/TBD Parked issue that required triaging/revisit in a near future. label Nov 17, 2023
@creydr creydr moved this to 📝 Draft in Eventing Sender Identity Nov 17, 2023
@creydr creydr self-assigned this Nov 17, 2023
@creydr
Copy link
Member Author

creydr commented Dec 4, 2023

Done in #7480

@creydr creydr closed this as completed Dec 4, 2023
@github-project-automation github-project-automation bot moved this from 📝 Draft to ✅ Done in Eventing Sender Identity Dec 4, 2023
@creydr creydr linked a pull request Dec 20, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/TBD Parked issue that required triaging/revisit in a near future.
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant