-
Notifications
You must be signed in to change notification settings - Fork 185
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
feat(collector): introduce new workflow to publish collector Lambda layer #1692
base: main
Are you sure you want to change the base?
feat(collector): introduce new workflow to publish collector Lambda layer #1692
Conversation
ce74d1e
to
ff1c49c
Compare
ClientIdList: | ||
- "sts.amazonaws.com" | ||
ThumbprintList: | ||
- "ffffffffffffffffffffffffffffffffffffffff" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we consider using the latest published thumbprints here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not necessary.
Note that the thumbprint below has been set to all F's because the thumbprint is not used when authenticating token.actions.githubusercontent.com. This is a special case used only when GitHub's OIDC is authenticating to IAM. IAM uses its library of trusted CAs to authenticate. The value is still the API, so it must be specified.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nvm, this is new to me (used to break)
Note that the thumbprint below has been set to all F's because the thumbprint is not used when authenticating token.actions.githubusercontent.com. This is a special case used only when GitHub's OIDC is authenticating to IAM. IAM uses its library of trusted CAs to authenticate. The value is still the API, so it must be specified.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. Only other suggestion is to ask a doc person to review the readme change. It's complicated and could probably use some extra eyes.
(Added the go label so we remember to group this in the collector release notes.) |
Fix for the failing job here: #1693 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few suggestions wordwise, but I at least am able to follow the instructions just fine 👍
Co-authored-by: Severin Neumann <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
96c97e2
to
bd9b36b
Compare
This PR introduces a new Github workflow (named
Publish Collector Lambda layer
) to allow users publishing their own custom Lambda collector layers from their own forks into their own (or any target) AWS accounts.