Merge pull request #85 from ContainerSolutions/release-with-github-actions

Release with GitHub actions

Author: csviri

.github/workflows/release.yml

@@ -0,0 +1,30 @@
+name: Release to Maven Central
+on:
+  release:
+    types: [created]
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Maven Central Repository
+        uses: actions/setup-java@v1
+        with:
+          java-version: 1.8
+      - name: change version to release version
+        # Assume that RELEASE_VERSION will have form like: "v1.0.1". So we cut the "v"
+        run: mvn versions:set "${RELEASE_VERSION:1}" versions:commit
+        env:
+          RELEASE_VERSION: ${{ github.event.release.tag_name }}
+      #      - name: Release Maven package
+      #        uses: samuelmeuli/action-maven-publish@v1
+      #        with:
+      #          maven_profiles: "release"
+      #          gpg_private_key: ${{ secrets.GPG_PRIVATE_KEY }}
+      #          gpg_passphrase: ${{ secrets.GPG_PASSPHRASE }}
+      #          nexus_username: ${{ secrets.OSSRH_USERNAME }}
+      #          nexus_password: ${{ secrets.OSSRH_TOKEN }}
+      - name: Set version to next snapshot version
+        run: mvn -q build-helper:parse-version versions:set -DnewVersion=\${parsedVersion.majorVersion}.\${parsedVersion.minorVersion}.\${parsedVersion.nextIncrementalVersion}-SNAPSHOT versions:commit
+      - name: Push changes to master
+        uses: tcitry/push-to-master@v1.0

operator-framework/src/main/java/com/github/containersolutions/operator/Operator.java

@@ -26,8 +26,6 @@
 public class Operator {
 
     private final static Logger log = LoggerFactory.getLogger(Operator.class);
-
-    private final Retry defaultRetry = GenericRetry.defaultLimitedExponentialRetry();
     private final KubernetesClient k8sClient;
     private Map<Class<? extends CustomResource>, CustomResourceOperationsImpl> customResourceClients = new HashMap<>();
 
@@ -37,15 +35,15 @@ public Operator(KubernetesClient k8sClient) {
 
 
     public <R extends CustomResource> void registerControllerForAllNamespaces(ResourceController<R> controller) throws OperatorException {
-        registerController(controller, true, defaultRetry);
+        registerController(controller, true, GenericRetry.defaultLimitedExponentialRetry());
     }
 
     public <R extends CustomResource> void registerControllerForAllNamespaces(ResourceController<R> controller, Retry retry) throws OperatorException {
         registerController(controller, true, retry);
     }
 
     public <R extends CustomResource> void registerController(ResourceController<R> controller, String... targetNamespaces) throws OperatorException {
-        registerController(controller, false, defaultRetry, targetNamespaces);
+        registerController(controller, false, GenericRetry.defaultLimitedExponentialRetry(), targetNamespaces);
     }
 
     public <R extends CustomResource> void registerController(ResourceController<R> controller, Retry retry, String... targetNamespaces) throws OperatorException {
@@ -99,7 +97,7 @@ public Map<Class<? extends CustomResource>, CustomResourceOperationsImpl> getCus
         return customResourceClients;
     }
 
-    public <T extends CustomResource, L extends CustomResourceList<T>, D extends CustomResourceDoneable<T>> CustomResourceOperationsImpl<T, L, D> 
+    public <T extends CustomResource, L extends CustomResourceList<T>, D extends CustomResourceDoneable<T>> CustomResourceOperationsImpl<T, L, D>
     getCustomResourceClients(Class<T> customResourceClass) {
         return customResourceClients.get(customResourceClass);
     }