Skip to content

[proofpoint_on_demand] Adapt definitions of nested subfields - #11031

Merged
jsoriano merged 2 commits into
elastic:mainfrom
jsoriano:proofpoint-on-demand-nested
Sep 9, 2024
Merged

[proofpoint_on_demand] Adapt definitions of nested subfields#11031
jsoriano merged 2 commits into
elastic:mainfrom
jsoriano:proofpoint-on-demand-nested

Conversation

@jsoriano

@jsoriano jsoriano commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

Same change as #11016 that I forgot to apply to this package.

Adapt definitions of subfields of nested objects so they are actually installed by Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was only installing empty nested objects. To workaround that, subfields can be moved to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround for older versions of the stack.

@jsoriano jsoriano self-assigned this Sep 6, 2024
@jsoriano
jsoriano requested a review from a team as a code owner September 6, 2024 15:16
@jsoriano jsoriano added Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:proofpoint_on_demand Proofpoint On Demand labels Sep 6, 2024
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@andrewkroh andrewkroh added the bugfix Pull request that fixes a bug issue label Sep 6, 2024
@elasticmachine

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

cc @jsoriano

@jsoriano
jsoriano merged commit 92f726e into elastic:main Sep 9, 2024
@jsoriano
jsoriano deleted the proofpoint-on-demand-nested branch September 9, 2024 12:16
@elasticmachine

Copy link
Copy Markdown

Package proofpoint_on_demand - 0.1.1 containing this change is available at https://epr.elastic.co/search?package=proofpoint_on_demand

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
…#11031)

Adapt definitions of subfields of nested objects so they are actually installed by
Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was
only installing empty nested objects. To workaround that, subfields can be moved
to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround
for older versions of the stack.
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
…#11031)

Adapt definitions of subfields of nested objects so they are actually installed by
Fleet. New mappings are otherwise equivalent.

These mappings were correctly defined as expected by the spec, but Fleet was
only installing empty nested objects. To workaround that, subfields can be moved
to have their own definitions.

Issue in Fleet is fixed in elastic/kibana#191730, but we can apply this workaround
for older versions of the stack.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue Integration:proofpoint_on_demand Proofpoint On Demand Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

4 participants