[bot] Update from tektoncd/operator to 229dca3f57e177ede377bc5657bc80…

…0db730cbc6

    $ git diff --stat 229dca3f57e177ede377bc5657bc800db730cbc6..1b02a97fcd9908f2dab006417375e2dc1fe1d58d
     components.yaml                                    |  6 +--
     docs/TektonConfig.md                               |  2 +
     docs/TektonPipeline.md                             |  7 +++
     docs/TektonResult.md                               | 14 ++++--
     ...operator-rh.clusterserviceversion.template.yaml |  6 +--
     .../operator/v1alpha1/tektonconfig_default_test.go |  3 +-
     .../operator/v1alpha1/tektonpipeline_defaults.go   |  9 ++--
     .../v1alpha1/tektonpipeline_defaults_test.go       | 36 +--------------
     pkg/apis/operator/v1alpha1/tektonpipeline_types.go |  7 +--
     pkg/apis/operator/v1alpha1/tektonresult_types.go   |  2 -
     .../operator/v1alpha1/zz_generated.deepcopy.go     | 15 ++----
     pkg/reconciler/common/transformers_test.go         |  4 +-
     .../tektonpipeline-feature-flags-base.yaml         | 31 -------------
     .../kubernetes/tektonpipeline/transform.go         |  5 --
     .../kubernetes/tektonpipeline/transform_test.go    | 54 ----------------------
     .../kubernetes/tektonresult/tektonresult.go        | 12 ++++-
     .../tektonresult/testdata/api-config.yaml          |  2 -
     .../kubernetes/tektonresult/transform_test.go      |  5 --
     pkg/reconciler/openshift/tektonresult/extension.go | 31 -------------
     .../openshift/tektonresult/extension_test.go       | 19 --------
     .../tektonresult/testdata/api-service.yaml         | 25 ----------
     tekton/build-publish-images-manifests.yaml         |  8 ++--
     tekton/operator-release-pipeline.yaml              |  2 +-
     tekton/task-fetch-components.yaml                  |  2 +-
     24 files changed, 57 insertions(+), 250 deletions(-)

https://github.com/tektoncd/operator/compare/229dca3f57e177ede377bc5657bc800db730cbc6..1b02a97fcd9908f2dab006417375e2dc1fe1d58d

head

@@ -1 +1 @@
-1b02a97fcd9908f2dab006417375e2dc1fe1d58d
+229dca3f57e177ede377bc5657bc800db730cbc6

upstream/components.yaml

@@ -1,6 +1,6 @@
 chains:
   github: tektoncd/chains
-  version: v0.22.1
+  version: v0.22.2
 dashboard:
   github: tektoncd/dashboard
   version: v0.50.0
@@ -12,13 +12,13 @@ manual-approval-gate:
   version: v0.3.0
 pipeline:
   github: tektoncd/pipeline
-  version: v0.63.0
+  version: v0.64.0
 pipelines-as-code:
   github: openshift-pipelines/pipelines-as-code
   version: v0.28.0
 results:
   github: tektoncd/results
-  version: v0.12.0
+  version: v0.12.1
 triggers:
   github: tektoncd/triggers
   version: v0.29.1

upstream/docs/TektonConfig.md

@@ -61,7 +61,6 @@ The TektonConfig CR provides the following features
       enable-param-enum: false
       enable-provenance-in-status: true
       enable-step-actions: false
-      enable-tekton-oci-bundles: false
       enforce-nonfalsifiability: none
       keep-pod-on-cancel: false
       max-result-size: 4096
@@ -218,7 +217,6 @@ pipeline:
   disable-working-directory-overwrite: true
   enable-api-fields: stable
   enable-custom-tasks: false
-  enable-tekton-oci-bundles: false
   metrics.pipelinerun.duration-type: histogram
   metrics.pipelinerun.level: pipelinerun
   metrics.taskrun.duration-type: histogram

upstream/docs/TektonPipeline.md

@@ -35,7 +35,6 @@ spec:
   enable-param-enum: false
   enable-provenance-in-status: true
   enable-step-actions: false
