search

stackabletech / druid-operator

An Operator for Apache Druid for Stackable Data Platform
Other
10 stars 0 forks source link

chore: Bump stackable-operator and product-config #585

Closed Techassi closed 4 months ago

Techassi commented 4 months ago

Part of https://github.com/stackabletech/issues/issues/601, https://github.com/stackabletech/issues/issues/557

# Reviewer
- [x] Changelog updated
- [x] Cargo.toml only contains references to git tags (not specific commits or branches)
Techassi commented 4 months ago

stackable-operator 0.70.0 needs re-tagging. Moving this back to "In Progress".

sbernauer commented 3 months ago

Hi @Techassi and @NickLarsenNZ,, I just wanted to ask why we added a changelog entry for this PR and https://github.com/stackabletech/druid-operator/pull/587? I though according to https://github.com/stackabletech/decisions/issues/17#issuecomment-2191148634 we

Only include dependency bumps in the changelog when they introduce breaking changes

which I can not see here (druid-op users should not notice this bump). I'm currently bumping to operator-rs 0.73.0 and need to adopt the changelog again. I would be in favor of removing this changelog entries (as we did in the past), WDYT?

NickLarsenNZ commented 3 months ago

@sbernauer, I can't answer that.