[Synthetics] Add warning if TLS config not set for Synthetics#195395
Merged
justinkambic merged 2 commits intoelastic:mainfrom Oct 9, 2024
Merged
Conversation
justinkambic
added
release_note:skip
Contributor
|
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
botelastic
Bot
added
the
ci:project-deploy-observability
Member
|
/oblt-deploy |
shahzad31
approved these changes
Oct 8, 2024
Contributor
💚 Build Succeeded
Metrics [docs]
History
|
Contributor
|
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/11260023565 |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Oct 9, 2024
…c#195395) ## Summary Recently while debugging a production issue where the Synthetics plugin was receiving 401 errors while trying to reach the Synthetics Service health endpoint, we isolated that there was an issue with the mTLS handshake between Kibana and the service. Unfortunately, we were unsure if there was some missing custom config (especially relevant in Serverless Kibana), or if the certificate values were not present in the first place. Adding this warning will help us make this determination better in the future when reviewing Kibana logs, as we will be assured if the config is not defined via this warning. (cherry picked from commit 8e986a6)
Contributor
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
kibanamachine
added a commit
that referenced
this pull request
Oct 9, 2024
…195395) (#195658) # Backport This will backport the following commits from `main` to `8.x`: - [[Synthetics] Add warning if TLS config not set for Synthetics (#195395)](#195395) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Justin Kambic","email":"jk@elastic.co"},"sourceCommit":{"committedDate":"2024-10-09T17:27:49Z","message":"[Synthetics] Add warning if TLS config not set for Synthetics (#195395)\n\n## Summary\r\n\r\nRecently while debugging a production issue where the Synthetics plugin\r\nwas receiving 401 errors while trying to reach the Synthetics Service\r\nhealth endpoint, we isolated that there was an issue with the mTLS\r\nhandshake between Kibana and the service.\r\n\r\nUnfortunately, we were unsure if there was some missing custom config\r\n(especially relevant in Serverless Kibana), or if the certificate values\r\nwere not present in the first place.\r\n\r\nAdding this warning will help us make this determination better in the\r\nfuture when reviewing Kibana logs, as we will be assured if the config\r\nis not defined via this warning.","sha":"8e986a6dd945160d80d9b400f4acb7f9181d962a","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0"],"title":"[Synthetics] Add warning if TLS config not set for Synthetics","number":195395,"url":"https://github.com/elastic/kibana/pull/195395","mergeCommit":{"message":"[Synthetics] Add warning if TLS config not set for Synthetics (#195395)\n\n## Summary\r\n\r\nRecently while debugging a production issue where the Synthetics plugin\r\nwas receiving 401 errors while trying to reach the Synthetics Service\r\nhealth endpoint, we isolated that there was an issue with the mTLS\r\nhandshake between Kibana and the service.\r\n\r\nUnfortunately, we were unsure if there was some missing custom config\r\n(especially relevant in Serverless Kibana), or if the certificate values\r\nwere not present in the first place.\r\n\r\nAdding this warning will help us make this determination better in the\r\nfuture when reviewing Kibana logs, as we will be assured if the config\r\nis not defined via this warning.","sha":"8e986a6dd945160d80d9b400f4acb7f9181d962a"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195395","number":195395,"mergeCommit":{"message":"[Synthetics] Add warning if TLS config not set for Synthetics (#195395)\n\n## Summary\r\n\r\nRecently while debugging a production issue where the Synthetics plugin\r\nwas receiving 401 errors while trying to reach the Synthetics Service\r\nhealth endpoint, we isolated that there was an issue with the mTLS\r\nhandshake between Kibana and the service.\r\n\r\nUnfortunately, we were unsure if there was some missing custom config\r\n(especially relevant in Serverless Kibana), or if the certificate values\r\nwere not present in the first place.\r\n\r\nAdding this warning will help us make this determination better in the\r\nfuture when reviewing Kibana logs, as we will be assured if the config\r\nis not defined via this warning.","sha":"8e986a6dd945160d80d9b400f4acb7f9181d962a"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Justin Kambic <jk@elastic.co>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Recently while debugging a production issue where the Synthetics plugin was receiving 401 errors while trying to reach the Synthetics Service health endpoint, we isolated that there was an issue with the mTLS handshake between Kibana and the service.
Unfortunately, we were unsure if there was some missing custom config (especially relevant in Serverless Kibana), or if the certificate values were not present in the first place.
Adding this warning will help us make this determination better in the future when reviewing Kibana logs, as we will be assured if the config is not defined via this warning.