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

timers continue when freezing operations #673

Closed
rtoma opened this issue Feb 8, 2021 · 2 comments · Fixed by #675
Closed

timers continue when freezing operations #673

rtoma opened this issue Feb 8, 2021 · 2 comments · Fixed by #675
Labels
bug Something isn't working

Comments

@rtoma
Copy link

rtoma commented Feb 8, 2021

Long story short

Thanks for creating kopf. I am currently developing an operator that updates Egress NetworkPolicy resources to sync its spec.egress[*].to[*].ipBlock.cidr keys with A records for DNS hostnames.

This operator used timers (looking into daemons to follow DNS TTL). I run an operator in k8s with cluster peering. For development I start a local instance with --dev.

I see the operator in k8s detects the development instance:

Freezing operations in favour of [<Peer rtoma@devsystem/20210208145159/nz5: priority=666, lifetime=60, lastseen='2021-02-08T14:52:01.160123'>].

The development instance's log show:

Resuming operations after the freeze. Conflicting operators with the same priority are gone.

Still both operator instances show Timer 'NAME' succeeded messages. I searched the docs, but could not find anything.

Description

The code snippet to reproduce the issue
import kopf
The exact command to reproduce the issue
kopf run ...
The full output of the command that failed

Environment

  • Kopf version: 1.29.1
  • Kubernetes version: 1.19.5
  • Python version: 3.9.1
  • OS/platform: k8s on pi4, local on arm mbp
Python packages installed
@rtoma rtoma added the bug Something isn't working label Feb 8, 2021
@nolar
Copy link
Owner

nolar commented Feb 8, 2021

Indeed, that’s possible. And that is clearly a bug. Thanks for reporting — I’ll take a look if it can be fixed easily.

@nolar
Copy link
Owner

nolar commented Feb 25, 2021

The fix is released in kopf>=1.30.0rc1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants