Update workflow to remove lint as there is no support from catalog-cd tool

[savitaashture]

Update the workflow as there is no lint command available in catalog-cd tool

[Senjuti256]

Why are we removing the portion of linting using Catlin? @savitaashture

[savitaashture]

good question!!

I did check for all the repos and i don't see anywhere we are doing linting using catalin (Not sure the reason) as well i remember @vdemeester saying that something related to catlin tool

So just removed from Openshift as well and thought to bring this point in the ecosystem discussion call to have a consistent linting across all the repos for yamls

[Senjuti256]

Yeah but I remember @vdemeester told that we require to run some form of linting on the results of the helm template and we used Catlin as it was upstream. This linting was added because there was a JIRA issue that required that our tasks (or the tasks in ecosystem catalog) should contain Red Hat supported images only and we decided to ensure this by making some changes to the portion of linting only. @savitaashture

[Senjuti256]

But I think @vdemeester can provide a better context to this than me@savitaashture

[savitaashture]

@Senjuti256

I removed only catalog-cd tool part as there is no lint and kept catalin lint as it is
Verified linting execution here https://github.com/savitaashture/task-openshift/actions/runs/8564598061/job/23471378385

[Senjuti256]

Don't we require any kind of linting for the YAML files? @savitaashture

[savitaashture]

same ans as abve

[Aneesh-M-Bhat]

This makes sense to me, the lint job has been failing since Jan end as I see it.
( actually lot of the other tests have also been failing, when i took a look just now, so not sure what's happening with the CI in this repo. )

[savitaashture]

( actually lot of the other tests have also been failing, when i took a look just now, so not sure what's happening with the CI in this repo. )

@Aneesh-M-Bhat The failure of CI on all pushed commit is because we don't have serverless installed for Kind after adding kn-apply & kn task

TaskRun started: kn-apply-run-2cd9s
Waiting for logs to be available...
[kn] Error: services.serving.knative.dev "hello" is forbidden: User "system:serviceaccount:default:default" cannot get resource "services" in API group "serving.knative.dev" in the namespace "default"
[kn] Run 'kn --help' for usage

[savitaashture]

@Aneesh-M-Bhat created issue : #34

[Senjuti256]

LGTM

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: savitaashture, Senjuti256

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [savitaashture]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment