Reintroduce dummy `.spec.clusterConfig.listenerClass`

stackabletech / hdfs-operator

Apache Hadoop HDFS operator for Stackable

Other

40 stars 4 forks source link

Reintroduce dummy `.spec.clusterConfig.listenerClass` #461

Closed nightkr closed 9 months ago

nightkr commented 9 months ago

There was only one valid value, but some of our tests and examples try to set it explicitly. After some discussion we agreed to add it back as a dummy field for now.

Regressed in #450, reported at https://stackable-workspace.slack.com/archives/C031A56R127/p1706516386716159.

lfrancke commented 9 months ago

Can you elaborate what the future of this is?

Do we want to remove this field in the next version or will it stay around forever?

nightkr commented 9 months ago

It should be dropped in v1alpha2.

lfrancke commented 9 months ago

@sbernauer Could you add this to your tracking issue for future changes?

sbernauer commented 9 months ago

It already is on https://github.com/stackabletech/issues/issues/504 - It was actually the trigger for me to create it :)