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

fix: remove deprecated squid filebeat module tutorial from integrations #191115

Conversation

pkoutsovasilis
Copy link
Contributor

@pkoutsovasilis pkoutsovasilis commented Aug 22, 2024

Summary

This PR removes the deprecated squid integration based on the Filebeat module (deprecation issue here). Instead, as of now, a user that wants to use the squad integration needs to tick the Display beta integrations and install the "new" squid integration

Risk Matrix

N/A

For maintainers

@pkoutsovasilis pkoutsovasilis added the release_note:skip Skip the PR/issue when compiling release notes label Aug 22, 2024
@pkoutsovasilis pkoutsovasilis self-assigned this Aug 22, 2024
@pkoutsovasilis pkoutsovasilis requested a review from a team as a code owner August 22, 2024 16:45
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @pkoutsovasilis

Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

LGTM

@pkoutsovasilis
Copy link
Contributor Author

Hi @afharo 👋 do we need to backport this one to 8.15 as well?

@afharo
Copy link
Member

afharo commented Aug 23, 2024

Hi @afharo 👋 do we need to backport this one to 8.15 as well?

Good question! I think that the changes are minimal enough that should be safe to backport (especially given that the modules were removed in 8.14).

@pkoutsovasilis pkoutsovasilis added v8.14.4 v8.15.1 backport:version Backport to applied version labels labels Aug 23, 2024
@pkoutsovasilis
Copy link
Contributor Author

pkoutsovasilis commented Aug 23, 2024

Good question! I think that the changes are minimal enough that should be safe to backport (especially given that the modules were removed in 8.14).

sorry but I am not quite familiar with the backporting process in this repo 😅 are my labels correct?

@afharo
Copy link
Member

afharo commented Aug 23, 2024

We only backport to the previous minor (and/or major, but it doesn't apply here).

I've corrected the labels to do so.

Once back ported & merged, the CI will add the versions that this PR will be released in.

@afharo afharo added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Integrations Team Label for Observability Integrations team and removed v8.14.4 v8.15.1 backport:version Backport to applied version labels labels Aug 23, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@pkoutsovasilis pkoutsovasilis merged commit bd9dde8 into elastic:main Aug 23, 2024
26 checks passed
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.15 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 191115

Questions ?

Please refer to the Backport tool documentation

@afharo
Copy link
Member

afharo commented Aug 23, 2024

💚 All backports created successfully

Status Branch Result
8.15

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

Questions ?

Please refer to the Backport tool documentation

afharo pushed a commit to afharo/kibana that referenced this pull request Aug 23, 2024
…ns (elastic#191115)

## Summary

This PR removes the deprecated `squid` integration based on the Filebeat
module (deprecation issue
[here](elastic/beats#37746)). Instead, as of
now, a user that wants to use the `squad` integration needs to tick the
`Display beta integrations` and install the "new" squid integration

### Risk Matrix

N/A

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit bd9dde8)

# Conflicts:
#	x-pack/plugins/translations/translations/fr-FR.json
#	x-pack/plugins/translations/translations/ja-JP.json
#	x-pack/plugins/translations/translations/zh-CN.json
afharo added a commit that referenced this pull request Aug 23, 2024
…egrations (#191115) (#191195)

# Backport

This will backport the following commits from `main` to `8.15`:
- [fix: remove deprecated squid filebeat module tutorial from
integrations (#191115)](#191115)

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

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

<!--BACKPORT [{"author":{"name":"Panos
Koutsovasilis","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-08-23T13:47:49Z","message":"fix:
remove deprecated squid filebeat module tutorial from integrations
(#191115)\n\n## Summary\n\nThis PR removes the deprecated `squid`
integration based on the Filebeat\nmodule (deprecation
issue\n[here](elastic/beats#37746)). Instead,
as of\nnow, a user that wants to use the `squad` integration needs to
tick the\n`Display beta integrations` and install the \"new\" squid
integration\n\n\n### Risk Matrix\n\nN/A\n\n\n### For maintainers\n\n- [
] This was checked for breaking API changes and was
[labeled\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"bd9dde8ba749a33a1d5b56dff77d2e47c6e7e17c","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","Team:Integrations","backport:prev-minor","v8.16.0"],"number":191115,"url":"https://github.com/elastic/kibana/pull/191115","mergeCommit":{"message":"fix:
remove deprecated squid filebeat module tutorial from integrations
(#191115)\n\n## Summary\n\nThis PR removes the deprecated `squid`
integration based on the Filebeat\nmodule (deprecation
issue\n[here](elastic/beats#37746)). Instead,
as of\nnow, a user that wants to use the `squad` integration needs to
tick the\n`Display beta integrations` and install the \"new\" squid
integration\n\n\n### Risk Matrix\n\nN/A\n\n\n### For maintainers\n\n- [
] This was checked for breaking API changes and was
[labeled\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"bd9dde8ba749a33a1d5b56dff77d2e47c6e7e17c"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","labelRegex":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/191115","number":191115,"mergeCommit":{"message":"fix:
remove deprecated squid filebeat module tutorial from integrations
(#191115)\n\n## Summary\n\nThis PR removes the deprecated `squid`
integration based on the Filebeat\nmodule (deprecation
issue\n[here](elastic/beats#37746)). Instead,
as of\nnow, a user that wants to use the `squad` integration needs to
tick the\n`Display beta integrations` and install the \"new\" squid
integration\n\n\n### Risk Matrix\n\nN/A\n\n\n### For maintainers\n\n- [
] This was checked for breaking API changes and was
[labeled\nappropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"bd9dde8ba749a33a1d5b56dff77d2e47c6e7e17c"}}]}]
BACKPORT-->

---------

Co-authored-by: Panos Koutsovasilis <[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) release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Integrations Team Label for Observability Integrations team v8.15.1 v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants