stackabletech / kafka-operator

Stackable Operator for Apache Kafka
Other
23 stars 6 forks source link

Stabilize the TLS test #719

Closed siegfriedweber closed 2 months ago

siegfriedweber commented 2 months ago

Description

The test scripts of the TLS test were copied in the TestStep 30-prepare-test-kafka to the pod test-kafka-broker-default-0. In the TestAssertion 30-assert, these scripts were executed. If the pod was restarted while the TestAssertion was running, the scripts could not be found anymore when the TestAssertion was retried.

It is not possible to simply move the copy steps into the TestAssertion because the working directory there is test/_work and the copy command would need the path to the scripts which is not known.

The solution in this pull request is to use a Job which mounts the test scripts from a ConfigMap.

Definition of Done Checklist

# Author
- [x] Changes are OpenShift compatible
- [x] Integration tests passed (for non trivial changes)
# Reviewer
- [x] Code contains useful comments
- [x] Code contains useful logging statements
- [x] (Integration-)Test cases added
- [x] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [x] Changelog updated
- [x] Cargo.toml only contains references to git tags (not specific commits or branches)
# Acceptance
- [ ] Feature Tracker has been updated
- [ ] Proper release label has been added
- [ ] [Roadmap](https://github.com/orgs/stackabletech/projects/25/views/1) has been updated
siegfriedweber commented 2 months ago

:white_check_mark: https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/kafka-operator-it-custom/61/