Skip to content
This repository was archived by the owner on Feb 18, 2021. It is now read-only.

WIP: always call update{ServiceNodes,ExitNodes} #246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcorbin
Copy link
Contributor

@jcorbin jcorbin commented Feb 18, 2016

Unsure if this, or a less heavy handed change is necessary to get parital
affinity working.

@Raynos
Copy link
Contributor

Raynos commented Feb 18, 2016

I think this is needed, but we should keep an eye on it. Can we add a custom logline here so we can see how often this happens.

This should be fine except for the CPU runaway case :(

@jcorbin
Copy link
Contributor Author

jcorbin commented Feb 18, 2016

Aye, stats on changes and updates would be great, will add.

Currently the only affect of updateServiceNodes is under the partial affinity feature, so noop until then.

@jcorbin jcorbin force-pushed the more_update_under_change branch from c33597b to ea2fa73 Compare February 26, 2016 17:45
@jcorbin jcorbin changed the title WIP: call update{ServiceNodes,ExitNodes} after changeTo{Exit,Forward} WIP: always call update{ServiceNodes,ExitNodes} Feb 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants