Skip to content

[Merged by Bors] - Centralize producer/consumer config for tests#2237

Closed
tjtelan wants to merge 19 commits intofluvio-community:masterfrom
tjtelan:fluvio-test-config
Closed

[Merged by Bors] - Centralize producer/consumer config for tests#2237
tjtelan wants to merge 19 commits intofluvio-community:masterfrom
tjtelan:fluvio-test-config

Conversation

@tjtelan
Copy link
Contributor

@tjtelan tjtelan commented Mar 8, 2022

@tjtelan tjtelan requested a review from sehz March 11, 2022 00:11
@sehz
Copy link

sehz commented Mar 11, 2022

Can you create issue for this?

Copy link

@sehz sehz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even thought this is just updating test generator, we should add changelog

@tjtelan tjtelan requested a review from sehz March 11, 2022 06:16
Copy link

@sehz sehz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Powerful work!

@sehz
Copy link

sehz commented Mar 11, 2022

rebase please

@sehz
Copy link

sehz commented Mar 11, 2022

bors r+

bors bot pushed a commit that referenced this pull request Mar 11, 2022
* Add data generator - Resolves #2244
* Adds producer, consumer configuration to top-level to share w/ all tests- Resolves #1538 
* Add topic configuration options to top-level so they can be shared by all tests - Resolves #2142 
* Add ability to disable global timeout over cli option and the fluvio_test macro - Resolves #2205
* Update hourly test to use the new features
@bors
Copy link

bors bot commented Mar 11, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Centralize producer/consumer config for tests [Merged by Bors] - Centralize producer/consumer config for tests Mar 11, 2022
@bors bors bot closed this Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants