Skip to content

Commit

Permalink
Merge pull request #72 from redpanda-data/clean-config
Browse files Browse the repository at this point in the history
topic: remove cleanup.policy filtering
  • Loading branch information
r-vasquez authored Apr 19, 2024
2 parents 57b8f92 + db1174e commit a2681d7
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions redpanda/resources/topic/resource_topic.go
Original file line number Diff line number Diff line change
Expand Up @@ -350,10 +350,7 @@ func filterDynamicConfig(configs []*dataplanev1alpha1.Topic_Configuration) []*da
var filtered []*dataplanev1alpha1.Topic_Configuration
for _, cfg := range configs {
if cfg != nil {
// cleanup.policy always report the source as Dynamic even if we are
// using the default. We can't manage this property for now.
// See https://github.com/redpanda-data/redpanda/issues/2225
if cfg.Source == dataplanev1alpha1.ConfigSource_CONFIG_SOURCE_DYNAMIC_TOPIC_CONFIG && cfg.Name != "cleanup.policy" {
if cfg.Source == dataplanev1alpha1.ConfigSource_CONFIG_SOURCE_DYNAMIC_TOPIC_CONFIG {
filtered = append(filtered, cfg)
}
}
Expand Down

0 comments on commit a2681d7

Please sign in to comment.