Skip to content

[Merged by Bors] - Add status field support#421

Closed
maltesander wants to merge 10 commits intomainfrom
feat/status
Closed

[Merged by Bors] - Add status field support#421
maltesander wants to merge 10 commits intomainfrom
feat/status

Conversation

@maltesander
Copy link
Copy Markdown
Member

@maltesander maltesander commented Mar 27, 2023 •
edited
Loading

Description

Part of stackabletech/issues#343

test https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/druid-operator-it-custom/104/

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
# 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
- [ ] Proper release label has been added

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

@maltesander maltesander requested a review from a team March 27, 2023 15:55
@maltesander maltesander mentioned this pull request Mar 27, 2023
Closed
17 tasks
@maltesander
Merge remote-tracking branch 'origin/main' into feat/status
e249f9e 
# Conflicts:
#	CHANGELOG.md
#	Cargo.lock
#	rust/crd/Cargo.toml
#	rust/operator-binary/Cargo.toml
#	rust/operator-binary/src/druid_controller.rs
@maltesander
add get for statefulset to clusterrole
7e16eb1
@maltesander
add cluster operation status
f9d5481
@maltesander
added tests
016947a
razvan
razvan approved these changes Apr 6, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors merge

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Apr 6, 2023

Merge conflict.

@razvan
Copy link
Copy Markdown
Member

razvan commented Apr 6, 2023

bors merge

bors bot pushed a commit that referenced this pull request Apr 6, 2023
@maltesander @razvan
Add status field support (#421)
11fe222 
# Description

Part of stackabletech/issues#343

test https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/druid-operator-it-custom/104/



Co-authored-by: Malte Sander <malte.sander.it@gmail.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Apr 6, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Add status field support [Merged by Bors] - Add status field support Apr 6, 2023
@bors bors bot closed this Apr 6, 2023
@bors bors bot deleted the feat/status branch April 6, 2023 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@razvan razvan razvan approved these changes

Assignees

@razvan razvan

@maltesander maltesander

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@maltesander @razvan