custom-metrics-apiserver icon indicating copy to clipboard operation
custom-metrics-apiserver copied to clipboard

Synchronize test-adapter and Getting Started Guide

Open olivierlemasle opened this issue 2 years ago • 6 comments
trafficstars

Currently, the Getting Started Guide includes pieces of code which, combined, give a sample adapter, different from the test-adapter.

That leads to two issues:

  • code in the Getting Started Guide is not compiled or tested. The last fix in #108 showed lots of inconsistencies (e.g. different name for a same variable in the code) or outdated code (e.g. the adapter had not been updated for the Interfaces changes).
  • there's 2 "test adapters" to maintain, and both are meant to serve as an example implementation (the "test-adapter" is also used for unit tests and is compiled during CI). These test adapters may diverge.

To tackle the first issue, we could enforce the Markdown from Getting Started Guide to be synced with Go code (e.g. kube-prometheus uses mdox to check that Jsonnet examples from its Markdown documentation is synced with Jsonnet files - cf here).

Ideally, we would have a single test implementation (test-adapter). However, test-adapter is more complex than the implementation from the Getting Started Guide...

What do you think?

olivierlemasle avatar Dec 10 '22 12:12 olivierlemasle

/assign @olivierlemasle /triage accepted

dashpole avatar Dec 15 '22 17:12 dashpole

/kind documentation

olivierlemasle avatar Dec 20 '22 18:12 olivierlemasle

Created PR #138 to try this mdox solution.

It "solves" issue 1 (compiling / testing code snippets in documentation), but not issue 2 (having 2 test adapters).

However, I wonder if it's really an issue. The current "test-adapter" may be better suited for tests (end to end tests to be created, unit tests), and this "sample-adapter" is an entry-level example.

olivierlemasle avatar Dec 22 '22 19:12 olivierlemasle

This issue has not been updated in over 1 year, and should be re-triaged.

You can:

  • Confirm that this issue is still relevant with /triage accepted (org members only)
  • Close this issue with /close

For more details on the triage process, see https://www.kubernetes.dev/docs/guide/issue-triage/

/remove-triage accepted

k8s-triage-robot avatar Jan 20 '24 00:01 k8s-triage-robot

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Apr 19 '24 00:04 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle rotten
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar May 19 '24 00:05 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

k8s-triage-robot avatar Jun 18 '24 01:06 k8s-triage-robot

@k8s-triage-robot: Closing this issue, marking it as "Not Planned".

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

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-sigs/prow repository.

k8s-ci-robot avatar Jun 18 '24 01:06 k8s-ci-robot