Skip to content

aws: move #20529 changelog entry to 8.1.0 and bump version - #20920

Merged
kcreddy merged 1 commit into
elastic:mainfrom
kcreddy:aws-fix-changelog
Aug 27, 2026
Merged

aws: move #20529 changelog entry to 8.1.0 and bump version#20920
kcreddy merged 1 commit into
elastic:mainfrom
kcreddy:aws-fix-changelog

Conversation

@kcreddy

@kcreddy kcreddy commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

aws: move #20529 changelog entry to 8.1.0 and bump version

The Identity Federation enhancement from #20529 was merged into the
already-released 7.4.0 changelog block instead of a new top version.
It merged the day after #14717 introduced 8.0.0, and the conflict
resolution left the entry in the stale 7.4.0 block, so the shipped
code carried no matching changelog entry.

Move the entry into a new 8.1.0 block and bump manifest.yml to 8.1.0
so the changelog top version matches the package version.

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

Screenshots

@kcreddy
aws: move elastic#20529 changelog entry to 8.1.0 and bump version
f311fcc 
The Identity Federation enhancement from elastic#20529 was merged into the
already-released 7.4.0 changelog block instead of a new top version.
It merged the day after elastic#14717 introduced 8.0.0, and the conflict
resolution left the entry in the stale 7.4.0 block, so the shipped
code carried no matching changelog entry.

Move the entry into a new 8.1.0 block and bump manifest.yml to 8.1.0
so the changelog top version matches the package version.
@kcreddy kcreddy added Integration:aws AWS Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] changelog-link-check:skip labels Aug 26, 2026
@kcreddy
kcreddy marked this pull request as ready for review August 26, 2026 17:04
@kcreddy
kcreddy requested review from a team as code owners August 26, 2026 17:04
Copilot AI lite review requested due to automatic review settings August 26, 2026 17:04
@infra-vault-gh-plugin-prod

infra-vault-gh-plugin-prod Bot commented Aug 26, 2026

Copy link
Copy Markdown

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

@kcreddy kcreddy self-assigned this Aug 26, 2026
Copilot AI reviewed Aug 26, 2026
View reviewed changes

Copilot AI left a comment

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.

Pull request overview

Adjusts the AWS integration package metadata so the package version and top changelog entry correctly reflect the Identity Federation enhancement that was previously listed under an already-released version.

Changes:

  • Bump packages/aws package version from 8.0.0 to 8.1.0.
  • Move the changelog entry for PR #20529 into a new top 8.1.0 changelog block.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/aws/manifest.yml Bumps the AWS package version to 8.1.0 to match the intended release notes.
packages/aws/changelog.yml Adds a new 8.1.0 section and relocates the #20529 enhancement entry to the top.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Changelog link mismatch — expected https://github.com/elastic/integrations/pull/20920 in the following file(s):

  • packages/aws/changelog.yml

Tip

If expected, add the changelog-link-check:skip label to skip this check. Or, if an issue link was intended, use .../issues/<n> instead.

View Buildkite build
@kcreddy

@infra-vault-gh-plugin-prod

infra-vault-gh-plugin-prod Bot commented Aug 26, 2026 •
edited
Loading

Copy link
Copy Markdown

💔 Build Failed

Failed CI Steps

History

cc @kcreddy

vera-review-bot[bot]
vera-review-bot Bot reviewed Aug 26, 2026
View reviewed changes
Comment thread packages/aws/changelog.yml
changes:
- description: Enable Identity Federation (Cloud Connectors) for the SecurityHub data streams (securityhub_findings, securityhub_findings_full_posture, securityhub_insights) by migrating their HTTPJSON templates from hand-rolled SigV4 signing to the input's auth.aws configuration. Fixes silent session-token breakage in the hand-rolled path.
type: enhancement
link: https://github.com/elastic/integrations/pull/20529

@vera-review-bot vera-review-bot Bot Aug 26, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Severity: 🟡 Medium confidence: high path: packages/aws/changelog.yml:6

Changelog links a different PR number

Details

This changelog entry's link: points at pull/20529, but it was added in PR #​20920. It is likely a leftover template placeholder or a copy from another PR.

Recommendation:

Point each added changelog entry's link at this PR:

link: https://github.com/elastic/integrations/pull/20920

🤖 AI-Generated Review | Vera Review Bot - v0.2.7 | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@vera-review-bot

vera-review-bot Bot commented Aug 26, 2026

Copy link
Copy Markdown

Review summary

Issues found across the latest commits f311fcc — 1 medium
  • 🟡 Changelog links a different PR number (link) (Unresolved)

A new commit triggers another review — at most once every 15 minutes. I skip the PR while it's approved or has merge conflicts.

🤖 AI-Generated Review | Vera Review Bot - v0.2.7 | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@mergify

mergify Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@kcreddy
kcreddy merged commit 75ab08a into elastic:main Aug 27, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@efd6 efd6 efd6 approved these changes
+1 more reviewer
@vera-review-bot vera-review-bot[bot] vera-review-bot[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@kcreddy kcreddy

Labels

changelog-link-check:skip Integration:aws AWS Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kcreddy @efd6