feat: Introduce v1beta1 API version

[odubajDT]

This PR

• adds v1beta1 API version with new resources
• adds E2E tests to check that new resources can be applied to cluster

Related Issues

Fixes #529

[codecov]

Codecov Report

Merging #535 (9118a6c) into main (32ddf00) will increase coverage by 0.02%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main     #535      +/-   ##
==========================================
+ Coverage   82.78%   82.81%   +0.02%     
==========================================
  Files          22       24       +2     
  Lines        1423     1437      +14     
==========================================
+ Hits         1178     1190      +12     
- Misses        207      209       +2     
  Partials       38       38              

Files	Coverage Δ
apis/core/v1beta1/featureflag_types.go	100.00% <100.00%> (ø)
apis/core/v1beta1/featureflagsource_types.go	83.33% <83.33%> (ø)

Flag	Coverage Δ
unit-tests	82.81% <85.71%> (+0.02%)	⬆️

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

[toddbaert]

Thanks @odubajDT . I will give this a thorough review and test in the next few days.

[Kavindu-Dodan]

Nice work. I left few comments but this IMO is important as we always try to validate flag configurations against the schema.

[toddbaert]

I'm ready to approve once this is adapted as in #352 (comment)

[odubajDT]

@toddbaert @Kavindu-Dodan @beeme1mr PR adapted according to the discussion in #352

[toddbaert]

Approved, but I noticed this (just a question).

Also I'm noticing a lint error in the CI which almost make it seems like a dependency might require a higher go version or something: https://github.com/open-feature/open-feature-operator/actions/runs/6730203336/job/18292441905?pr=535#step:6:493

[odubajDT]

Approved, but I noticed this (just a question).

Also I'm noticing a lint error in the CI which almost make it seems like a dependency might require a higher go version or something: https://github.com/open-feature/open-feature-operator/actions/runs/6730203336/job/18292441905?pr=535#step:6:493

Thank you!

The error is present also in the main branch and is not related to this PR. I would say it should be resolved in a separate PR. I can have a look at it.

[Kavindu-Dodan]

Approved, but I noticed this (just a question).
Also I'm noticing a lint error in the CI which almost make it seems like a dependency might require a higher go version or something: https://github.com/open-feature/open-feature-operator/actions/runs/6730203336/job/18292441905?pr=535#step:6:493

Thank you!

The error is present also in the main branch and is not related to this PR. I would say it should be resolved in a separate PR. I can have a look at it.

The error comes with the usage of golangci-lint@latest - From version v1.54.2 onwards, there's a transitive dependency which require Go version 1.20.x+ .

The quick fix for this PR is to fix makefile to v1.54.1 : https://github.com/open-feature/open-feature-operator/blob/main/Makefile#L88-L89

[odubajDT]

Approved, but I noticed this (just a question).
Also I'm noticing a lint error in the CI which almost make it seems like a dependency might require a higher go version or something: https://github.com/open-feature/open-feature-operator/actions/runs/6730203336/job/18292441905?pr=535#step:6:493

Thank you!
The error is present also in the main branch and is not related to this PR. I would say it should be resolved in a separate PR. I can have a look at it.

The error comes with the usage of golangci-lint@latest - From version v1.54.2 onwards, there's a transitive dependency which require Go version 1.20.x+ .

The quick fix for this PR is to fix makefile to v1.54.1 : https://github.com/open-feature/open-feature-operator/blob/main/Makefile#L88-L89

I already started working on a proper fix by moving the check to a separate workflow and also using a dedicated golangci-lint action for it #538

Hopefully it will be ready soon