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

run tests for operators which are relevant to the changes in the PR #147

Open
zmalik opened this issue Dec 5, 2019 · 0 comments
Open

run tests for operators which are relevant to the changes in the PR #147

zmalik opened this issue Dec 5, 2019 · 0 comments

Comments

@zmalik
Copy link
Member

zmalik commented Dec 5, 2019

right now we are running some tests for any change in the repository. Those tests are for just for zookeeper operator. In case we change some other operator logic, these tests are irrelevant.
we should only run the tests related to that operator or no tests in case we don't have any for that operator. We need to ensure that:

  • CI run tests that are relevant to the changes done in the PR
  • PR status provide feedback on which tests are being run
  • Provide documentation/examples on how to add tests for an operator

In all cases, the PR checks should provide information around which tests have been run for a certain PR. To enable users to write tests for their own operators that can live in this repository.

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

No branches or pull requests

1 participant