Skip to content

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

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

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

Conversation

@maltesander
Copy link
Member

@maltesander maltesander commented Apr 5, 2023

Description

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

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 marked this pull request as ready for review April 5, 2023 11:00
Copy link
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 bot pushed a commit that referenced this pull request Apr 6, 2023
@bors
Copy link
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:09
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.

2 participants