Skip to content

Add false_allow_templates to DynamicMapping - #1178

Open
Truta446 wants to merge 1 commit into
opensearch-project:mainfrom
Truta446:feat/dynamic-mapping-false-allow-templates
Open

Add false_allow_templates to DynamicMapping#1178
Truta446 wants to merge 1 commit into
opensearch-project:mainfrom
Truta446:feat/dynamic-mapping-false-allow-templates

Conversation

@Truta446

Copy link
Copy Markdown

Description

Adds the false_allow_templates value to the DynamicMapping schema.

OpenSearch's dynamic mapping parameter accepts false_allow_templates (new fields are only indexed when they match a dynamic template, otherwise they are not indexed), but the spec's DynamicMapping enum was missing it — it only listed false, strict, strict_allow_templates, true, and boolean. As a result, generated clients reject the value at the type level even though it works at runtime.

The value was introduced in OpenSearch 3.3.0 (opensearch-project/OpenSearch#19065), so the added integration test case is gated with version: '>= 3.3', mirroring the existing strict_allow_templates case.

Reported downstream in the JS client: .

Issues Resolved

Fixes the missing false_allow_templates value in DynamicMapping.

Changes

  • spec/schemas/_common.mapping.yaml: add false_allow_templates to the DynamicMapping string enum.
  • tests/default/indices/mapping/mapping.yaml: add an integration test setting dynamic: false_allow_templates (gated >= 3.3).
  • CHANGELOG.md: changelog entry.

Check List

  • All tests pass (npm run lint:spec passes; npm run merge succeeds and the bundled spec includes the value).
  • Updated the CHANGELOG.
  • Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Juan Versolato Lopes <juan.versolato@atlas.red>
@Truta446
Truta446 force-pushed the feat/dynamic-mapping-false-allow-templates branch from 4b5e625 to c056152 Compare July 23, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants