Skip to content

chore: bump operator-rs to 0.60.1#366

Merged
NickLarsenNZ merged 31 commits intomainfrom
bump/operator-rs-0.60.0
Jan 5, 2024
Merged

chore: bump operator-rs to 0.60.1#366
NickLarsenNZ merged 31 commits intomainfrom
bump/operator-rs-0.60.0

Conversation

@NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented Jan 3, 2024 •
edited
Loading

Description

Supersedes #364. (I renamed the branch, then Github closed the old PR, oops).

  • BREAKING Removed legacy node selector on roleGroups.
  • Use new label builders.
  • Use new ldap::AuthorizationClassProvider endpoint_url() method

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
- [ ] CRD documentation for all fields, following the [style guide](https://docs.stackable.tech/home/nightly/contributor/style-guide).
- [ ] Helm chart can be installed and deployed operator works
- [ ] Integration tests passed (for non trivial changes)
- [ ] Changes need to be "offline" compatible

# Reviewer
- [x] Code contains useful comments
- [x] (Integration-)Test cases added
- [x] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/style-guide).
- [x] Changelog updated
- [x] 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

NickLarsenNZ and others added 24 commits November 23, 2023 13:42
@NickLarsenNZ
bump: operator-rs to 0.56.1
46bafbf
@NickLarsenNZ
upgrade operator-rs to 0.59.0
82330b0
@NickLarsenNZ
remove legacy selector for rolegoup
336657a
@NickLarsenNZ
error out for unsupported authclass providers
6a213bd
@NickLarsenNZ
rename labels module import to kvp
555d21b
@NickLarsenNZ
refactor ldap auth class provider usage
f9d480f
@NickLarsenNZ
use new labels stuff, and break up manifest objects
4161d6e
@NickLarsenNZ
new version parameter for print_yaml_schema
245ae2f
@NickLarsenNZ
add fallable volume mounts for ldap backend
32e7342
@NickLarsenNZ
remove legacy node affinity test
2373572
@NickLarsenNZ
docs: changelog
d62da7f
@NickLarsenNZ
doc: changelog updates and corrections
df2ccb0
@NickLarsenNZ
update crds
e117749
@NickLarsenNZ
Merge remote-tracking branch 'origin/main' into bump/operator-rs-0.59.0
5d5d82b
@NickLarsenNZ
update crds (after merge)
00c7a60
@NickLarsenNZ
you got it dogg
68c7a8e
@NickLarsenNZ
keep AuthenticationClassProvider matches explicit
eda48f4
@NickLarsenNZ
move the changelog entry into unreleased
2097e68
@NickLarsenNZ
no more tracking operator upgrades (was in the wrong section anyway)
3b6e532
@NickLarsenNZ
move another changelog entry into unreleased
81d42fc
@NickLarsenNZ
no more tracking operator upgrades (was in the wrong section anyway)
3f32d14
@NickLarsenNZ @sbernauer
Update CHANGELOG.md
7ac61fc 
Co-authored-by: Sebastian Bernauer <[email protected]>
@NickLarsenNZ
bump operator-rs to 0.60.0
8e79917
@NickLarsenNZ
Use new PR after branch rename mishap
6e8fbdf
github-actions[bot]
github-actions bot reviewed Jan 3, 2024
View reviewed changes
NickLarsenNZ
NickLarsenNZ commented Jan 3, 2024
View reviewed changes
@NickLarsenNZ NickLarsenNZ requested a review from sbernauer January 4, 2024 08:53
sbernauer
sbernauer previously approved these changes Jan 4, 2024
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Started test. LGTM when test passes
https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/airflow-operator-it-custom/116/

@sbernauer
Copy link
Member

sbernauer commented Jan 4, 2024

Retry https://ci.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/airflow-operator-it-custom/117/

@NickLarsenNZ NickLarsenNZ requested a review from sbernauer January 4, 2024 11:29
@sbernauer sbernauer mentioned this pull request Jan 4, 2024
Merged
@sbernauer
Copy link
Member

sbernauer commented Jan 5, 2024

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

@NickLarsenNZ NickLarsenNZ changed the title chore: bump operator-rs to 0.60.0 chore: bump operator-rs to 0.60.1 Jan 5, 2024
sbernauer
sbernauer approved these changes Jan 5, 2024
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Test passed 🥳

@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Jan 5, 2024
@NickLarsenNZ NickLarsenNZ requested a review from a team January 5, 2024 14:24
Merged via the queue into main with commit 4758707 Jan 5, 2024
@NickLarsenNZ NickLarsenNZ deleted the bump/operator-rs-0.60.0 branch January 5, 2024 14:26
@NickLarsenNZ NickLarsenNZ mentioned this pull request Jan 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@sbernauer sbernauer sbernauer approved these changes

Assignees

@NickLarsenNZ NickLarsenNZ

Labels

None yet

Projects

Stackable Engineering
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NickLarsenNZ @sbernauer