New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(kubectl): Extract interface for KubectlJobExecutor and use it. #6076
Conversation
…stead of the implementation.
Please delete the pull request instructions from the body of your pull request message. The instructions start with the line:
You can reopen your pull request when this has been addressed. |
import java.util.List; | ||
import javax.annotation.Nonnull; | ||
|
||
public interface KubectlJobExecutorInterface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's rename it to KubectlJobExecutor
and the implementation to DefaultKubectlJobExecutor
…stead of the implementation.
…stead of the implementation.
…stead of the implementation.
...es/src/main/java/com/netflix/spinnaker/clouddriver/kubernetes/op/job/KubectlJobExecutor.java
Outdated
Show resolved
Hide resolved
…stead of the implementation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving since this is only adding a marker interface that will improve the extensibility of the class, allowing downstream consumers to customize some parts of the OSS implementation
@mergify backport release-1.28.x |
❌ Sorry but I didn't understand the command. Please consult the commands documentation 📚. |
✅ Backports have been created
|
…6076) * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. (cherry picked from commit bb04878) # Conflicts: # clouddriver-kubernetes/src/main/java/com/netflix/spinnaker/clouddriver/kubernetes/op/handler/CanDeploy.java # clouddriver-kubernetes/src/main/java/com/netflix/spinnaker/clouddriver/kubernetes/op/job/KubectlJobExecutor.java # clouddriver-kubernetes/src/test/groovy/com/netflix/spinnaker/clouddriver/kubernetes/provider/view/KubernetesJobProviderSpec.groovy # clouddriver-kubernetes/src/test/java/com/netflix/spinnaker/clouddriver/kubernetes/op/job/DefaultKubectlJobExecutorTest.java
…pinnaker#6076) * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. * feat(kubectl): Extract interface for KubectlJobExecutor and use it instead of the implementation. (cherry picked from commit bb04878)
…se it. (spinnaker#6076)" This reverts commit bb04878.
This change is made to enhance the kubernetes module to add the possibility to decorate the class. This will be usefull when extending the functionalities of the the KubectlJobExecutor in the community or using plugins.