Skip to content

Reverting fadddccd53c4e8f41553fef6ca0630f0124ed4a0 and fixing up b6f4227fe3f4b9ce344f159ac505258228fa8aeb#4503

Merged
zwoop merged 1 commit intoapache:7.1.xfrom
ezelkow1:715x_revert
Oct 27, 2018
Merged

Reverting fadddccd53c4e8f41553fef6ca0630f0124ed4a0 and fixing up b6f4227fe3f4b9ce344f159ac505258228fa8aeb#4503
zwoop merged 1 commit intoapache:7.1.xfrom
ezelkow1:715x_revert

Conversation

@ezelkow1
Copy link
Member

@ezelkow1 ezelkow1 commented Oct 26, 2018

Testing the latest 7.1.x we saw a crash that kept coming up, https://gist.github.com/ezelkow1/6514f3ddb7a68e2b3279ae3a26cb0021

This seemed to be triggered by an unknown event in the state machine:
[Oct 26 18:25:07.032] Server {0x2b9b78a09700} ERROR: <HttpSM.cc:1803 (state_http_server_open)> [HttpSM::state_http_server_open] Unknown event: 105 (VC_EVENT_INACTIVITY_TIMEOUT)

I tracked it down to fadddc, and to do the revert I also had to back out a few changes from b6f422

The crash would be triggered within a few minutes of running synthetic-production traffic. With this revert I ran for 30 minutes and never got the crash

@zwoop
Copy link
Contributor

zwoop commented Oct 27, 2018

[approve ci]
[add to whitelist]

@zwoop zwoop added this to the 7.1.5 milestone Oct 27, 2018
@zwoop zwoop added the Cleanup label Oct 27, 2018
@zwoop zwoop self-requested a review October 27, 2018 03:29
@zwoop zwoop merged commit eb3d15a into apache:7.1.x Oct 27, 2018
@zwoop
Copy link
Contributor

zwoop commented Oct 27, 2018

Removing the Milestone from this, since this is a no-op now.

@zwoop zwoop removed this from the 7.1.5 milestone Oct 27, 2018
@zwoop zwoop added Revert and removed Revert labels Jan 7, 2019
@ezelkow1 ezelkow1 deleted the 715x_revert branch April 3, 2020 17:07
@ezelkow1 ezelkow1 modified the milestones: Old, Reverted Feb 23, 2022
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