Skip to content

mwi/unsolicited/incoming: Fix several issues causing instability.#146

Merged
github-actions[bot] merged 1 commit into
asterisk:masterfrom
gtjoseph:master-mwi-incoming
Jun 22, 2026
Merged

mwi/unsolicited/incoming: Fix several issues causing instability.#146
github-actions[bot] merged 1 commit into
asterisk:masterfrom
gtjoseph:master-mwi-incoming

Conversation

@gtjoseph

Copy link
Copy Markdown
Member

There are two sipp scenarios "allowed" and "not allowed", each generating a
TestEvent. The test is set to end when the TestEvent for "not allowed"
arrives. The scenarios were being run in parallel however so if the
"not allowed" scenario happens to finish first and generates its TestEvent,
the test will fail with an "Event occurred 0 times." error because the
"allowed" scenario hasn't had its chance yet.

  • The sipp scenarios are now run seqauentially, "allowed" first, then
    "not allowed".
  • Both scenario files were missing the closing </scenario> tag for some
    reason so that's been fixed.

There are two sipp scenarios "allowed" and "not allowed", each generating a
TestEvent.  The test is set to end when the TestEvent for "not allowed"
arrives.  The scenarios were being run in parallel however so if the
"not allowed" scenario happens to finish first and generates its TestEvent,
the test will fail with an "Event occurred 0 times." error because the
"allowed" scenario hasn't had its chance yet.

* The sipp scenarios are now run seqauentially, "allowed" first, then
"not allowed".
* Both scenario files were missing the closing `</scenario>` tag for some
reason so that's been fixed.
@gtjoseph

Copy link
Copy Markdown
Member Author

cherry-pick-to: 20
cherry-pick-to: 21
cherry-pick-to: 22
cherry-pick-to: 23
cherry-pick-to: certified/20.7
cherry-pick-to: certified/22.8

@github-actions

Copy link
Copy Markdown

Workflow Check completed successfully

@github-actions github-actions Bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-passed Cherry-Pick checks passed and removed cherry-pick-test cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Jun 22, 2026
@github-actions

Copy link
Copy Markdown

Workflow CPCheck completed successfully

@github-actions github-actions Bot merged commit 379b3a5 into asterisk:master Jun 22, 2026
49 checks passed
@github-actions

Copy link
Copy Markdown

Successfully merged to branch master and cherry-picked to ["20","21","22","23","certified/20.7","certified/22.8"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants