Allow to skip dockerfilegen

Author: cardil

pkg/config/defaults.go

@@ -37,7 +37,9 @@ func newDefaults(project Project) Config {
 			ApplyForkFiles:  ":open_file_folder: Apply fork specific files",
 			ImagesGenerated: ":vhs: Images generated",
 		},
-		SyncLabels:    []string{"kind/sync-fork-to-upstream"},
-		DockerfileGen: dockerfilegen.DefaultParams(project.Path),
+		SyncLabels: []string{"kind/sync-fork-to-upstream"},
+		DockerfileGen: DockerfileGen{
+			Params: dockerfilegen.DefaultParams(project.Path),
+		},
 	}
 }

pkg/config/new_test.go

@@ -0,0 +1,39 @@
+package config_test
+
+import (
+	"os"
+	"path"
+	"testing"
+
+	"github.com/openshift-knative/deviate/pkg/config"
+	"github.com/openshift-knative/deviate/pkg/log"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestNew(t *testing.T) {
+	configYaml := `upstream: https://github.com/example/project.git
+dockerfileGen:
+  skip: true
+  images-from:
+    - eventing`
+	tmp := t.TempDir()
+	configPath := path.Join(tmp, ".deviate.yaml")
+	if err := os.WriteFile(configPath, []byte(configYaml), 0o650); err != nil {
+		require.NoError(t, err)
+	}
+	project := config.Project{
+		Path:       tmp,
+		ConfigPath: configPath,
+	}
+	cfg, err := config.New(project, log.TestingLogger{T: t}, noopInformer{})
+	require.NoError(t, err)
+	assert.True(t, cfg.DockerfileGen.Skip)
+	assert.Equal(t, cfg.DockerfileGen.ImagesFromRepositories, []string{"eventing"})
+}
+
+type noopInformer struct{}
+
+func (n noopInformer) Remote(name string) (string, error) {
+	return name, nil
+}

pkg/config/structure.go

@@ -9,7 +9,7 @@ type Config struct {
 	DryRun                     bool                 `json:"dryRun"`
 	GithubWorkflowsRemovalGlob string               `json:"githubWorkflowsRemovalGlob" valid:"required"`
 	SyncLabels                 []string             `json:"syncLabels"                 valid:"required"`
-	DockerfileGen              dockerfilegen.Params `json:"dockerfileGen"`
+	DockerfileGen              DockerfileGen        `json:"dockerfileGen"`
 	ResyncReleases             `json:"resyncReleases"`
 	Branches                   `json:"branches"`
 	Tags                       `json:"tags"`
@@ -56,3 +56,9 @@ type Searches struct {
 	UpstreamReleases   string `json:"upstreamReleases"   valid:"required"`
 	DownstreamReleases string `json:"downstreamReleases" valid:"required"`
 }
+
+// DockerfileGen wraps dockerfilegen.Params adding a skip param.
+type DockerfileGen struct {
+	dockerfilegen.Params
+	Skip bool `json:"skip"`
+}

pkg/log/testing.go

@@ -0,0 +1,18 @@
+package log
+
+import "testing"
+
+// TestingLogger implements a testing logger.
+type TestingLogger struct {
+	T testing.TB
+}
+
+func (t TestingLogger) Println(v ...interface{}) {
+	t.T.Log(v...)
+}
+
+func (t TestingLogger) Printf(format string, v ...interface{}) {
+	t.T.Logf(format, v...)
+}
+
+var _ Logger = TestingLogger{}

pkg/sync/generate_images.go

@@ -12,16 +12,20 @@ import (
 
 func (o Operation) generateImages(rel release) step {
 	return func() error {
-		o.Println("- Generating images")
 		params := o.Config.DockerfileGen
+		if params.Skip {
+			o.Println("- Skipping image generation")
+			return nil
+		}
+		o.Println("- Generating images")
 		var closer func()
 		var err error
 		params.ProjectFilePath, closer, err = tempProjectFile(o, rel)
 		defer closer()
 		if err != nil {
 			return err
 		}
-		return dockerfilegen.GenerateDockerfiles(params)
+		return dockerfilegen.GenerateDockerfiles(params.Params)
 	}
 }