From c8b6d66439f6161e77c792a3a38395ce89968816 Mon Sep 17 00:00:00 2001
From: Chris Suszynski <ksuszyns@redhat.com>
Date: Mon, 25 Jul 2022 14:51:24 +0200
Subject: [PATCH] Removal of deprecated, unused functions. (#365)

---
 pkg/environment/magic.go  | 10 ----------
 pkg/manifest/installer.go | 38 --------------------------------------
 2 files changed, 48 deletions(-)

diff --git a/pkg/environment/magic.go b/pkg/environment/magic.go
index 0990e8fa..5da25b62 100644
--- a/pkg/environment/magic.go
+++ b/pkg/environment/magic.go
@@ -194,16 +194,6 @@ func (mr *MagicGlobalEnvironment) Environment(opts ...EnvOpts) (context.Context,
 	return ctx, env
 }
 
-func (mr *MagicEnvironment) Images() map[string]string {
-	ctx := mr.c
-	if refs, err := ProduceImages(ctx); err != nil {
-		logging.FromContext(ctx).Fatal(err)
-		return nil
-	} else {
-		return refs
-	}
-}
-
 func (mr *MagicEnvironment) TemplateConfig(base map[string]interface{}) map[string]interface{} {
 	cfg := make(map[string]interface{})
 	for k, v := range base {
diff --git a/pkg/manifest/installer.go b/pkg/manifest/installer.go
index bd368e47..add00096 100644
--- a/pkg/manifest/installer.go
+++ b/pkg/manifest/installer.go
@@ -21,16 +21,11 @@ import (
 	"context"
 	"encoding/json"
 	"io/fs"
-	"os"
-	"path"
-	"runtime"
 	"strings"
 
 	"k8s.io/client-go/util/retry"
 	"knative.dev/pkg/injection/clients/dynamicclient"
 	"knative.dev/pkg/logging"
-	testlog "knative.dev/reconciler-test/pkg/logging"
-
 	"knative.dev/reconciler-test/pkg/environment"
 	"knative.dev/reconciler-test/pkg/feature"
 )
@@ -38,14 +33,6 @@ import (
 // CfgFn is the function signature of configuration mutation options.
 type CfgFn func(map[string]interface{})
 
-// Deprecated: use InstallYamlFS instead.
-func InstallYaml(ctx context.Context, dir string, base map[string]interface{}) (Manifest, error) {
-	log := loggingFrom(ctx, "InstallYaml")
-	log.Warn("InstallYaml is deprecated and will be removed, " +
-		"use InstallYamlFS instead.")
-	return InstallYamlFS(ctx, os.DirFS(dir), base)
-}
-
 func InstallYamlFS(ctx context.Context, fsys fs.FS, base map[string]interface{}) (Manifest, error) {
 	env := environment.FromContext(ctx)
 	images, err := environment.ProduceImages(ctx)
@@ -95,31 +82,6 @@ func InstallYamlFS(ctx context.Context, fsys fs.FS, base map[string]interface{})
 	return manifest, nil
 }
 
-// Deprecated: use InstallYamlFS instead.
-func InstallLocalYaml(ctx context.Context, base map[string]interface{}) (Manifest, error) {
-	log := loggingFrom(ctx, "InstallLocalYaml")
-	pwd, _ := os.Getwd()
-	log.Debug("PWD: ", pwd)
-	_, filename, _, _ := runtime.Caller(1)
-	log.Debug("FILENAME: ", filename)
-
-	return InstallYamlFS(ctx, os.DirFS(path.Dir(filename)), base)
-}
-
-// Deprecated: use ImagesFromFS instead.
-func ImagesLocalYaml() []string {
-	ctx := testlog.NewContext()
-	log := loggingFrom(ctx, "ImagesLocalYaml")
-	log.Warn("ImagesLocalYaml is deprecated and will be removed, " +
-		"use ImagesFromFS instead.")
-	pwd, _ := os.Getwd()
-	log.Debug("PWD: ", pwd)
-	_, filename, _, _ := runtime.Caller(1)
-	log.Debug("FILENAME: ", filename)
-
-	return ImagesFromFS(ctx, os.DirFS(path.Dir(filename)))
-}
-
 func ImagesFromFS(ctx context.Context, fsys fs.FS) []string {
 	log := logging.FromContext(ctx)
 	var images []string