Skip to content

Adding KIND provider for the prombench#390

Merged
geekodour merged 21 commits intoprometheus:masterfrom
imrajdas:kind-provider
Sep 2, 2020
Merged

Adding KIND provider for the prombench#390
geekodour merged 21 commits intoprometheus:masterfrom
imrajdas:kind-provider

Conversation

@imrajdas
Copy link
Contributor

@imrajdas imrajdas commented Jun 24, 2020

Proposed Changes

  • Adding kind provider to deploy KIND and prombench.
  • Adding new commands to infra CLI to do prombench operations using KIND
  • Adding a doc to use prombench with KIND

Issue: #333

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@imrajdas imrajdas marked this pull request as ready for review July 6, 2020 19:19
@imrajdas imrajdas changed the title (WIP): Adding KIND provider for the prombench chore(kind-provider): Adding KIND provider for the prombench Jul 6, 2020
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@imrajdas
Copy link
Contributor Author

imrajdas commented Jul 6, 2020

cc @geekodour This PR is ready for review

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
imrajdas added 2 commits July 17, 2020 00:55
Signed-off-by: Raj Babu Das <raj.das@mayadata.io>
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@imrajdas
Copy link
Contributor Author

@geekodour circleCI has been passed

@imrajdas imrajdas force-pushed the kind-provider branch 2 times, most recently from 5292317 to de43323 Compare July 23, 2020 16:45
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@geekodour
Copy link
Contributor

The tests seem to be failing aswell 🤔

@imrajdas
Copy link
Contributor Author

imrajdas commented Aug 8, 2020

The tests seem to be failing aswell

Doc format failure

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@imrajdas
Copy link
Contributor Author

imrajdas commented Aug 8, 2020

@geekodour , I made the changes as per your suggestion and build is also passed. PTAL

Copy link
Contributor

@geekodour geekodour left a comment

Choose a reason for hiding this comment

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

Few more changes and suggestions about logging and validation.

Copy link
Contributor

@geekodour geekodour left a comment

Choose a reason for hiding this comment

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

few more suggestions and improvements :)

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
Copy link
Contributor

@geekodour geekodour left a comment

Choose a reason for hiding this comment

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

Some nits :)

Let's wait for #425 to get merged and come back again.

@imrajdas imrajdas changed the title chore(kind-provider): Adding KIND provider for the prombench Adding KIND provider for the prombench Aug 21, 2020
@imrajdas imrajdas requested a review from geekodour August 21, 2020 16:53
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@geekodour geekodour requested a review from gouthamve August 26, 2020 20:22
@geekodour
Copy link
Contributor

I remember @gouthamve requesting local running of prombench, It runs locally now with this PR. :) but still need improvements, also have to figure out running prombench from local images thing. Do you have any other suggestions/requests for local running of prombench?

But this should make adding and testing new components in the test-infra easier ✨

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
@geekodour geekodour merged commit 2f8f9f9 into prometheus:master Sep 2, 2020
@geekodour
Copy link
Contributor

🎉 merged. thanks for all your work! :)

Will add info on E2E test-infra tests and supporting local images in another PR.

@imrajdas imrajdas deleted the kind-provider branch September 2, 2020 16:06
kushalShukla-web pushed a commit to kushalShukla-web/test-infra that referenced this pull request Feb 8, 2025
This PR adds support of running test-infra components locally using the
KIND provider. KIND specific commands are added to the infra CLI.

When running locally, we don't have lot of CPU and memory resources.
LOADGEN_SCALE_UP_REPLICAS deployment variable was added which makes sure
that when run locally we're only scale to 2 fake-webservers.

Signed-off-by: Raj Babu Das <mail.rajdas@gmail.com>
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.

2 participants