Skip to content

Conversation

@chemamartinez
Copy link
Contributor

Proposed commit message

Converting to long an empty string field leads to the following pipeline error:

Processor 'convert' with tag '' failed with message 'For input string: ""'

Fixed all the possible cases in the Akamai ingest pipeline.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

@chemamartinez chemamartinez added Integration:akamai Akamai (Community supported) bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jan 8, 2025
@chemamartinez chemamartinez self-assigned this Jan 8, 2025
@chemamartinez chemamartinez marked this pull request as ready for review January 8, 2025 17:32
@chemamartinez chemamartinez requested a review from a team as a code owner January 8, 2025 17:32
@elasticmachine
Copy link

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

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

Package akamai 👍(0) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
siem 1893.94 1540.83 -353.11 (-18.64%) 💔

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

cc @chemamartinez

@elastic-sonarqube
Copy link

@chemamartinez chemamartinez merged commit de2fbe9 into elastic:main Jan 8, 2025
5 checks passed
@elastic-vault-github-plugin-prod

Package akamai - 2.27.1 containing this change is available at https://epr.elastic.co/package/akamai/2.27.1/

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
Fix pipeline error when converting an empty numerical field.
@chemamartinez chemamartinez deleted the akamai-fix-pipeline-convert-condition branch February 4, 2025 13:33
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
Fix pipeline error when converting an empty numerical field.
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:akamai Akamai (Community supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants