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

add archive restore retry on mapper parsing exception (6.1) #21258

Open
wants to merge 1 commit into
base: 6.1
Choose a base branch
from

Conversation

graylog-internal-actions-access[bot]
Copy link

@graylog-internal-actions-access graylog-internal-actions-access bot commented Jan 2, 2025

Note: This is a backport of #21197 to 6.1.

Description

Since this PR, we save the mapping in the archive-metadata.json file and use this mapping to create an index. The problem is that if a mapping contains field properties that reference a field from the index settings, this leads to an error. We have therefore started to also store the index settings in the archive-metadata.json and apply them during index creation. This has been done in this PR. Unfortunately, there is still an error case for archives that do not contain any settings. For this scenario, this PR implements a retry with default mappings/settings if such an error occurs.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

companion PR:
/jenkins-pr-deps Graylog2/graylog-plugin-enterprise#9455

* add restore retry on mapper parsing exception

* add changelog

* fix changelog

* update changelog

Co-authored-by: Patrick Mann <[email protected]>

---------

Co-authored-by: Patrick Mann <[email protected]>
(cherry picked from commit 7664ddf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant