Skip to content

[cliDevMode] set server ready status to false when restarting#97575

Merged
spalger merged 1 commit intoelastic:masterfrom
spalger:fix/cli-dev-mode-restart-not-ready
Apr 20, 2021
Merged

[cliDevMode] set server ready status to false when restarting#97575
spalger merged 1 commit intoelastic:masterfrom
spalger:fix/cli-dev-mode-restart-not-ready

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Apr 20, 2021

Set devServer.ready$ to false when server is first starting up to prevent 502 errors and instead pause requests in the basePathProxy. This used to work but I'm assuming at the extraction of the CliDevMode to a module the ready status stopped getting reset to false when the server was starting up.

@spalger spalger added Team:Operations Kibana-Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.13.0 labels Apr 20, 2021
@spalger spalger requested a review from a team as a code owner April 20, 2021 00:50
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger merged commit 39894c5 into elastic:master Apr 20, 2021
@spalger spalger deleted the fix/cli-dev-mode-restart-not-ready branch April 20, 2021 17:20
@spalger spalger added the auto-backport Deprecated - use backport:version if exact versions are needed label Apr 20, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 20, 2021
…c#97575)

Co-authored-by: spalger <spalger@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Apr 20, 2021
#97683)

Co-authored-by: spalger <spalger@users.noreply.github.com>

Co-authored-by: Spencer <email@spalger.com>
Co-authored-by: spalger <spalger@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.13.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants