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

Include CloudEvents proto to generated bundle #86

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

jalseth
Copy link
Contributor

@jalseth jalseth commented Jul 8, 2023

Summary

Sigstore will follow the CloudEvents specification when publishing events to Pub/Sub topics. Protobuf events can be used for better efficiency compared to JSON.

Release Note

  • Added CloudEvents message to be used by Sigstore projects when publishing events.

Documentation

N/A, all docs on events usage will be included in the relevant project (rekor, rekor-monitor, etc.).

protos/events.proto Outdated Show resolved Hide resolved
bobcallaway
bobcallaway previously approved these changes Jul 28, 2023
@jalseth
Copy link
Contributor Author

jalseth commented Jul 28, 2023

Not sure why the check failed. I used make all to generate the code for each language and this only adds new code, it does not modify any existing code.

@haydentherapper
Copy link
Collaborator

haydentherapper commented Jul 28, 2023

Have you rebased against main, before running make all?

Sigstore will follow the CloudEvents specification when publishing
events to Pub/Sub topics.

Signed-off-by: James Alseth <james@jalseth.me>
@jalseth
Copy link
Contributor Author

jalseth commented Jul 29, 2023

Oh, thanks! I didn't realize 20+ commits had been made since I opened this. Should be fixed now.

@haydentherapper
Copy link
Collaborator

Thanks @jalseth!

@haydentherapper haydentherapper merged commit e728c60 into sigstore:main Jul 31, 2023
24 checks passed
@jalseth jalseth deleted the ce-proto branch August 16, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants