infinispan / infinispan-operator

Infinispan Operator
https://infinispan.org/docs/infinispan-operator/main/operator.html
Apache License 2.0
48 stars 54 forks source link

Remove deprecated operand check #2131

Closed Crumby closed 2 months ago

Crumby commented 2 months ago

TestOperandUpgrade expects the oldest operand version to be deprecated which may not be true.

ryanemerson commented 2 months ago

@Crumby We need this to make sure that the deprecated field in the status is set correctly. Upstream we ensure that the oldest version in the testing Operand list is deprecated specifically for this check. Instead of removing it, could you surround the assertion with an if statement checking whether the oldest Operand is deprecated, and if not skip?

Crumby commented 2 months ago

@ryanemerson Fixed. I didn't realize it's actually checking for status field

ryanemerson commented 2 months ago

Thanks @Crumby