Read component version from configmap instead of CRD label

[sm43]

In the recent refactoring while starting the controller we read the version of component from CRD label
but we also have a config map added to all components which has the version of that component.
The configmap was added so that client like tkn can read the version of component.

In Operator we can do the same instead of reading from CRD we should read from configmap

from controller we make a call

operator/pkg/reconciler/kubernetes/tektonpipeline/controller.go

Line 70 in a3d2802

releaseVersion, err := common.FetchVersionFromCRD(manifest, releaseLabel)

and the func is implemented

operator/pkg/reconciler/common/utils.go

Line 30 in a3d2802

func FetchVersionFromCRD(manifest mf.Manifest, releaseLabel string) (string, error) {

Update the func FetchVersionFromCRD to have param for configmap name
and the func would find the version from that configmap and return it

for pipelines the configmap is pipelines-info https://github.com/tektoncd/pipeline/blob/main/config/config-info.yaml
similarly they re present for other components too

/kind feature
/good-first-issue
/help-wanted

[tekton-robot]

@sm43:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

Details

In response to this:

In the recent refactoring while starting the controller we read the version of component from CRD label
but we also have a config map added to all components which has the version of that component.
The configmap was added so that client like tkn can read the version of component.

In Operator we can do the same instead of reading from CRD we should read from configmap

from controller we make a call

operator/pkg/reconciler/kubernetes/tektonpipeline/controller.go

Line 70 in a3d2802

releaseVersion, err := common.FetchVersionFromCRD(manifest, releaseLabel)

and the func is implemented

operator/pkg/reconciler/common/utils.go

Line 30 in a3d2802

func FetchVersionFromCRD(manifest mf.Manifest, releaseLabel string) (string, error) {

Update the func FetchVersionFromCRD to have param for configmap name
and the func would find the version from that configmap and return it

for pipelines the configmap is pipelines-info https://github.com/tektoncd/pipeline/blob/main/config/config-info.yaml
similarly they re present for other components too

/kind feature
/good-first-issue
/help-wanted

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[vinamra28]

/assign