Review if patch(context, string) should handle the rolling case

[shawkins]

kubernetes-client/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/internal/apps/v1/RollableScalableResourceOperation.java

Line 171 in 74cc63d

return getRollingUpdater(rollingTimeout, rollingTimeUnit).rollUpdate(getMandatory(), t);

the old patch and other operations consider the rolling case separately. This should be reviewed if it is needed for the new patch(context, string) method and/or if the rolling case could be handled differently given the additional base logic that's now available.

This is related to #3070

[shawkins]

I opened #3221 after reviewing this issue - the changes are just to make the operation logic as consistent as possible. I saw that openshift operations should override the builder method and the other logic dealing with special case or modifying the item should better account for the patch logic.