Skip to content

Comments

update package testing script after /test was moved to src/platform/test#214730

Merged
delanni merged 1 commit intomainfrom
fix-artifact-testing-post-move
Mar 17, 2025
Merged

update package testing script after /test was moved to src/platform/test#214730
delanni merged 1 commit intomainfrom
fix-artifact-testing-post-move

Conversation

@delanni
Copy link
Contributor

@delanni delanni commented Mar 17, 2025

Summary

After #210956 - the artifact snapshot testing code is broken because a path reference wasn't updated. This PR updates that.

Fixes: https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5745

The checks started (https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5764) but it failed for a different reason. Probably some real errors?

@delanni delanni added backport:all-open Backport to all branches that could still receive a release Team:Operations Kibana-Operations Team release_note:skip Skip the PR/issue when compiling release notes labels Mar 17, 2025
@delanni delanni marked this pull request as ready for review March 17, 2025 16:27
@delanni delanni requested a review from a team as a code owner March 17, 2025 16:27
@elasticmachine
Copy link
Contributor

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

@delanni delanni merged commit 8f0cd73 into main Mar 17, 2025
16 checks passed
@delanni delanni deleted the fix-artifact-testing-post-move branch March 17, 2025 16:27
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 7.17, 8.16, 8.17, 8.18, 8.x, 9.0

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 17, 2025
…est (elastic#214730)

## Summary
After elastic#210956 - the artifact
snapshot testing code is broken because a path reference wasn't updated.
This PR updates that.

Fixes:
https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5745

The checks started
(https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5764)
but it failed for a different reason. Probably some real errors?

(cherry picked from commit 8f0cd73)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts
8.16 Backport failed because of merge conflicts
8.17 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.17:
- [Obs Onboarding][Auto Detect] Fix auto detect script failing on systems with older curl versions (#214552)
- [Security Solution][Detection Engine] fixes rule preview works for form's invalid state (#173930) (#213801)
8.18 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.18:
- [Security Solution] Flyout navigation in alert/events: update tooltip copy (#214264)
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- Unauthorized route migration for routes owned by response-ops (#214785)
- Unauthorized route migration for routes owned by appex-ai-infra (#214777)
- Unauthorized route migration for routes owned by stack-monitoring (#214787)
- [Obs Overview] Replace clickable tooltip with external link (#213741)
- [ResponseOps][Rules] Allow users to delete snooze schedule from a rule (#213247)
- Upgrade to Storybook 8 (#195148)
- [Response Ops] Es query rule "size" field re-initializes to 100 if set to 0 when editing (#213636)
- [Space time] extending Scout with perfTracker fixture (#212397)
- SKA: Relocate /test to /src/platform/test (#210956)
9.0

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 214730

Questions ?

Please refer to the Backport tool documentation

@jbudz
Copy link
Contributor

jbudz commented Mar 17, 2025

^ Looks like we only want this in 8.x, and possibly not needed if included in #214791

@delanni delanni added backport:version Backport to applied version labels v8.19.0 backport:skip This PR does not require backporting and removed backport:all-open Backport to all branches that could still receive a release labels Mar 18, 2025
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@delanni delanni removed backport:version Backport to applied version labels v8.19.0 labels Mar 18, 2025
jbudz added a commit that referenced this pull request Mar 18, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to #214730 and
#210956
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 18, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to elastic#214730 and
elastic#210956

(cherry picked from commit ca04dba)
jbudz added a commit that referenced this pull request Mar 18, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [[package testing] Skip FTR
(#215070)](#215070)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT
[{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2025-03-18T21:33:50Z","message":"[package
testing] Skip FTR (#215070)\n\nThe functional test run during package
testing is currently erroring\npost folder migration, but after fixing
the path it's still not going to\nmatch on any tests.\n\nThis
temporarily disables the step while alternatives are
considered.\n\nRelated to #214730
and\nhttps://github.com//pull/210956","sha":"ca04dbafd285b99ad70e3e74683c88617352ca73","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","skip-ci","backport:version","v9.1.0","v8.19.0"],"title":"[package
testing] Skip
FTR","number":215070,"url":"https://github.com/elastic/kibana/pull/215070","mergeCommit":{"message":"[package
testing] Skip FTR (#215070)\n\nThe functional test run during package
testing is currently erroring\npost folder migration, but after fixing
the path it's still not going to\nmatch on any tests.\n\nThis
temporarily disables the step while alternatives are
considered.\n\nRelated to #214730
and\nhttps://github.com//pull/210956","sha":"ca04dbafd285b99ad70e3e74683c88617352ca73"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/215070","number":215070,"mergeCommit":{"message":"[package
testing] Skip FTR (#215070)\n\nThe functional test run during package
testing is currently erroring\npost folder migration, but after fixing
the path it's still not going to\nmatch on any tests.\n\nThis
temporarily disables the step while alternatives are
considered.\n\nRelated to #214730
and\nhttps://github.com//pull/210956","sha":"ca04dbafd285b99ad70e3e74683c88617352ca73"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jon <jon@elastic.co>
clintandrewhall pushed a commit to clintandrewhall/kibana that referenced this pull request Mar 20, 2025
…est (elastic#214730)

## Summary
After elastic#210956 - the artifact
snapshot testing code is broken because a path reference wasn't updated.
This PR updates that.

Fixes:
https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5745

The checks started
(https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5764)
but it failed for a different reason. Probably some real errors?
clintandrewhall pushed a commit to clintandrewhall/kibana that referenced this pull request Mar 20, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to elastic#214730 and
elastic#210956
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
…est (elastic#214730)

## Summary
After elastic#210956 - the artifact
snapshot testing code is broken because a path reference wasn't updated.
This PR updates that.

Fixes:
https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5745

The checks started
(https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/5764)
but it failed for a different reason. Probably some real errors?
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to elastic#214730 and
elastic#210956
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Mar 24, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to elastic#214730 and
elastic#210956
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Mar 31, 2025
The functional test run during package testing is currently erroring
post folder migration, but after fixing the path it's still not going to
match on any tests.

This temporarily disables the step while alternatives are considered.

Related to elastic#214730 and
elastic#210956
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants