stackabletech / spark-k8s-operator

Operator for Apache Spark-on-Kubernetes for Stackable Data Platform
https://stackable.tech
Other
51 stars 3 forks source link

[Merged by Bors] - fix: Typo in executor cpu limit property #263

Closed sbernauer closed 1 year ago

sbernauer commented 1 year ago

Description

Introduced in https://github.com/stackabletech/spark-k8s-operator/pull/256

Definition of Done Checklist

# Author
- [ ] Changes are OpenShift compatible
- [ ] CRD changes approved
- [ ] Helm chart can be installed and deployed operator works
- [ ] Integration tests passed (for non trivial changes)
# Reviewer
- [ ] Code contains useful comments
- [ ] (Integration-)Test cases added
- [ ] Documentation added or updated
- [ ] Changelog updated
- [ ] Cargo.toml only contains references to git tags (not specific commits or branches)
# Acceptance
- [ ] Feature Tracker has been updated
- [x] Proper release label has been added

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

sbernauer commented 1 year ago

getting started script passed, starting kuttl tests...

sbernauer commented 1 year ago

Started https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/spark-k8s-operator-it-custom/93/

sbernauer commented 1 year ago

bors r+ tests passed

bors[bot] commented 1 year ago

Pull request successfully merged into main.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here. For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

lfrancke commented 1 year ago

Same question: This made it into 23.7?

maltesander commented 1 year ago

Yes this was a bug an was cherry picked into 23.7 branch (other releases were not affected).