Skip to content
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

Use getUrlPartsWithStrippedDefaultPort instead of getUrlParts #199264

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

maryam-saeidi
Copy link
Member

@maryam-saeidi maryam-saeidi commented Nov 7, 2024

Summary

Use getUrlPartsWithStrippedDefaultPort to avoid this issue on MKI:

Serverless Observability - Deployment-agnostic api integration - Custom Threshold rule AVG - PCT - FIRED Rule creation should set correct action variables

Error: expected 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'
to sort of equal 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'

@maryam-saeidi maryam-saeidi added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Nov 7, 2024
@maryam-saeidi maryam-saeidi self-assigned this Nov 7, 2024
@maryam-saeidi maryam-saeidi requested a review from a team as a code owner November 7, 2024 09:01
Copy link
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Also checked these changes against an MKI project (with parameters how the pipeline is calling it) and it passed ✔️

@maryam-saeidi maryam-saeidi enabled auto-merge (squash) November 7, 2024 09:39
@wayneseymour wayneseymour added the Team:obs-ux-management Observability Management User Experience Team label Nov 7, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@maryam-saeidi maryam-saeidi merged commit 996104f into elastic:main Nov 7, 2024
28 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11721434266

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @maryam-saeidi

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 7, 2024
…c#199264)

## Summary

Use getUrlPartsWithStrippedDefaultPort to avoid this
[issue](elastic#195902 (comment))
on MKI:

```
Serverless Observability - Deployment-agnostic api integration - Custom Threshold rule AVG - PCT - FIRED Rule creation should set correct action variables

Error: expected 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'
to sort of equal 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'
```

(cherry picked from commit 996104f)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@maryam-saeidi maryam-saeidi deleted the fix-test-port branch November 7, 2024 10:57
kibanamachine added a commit that referenced this pull request Nov 7, 2024
…199264) (#199288)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Use getUrlPartsWithStrippedDefaultPort instead of getUrlParts
(#199264)](#199264)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Maryam
Saeidi","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-07T10:45:33Z","message":"Use
getUrlPartsWithStrippedDefaultPort instead of getUrlParts
(#199264)\n\n## Summary\r\n\r\nUse getUrlPartsWithStrippedDefaultPort to
avoid
this\r\n[issue](https://github.com/elastic/kibana/pull/195902#issuecomment-2411667499)\r\non
MKI:\r\n\r\n```\r\nServerless Observability - Deployment-agnostic api
integration - Custom Threshold rule AVG - PCT - FIRED Rule creation
should set correct action variables\r\n\r\nError: expected
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\nto
sort of equal
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\n```","sha":"996104f2eae2a3eb2412d5bb4ca52e89a5dbb1db","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-minor","Team:obs-ux-management"],"title":"Use
getUrlPartsWithStrippedDefaultPort instead of
getUrlParts","number":199264,"url":"https://github.com/elastic/kibana/pull/199264","mergeCommit":{"message":"Use
getUrlPartsWithStrippedDefaultPort instead of getUrlParts
(#199264)\n\n## Summary\r\n\r\nUse getUrlPartsWithStrippedDefaultPort to
avoid
this\r\n[issue](https://github.com/elastic/kibana/pull/195902#issuecomment-2411667499)\r\non
MKI:\r\n\r\n```\r\nServerless Observability - Deployment-agnostic api
integration - Custom Threshold rule AVG - PCT - FIRED Rule creation
should set correct action variables\r\n\r\nError: expected
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\nto
sort of equal
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\n```","sha":"996104f2eae2a3eb2412d5bb4ca52e89a5dbb1db"}},"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/199264","number":199264,"mergeCommit":{"message":"Use
getUrlPartsWithStrippedDefaultPort instead of getUrlParts
(#199264)\n\n## Summary\r\n\r\nUse getUrlPartsWithStrippedDefaultPort to
avoid
this\r\n[issue](https://github.com/elastic/kibana/pull/195902#issuecomment-2411667499)\r\non
MKI:\r\n\r\n```\r\nServerless Observability - Deployment-agnostic api
integration - Custom Threshold rule AVG - PCT - FIRED Rule creation
should set correct action variables\r\n\r\nError: expected
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\nto
sort of equal
'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'\r\n```","sha":"996104f2eae2a3eb2412d5bb4ca52e89a5dbb1db"}}]}]
BACKPORT-->

Co-authored-by: Maryam Saeidi <[email protected]>
mbondyra pushed a commit to mbondyra/kibana that referenced this pull request Nov 8, 2024
…c#199264)

## Summary

Use getUrlPartsWithStrippedDefaultPort to avoid this
[issue](elastic#195902 (comment))
on MKI:

```
Serverless Observability - Deployment-agnostic api integration - Custom Threshold rule AVG - PCT - FIRED Rule creation should set correct action variables

Error: expected 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'
to sort of equal 'https://bk-serverless-ftr-3067-e697d43e3ad9-e0ac80.kb.eu-west-1.aws.qa.elastic.cloud:443/app/observability/alerts/1e0c2d3e-e5c2-4bfe-9df0-d46681253b9f'
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-management Observability Management User Experience Team v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants