Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workload ID: Add Protos for WorkloadIdentity resource, CRUD and events #49650

Merged
merged 6 commits into from
Dec 6, 2024

Conversation

strideynet
Copy link
Contributor

@strideynet strideynet commented Dec 2, 2024

Adds protos for the new WorkloadIdentity resource, along with the Audit Event protos and similar.

I've gone with the most basic version of the protos presented in the RFD for now. I figure it makes sense to move forward with this boilerplatery work whilst the finer points of the RFD are discussed.

As per RFD: #49133

Part of #49572

@strideynet strideynet added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v16 backport/branch/v17 labels Dec 2, 2024
@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log size/md labels Dec 2, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-49650.d3pp5qlev8mo18.amplifyapp.com

@strideynet strideynet force-pushed the strideynet/workload-identity-pbuf-and-events branch 3 times, most recently from f560fad to 1c132aa Compare December 5, 2024 10:39
@strideynet strideynet force-pushed the strideynet/workload-identity-pbuf-and-events branch from 1c132aa to 36d4bb2 Compare December 5, 2024 10:43
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from atburke December 5, 2024 18:15
Copy link
Contributor

@capnspacehook capnspacehook left a comment

Choose a reason for hiding this comment

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

Thanks for adding the TrimToMaxSize impl!

@strideynet strideynet added this pull request to the merge queue Dec 6, 2024
Merged via the queue into master with commit 7a3fce9 Dec 6, 2024
44 checks passed
@strideynet strideynet deleted the strideynet/workload-identity-pbuf-and-events branch December 6, 2024 12:00
@public-teleport-github-review-bot

@strideynet See the table below for backport results.

Branch Result
branch/v16 Failed
branch/v17 Failed

strideynet added a commit that referenced this pull request Dec 9, 2024
#49650)

* Add protos for resource and resource svc

* Add field for encoding the created/updated workloadidentity in audit
events

* Remove redundant package qualifier

* Add UpsertWorkloadIdentity

* Appease linter

* Add TrimToMaxSize implementation
strideynet added a commit that referenced this pull request Dec 9, 2024
#49650)

* Add protos for resource and resource svc

* Add field for encoding the created/updated workloadidentity in audit
events

* Remove redundant package qualifier

* Add UpsertWorkloadIdentity

* Appease linter

* Add TrimToMaxSize implementation
github-merge-queue bot pushed a commit that referenced this pull request Dec 9, 2024
#49650) (#49926)

* Add protos for resource and resource svc

* Add field for encoding the created/updated workloadidentity in audit
events

* Remove redundant package qualifier

* Add UpsertWorkloadIdentity

* Appease linter

* Add TrimToMaxSize implementation
github-merge-queue bot pushed a commit that referenced this pull request Dec 9, 2024
#49650) (#49927)

* Add protos for resource and resource svc

* Add field for encoding the created/updated workloadidentity in audit
events

* Remove redundant package qualifier

* Add UpsertWorkloadIdentity

* Appease linter

* Add TrimToMaxSize implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit-log Issues related to Teleports Audit Log backport/branch/v16 backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants