-
Notifications
You must be signed in to change notification settings - Fork 9
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
Removed archived evcs from self.circuits
#433
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done @Alopalao, it's almost there, just trivial details remaining. Also, look forward to the e2e test results.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll leave this pre-approved, check out the last thread that I opened though, there's a minor refactoring and lock that can get removed.
@Alopalao, I'll also recommend that you explore disabling/enabling EVCs and perform a few link flaps low frequency stuff should suffice. Also, when e2e tests are passing and the opened thread on this PR is addressed it can get merged.
Performed the tests with link flaps and disabling/enabling EVCs. No errors and no warning others than the recently implemented: |
@Alopalao, 3800 more or less messages on avg over 5 secs queued on the I also ran here 50 flaps per iteration waiting for 10 secs with 100 EVCs and no warnings on my environment at least:
If you still keep seeing this, ping me we can pair on it and analyze your environment, let's see if we might want increase a bit more the default params of queue monitors. From what I'm seeing here it detected correctly. Other than that, I'll go ahead and merge this PR, but let's keep this other discussion here going in the background. OK? |
Closes #369
Summary
Removed archived evcs from
self.circuits
.Local Tests
Tested
link_down
andlink_up
events.Tried to delete, patch a deleted EVC.
Tried to create an schedule for a deleted EVC.
End-to-End Tests
The failures are popped because of an outdated
kytos-docker
without the removal offlow_stats