Skip to content

[Merged by Bors] - Bump operator-rs to 0.27.1#116

Closed
sbernauer wants to merge 2 commits intomainfrom
operator-rs-0.27.0
Closed

[Merged by Bors] - Bump operator-rs to 0.27.1#116
sbernauer wants to merge 2 commits intomainfrom
operator-rs-0.27.0

Conversation

@sbernauer
Copy link
Copy Markdown
Member

@sbernauer sbernauer commented Nov 17, 2022 •
edited by maltesander
Loading

Description

Fixes #113

Review Checklist

  • Code contains useful comments
  • CRD change approved (or not applicable)
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)
  • Helm chart can be installed and deployed operator works (or not applicable)

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

@sbernauer sbernauer requested a review from nightkr November 17, 2022 08:32
@maltesander
Copy link
Copy Markdown
Member

maltesander commented Nov 17, 2022

Can you bump to 0.27.1? 0.27.0 had a label bug stackabletech/operator-rs#507

@sbernauer
Copy link
Copy Markdown
Member Author

sbernauer commented Nov 17, 2022

Sure, thanks!

@maltesander maltesander self-requested a review November 18, 2022 13:52
@maltesander maltesander changed the title Bump operator-rs to 0.27.0 Bump operator-rs to 0.27.1 Nov 18, 2022
maltesander
maltesander approved these changes Nov 18, 2022
View reviewed changes
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.

LGTM!

@sbernauer
Copy link
Copy Markdown
Member Author

sbernauer commented Nov 23, 2022

bors r+

bors Bot pushed a commit that referenced this pull request Nov 23, 2022
@sbernauer
Bump operator-rs to 0.27.1 (#116)
5aee855 
# Description

Fixes #113
@bors
Copy link
Copy Markdown

bors Bot commented Nov 23, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors Bot changed the title Bump operator-rs to 0.27.1 [Merged by Bors] - Bump operator-rs to 0.27.1 Nov 23, 2022
@bors bors Bot closed this Nov 23, 2022
@bors bors Bot deleted the operator-rs-0.27.0 branch November 23, 2022 08:13
nightkr
nightkr reviewed Nov 23, 2022
View reviewed changes
Comment thread rust/operator-binary/src/restart_controller/statefulset.rs
Comment thread rust/operator-binary/src/restart_controller/statefulset.rs
@nightkr nightkr mentioned this pull request Nov 23, 2022
Closed
7 tasks
bors Bot pushed a commit that referenced this pull request Nov 23, 2022
@nightkr
Revert to also triggering when cms/secrets are deleted (#117)
9e14f90 
# Description

The equivalent for `try_flatten_touched` is `WatchStreamExt::touched_objects`, `WatchStreamExt::applied_objects` corresponds to `try_flatten_applied`.

No changelog entry for this since it's just fixing a regression introduced by #116.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maltesander maltesander maltesander approved these changes

+1 more reviewer

@nightkr nightkr nightkr left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bump operator-rs to 0.27.1

3 participants

@sbernauer @maltesander @nightkr