docs: add htmltest verification for documentation

[aepfli]

Fixed HTMLTest make target, and added a verification step for GitHub Actions.

[codecov]

Codecov Report

Merging #932 (0b534c3) into main (2761a4d) will increase coverage by 0.60%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #932      +/-   ##
==========================================
+ Coverage   58.02%   58.62%   +0.60%     
==========================================
  Files          89       97       +8     
  Lines        7076     7556     +480     
==========================================
+ Hits         4106     4430     +324     
- Misses       2821     2935     +114     
- Partials      149      191      +42     

Impacted Files	Coverage Δ
klt-cert-manager/kubeutils/certificates.go	0.00% <0.00%> (ø)
...anager/controllers/keptnwebhookcontroller/certs.go	70.96% <0.00%> (ø)
klt-cert-manager/kubeutils/secret.go	67.39% <0.00%> (ø)
klt-cert-manager/eventfilter/eventfilter.go	100.00% <0.00%> (ø)
klt-cert-manager/kubeutils/query.go	100.00% <0.00%> (ø)
...llers/keptnwebhookcontroller/certificate_secret.go	85.88% <0.00%> (ø)
...rs/keptnwebhookcontroller/webhook_configuration.go	100.00% <0.00%> (ø)
...okcontroller/keptnwebhookcertificate_controller.go	52.00% <0.00%> (ø)

Flag	Coverage Δ
component-tests	42.30% <ø> (ø)
keptn-lifecycle-operator	54.69% <ø> (ø)
klt-cert-manager	67.50% <ø> (?)
scheduler	21.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[aepfli]

currently the build fails for following issues, i will check if this is something fix worthy

docs/crd-ref/lifecycle/v1alpha2/index.html
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha2/index.html --> #keptnevaluationstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha2/index.html --> #keptntaskdefinitionstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha2/index.html --> #keptnappversionstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha2/index.html --> #keptnworkloadinstancestatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha2/index.html --> #keptnappversionstatus
docs/crd-ref/lifecycle/v1alpha1/index.html
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnappversionstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnworkloadinstancestatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnevaluationstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptntaskdefinitionstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnappversionstatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnworkloadinstancestatus
  hash does not exist --- docs/crd-ref/lifecycle/v1alpha1/index.html --> #keptnappversionstatus

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[aepfli]

@StackScribe @thisthat @thschue @mowies - your input would be highly appreciated

[thisthat]

LGTM

[mowies]

I don't know what to do with the failing links right now. Status objects are not rendered into the CRD docs, because they are not really needed but the link seem to still be rendered. But I don't link ignore the generated files altogether since we will add other links in there that should be checked.

[aepfli]

I suggest we merge it, with the current configuration and create a follow up to take a look at crd-ref-docs.

I think html test adds more benefit, as it would bring us to find a crd-ref solution immediately.

WDYT?

[mowies]

If merging means that all master builds are red then I'm not voting for it 😓

[thschue]

lgtm

[aepfli]

no worries @mowies, i excluded the lifecyle crd ref for now within the .htmltest.yml file - we can add more any time