Added tristate flags and changed experimental config map name

[slinkydeveloper]

Signed-off-by: Francesco Guardiani francescoguard@gmail.com

Address suggestions on the mail thread https://groups.google.com/u/0/g/knative-dev/c/M_10Rh_30ho

Proposed Changes

• 🧹 Renamed config-experimental-features CM to config-features
• 🧹 Added the tristate Flag value
• 🧹 Renamed experimental package to feature package, and renamed test packages to feature_test (to avoid circular refs with other api packages)
• 🎁 Added the flag_names file, where we should place all the different flags

Pre-review Checklist

• At least 80% unit test coverage
• E2E tests for any new behavior: N/A
• Docs PR for any user-facing impact: Added the experimental features page docs#3676
• Spec PR for any new API feature: N/A
• Conformance test for any change to the spec: N/A

[codecov]

Codecov Report

Merging #5459 (3787ec8) into main (d1106c8) will decrease coverage by 0.00%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##             main    #5459      +/-   ##
==========================================
- Coverage   82.76%   82.75%   -0.01%     
==========================================
  Files         197      197              
  Lines        6086     6090       +4     
==========================================
+ Hits         5037     5040       +3     
- Misses        724      725       +1     
  Partials      325      325              

Impacted Files	Coverage Δ
pkg/apis/feature/api_validation.go	93.10% <ø> (ø)
pkg/apis/feature/store.go	87.50% <50.00%> (ø)
pkg/apis/feature/features.go	87.50% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1106c8...3787ec8. Read the comment docs.

[devguyio]

/assign

[knative-metrics-robot]

The following is the coverage report on the affected files.
Say /test pull-knative-eventing-go-coverage to re-run this coverage report

File	Old Coverage	New Coverage	Delta
pkg/apis/feature/api_validation.go	Do not exist	95.5%
pkg/apis/feature/features.go	Do not exist	93.8%
pkg/apis/feature/store.go	Do not exist	88.2%

[julz]

/lgtm from a serving POV

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: julz, slinkydeveloper

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [slinkydeveloper]

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