Revert "Directly specify next step following a response rather than use chaining"

This reverts commit d522d00.

Author: rjeberhard

operator/src/main/java/oracle/kubernetes/operator/ClusterResourceStatusUpdater.java

@@ -129,12 +129,14 @@ public Result onFailure(Packet packet, KubernetesApiResponse<ClusterResource> ca
     }
 
     private Step createRetry() {
-      return createClusterResourceRefreshStep(new SingleClusterResourceStatusUpdateStep(context.getClusterName()));
+      return Step.chain(
+          createClusterResourceRefreshStep(),
+          new SingleClusterResourceStatusUpdateStep(context.getClusterName()));
     }
 
-    private Step createClusterResourceRefreshStep(Step next) {
+    private Step createClusterResourceRefreshStep() {
       return RequestBuilder.CLUSTER.get(
-          context.getNamespace(), context.getClusterResourceName(), new ReadClusterResponseStep(next));
+          context.getNamespace(), context.getClusterResourceName(), new ReadClusterResponseStep());
     }
   }
 
@@ -284,11 +286,6 @@ private Step createReplaceClusterStatusAsyncStep() {
   }
 
   private static class ReadClusterResponseStep extends ResponseStep<ClusterResource> {
-
-    private ReadClusterResponseStep(Step step) {
-      super(step);
-    }
-
     @Override
     public Result onSuccess(Packet packet, KubernetesApiResponse<ClusterResource> callResponse) {
       if (callResponse.getObject() != null) {

operator/src/main/java/oracle/kubernetes/operator/steps/ShutdownManagedServerStep.java

@@ -362,12 +362,12 @@ public Result onFailure(Packet packet, HttpResponse<String> response) {
         LOGGER.info(MessageKeys.SERVER_SHUTDOWN_REST_FAILURE, serverName, response);
       }
       removeShutdownRequestRetryCount(packet);
-      return doNext(createDomainRefreshStep(getDomainPresenceInfo(packet).getDomainName(),
-          getDomainPresenceInfo(packet).getNamespace(), getNext()), packet);
+      return doNext(Step.chain(createDomainRefreshStep(getDomainPresenceInfo(packet).getDomainName(),
+          getDomainPresenceInfo(packet).getNamespace()), getNext()), packet);
     }
 
-    private Step createDomainRefreshStep(String domainName, String namespace, Step next) {
-      return RequestBuilder.DOMAIN.get(namespace, domainName, new DomainUpdateStep(next));
+    private Step createDomainRefreshStep(String domainName, String namespace) {
+      return RequestBuilder.DOMAIN.get(namespace, domainName, new DomainUpdateStep());
     }
 
     private boolean shouldRetry(Packet packet) {
@@ -402,11 +402,6 @@ void setHttpAsyncRequestStep(HttpRequestStep requestStep) {
   }
 
   static class DomainUpdateStep extends DefaultResponseStep<DomainResource> {
-
-    DomainUpdateStep(Step next) {
-      super(next);
-    }
-
     @Override
     public Result onSuccess(Packet packet, KubernetesApiResponse<DomainResource> callResponse) {
       if (callResponse.getObject() != null) {