format two yaml files and exclude all upstream folders

Signed-off-by: juliusvonkohout <[email protected]>
kubeflow · Nov 19, 2024 · 383425e · 383425e
1 parent 8f1724a
commit 383425e
Show file tree

Hide file tree

Showing 4 changed files with 55 additions and 50 deletions.
diff --git a/.github/workflows/linting_bash_python_yaml_files.yaml b/.github/workflows/linting_bash_python_yaml_files.yaml
@@ -71,7 +71,10 @@ jobs:
     - name: Set up changed files
       id: changed_files
       run: |
-        git diff --name-only --diff-filter=AM origin/master...HEAD | grep -E '^common/.*\.ya?ml$|^example/.*\.ya?ml$|^hack/.*\.ya?ml$|^tests/.*\.ya?ml$|^.github/.*\.ya?ml$' > changed_files_in_PR.txt || true
+        git diff --name-only --diff-filter=AM origin/master...HEAD \
+        | grep -v 'upstream' \
+        | grep -E '^common/.*\.ya?ml$|^example/.*\.ya?ml$|^hack/.*\.ya?ml$|^tests/.*\.ya?ml$|^.github/.*\.ya?ml$' \
+        > changed_files_in_PR.txt || true
         if [ ! -s changed_files_in_PR.txt ]; then
           echo "No YAML files have changed in this PR." > changed_files_in_PR.txt
         fi

diff --git a/changed_files_in_PR.txt b/changed_files_in_PR.txt
@@ -0,0 +1,2 @@
+common/knative/knative-eventing-post-install-jobs/base/eventing-post-install.yaml
+common/knative/knative-serving-post-install-jobs/base/serving-post-install-jobs.yaml
diff --git a/common/knative/knative-eventing-post-install-jobs/base/eventing-post-install.yaml b/common/knative/knative-eventing-post-install-jobs/base/eventing-post-install.yaml
@@ -26,28 +26,28 @@ spec:
       serviceAccountName: knative-eventing-post-install-job
       restartPolicy: OnFailure
       containers:
-        - name: migrate
-          image: gcr.io/knative-releases/knative.dev/pkg/apiextensions/storageversion/cmd/migrate@sha256:f1786ed71c979b93e3fba02c4cfb3df33d97be0cce2c9ef994bfba4cc15a5558
-          args:
-            - "apiserversources.sources.knative.dev"
-            - "brokers.eventing.knative.dev"
-            - "channels.messaging.knative.dev"
-            - "containersources.sources.knative.dev"
-            - "eventtypes.eventing.knative.dev"
-            - "inmemorychannels.messaging.knative.dev"
-            - "parallels.flows.knative.dev"
-            - "pingsources.sources.knative.dev"
-            - "sequences.flows.knative.dev"
-            - "sinkbindings.sources.knative.dev"
-            - "subscriptions.messaging.knative.dev"
-            - "triggers.eventing.knative.dev"
-            - "jobsinks.sinks.knative.dev"
-          securityContext:
-            allowPrivilegeEscalation: false
-            readOnlyRootFilesystem: true
-            runAsNonRoot: true
-            capabilities:
-              drop:
-                - ALL
-            seccompProfile:
-              type: RuntimeDefault
+      - name: migrate
+        image: gcr.io/knative-releases/knative.dev/pkg/apiextensions/storageversion/cmd/migrate@sha256:f1786ed71c979b93e3fba02c4cfb3df33d97be0cce2c9ef994bfba4cc15a5558
+        args:
+        - "apiserversources.sources.knative.dev"
+        - "brokers.eventing.knative.dev"
+        - "channels.messaging.knative.dev"
+        - "containersources.sources.knative.dev"
+        - "eventtypes.eventing.knative.dev"
+        - "inmemorychannels.messaging.knative.dev"
+        - "parallels.flows.knative.dev"
+        - "pingsources.sources.knative.dev"
+        - "sequences.flows.knative.dev"
+        - "sinkbindings.sources.knative.dev"
+        - "subscriptions.messaging.knative.dev"
+        - "triggers.eventing.knative.dev"
+        - "jobsinks.sinks.knative.dev"
+        securityContext:
+          allowPrivilegeEscalation: false
+          readOnlyRootFilesystem: true
+          runAsNonRoot: true
+          capabilities:
+            drop:
+            - ALL
+          seccompProfile:
+            type: RuntimeDefault
diff --git a/common/knative/knative-serving-post-install-jobs/base/serving-post-install-jobs.yaml b/common/knative/knative-serving-post-install-jobs/base/serving-post-install-jobs.yaml
@@ -24,27 +24,27 @@ spec:
       serviceAccountName: controller
       restartPolicy: OnFailure
       containers:
-        - name: migrate
-          image: gcr.io/knative-releases/knative.dev/pkg/apiextensions/storageversion/cmd/migrate@sha256:c2f7830569ab0b9f40ba785796d7a1d3e2069987528f5ca945ab7a339b0d96e7
-          args:
-            - "services.serving.knative.dev"
-            - "configurations.serving.knative.dev"
-            - "revisions.serving.knative.dev"
-            - "routes.serving.knative.dev"
-            - "domainmappings.serving.knative.dev"
-          resources:
-            requests:
-              cpu: 100m
-              memory: 100Mi
-            limits:
-              cpu: 1000m
-              memory: 1000Mi
-          securityContext:
-            allowPrivilegeEscalation: false
-            readOnlyRootFilesystem: true
-            runAsNonRoot: true
-            capabilities:
-              drop:
-                - ALL
-            seccompProfile:
-              type: RuntimeDefault
+      - name: migrate
+        image: gcr.io/knative-releases/knative.dev/pkg/apiextensions/storageversion/cmd/migrate@sha256:c2f7830569ab0b9f40ba785796d7a1d3e2069987528f5ca945ab7a339b0d96e7
+        args:
+        - "services.serving.knative.dev"
+        - "configurations.serving.knative.dev"
+        - "revisions.serving.knative.dev"
+        - "routes.serving.knative.dev"
+        - "domainmappings.serving.knative.dev"
+        resources:
+          requests:
+            cpu: 100m
+            memory: 100Mi
+          limits:
+            cpu: 1000m
+            memory: 1000Mi
+        securityContext:
+          allowPrivilegeEscalation: false
+          readOnlyRootFilesystem: true
+          runAsNonRoot: true
+          capabilities:
+            drop:
+            - ALL
+          seccompProfile:
+            type: RuntimeDefault
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		common/knative/knative-eventing-post-install-jobs/base/eventing-post-install.yaml
		common/knative/knative-serving-post-install-jobs/base/serving-post-install-jobs.yaml