-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Synthetics] URL validation softens to allow vars usage !! #197797
Conversation
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
|
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.
LGTM
Starting backport for target branches: 8.15, 8.16, 8.x |
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…97797) ## Summary URL validation softens to allow vars usage !! For example ` should urls: "${url}" interpolate --params '{"url": "my-url"}'` (cherry picked from commit 3b05b6a) # Conflicts: # x-pack/plugins/observability_solution/synthetics/server/synthetics_service/project_monitor/normalizers/common_fields.ts
…7797) (#197839) # Backport This will backport the following commits from `main` to `8.x`: - [[Synthetics] URL validation softens to allow vars usage !! (#197797)](#197797) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-25T14:33:26Z","message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","Team:obs-ux-management"],"title":"[Synthetics] URL validation softens to allow vars usage !!","number":197797,"url":"https://github.com/elastic/kibana/pull/197797","mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197797","number":197797,"mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}}]}] BACKPORT--> Co-authored-by: Shahzad <[email protected]>
…97797) (#197841) # Backport This will backport the following commits from `main` to `8.15`: - [[Synthetics] URL validation softens to allow vars usage !! (#197797)](#197797) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-25T14:33:26Z","message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","Team:obs-ux-management"],"number":197797,"url":"https://github.com/elastic/kibana/pull/197797","mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197797","number":197797,"mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}},{"url":"https://github.com/elastic/kibana/pull/197838","number":197838,"branch":"8.16","state":"OPEN"},{"url":"https://github.com/elastic/kibana/pull/197839","number":197839,"branch":"8.x","state":"OPEN"}]}] BACKPORT-->
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
3 similar comments
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
…97797) (#197838) # Backport This will backport the following commits from `main` to `8.16`: - [[Synthetics] URL validation softens to allow vars usage !! (#197797)](#197797) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-25T14:33:26Z","message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","Team:obs-ux-management"],"title":"[Synthetics] URL validation softens to allow vars usage !!","number":197797,"url":"https://github.com/elastic/kibana/pull/197797","mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197797","number":197797,"mergeCommit":{"message":"[Synthetics] URL validation softens to allow vars usage !! (#197797)\n\n## Summary\r\n\r\nURL validation softens to allow vars usage !!\r\n\r\nFor example \r\n\r\n` should urls: \"${url}\" interpolate --params '{\"url\": \"my-url\"}'`","sha":"3b05b6a7a83df7ff8b929dbbb60c639cc11101d8"}}]}] BACKPORT--> Co-authored-by: Shahzad <[email protected]> Co-authored-by: Maryam Saeidi <[email protected]>
Summary
URL validation softens to allow vars usage !!
For example
should urls: "${url}" interpolate --params '{"url": "my-url"}'