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.x] [saved objects] enable deletion of saved objects by type if configured #6716

Merged
merged 3 commits into from
May 28, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 7a8c281 from #6443.

ZilongX
ZilongX previously approved these changes May 5, 2024
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.42%. Comparing base (9ff9d51) to head (d5c14e7).

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.x    #6716      +/-   ##
==========================================
- Coverage   67.42%   67.42%   -0.01%     
==========================================
  Files        3442     3442              
  Lines       67802    67821      +19     
  Branches    11018    11022       +4     
==========================================
+ Hits        45716    45726      +10     
- Misses      19420    19427       +7     
- Partials     2666     2668       +2     
Flag Coverage Δ
Linux_1 33.09% <0.00%> (-0.02%) ⬇️
Linux_2 55.16% <57.14%> (-0.01%) ⬇️
Linux_3 45.17% <0.00%> (-0.03%) ⬇️
Linux_4 34.95% <0.00%> (-0.02%) ⬇️
Windows_1 33.11% <0.00%> (-0.02%) ⬇️
Windows_2 55.13% <57.14%> (-0.01%) ⬇️
Windows_3 45.18% <0.00%> (-0.03%) ⬇️
Windows_4 34.95% <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
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.

ananzh
ananzh previously approved these changes May 5, 2024
#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>
@kavilla kavilla dismissed stale reviews from ZilongX and ananzh via e676653 May 7, 2024 23:31
@kavilla kavilla force-pushed the backport/backport-6443-to-2.x branch from 7d32df1 to e676653 Compare May 7, 2024 23:31
kavilla
kavilla previously approved these changes May 7, 2024
Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

backport to 2.14 already merged in rebased this.

abbyhu2000
abbyhu2000 previously approved these changes May 14, 2024
Signed-off-by: Qingyang(Abby) Hu <abigailhu2000@gmail.com>
@abbyhu2000 abbyhu2000 dismissed stale reviews from kavilla and themself via 0622874 May 14, 2024 18:19
@ZilongX
Copy link
Collaborator

ZilongX commented May 28, 2024

Update branch and re-trigger all checks now

@ZilongX ZilongX self-requested a review May 28, 2024 19:48
@ZilongX ZilongX merged commit 022ed35 into 2.x May 28, 2024
67 of 69 checks passed
@ZilongX ZilongX deleted the backport/backport-6443-to-2.x branch May 28, 2024 19:51
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.

5 participants