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

Handle more deadlocks and races + allow empty config values. #71

Merged
merged 6 commits into from
Oct 18, 2022

Conversation

ppalucki
Copy link
Contributor

@ppalucki ppalucki commented Oct 17, 2022

There were some issues when configuration was changed from:

  • null -> something
  • something -> null
  • somtheing -> force
  • null -> force

Now the installation support changing policy and resolves #29

Works for both static and dynamic configuraiton.

Also update example files and add some workloads for testing/demo.

@ppalucki ppalucki added bug Something isn't working enhancement New feature or request labels Oct 17, 2022
@ppalucki ppalucki added this to the 1.0 milestone Oct 17, 2022
@ppalucki ppalucki self-assigned this Oct 17, 2022
pkg/configs/configs.go Outdated Show resolved Hide resolved
…age only when there are actually extensions to reconcile (was confusing that log was there but nothing were happening)
@ppalucki ppalucki merged commit d7bb03d into master Oct 18, 2022
@ppalucki ppalucki deleted the ppalucki-demo-configs branch December 20, 2023 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for dynamically changing a policy when nen config arrives.
2 participants