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

Fix addition/deletion of namespaces and exception on startup #133

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

VelorumS
Copy link

@VelorumS VelorumS commented Jun 7, 2024

Fix exception in startup_fn() because of no namespace name in clustersecrets metadata.

Fix exception in on_field_data() when encountering a deleted namespace.

Fix cache update in on_field_data() so a newly added namespaces could get an up-to-date secret data.

Fix the addition of new namespaces to the list for synchronization.

Fix storing body in cache so the new namespaces could get the data.

Fix storing clustersecret's namespace name in cache in create_fn().

Update kopf to fix mutable default <class 'kopf._cogs.structs.diffs.Diff'> for field diff is not allowed: use default_factory with Python 3.11+.

Fixes #117
Fixes #116

@zakkg3
Copy link
Owner

zakkg3 commented Sep 20, 2024

I have to fix the e2e test and if that passes ill merge this. I guess in the next days I hope I find the time for this.
sorry for the delay.

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