Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Big issues with latest version #65

Open
acamilleri opened this issue Feb 17, 2025 · 5 comments
Open

Big issues with latest version #65

acamilleri opened this issue Feb 17, 2025 · 5 comments

Comments

@acamilleri
Copy link
Contributor

Hello

Following #63, I tested changes on a test cluster and I encountered a lot of issues with your latest version.

  • CRDs deployed with the Helm charts are desync.
  • When the default config is enabled a caCertSecretRef is created with a caCertificates.
  • AllowedNamespaces is not handled as slices.
  • Etc...

I fixes lot of issues, the PR is here: #64

@acamilleri acamilleri changed the title Big issues with latest versions Big issues with latest version Feb 17, 2025
@acamilleri
Copy link
Contributor Author

#66

@acamilleri
Copy link
Contributor Author

#67

@acamilleri
Copy link
Contributor Author

If you agree with all the PRs, you can merge fixes first and I can rebase the access policy bucket feature later.

@Donatien26
Copy link
Collaborator

Sorry for the inconvenience. I tried adding some unit tests, but it's not enough ^^" I'll review your merge requests as soon as possible. I'm also working on improving the CI so we can implement evolutions and release candidates to avoid releasing unstable versions!

Apologies again!

@Donatien26
Copy link
Collaborator

I made some tests on my side, i don't find any remaining bugs. Can you test the last version and tell me if you find some remaining bugs ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants