Added script for checking diff of generated code (flyteorg#303)

* Added ci check for go generate

Signed-off-by: Yuvraj <[email protected]>

Signed-off-by: Katrina Rogan <[email protected]>

Co-authored-by: Katrina Rogan <[email protected]>

.github/workflows/pull_request.yml

@@ -133,3 +133,13 @@ jobs:
           GO111MODULE: "on"
         with:
           args: make install && make lint
+
+  generate:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - uses: actions/setup-go@v2
+        with:
+          go-version: '1.16'
+      - name: Go generate and diff
+        run: DELTA_CHECK=true make generate

Makefile

@@ -69,5 +69,3 @@ seed_projects:
 
 all: compile
 
-generate: download_tooling
-	@go generate ./...

boilerplate/flyte/golang_test_targets/Makefile

@@ -8,6 +8,10 @@
 download_tooling: #download dependencies (including test deps) for the package
 	@boilerplate/flyte/golang_test_targets/download_tooling.sh
 
+.PHONY: generate
+generate: download_tooling #generate go code
+	@boilerplate/flyte/golang_test_targets/go-gen.sh
+
 .PHONY: lint
 lint: download_tooling #lints the package for common code smells
 	GL_DEBUG=linters_output,env golangci-lint run --deadline=5m --exclude deprecated -v

boilerplate/flyte/golang_test_targets/go-gen.sh

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+
+set -ex
+
+echo "Generating pflags"
+go generate ./...
+
+# This section is used by GitHub workflow to ensure that the generation step was run
+if [ -n "$DELTA_CHECK" ]; then
+  DIRTY=$(git status --porcelain)
+  if [ -n "$DIRTY" ]; then
+    echo "FAILED: Go code updated without commiting generated code."
+    echo "Ensure make generate has run and all changes are committed."
+    DIFF=$(git diff)
+    echo "diff detected: $DIFF"
+    DIFF=$(git diff --name-only)
+    echo "files different: $DIFF"
+    exit 1
+  else
+    echo "SUCCESS: Generated code is up to date."
+  fi
+fi

pkg/runtime/interfaces/application_configuration.go

@@ -312,7 +312,7 @@ type SignedURL struct {
 }
 
 //go:generate enumer -type=InlineEventDataPolicy -trimprefix=InlineEventDataPolicy
-type InlineEventDataPolicy = int
+type InlineEventDataPolicy int
 
 const (
 	// InlineEventDataPolicyOffload specifies that inline execution event data (e.g. outputs) should be offloaded to the