Skip to content

[Merged by Bors] - Implement pause/stop#336

Closed
razvan wants to merge 7 commits intomainfrom
feat/cluster-operation
Closed

[Merged by Bors] - Implement pause/stop#336
razvan wants to merge 7 commits intomainfrom
feat/cluster-operation

Conversation

@razvan
Copy link
Copy Markdown
Member

@razvan razvan commented Mar 30, 2023

Description

CI failed but the failed tests work locally (on kind).

Please add a description here. This will become the commit message of the merge request later.

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

@razvan razvan self-assigned this Mar 30, 2023
@razvan razvan requested a review from a team March 30, 2023 13:01
Comment thread Cargo.toml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I remember from the back of my head that it's intentional and needed. You can try to remove it and see how it goes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Works with both for me :) But I changed the other tomls too.

Copy link
Copy Markdown
Member Author

@razvan razvan Apr 3, 2023

Choose a reason for hiding this comment

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

Hm when I removed the // cargo complained that I have the same origin multiple times (which was true). It only worked with // (in the patch section)

@maltesander maltesander self-requested a review March 31, 2023 15:02
@maltesander
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Member

@maltesander maltesander 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 3, 2023
# Description

[CI](https://ci.stackable.tech/job/hbase-operator-it-custom/79/) failed but the failed tests work locally (on kind).

*Please add a description here. This will become the commit message of the merge request later.*



Co-authored-by: Malte Sander <malte.sander.it@gmail.com>
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Apr 3, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Implement pause/stop [Merged by Bors] - Implement pause/stop Apr 3, 2023
@bors bors bot closed this Apr 3, 2023
@bors bors bot deleted the feat/cluster-operation branch April 3, 2023 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants