Fix wrong entry in development documentation and other minor documentation corrections.

[twoGiants]

Changes

Change the docker registry entry for KO_DOCKER_REPO to localhost:5000. The former entry kind.local (here) lead to an error when deploying the local pipeline build.

Fix a broken link in the testing documentation and remove a leftover comment from the makefile which was targeted to a non existing configuration.

Add optional installation recommendation for yamllint so that it can be used when running pre-commit locally otherwise it will show a Failed message when linting yaml files.

Closes #8660
Closes #8486

/kind bug

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

• Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
• Has Tests included if any functionality added or changed
• pre-commit Passed
• Follows the commit message standard
• Meets the Tekton contributor standards (including functionality, content, code)
• Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
• Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
• Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

[vdemeester]

/lgtm

[tekton-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

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 [vdemeester]

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

[afrittoli]

@twoGiants Thanks for this!
The check-pr-has-kind-label CI job expects a single kind label, so it fails now.

About the kind.local registry setting, it is supported by ko to inject the images directly into the kind cluster.
It used to work, do you have details on the failure you're getting?

[twoGiants]

@twoGiants Thanks for this! The check-pr-has-kind-label CI job expects a single kind label, so it fails now.

Thanks, I removed the "documentation" label.

About the kind.local registry setting, it is supported by ko to inject the images directly into the kind cluster. It used to work, do you have details on the failure you're getting?

Yes, I created a small issue #8660. The error is:

Error: error processing import paths in "config/webhook.yaml": error resolving image references: no nodes found for cluster "kind"

I am investigating a bit further. I will get back at you here.

[twoGiants]

@afrittoli I found the issue with kind.local. It is not working if one starts the cluster with the tekton_in_kind.sh plumbing script which I did. In the plumbing script a local registry is deployed outside kind and the configuration is patched to use it. ko must then be configured to use that registry because that is what kind uses.

But when I start the kind cluster without the local registry kind.local for ko works. I will open another tiny PR with the updated documentation which mentions this.

Thanks for pointing that out @afrittoli, it helped me to dig a bit deeper and find the real issue!

[twoGiants]

@afrittoli updated the documentation in this tiny PR #8662. Thank you for the review!