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

Revert "[dashboard] fix Reset dashboard breaks panel state when ran after editor edits (#201687)" #201902

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Nov 27, 2024

This reverts commit 289bb16.

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#7498

[✅] x-pack/test_serverless/functional/test_suites/search/common_configs/config.group4.ts: 25/25 tests passed.

see run history

@jbudz jbudz marked this pull request as ready for review November 27, 2024 04:53
@jbudz jbudz requested a review from a team as a code owner November 27, 2024 04:53
@jbudz jbudz merged commit a0f88d9 into elastic:main Nov 27, 2024
7 of 10 checks passed
@kibanamachine kibanamachine added v9.0.0 backport:skip This commit does not require backporting labels Nov 27, 2024
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
dashboard 682.3KB 682.3KB +42.0B

@jbudz jbudz added backport:version Backport to applied version labels v8.17.0 v8.16.2 and removed backport:skip This commit does not require backporting labels Nov 27, 2024
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.17

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

@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.17

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

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.17

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 Nov 27, 2024
…e when ran after editor edits (#201687)" (#201902) (#202019)

# Backport

This will backport the following commits from `main` to `8.17`:
- [Revert "[dashboard] fix Reset dashboard breaks panel state when
ran after editor edits (#201687)"
(#201902)](#201902)

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

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

<!--BACKPORT
[{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2024-11-27T04:53:22Z","message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["v9.0.0","backport:version","v8.17.0","v8.16.2"],"title":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits
(#201687)\"","number":201902,"url":"https://github.com/elastic/kibana/pull/201902","mergeCommit":{"message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201902","number":201902,"mergeCommit":{"message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jon <jon@elastic.co>
kibanamachine added a commit that referenced this pull request Nov 27, 2024
…e when ran after editor edits (#201687)&quot; (#201902) (#202018)

# Backport

This will backport the following commits from `main` to `8.16`:
- [Revert &quot;[dashboard] fix Reset dashboard breaks panel state when
ran after editor edits (#201687)&quot;
(#201902)](#201902)

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

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

<!--BACKPORT
[{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2024-11-27T04:53:22Z","message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["v9.0.0","backport:version","v8.17.0","v8.16.2"],"title":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits
(#201687)\"","number":201902,"url":"https://github.com/elastic/kibana/pull/201902","mergeCommit":{"message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201902","number":201902,"mergeCommit":{"message":"Revert
\"[dashboard] fix Reset dashboard breaks panel state when ran after
editor edits (#201687)\" (#201902)\n\nThis reverts commit
289bb16.","sha":"a0f88d90134f8c49b32cffe23f438a91f77c7055"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jon <jon@elastic.co>
dej611 added a commit that referenced this pull request Nov 29, 2024
## Summary
Fixes: #190737, #190802, #190990, #179307

This test started failing after @nreese Reset PR was merged here:
#201687 and the Lens embeddable
refactor PR introducing a regression which affected the test.
Later on the PR was reverted with
#201902 .


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
…2275)

## Summary
Fixes: elastic#190737, elastic#190802, elastic#190990, elastic#179307

This test started failing after @nreese Reset PR was merged here:
elastic#201687 and the Lens embeddable
refactor PR introducing a regression which affected the test.
Later on the PR was reverted with
elastic#201902 .


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
…2275)

## Summary
Fixes: elastic#190737, elastic#190802, elastic#190990, elastic#179307

This test started failing after @nreese Reset PR was merged here:
elastic#201687 and the Lens embeddable
refactor PR introducing a regression which affected the test.
Later on the PR was reverted with
elastic#201902 .


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels v8.16.2 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants