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

[eem] disable entity discovery on plugin startup #204536

Merged
merged 19 commits into from
Dec 19, 2024

Conversation

klacabane
Copy link
Contributor

@klacabane klacabane commented Dec 17, 2024

Disable entity discovery on plugin startup and remove some related files.

We'll need a follow up to cleanup endpoints.

Testing

  • enable entity discovery from main with PUT kbn:/internal/entities/managed/enablement, preferably in a cluster with data so you can verify that the .entities* indices are also deleted
  • checkout that branch and start kibana
  • verify all v1 builtin definitions with their transforms/templates/pipelines are deleted. .entities indices for these definitions should also be gone

@klacabane klacabane 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) Team:obs-entities Observability Entities Team labels Dec 17, 2024
@klacabane klacabane self-assigned this Dec 17, 2024
@klacabane klacabane marked this pull request as ready for review December 18, 2024 12:40
@klacabane klacabane requested a review from a team as a code owner December 18, 2024 12:40
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-entities (Team:obs-entities)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

References to deprecated APIs

id before after diff
entityManager 10 12 +2

History

cc @klacabane

@klacabane klacabane merged commit 505cc0f into elastic:main Dec 19, 2024
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@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:
- [[ES

Manual backport

To create the backport manually run:

node scripts/backport --pr 204536

Questions ?

Please refer to the Backport tool documentation

JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Dec 19, 2024
Disable entity discovery on plugin startup and remove some related
files.

We'll need a follow up to cleanup endpoints.

### Testing

- enable entity discovery from `main` with `PUT
kbn:/internal/entities/managed/enablement`, preferably in a cluster with
data so you can verify that the `.entities*` indices are also deleted
- checkout that branch and start kibana
- verify all v1 builtin definitions with their
transforms/templates/pipelines are deleted. `.entities` indices for
these definitions should also be gone

---------

Co-authored-by: kibanamachine <[email protected]>
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Dec 20, 2024
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 204536 locally

@klacabane
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

klacabane added a commit to klacabane/kibana that referenced this pull request Dec 20, 2024
Disable entity discovery on plugin startup and remove some related
files.

We'll need a follow up to cleanup endpoints.

### Testing

- enable entity discovery from `main` with `PUT
kbn:/internal/entities/managed/enablement`, preferably in a cluster with
data so you can verify that the `.entities*` indices are also deleted
- checkout that branch and start kibana
- verify all v1 builtin definitions with their
transforms/templates/pipelines are deleted. `.entities` indices for
these definitions should also be gone

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 505cc0f)

# Conflicts:
#	x-pack/platform/plugins/shared/entity_manager/server/lib/entities/built_in/services_from_ecs_data.ts
klacabane added a commit that referenced this pull request Dec 23, 2024
…5059)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[eem] disable entity discovery on plugin startup
(#204536)](#204536)

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

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

<!--BACKPORT [{"author":{"name":"Kevin
Lacabane","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-19T13:35:12Z","message":"[eem]
disable entity discovery on plugin startup (#204536)\n\nDisable entity
discovery on plugin startup and remove some related\nfiles.\n\nWe'll
need a follow up to cleanup endpoints.\n\n### Testing\n\n- enable entity
discovery from `main` with
`PUT\nkbn:/internal/entities/managed/enablement`, preferably in a
cluster with\ndata so you can verify that the `.entities*` indices are
also deleted\n- checkout that branch and start kibana\n- verify all v1
builtin definitions with their\ntransforms/templates/pipelines are
deleted. `.entities` indices for\nthese definitions should also be
gone\n\n---------\n\nCo-authored-by: kibanamachine
<[email protected]>","sha":"505cc0fbbb901263b3a07d2b1a2f7b310a690e18","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport
missing","v9.0.0","backport:prev-minor","Team:obs-entities"],"number":204536,"url":"https://github.com/elastic/kibana/pull/204536","mergeCommit":{"message":"[eem]
disable entity discovery on plugin startup (#204536)\n\nDisable entity
discovery on plugin startup and remove some related\nfiles.\n\nWe'll
need a follow up to cleanup endpoints.\n\n### Testing\n\n- enable entity
discovery from `main` with
`PUT\nkbn:/internal/entities/managed/enablement`, preferably in a
cluster with\ndata so you can verify that the `.entities*` indices are
also deleted\n- checkout that branch and start kibana\n- verify all v1
builtin definitions with their\ntransforms/templates/pipelines are
deleted. `.entities` indices for\nthese definitions should also be
gone\n\n---------\n\nCo-authored-by: kibanamachine
<[email protected]>","sha":"505cc0fbbb901263b3a07d2b1a2f7b310a690e18"}},"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/204536","number":204536,"mergeCommit":{"message":"[eem]
disable entity discovery on plugin startup (#204536)\n\nDisable entity
discovery on plugin startup and remove some related\nfiles.\n\nWe'll
need a follow up to cleanup endpoints.\n\n### Testing\n\n- enable entity
discovery from `main` with
`PUT\nkbn:/internal/entities/managed/enablement`, preferably in a
cluster with\ndata so you can verify that the `.entities*` indices are
also deleted\n- checkout that branch and start kibana\n- verify all v1
builtin definitions with their\ntransforms/templates/pipelines are
deleted. `.entities` indices for\nthese definitions should also be
gone\n\n---------\n\nCo-authored-by: kibanamachine
<[email protected]>","sha":"505cc0fbbb901263b3a07d2b1a2f7b310a690e18"}}]}]
BACKPORT-->
@kibanamachine kibanamachine added v8.18.0 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Dec 23, 2024
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
Disable entity discovery on plugin startup and remove some related
files.

We'll need a follow up to cleanup endpoints.

### Testing

- enable entity discovery from `main` with `PUT
kbn:/internal/entities/managed/enablement`, preferably in a cluster with
data so you can verify that the `.entities*` indices are also deleted
- checkout that branch and start kibana
- verify all v1 builtin definitions with their
transforms/templates/pipelines are deleted. `.entities` indices for
these definitions should also be gone

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
Disable entity discovery on plugin startup and remove some related
files.

We'll need a follow up to cleanup endpoints.

### Testing

- enable entity discovery from `main` with `PUT
kbn:/internal/entities/managed/enablement`, preferably in a cluster with
data so you can verify that the `.entities*` indices are also deleted
- checkout that branch and start kibana
- verify all v1 builtin definitions with their
transforms/templates/pipelines are deleted. `.entities` indices for
these definitions should also be gone

---------

Co-authored-by: kibanamachine <[email protected]>
viduni94 pushed a commit to viduni94/kibana that referenced this pull request Jan 23, 2025
Disable entity discovery on plugin startup and remove some related
files.

We'll need a follow up to cleanup endpoints.

### Testing

- enable entity discovery from `main` with `PUT
kbn:/internal/entities/managed/enablement`, preferably in a cluster with
data so you can verify that the `.entities*` indices are also deleted
- checkout that branch and start kibana
- verify all v1 builtin definitions with their
transforms/templates/pipelines are deleted. `.entities` indices for
these definitions should also be gone

---------

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) release_note:skip Skip the PR/issue when compiling release notes Team:obs-entities Observability Entities Team v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[EEM] disable all built-in observability entity definitions at kibana startup
4 participants