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

[Logs] Deprecate configuration settings #201625

Merged

Conversation

tonyghiani
Copy link
Contributor

@tonyghiani tonyghiani commented Nov 25, 2024

📓 Summary

Closes #200898

These changes deprecate some unused configurations and update the implementation where required in preparation for the Kibana v9 upgrade.

Screenshot 2024-11-25 at 12 54 14

@tonyghiani tonyghiani added release_note:deprecation Team:obs-ux-logs Observability Logs User Experience Team backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Nov 25, 2024
@tonyghiani tonyghiani marked this pull request as ready for review November 26, 2024 09:06
@tonyghiani tonyghiani requested review from a team as code owners November 26, 2024 09:06
@tonyghiani tonyghiani requested a review from a team November 26, 2024 09:06
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@cauemarcondes cauemarcondes left a comment

Choose a reason for hiding this comment

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

LGTM, @tonyghiani do you really want to backport it to 8.x and 8.17?

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 26, 2024
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@tonyghiani
Copy link
Contributor Author

Thanks for the review @cauemarcondes

do you really want to backport it to 8.x and 8.17?

I just asked to the core team regarding this kind of deprecations and it si correct to backport them to 8.x 👌

Copy link
Contributor

@mykolaharmash mykolaharmash left a comment

Choose a reason for hiding this comment

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

Onboarding changes are looking good ✨ Thank you @tonyghiani!

@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 4, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: e93ac47
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-201625-e93ac47ffa86

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
observabilityOnboarding 272.1KB 272.1KB -5.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
observabilityLogsExplorer 15.3KB 15.2KB -115.0B
observabilityOnboarding 10.5KB 10.5KB -22.0B
total -137.0B

History

@tonyghiani tonyghiani merged commit 2994b00 into elastic:main Dec 5, 2024
8 checks passed
@tonyghiani tonyghiani deleted the 200898-deprecate-configuration-settings branch December 5, 2024 09:24
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- [Infra] Exclude frozen/cold data tiers from source queries (#201804)

Manual backport

To create the backport manually run:

node scripts/backport --pr 201625

Questions ?

Please refer to the Backport tool documentation

@tonyghiani
Copy link
Contributor Author

💚 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

tonyghiani added a commit to tonyghiani/kibana that referenced this pull request Dec 5, 2024
## 📓 Summary

Closes elastic#200898

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 2994b00)

# Conflicts:
#	x-pack/plugins/observability_solution/infra/tsconfig.json
tonyghiani added a commit that referenced this pull request Dec 5, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Logs] Deprecate configuration settings
(#201625)](#201625)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Marco Antonio
Ghiani","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-05T09:23:54Z","message":"[Logs]
Deprecate configuration settings (#201625)\n\n## 📓 Summary\r\n\r\nCloses
#200898 \r\n\r\nThese changes deprecate some unused configurations and
update the\r\nimplementation where required in preparation for the
Kibana v9 upgrade.\r\n\r\n<img width=\"3004\" alt=\"Screenshot
2024-11-25 at 12 54
14\"\r\nsrc=\"https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Marco Antonio Ghiani <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"2994b0001cbae6bc1528bc1ad77c435597e0d5a2","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:deprecation","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-logs"],"number":201625,"url":"https://github.com/elastic/kibana/pull/201625","mergeCommit":{"message":"[Logs]
Deprecate configuration settings (#201625)\n\n## 📓 Summary\r\n\r\nCloses
#200898 \r\n\r\nThese changes deprecate some unused configurations and
update the\r\nimplementation where required in preparation for the
Kibana v9 upgrade.\r\n\r\n<img width=\"3004\" alt=\"Screenshot
2024-11-25 at 12 54
14\"\r\nsrc=\"https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Marco Antonio Ghiani <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"2994b0001cbae6bc1528bc1ad77c435597e0d5a2"}},"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/201625","number":201625,"mergeCommit":{"message":"[Logs]
Deprecate configuration settings (#201625)\n\n## 📓 Summary\r\n\r\nCloses
#200898 \r\n\r\nThese changes deprecate some unused configurations and
update the\r\nimplementation where required in preparation for the
Kibana v9 upgrade.\r\n\r\n<img width=\"3004\" alt=\"Screenshot
2024-11-25 at 12 54
14\"\r\nsrc=\"https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Marco Antonio Ghiani <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"2994b0001cbae6bc1528bc1ad77c435597e0d5a2"}}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <[email protected]>
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/kibana that referenced this pull request Dec 10, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
mykolaharmash pushed a commit to mykolaharmash/kibana that referenced this pull request Dec 11, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## 📓 Summary

Closes elastic#200898 

These changes deprecate some unused configurations and update the
implementation where required in preparation for the Kibana v9 upgrade.

<img width="3004" alt="Screenshot 2024-11-25 at 12 54 14"
src="https://github.com/user-attachments/assets/cfa56d25-a270-4ec5-a97a-e72e7a7478a4">

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
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) ci:project-deploy-observability Create an Observability project release_note:deprecation Team:obs-ux-logs Observability Logs User Experience Team v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Logs UI] Deprecate legacy/unused configurations
6 participants