Skip to content

feat(proto) Update Plugin SDK for Cross Account Discovery - #6779

Merged
matt-romanelli merged 1 commit into
llb-icu-19968from
ICU-19969
Aug 18, 2026
Merged

feat(proto) Update Plugin SDK for Cross Account Discovery#6779
matt-romanelli merged 1 commit into
llb-icu-19968from
ICU-19969

Conversation

@matt-romanelli

Copy link
Copy Markdown

Problem

  • The Boundary AWS plugin is dependent on proto contracts from Boundary CE

Solution

Test

  • Successfully built Boundary
  • No unit tests at this time until the code is wired up to use the new fields

Description

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've documented the impact of any changes to security controls.
    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

**Problem**
- The Boundary AWS plugin is dependent on proto contracts from Boundary CE

**Solution**
- These are the same changes that were made to Boundary Enterprise in this PR: hashicorp/boundary-enterprise#2960

**Test**
- Successfully built Boundary
- No unit tests at this time until the code is wired up to use the new fields

@ryanflynn9299 ryanflynn9299 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@delainerogers delainerogers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@matt-romanelli
matt-romanelli merged commit 2b8c212 into llb-icu-19968 Aug 18, 2026
53 of 55 checks passed
@matt-romanelli
matt-romanelli deleted the ICU-19969 branch August 18, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants