Add no-push parameter to build composite (#195)

* add no-push to builder arg

* make repo and registry optional

* fix shebang issue in idp shell

* convert input to json for if

* change eval criteria

* switch to fromJSON

Author: Tyler Titsworth

.github/action.yml

@@ -13,12 +13,19 @@ inputs:
     type: string
   registry:
     description: Container Registry URL
-    required: true
+    required: false
+    default: 'intel'
     type: string
   repo:
     description: Container Project Repository
-    required: true
+    required: false
+    default: 'ai-containers'
     type: string
+  no-push:
+    description: Do not push to Registry
+    required: false
+    default: false
+    type: boolean
 runs:
   using: composite
   steps:
@@ -35,6 +42,7 @@ runs:
       working-directory: ${{ inputs.group_dir }}
     - name: Push Containers
       shell: bash
+      if: ${{ !fromJson(inputs.no-push) }}
       run: |
         REGISTRY=${{ inputs.registry }} \
         REPO=${{ inputs.repo }} \

.github/linters/.hadolint.yaml

@@ -8,3 +8,4 @@ ignored:
   - DL4006
   - SC1008
   - SC2140
+  - SC2239

.github/workflows/container-pipeline-tester.yaml

@@ -87,6 +87,7 @@ jobs:
           env_overrides: ${{ inputs.env_overrides }}
           registry: ${{ vars.REGISTRY }}
           repo: ${{ vars.REPO }}
+          no-push: false
   setup-test:
     needs: [ build-containers ]
     runs-on: [ k8-runners ]

python/Dockerfile

@@ -65,7 +65,7 @@ ENV PATH /opt/conda/envs/idp/bin:/opt/conda/condabin:/opt/conda/bin/:${PATH}
 ENV LD_LIBRARY_PATH /lib64/:/usr/lib64/:/usr/local/lib64:/opt/conda/envs/idp/lib:${LD_LIBRARY_PATH}
 
 RUN echo "conda activate idp" >> ~/.bashrc
-SHELL ["/opt/conda", "run", "-n", "idp", "/bin/bash", "-c"]
+SHELL ["conda", "run", "-n", "idp", "/bin/bash", "-c"]
 
 RUN conda run -n idp python -m pip install --no-cache-dir numpy==1.22.4