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

[YUNIKORN-2138] Implement a lightweight monitor tool #163

Closed
wants to merge 2 commits into from

Conversation

wusamzong
Copy link
Contributor

What is this PR for?

While existing [performance testing tools|https://github.com/apache/yunikorn-release/tree/master/perf-tools] are well-established, this test aimed to broaden the scope of test items to include diverse Kubernetes features such as Priority class, toleration, etc., which also involve stochasticity.
Consequently, there is a need to implement a lightweight throughput monitoring tool.

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-2138/

How should this be tested?

Screenshots (if appropriate)

Questions:

  • - The licenses files need update.
  • - There is breaking changes for older versions.
  • - It needs documentation.

@craigcondit
Copy link
Contributor

I'm not sure I see the value in this. It seems like this tool just generates some (configurable) number of pods in a namespace and measures what.. cpu / memory usage of yunikorn? This is probably much better handled in some sort of integration test in the k8shim repo for one thing. Also, rather than generating pods directly (and not cleaning them up) it would be far better to just submit a Job with 'n' parallelism. Let's keep the release repo clear of these sorts of things. Closing this.

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