Skip to content

[kubernetes/container_logs] - Add system tests #19179

Merged
VihasMakwana merged 5 commits into
elastic:mainfrom
VihasMakwana:add-base-fields
May 27, 2026
Merged

[kubernetes/container_logs] - Add system tests #19179
VihasMakwana merged 5 commits into
elastic:mainfrom
VihasMakwana:add-base-fields

Conversation

@VihasMakwana
Copy link
Copy Markdown
Contributor

Proposed commit message

This PR adds missing system tests and updates base-fields.yml to include the fields that are exported.

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

@VihasMakwana VihasMakwana requested a review from a team as a code owner May 23, 2026 13:06
@VihasMakwana VihasMakwana self-assigned this May 23, 2026
@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

TL;DR

Buildkite failed in Check integrations kubernetes because package validation detected a version/changelog mismatch (current manifest version doesn't have changelog entry). Update packages/kubernetes/changelog.yml to include the exact current packages/kubernetes/manifest.yml version (or align the manifest version).

Remediation

  • Add a top changelog entry whose version exactly matches packages/kubernetes/manifest.yml version in this PR.
  • Re-run the package check job (.buildkite/scripts/test_one_package.sh packages/kubernetes ...) after syncing those two files.
Investigation details

Root Cause

The failing step is package lint/validation, not test/runtime logic. The validator exits with:

checking package failed: linting package failed: found 1 validation error: current manifest version doesn't have changelog entry

This means the kubernetes package manifest version and changelog versions are out of sync in the PR head.

Evidence

Error: checking package failed: linting package failed: found 1 validation error:

  1. current manifest version doesn't have changelog entry

[kubernetes] run_tests_package failed
  • Relevant files in repo:
    • packages/kubernetes/manifest.yml (version field)
    • packages/kubernetes/changelog.yml (top - version: entries)

Verification

  • Not run locally against the PR head SHA because that specific commit is not available in this checkout.

Follow-up

If you intentionally bumped the package version in this PR, add the matching changelog entry at the top of changelog.yml (newest first).

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.

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

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

Comment thread packages/kubernetes/changelog.yml Outdated
Co-authored-by: Andrew Gizas <andreas.gkizas@elastic.co>
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @VihasMakwana

@VihasMakwana VihasMakwana merged commit 76a9081 into elastic:main May 27, 2026
8 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:kubernetes Kubernetes Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants