-
Notifications
You must be signed in to change notification settings - Fork 4
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
ci: move signing packages to automation page #105
base: main
Are you sure you want to change the base?
ci: move signing packages to automation page #105
Conversation
782f352
to
2356e4c
Compare
2356e4c
to
b1b40c2
Compare
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.
have you checked already that all the repos share the same script?
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.
Adding some context for the scripts:
These are only needed for FIPS. As we need to use Ubuntu 16 for building FIPS packages at this time (it might change with go 1.24), and there is some issue with using goreleaser signing with Ubuntu 16 as it prompts for password and non-interactive doesn't work (multiple tests done for this).
TL;DR
This script will be same for all the OHIs that need the packages to be signed.
@@ -47,5 +47,6 @@ ENV GOFLAGS="-buildvcs=false" | |||
# Since the user does not match the owners of the repo "git rev-parse --is-inside-work-tree" fails and goreleaser does not populate projectName | |||
# https://stackoverflow.com/questions/72978485/git-submodule-update-failed-with-fatal-detected-dubious-ownership-in-repositor | |||
RUN git config --global --add safe.directory '*' | |||
COPY ./sign.sh ./sign_deb.exp ./sign_rpm.exp ./sign_tar.exp /usr/local/bin |
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.
We are only copying the files since they are executed already in the makefile? https://github.com/newrelic/nri-redis/pull/212/files#diff-553b9a740f0151f2e959512198dddbdd0d69c398d29583ca0a4e011d5598932e
Modify signing packages to SHA-256 and move the signing files from integrations to automations repo.