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

chore: remove old event stream from codebase #1008

Closed
wants to merge 1 commit into from

Conversation

mkanoor
Copy link
Contributor

@mkanoor mkanoor commented Aug 15, 2024

https://issues.redhat.com/browse/AAP-27632
Remove the old EventStream from fanout

@mkanoor mkanoor requested a review from a team as a code owner August 15, 2024 13:51
@mkanoor mkanoor force-pushed the remove_event_stream branch 5 times, most recently from 9bbffc0 to 5b65947 Compare August 15, 2024 14:13
@mkanoor mkanoor added run-e2e run-ui-e2e Run UI E2E tests on a PR and removed run-e2e run-ui-e2e Run UI E2E tests on a PR labels Aug 15, 2024
@Dostonbek1
Copy link
Member

@mkanoor Do you have a PR in plan for updating the e2e tests for these changes?

@mkanoor mkanoor force-pushed the remove_event_stream branch 2 times, most recently from b761e11 to 1812108 Compare August 16, 2024 21:27
hsong-rh
hsong-rh previously approved these changes Aug 16, 2024
@mkanoor
Copy link
Contributor Author

mkanoor commented Aug 16, 2024

@Dostonbek1 I have generated a new API client in this PR https://github.com/ansible/eda-qa/pull/420

@mkanoor
Copy link
Contributor Author

mkanoor commented Aug 16, 2024

@Dostonbek1
Copy link
Member

@mkanoor mkanoor force-pushed the remove_event_stream branch 3 times, most recently from a97c3d3 to faa628c Compare August 21, 2024 13:02
@mkanoor mkanoor force-pushed the remove_event_stream branch from faa628c to 92bceb9 Compare August 22, 2024 15:12
Copy link

@rooftopcellist
Copy link
Member

Closing this PR as the changes have landed as a part of #1011.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants