Added a check to watch on secret pipelines-as-code-secret to decide GithubApp or Webhook

[savitaashture]

Changes

Fixes: https://issues.redhat.com/browse/SRVKP-2885

tkn-pac create repo should not configure webhook when GithubApp is installed but right now PAC relaying on pipelines-as-code-info configmap for Provider key here and if Provider is "" even though secret pipelines-as-code-secret present tkn-pac create repo does not know about secret info and it tries for Webhook

Submitter Checklist

• ♽ Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
• 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
• 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
• 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
• 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

[chmouel]

this looks good but we really need to take the opportunity to add unittests for this file (configmap)

[savitaashture]

this looks good but we really need to take the opportunity to add unittests for this file (configmap)

Yes sure agree

Shall i do that as a separate PR

[chmouel]

let's do this in this PR, what do you think?

[savitaashture]

let's do this in this PR, what do you think?

Yes works for me