Skip to content

feat(q_dev): add Account ID to scope config and auto-construct both r…#8725

Merged
klesh merged 1 commit intomainfrom
feat/q-dev-account-id-scope-redesign
Feb 20, 2026
Merged

feat(q_dev): add Account ID to scope config and auto-construct both r…#8725
klesh merged 1 commit intomainfrom
feat/q-dev-account-id-scope-redesign

Conversation

@warren830
Copy link
Contributor

…eport paths

User now provides basePath and accountId instead of manually typing the full S3 prefix. The collector automatically constructs and scans both by_user_analytic and user_report paths under
{basePath}/AWSLogs/{accountId}/KiroLogs/…/{region}/{year}[/{month}].

Includes migration, updated blueprint, multi-prefix collector iteration, user_report model/extractor/dashboard, and frontend Account ID input field. Old scopes without accountId continue to work unchanged.

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?

Does this close any open issues?

Closes xx

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

…eport paths

User now provides basePath and accountId instead of manually typing the full
S3 prefix. The collector automatically constructs and scans both
by_user_analytic and user_report paths under
{basePath}/AWSLogs/{accountId}/KiroLogs/…/{region}/{year}[/{month}].

Includes migration, updated blueprint, multi-prefix collector iteration,
user_report model/extractor/dashboard, and frontend Account ID input field.
Old scopes without accountId continue to work unchanged.
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. component/config-ui This issue or PR relates to config-ui component/ext This issue or PR relates to external components, such as Grafana component/plugins This issue or PR relates to plugins pr-type/feature-development This PR is to develop a new feature priority/high This issue is very important labels Feb 20, 2026
Copy link
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

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

LGTM

@klesh klesh merged commit b1cf560 into main Feb 20, 2026
17 checks passed
@klesh klesh deleted the feat/q-dev-account-id-scope-redesign branch February 20, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/config-ui This issue or PR relates to config-ui component/ext This issue or PR relates to external components, such as Grafana component/plugins This issue or PR relates to plugins pr-type/feature-development This PR is to develop a new feature priority/high This issue is very important size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants