Skip to content

[Axonius][identity] Update changelog description#18839

Merged
efd6 merged 3 commits into
elastic:feature/axonius-0.1.0from
muskan-agarwal26:datastream-identity_update_changelog_description
May 11, 2026
Merged

[Axonius][identity] Update changelog description#18839
efd6 merged 3 commits into
elastic:feature/axonius-0.1.0from
muskan-agarwal26:datastream-identity_update_changelog_description

Conversation

@muskan-agarwal26
Copy link
Copy Markdown
Contributor

Proposed commit message

  • Updated changelog description as mentioned in the comment

@muskan-agarwal26 muskan-agarwal26 requested a review from a team as a code owner May 6, 2026 08:29
@muskan-agarwal26 muskan-agarwal26 changed the base branch from main to feature/axonius-0.1.0 May 6, 2026 08:29
@muskan-agarwal26 muskan-agarwal26 requested a review from a team as a code owner May 6, 2026 08:29
@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 the Crest Contributions from Crest developement team. label May 6, 2026
Comment on lines -4 to 6
- description: Initial release.
- description: Add support for identity data stream.
type: enhancement
link: https://github.com/elastic/integrations/pull/16599
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.

    - description: Initial release.
      type: enhancement
      link: https://github.com/elastic/integrations/pull/16599
    - description: Add support for identity data stream.
      type: enhancement
      link: https://github.com/elastic/integrations/pull/16620

@muskan-agarwal26 muskan-agarwal26 requested a review from efd6 May 7, 2026 06:40
Copy link
Copy Markdown
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Thanks

@muskan-agarwal26
Copy link
Copy Markdown
Contributor Author

@efd6 , could you please merge this PR as well.

@efd6
Copy link
Copy Markdown
Contributor

efd6 commented May 7, 2026

/test

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

TL;DR

The Check integrations axonius job failed, but the provided log excerpt only contains teardown/artifact-upload output and does not include the failing assertion. Based on current evidence, this looks like a transient system/infrastructure failure rather than a regression from this PR’s changelog-only change.

Remediation

  • Re-run the failed Buildkite step (Check integrations axonius) to confirm whether this was transient.
  • If it fails again, attach the failing JUnit artifact content (especially axonius-system-*.xml) or the full pre-teardown log segment so we can pinpoint the exact failing test/assertion.
Investigation details

Root Cause

No deterministic code-level root cause is visible in the available Buildkite log capture. The log starts at teardown and ends with generic exit status output, without the original error stack/assertion.

Also, the PR delta is changelog-only (packages/axonius/changelog.yml), and package validation succeeds locally for this branch/commit.

Evidence

  • Build: https://buildkite.com/elastic/integrations/builds/42463
  • Job/step: Check integrations axonius
  • Key log excerpts:
    • /tmp/gh-aw/buildkite-logs/integrations-check-integrations-axonius.txt:73--- [axonius] failed
    • /tmp/gh-aw/buildkite-logs/integrations-check-integrations-axonius.txt:76-78 → generic exit only (Error: The command exited with status 1)
    • /tmp/gh-aw/buildkite-logs/integrations-check-integrations-axonius.txt:91-119 → artifacts uploaded after failure (including axonius-system-*.xml), but failing assertion text is not present in this log file.
  • Code change inspected:
    • packages/axonius/changelog.yml:1-9 (only PR-modified file)

Verification

  • Ran local package lint/build check for the PR branch via CI helper functions (use_elastic_package + check_package axonius): passed.
  • Full Buildkite package test flow could not be faithfully replayed in this environment due missing Buildkite runtime env setup in the wrapper script (YQ_VERSION/pipeline env expectations).

Follow-up

If the retry still fails, we should classify based on the first failing assertion from axonius-system-*.xml (test bug vs infra). Right now, available evidence best fits infrastructure/transient or flaky system test.

Note

🔒 Integrity filter blocked 3 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.

@efd6
Copy link
Copy Markdown
Contributor

efd6 commented May 7, 2026

@muskan-agarwal26 This is failing due to a mismatched assert.hit_count. This suggest that the situation on feature/axonius-0.1.0 is incorrect. Can you look into that?

@muskan-agarwal26
Copy link
Copy Markdown
Contributor Author

muskan-agarwal26 commented May 8, 2026

Thanks @efd6 , the failure was due to mismatched assert.hit_count in network, I have resolved it.

@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

@muskan-agarwal26
Copy link
Copy Markdown
Contributor Author

@efd6 , could you please merge this now?

@efd6 efd6 merged commit ca3939a into elastic:feature/axonius-0.1.0 May 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Crest Contributions from Crest developement team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants