Skip to content

fix: updated kibana version & enable_request_tracer default set to false#19251

Merged
efd6 merged 2 commits into
elastic:mainfrom
metron-labs:develop-02
Jun 1, 2026
Merged

fix: updated kibana version & enable_request_tracer default set to false#19251
efd6 merged 2 commits into
elastic:mainfrom
metron-labs:develop-02

Conversation

@Bharatkk-metron
Copy link
Copy Markdown
Contributor

Change Type

  • Bug

Proposed commit message

In Ref to : #19246

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

@Bharatkk-metron Bharatkk-metron requested a review from a team as a code owner May 28, 2026 05:48
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Reviewers

Buildkite won't run for external contributors automatically; you need to add a comment:

  • /test : will kick off a build in Buildkite.

NOTE: https://github.com/elastic/integrations/blob/main/.buildkite/pull-requests.json contains all those details.

@andrewkroh andrewkroh added Integration:doppel Doppel (Partner supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels May 28, 2026
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

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

@efd6
Copy link
Copy Markdown
Contributor

efd6 commented May 28, 2026

/test

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

TL;DR

Buildkite failed in Check integrations doppel because package lint could not find a changelog entry for the current manifest version. The immediate fix is to reorder packages/doppel/changelog.yml so the newest version (0.1.1) is first.

Remediation

  • Move the 0.1.1 block above 0.1.0 in packages/doppel/changelog.yml (this repo expects newest versions at the top).
  • Re-run package lint for doppel (or rerun the Buildkite job) after reordering.
Investigation details

Root Cause

This is a configuration/metadata failure in package changelog ordering, not a code runtime failure.

  • packages/doppel/manifest.yml:4 sets version: 0.1.1.
  • packages/doppel/changelog.yml:1 starts with 0.1.0, and 0.1.1 appears later at packages/doppel/changelog.yml:6.
  • In this repository, changelogs are expected with newer versions first (see header convention in other packages: # newer versions go on top).

With 0.1.1 not first, lint reports that the current manifest version has no matching changelog entry.

Evidence

  • Build: https://buildkite.com/elastic/integrations/builds/43723
  • Job/step: Check integrations doppel (.buildkite/scripts/test_one_package.sh packages/doppel origin/main 2e385ea20b28aed81b362a4f3d234ec240708572)
  • Key log excerpt:
    Error: checking package failed: linting package failed: found 1 validation error:
       1. current manifest version doesn't have changelog entry
    

Verification

  • Not run locally end-to-end in this environment because the Buildkite helper script depends on CI-provided environment variables (YQ_VERSION).

Follow-up

If this still fails after reorder, check for formatting drift in changelog.yml (unexpected indentation or malformed YAML around the 0.1.1 entry).

Note

🔒 Integrity filter blocked 2 items

The following items were blocked because they don't meet the GitHub integrity level.

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

Comment on lines +6 to +13
- version: "0.1.1"
changes:
- description: Fix kibana.version constraints to allow v9 stack.
type: bugfix
link: https://github.com/elastic/integrations/pull/19251
- description: Fix default request trace enablement value.
type: bugfix
link: https://github.com/elastic/integrations/pull/19251
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

New version at the top.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved the new version at top. Thank you!

@Bharatkk-metron
Copy link
Copy Markdown
Contributor Author

@efd6 Could you please run test again? Thank you!

@efd6
Copy link
Copy Markdown
Contributor

efd6 commented Jun 1, 2026

/test

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@efd6 efd6 merged commit 3788ca8 into elastic:main Jun 1, 2026
4 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

@Bharatkk-metron
Copy link
Copy Markdown
Contributor Author

Hi @efd6 ,

Thank you so much for you help in getting this PR merged.

We've performed the sanity testing and now all app features and working as expected.

Could you please help us to resolve this issue #19227 and get the app ready for users in production?

Thanks!

herrBez pushed a commit to herrBez/integrations that referenced this pull request Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:doppel Doppel (Partner 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.

4 participants