Skip to content

Conversation

@HoustonPutman
Copy link
Contributor

@HoustonPutman HoustonPutman commented Dec 16, 2022

Resolves #134

This is the first of hopefully many integration testing tickets, mostly to build the start of a framework so that other work can be done iteratively.

Goals/Progress:

  • Integration testing with the Solr Operator Helm Chart
  • At least 1 integration test for each Solr resource
  • Documentation
  • Good cleanup while retaining ability to debug when things didn't work
  • The go code should not be calling external programs. See usage of Helm golang APIs.

Future Goals, not required for this ticket:

  • TLS testing
  • Being able to use this component for the release smoke tests
  • Well supported ability to use a non-KiND cluster
  • Be able to run without access to the internet

The test suite will install the solr operator via the helm chart.

The first pass will not be for release testing yet.
@HoustonPutman
Copy link
Contributor Author

Ok smoketester usage is now completely covered by the e2e tests. This should be pretty close to merging in early january once we have some other eyes on it!

Btw #511 is required for the tests to pass, so we should decide on that before merging this in.

@HoustonPutman
Copy link
Contributor Author

Ok, big win. Much like the solr tests, the e2e tests are now completely reproducable with all possible inputs (random seed, operator/solr image, kubernetes version, etc), and accept easy filtering.

On test failure each individual test will also output a command to reproduce the test failure.

Screen Shot 2023-01-05 at 3 41 05 PM

This should be ported to the unit tests in a separate PR, because its an awesome feature and very much missing from the operator tests.

We are reaching the finish line here. I am going to document the tests and clean up e2e-tests and e2e-tests-run, so that there is only 1 task. Then I think this will be ready to merge, hopefully next week.

@HoustonPutman
Copy link
Contributor Author

Ok I don't think Github Action runners have the resources to run the e2e tests, so we will have to disable those for now. We can have them run in Jenkins instead for now.

@HoustonPutman
Copy link
Contributor Author

I think this is ready to go and plan to commit it early next week!

@HoustonPutman HoustonPutman merged commit 68fd75d into apache:main Feb 6, 2023
@HoustonPutman HoustonPutman deleted the e2e-tests branch February 6, 2023 17:28
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.

Introduce comprehensive integration tests

1 participant