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

[Backport 2.14] [saved objects] enable deletion of saved objects by type if configured #6717

Merged
merged 1 commit into from
May 5, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 7a8c281 from #6443.

#6443)

* [saved objects] enable deletion of saved objects by type if configured

Adds the following settings:
```
migrations.delete.enabled
migrations.delete.types
```

`unknown` types already exist but the purpose of this type is for plugins
that are disabled. OpenSearch Dashboards gets confused when a plugin is not
defining a saved object type but the saved object exists. This can occur
when migrating from a non-OSD version and there exists non-compatiable
saved objects.

If OSD is failing to migrate an index because of a document, I can now
configure OSD to delete types of saved objects that I specified because
I know that these types should not be carried over.

resolves: #1040

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

* address comments

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

* Changeset file for PR #6443 created/updated

---------

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 7a8c281)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented May 5, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 67.73%. Comparing base (21f2d9f) to head (0713559).

Files Patch % Lines
...er/saved_objects/migrations/core/index_migrator.ts 33.33% 5 Missing and 1 partial ⚠️
.../core/server/saved_objects/saved_objects_config.ts 0.00% 2 Missing ⚠️
...earch_dashboards/opensearch_dashboards_migrator.ts 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             2.14    #6717      +/-   ##
==========================================
- Coverage   67.73%   67.73%   -0.01%     
==========================================
  Files        3414     3414              
  Lines       66922    66941      +19     
  Branches    10870    10874       +4     
==========================================
+ Hits        45331    45341      +10     
- Misses      18953    18960       +7     
- Partials     2638     2640       +2     
Flag Coverage Δ
Linux_1 33.18% <0.00%> (-0.02%) ⬇️
Linux_2 55.66% <57.14%> (-0.01%) ⬇️
Linux_3 ?
Linux_4 34.98% <0.00%> (-0.02%) ⬇️
Windows_1 33.20% <0.00%> (-0.02%) ⬇️
Windows_2 55.63% <57.14%> (-0.01%) ⬇️
Windows_3 45.24% <0.00%> (-0.03%) ⬇️
Windows_4 34.98% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ananzh ananzh merged commit 1c9967d into 2.14 May 5, 2024
88 of 91 checks passed
@github-actions github-actions bot deleted the backport/backport-6443-to-2.14 branch May 5, 2024 04:06
@ananzh
Copy link
Member

ananzh commented May 5, 2024

ciGroup3 fails on src/plugins/dashboard/public/application/components/dashboard_listing/dashboard_listing.test.tsx, which seems flaky. This PR has no change on dashboard. Also the snapshot update is not valid. Local can't reproduce.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports distinguished-contributor operational excellence v2.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants