-
Notifications
You must be signed in to change notification settings - Fork 182
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
Adding Binary to Release #1553
Adding Binary to Release #1553
Conversation
One of the issues I am having is I am not sure how to test this change. I do not have the infrastructure setup in my fork to test the workflows. Does anyone have any suggestions on how to test this workflow change? |
Okay, you can see releases using this code here: https://github.com/webfinesse/opentelemetry-lambda/releases I want to delete these releases as they are obviously unofficial, however it shows that my changes work. Workflow runs can be seen here: https://github.com/webfinesse/opentelemetry-lambda/actions |
Thanks for the contribution. We'll wait for the next release to see how well this works. |
Thanks for adding this. It worked great! https://github.com/open-telemetry/opentelemetry-lambda/releases |
This pull request includes updates to several GitHub Actions workflows to enhance their functionality and permissions. The most important changes involve modifying permissions and adding steps to upload binaries to releases.
Permissions updates:
contents
permission fromread
towrite
in.github/workflows/release-layer-collector.yml
,.github/workflows/release-layer-java.yml
,.github/workflows/release-layer-nodejs.yml
,.github/workflows/release-layer-python.yml
, and.github/workflows/release-layer-ruby.yml
to allow for writing content. [1] [2] [3] [4] [5]Job updates:
.github/workflows/release-layer-collector.yml
..github/workflows/release-layer-java.yml
..github/workflows/release-layer-nodejs.yml
..github/workflows/release-layer-python.yml
..github/workflows/release-layer-ruby.yml
.resolves #1548
secrets.GITHUB_TOKEN
is set. If this is setup differently, please let me know.