-  enable-tekton-oci-bundles: false
   enforce-nonfalsifiability: none
   keep-pod-on-cancel: false
   max-result-size: 4096
@@ -113,12 +112,6 @@ injected sidecars, setting this option to false can lead to unexpected behavior.
     See more info [here](https://github.com/tektoncd/pipeline/issues/2981).
 
 
-- `enable-tekton-oci-bundles` (Default: `false`)
-
-    Setting this flag to "true" enables the use of Tekton OCI bundle. This is an experimental feature and thus should
-still be considered an alpha feature.
-
-
 - `enable-custom-tasks` (Default: `false`)
 
     Setting this flag to "true" enables the use of custom tasks from within pipelines. This is an experimental feature

upstream/docs/TektonResult.md

@@ -228,19 +228,11 @@ LOGGING_PLUGIN_NAMESPACE_KEY
 LOGGING_PLUGIN_STATIC_LABELS
 LOGGING_PLUGIN_TLS_VERIFICATION_DISABLE
 LOGGING_PLUGIN_FORWARDER_DELAY_DURATION
+LOGGING_PLUGIN_QUERY_PARAMS
+LOGGING_PLUGIN_QUERY_LIMIT
 ```
 
-- `LOGGING_PLUGIN_PROXY_PATH`: The path to the proxy used for logging. These reverse proxyies like NGinx is used for authentication.
-
-- `LOGGING_PLUGIN_API_URL`: The Base URL for quering Loki.
-
-- `LOGGING_PLUGIN_TOKEN_PATH`: The path to the token used for authentication with the logging service. `/var/run/secrets/kubernetes.io/serviceaccount/token` can be used if results api service account token can be used as Authorization jwt token for quering Loki.
-
-- `LOGGING_PLUGIN_NAMESPACE_KEY`: The key used to identify the namespace in log queries.
-
-- `LOGGING_PLUGIN_STATIC_LABELS`: Any static labels to be added to all log queries. It's necessary to have some fields to filter out tekton logs.
-
-- `LOGGING_PLUGIN_TLS_VERIFICATION_DISABLE`: A boolean flag to disable TLS verification. This should be set to "false" in production environments to ensure secure connections.
+Please consult the docs [here](https://github.com/tektoncd/results/blob/main/docs/logging-support.md) for information on these fields.
 
 These fields allow you to configure how Tekton Results interacts with your Loki backend.
 

upstream/operatorhub/openshift/manifests/bases/openshift-pipelines-operator-rh.clusterserviceversion.template.yaml

@@ -306,12 +306,12 @@ spec:
 
     ### Components
 
-    - Tekton Pipelines: v0.63.0
+    - Tekton Pipelines: v0.64.0
     - Tekton Triggers: v0.29.1
     - Pipelines as Code: v0.28.0
-    - Tekton Chains: v0.22.1
+    - Tekton Chains: v0.22.2
     - Tekton Hub (tech-preview): v1.18.0
-    - Tekton Results (tech-preview): v0.12.0
+    - Tekton Results (tech-preview): v0.12.1
     - Manual Approval Gate (tech-preview): v0.3.0
 
     ## Getting Started

upstream/pkg/apis/operator/v1alpha1/tektonconfig_default_test.go

@@ -69,8 +69,7 @@ func Test_SetDefaults_Pipeline_Properties(t *testing.T) {
 	}
 
 	tc.SetDefaults(context.TODO())
-	if *tc.Spec.Pipeline.SendCloudEventsForRuns != true ||
-		*tc.Spec.Pipeline.EnableTektonOciBundles != false {
+	if *tc.Spec.Pipeline.SendCloudEventsForRuns != true {
 		t.Error("Setting default failed for TektonConfig (spec.pipeline.pipelineProperties)")
 	}
 }

upstream/pkg/apis/operator/v1alpha1/tektonpipeline_defaults.go

@@ -48,9 +48,12 @@ func (p *Pipeline) setDefaults() {
 	if p.RequireGitSshSecretKnownHosts == nil {
 		p.RequireGitSshSecretKnownHosts = ptr.Bool(config.DefaultRequireGitSSHSecretKnownHosts)
 	}
-	if p.EnableTektonOciBundles == nil {
-		p.EnableTektonOciBundles = ptr.Bool(config.DefaultEnableTektonOciBundles)
-	}
+
+	// not in use, see: https://github.com/tektoncd/pipeline/pull/7789
+	// this field is removed from pipeline component
+	// keeping here to maintain the API compatibility
+	p.EnableTektonOciBundles = nil
+
 	if p.EnableCustomTasks == nil {
 		// EnableCustomTask is always enable
 		p.EnableCustomTasks = ptr.Bool(true)

upstream/pkg/apis/operator/v1alpha1/tektonpipeline_defaults_test.go

@@ -21,6 +21,7 @@ import (
 	"testing"
 
 	"github.com/google/go-cmp/cmp"
+	"github.com/stretchr/testify/assert"
 	"github.com/tektoncd/pipeline/pkg/apis/config"
 	"github.com/tektoncd/pipeline/test/diff"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -45,7 +46,6 @@ func Test_SetDefaults_PipelineProperties(t *testing.T) {
 		AwaitSidecarReadiness:                    ptr.Bool(true),
 		RunningInEnvironmentWithInjectedSidecars: ptr.Bool(true),
 		RequireGitSshSecretKnownHosts:            ptr.Bool(false),
-		EnableTektonOciBundles:                   ptr.Bool(false),
 		EnableCustomTasks:                        ptr.Bool(true),
 		EnableApiFields:                          "beta",
 		EmbeddedStatus:                           "",
@@ -84,3 +84,37 @@ func Test_SetDefaults_PipelineProperties(t *testing.T) {
 		t.Errorf("failed to update deployment %s", diff.PrintWantGot(d))
 	}
 }
+
+// not in use, see: https://github.com/tektoncd/pipeline/pull/7789
+// this field is removed from pipeline component
+// keeping in types to maintain the API compatibility
+// this test verifies that, "EnableTektonOciBundles" always keeps nil on defaults
+func TestEnableTektonOciBundlesIgnored(t *testing.T) {
+	tp := &TektonPipeline{
+		Spec: TektonPipelineSpec{
+			Pipeline: Pipeline{
+				PipelineProperties: PipelineProperties{
+					EnableTektonOciBundles: ptr.Bool(true),
+				},
+			},
+		},
+	}
+	ctx := context.TODO()
+
+	tests := []struct {
+		name                   string
+		enableTektonOciBundles *bool
+	}{
+		{name: "with-true", enableTektonOciBundles: ptr.Bool(true)},
+		{name: "with-false", enableTektonOciBundles: ptr.Bool(false)},
+		{name: "with-nil", enableTektonOciBundles: nil},
+	}
+
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			tp.Spec.Pipeline.EnableTektonOciBundles = test.enableTektonOciBundles
+			tp.SetDefaults(ctx)
+			assert.Nil(t, tp.Spec.Pipeline.EnableTektonOciBundles, "EnableTektonOciBundles removed from pipeline and should be nil on defaulting")
+		})
+	}
+}

upstream/pkg/apis/operator/v1alpha1/tektonpipeline_types.go

@@ -93,7 +93,6 @@ type PipelineProperties struct {
 	AwaitSidecarReadiness                    *bool  `json:"await-sidecar-readiness,omitempty"`
 	RunningInEnvironmentWithInjectedSidecars *bool  `json:"running-in-environment-with-injected-sidecars,omitempty"`
 	RequireGitSshSecretKnownHosts            *bool  `json:"require-git-ssh-secret-known-hosts,omitempty"`
-	EnableTektonOciBundles                   *bool  `json:"enable-tekton-oci-bundles,omitempty"`
 	EnableCustomTasks                        *bool  `json:"enable-custom-tasks,omitempty"`
 	EnableApiFields                          string `json:"enable-api-fields,omitempty"`
 	EmbeddedStatus                           string `json:"embedded-status,omitempty"`
@@ -102,6 +101,12 @@ type PipelineProperties struct {
 	// This field will be removed, see https://github.com/tektoncd/operator/issues/1497
 	// originally this field was removed in https://github.com/tektoncd/operator/pull/1481
 	// there is no use with this field, just adding back to unblock the upgrade
+
+	// not in use, see: https://github.com/tektoncd/pipeline/pull/7789
+	// this field is removed from pipeline component
+	// keeping here to maintain the API compatibility
+	EnableTektonOciBundles *bool `json:"enable-tekton-oci-bundles,omitempty"`
+
 	VerificationMode          string `json:"verification-mode,omitempty"`
 	VerificationNoMatchPolicy string `json:"trusted-resources-verification-no-match-policy,omitempty"`
 	EnableProvenanceInStatus  *bool  `json:"enable-provenance-in-status,omitempty"`

upstream/pkg/apis/operator/v1alpha1/tektonresult_types.go

@@ -97,6 +97,8 @@ type ResultsAPIProperties struct {
 	LoggingPluginStaticLabels           string `json:"logging_plugin_static_labels,omitempty"`
 	LoggingPluginCACert                 string `json:"logging_plugin_ca_cert,omitempty"`
 	LoggingPluginForwarderDelayDuration *uint  `json:"logging_plugin_forwarder_delay_duration,omitempty"`
+	LoggingPluginQueryLimit             *uint  `json:"logging_plugin_query_limit,omitempty"`
+	LoggingPluginQueryParams            string `json:"logging_plugin_query_params,omitempty"`
 	// Options holds additions fields and these fields will be updated on the manifests
 	Options AdditionalOptions `json:"options"`
 }

upstream/pkg/reconciler/common/transformers_test.go

@@ -515,8 +515,7 @@ func TestAddConfigMapValues_PipelineProperties(t *testing.T) {
 	assertNoEror(t, err)
 
 	prop := v1alpha1.PipelineProperties{
-		EnableTektonOciBundles: ptr.Bool(true),
-		EnableApiFields:        "stable",
+		EnableApiFields: "stable",
 	}
 
 	manifest, err = manifest.Transform(AddConfigMapValues("test1", prop))
@@ -527,7 +526,6 @@ func TestAddConfigMapValues_PipelineProperties(t *testing.T) {
 	assertNoEror(t, err)
 
 	assert.Equal(t, cm.Data["foo"], "bar")
-	assert.Equal(t, cm.Data["enable-tekton-oci-bundles"], "true")
 	assert.Equal(t, cm.Data["enable-api-fields"], "stable")
 }
 

upstream/pkg/reconciler/kubernetes/tektonpipeline/testdata/tektonpipeline-feature-flags-base.yaml

@@ -0,0 +1,31 @@
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: feature-flags
+  namespace: tekton-pipelines
+  labels:
+    app.kubernetes.io/instance: default
+    app.kubernetes.io/part-of: tekton-pipelines
+data:
+  disable-affinity-assistant: "false"
+  coschedule: "workspaces"
+  disable-creds-init: "false"
+  await-sidecar-readiness: "true"
+  running-in-environment-with-injected-sidecars: "true"
+  require-git-ssh-secret-known-hosts: "false"
+  enable-tekton-oci-bundles: "false"
+  enable-api-fields: "beta"
+  send-cloudevents-for-runs: "false"
+  trusted-resources-verification-no-match-policy: "ignore"
+  enable-provenance-in-status: "true"
+  enforce-nonfalsifiability: "none"
+  results-from: "termination-message"
+  set-security-context: "false"
+  keep-pod-on-cancel: "false"
+  enable-cel-in-whenexpression: "false"
+  enable-step-actions: "false"
+  enable-artifacts: "false"
+  enable-param-enum: "false"
+  disable-inline-spec: ""
+  enable-concise-resolver-syntax: "false"
+  enable-kubernetes-sidecar: "false"

upstream/pkg/reconciler/kubernetes/tektonpipeline/transform.go

@@ -56,6 +56,11 @@ func filterAndTransform(extension common.Extension) client.FilterAndTransform {
 	return func(ctx context.Context, manifest *mf.Manifest, comp v1alpha1.TektonComponent) (*mf.Manifest, error) {
 		pipeline := comp.(*v1alpha1.TektonPipeline)
 
+		// not in use, see: https://github.com/tektoncd/pipeline/pull/7789
+		// this field is removed from pipeline component
+		// still keeping types to maintain the API compatibility
+		pipeline.Spec.Pipeline.EnableTektonOciBundles = nil
+
 		images := common.ToLowerCaseKeys(common.ImagesFromEnv(common.PipelinesImagePrefix))
 		instance := comp.(*v1alpha1.TektonPipeline)
 		// adding extension's transformers first to run them before `extra` transformers

upstream/pkg/reconciler/kubernetes/tektonpipeline/transform_test.go

@@ -17,13 +17,15 @@ limitations under the License.
 package tektonpipeline
 
 import (
+	"context"
 	"encoding/json"
 	"fmt"
 	"reflect"
 	"testing"
 
 	"github.com/google/go-cmp/cmp"
 	"github.com/tektoncd/operator/pkg/apis/operator/v1alpha1"
+	"github.com/tektoncd/operator/pkg/reconciler/common"
 	"github.com/tektoncd/pipeline/test/diff"
 	"gotest.tools/v3/assert"
 	appsv1 "k8s.io/api/apps/v1"
@@ -365,3 +367,55 @@ func TestUpdateResolverConfigEnvironmentsInDeployment(t *testing.T) {
 		})
 	}
 }
+
+// not in use, see: https://github.com/tektoncd/pipeline/pull/7789
+// this field is removed from pipeline component
+// keeping in types to maintain the API compatibility
+// this test verifies that, "EnableTektonOciBundles" always not present in the feature flags config map
+func TestEnableTektonOciBundlesFeatureFlag(t *testing.T) {
+	tp := &v1alpha1.TektonPipeline{
+		Spec: v1alpha1.TektonPipelineSpec{
+			Pipeline: v1alpha1.Pipeline{
+				PipelineProperties: v1alpha1.PipelineProperties{
+					EnableTektonOciBundles: ptr.Bool(true),
+				},
+			},
+		},
+	}
+	ctx := context.TODO()
+
+	tests := []struct {
+		name                   string
+		enableTektonOciBundles *bool
+		expectedValue          string
+	}{
+		{name: "with-true", enableTektonOciBundles: ptr.Bool(true), expectedValue: "false"},
+		{name: "with-false", enableTektonOciBundles: ptr.Bool(false), expectedValue: "false"},
+		{name: "with-nil", enableTektonOciBundles: nil, expectedValue: "false"},
+	}
+
+	for _, test := range tests {
+		t.Run(test.name, func(t *testing.T) {
+			tp.Spec.Pipeline.EnableTektonOciBundles = test.enableTektonOciBundles
+
+			// get manifests
+			manifest, err := common.Fetch("./testdata/tektonpipeline-feature-flags-base.yaml")
+			assert.NilError(t, err, "error on fetching testdata")
+
+			transformers := filterAndTransform(common.NoExtension(ctx))
+			_, err = transformers(ctx, &manifest, tp)
+			assert.NilError(t, err)
+
+			resources := manifest.Resources()
+			assert.Assert(t, len(resources) > 0)
+
+			featureFlagsMap := corev1.ConfigMap{}
+			err = apimachineryRuntime.DefaultUnstructuredConverter.FromUnstructured(resources[0].Object, &featureFlagsMap)
+			assert.NilError(t, err)
+
+			flagValue, found := featureFlagsMap.Data["enable-tekton-oci-bundles"]
+			assert.Assert(t, found == true, "'enable-tekton-oci-bundles' not found")
+			assert.Assert(t, flagValue == test.expectedValue, "'enable-tekton-oci-bundles' is not '%s'", test.expectedValue)
+		})
+	}
+}