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

Add backend WorkloadIdentityX509Revocations service #51997

Merged
merged 11 commits into from
Feb 12, 2025

Conversation

strideynet
Copy link
Contributor

@strideynet strideynet commented Feb 10, 2025

Part of #51504

Depends on #51996

Adds backend service implementation for WorkloadIdentityX509Revocation service & tests, as well as support for the events and audit events. Will follow up with GRPC service implementation.

@strideynet strideynet added backport/branch/v16 backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry labels Feb 11, 2025
@strideynet strideynet changed the title Strideynet/revocations service Add backend WorkloadIdentityX509Revocations service Feb 11, 2025
@strideynet strideynet marked this pull request as ready for review February 11, 2025 10:57
@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log size/lg labels Feb 11, 2025
Base automatically changed from strideynet/revocations-protos to master February 11, 2025 11:17
@strideynet strideynet force-pushed the strideynet/revocations-service branch from b6c087e to a3478fe Compare February 11, 2025 12:08
lib/services/local/events.go Outdated Show resolved Hide resolved
lib/services/workload_identity_x509_revocation.go Outdated Show resolved Hide resolved
@strideynet strideynet added this pull request to the merge queue Feb 12, 2025
Merged via the queue into master with commit d1d81ee Feb 12, 2025
42 checks passed
@strideynet strideynet deleted the strideynet/revocations-service branch February 12, 2025 11:26
@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 Feb 12, 2025
* Add `lib/services` implementation

* Fix GoDoc comments

* Wire up events support

* Add TestValidateWorkloadIdentityX509Revocation

* Wire up new WorkoadIdentityX509Revocation service

* Add TestWorkloadIdentityX509RevocationMarshaling

* Add tests for WorkloadIdentityX509RevocationService

* Avoid lib/services/local/events.go exploding in size

* Switch to regex based validation

* Fix TestWorkloadIdentityX509RevocationService_List

* Actually fix TestWorkloadIdentityX509RevocationService_List
strideynet added a commit that referenced this pull request Feb 12, 2025
* Add `lib/services` implementation

* Fix GoDoc comments

* Wire up events support

* Add TestValidateWorkloadIdentityX509Revocation

* Wire up new WorkoadIdentityX509Revocation service

* Add TestWorkloadIdentityX509RevocationMarshaling

* Add tests for WorkloadIdentityX509RevocationService

* Avoid lib/services/local/events.go exploding in size

* Switch to regex based validation

* Fix TestWorkloadIdentityX509RevocationService_List

* Actually fix TestWorkloadIdentityX509RevocationService_List
github-merge-queue bot pushed a commit that referenced this pull request Feb 12, 2025
…52056)

* Add backend WorkloadIdentityX509Revocations service (#51997)

* Add `lib/services` implementation

* Fix GoDoc comments

* Wire up events support

* Add TestValidateWorkloadIdentityX509Revocation

* Wire up new WorkoadIdentityX509Revocation service

* Add TestWorkloadIdentityX509RevocationMarshaling

* Add tests for WorkloadIdentityX509RevocationService

* Avoid lib/services/local/events.go exploding in size

* Switch to regex based validation

* Fix TestWorkloadIdentityX509RevocationService_List

* Actually fix TestWorkloadIdentityX509RevocationService_List

* Fix backport
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/lg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants