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

Update the rpm package signing to work with newer rpm version #892

Merged
merged 2 commits into from
Dec 31, 2019

Conversation

twk3
Copy link
Contributor

@twk3 twk3 commented Jun 15, 2019

Description

Newer versions of rpmsign (rpm 4.13+), no longer prompt for the pass
phrase, but instead provide gpg with access to stdin/out to request it.

See: rpm-software-management/rpm@0bce5fc#diff-c9c5aab99001a72b592ce7465ebff8a6

Because this process is deferred, you will also often end up with
warning output from rpmbuild before the prompt, so we can no longer
assume the prompt will be the first output.

This issue was discovered while trying to build rpm packages for
OpenSUSE 15 which has the newer rpm.

Maintainers

Please ensure that you check for:

  • [] If this change impacts git cache validity, it bumps the git cache
    serial number
  • [] If this change impacts compatibility with omnibus-software, the
    corresponding change is reviewed and there is a release plan
  • [] If this change impacts compatibility with the omnibus cookbook, the
    corresponding change is reviewed and there is a release plan

Newer versions of rpmsign (rpm 4.13+), no longer prompt for the pass
phrase, but instead provide gpg with access to stdin/out to request it.

Because this process is deferred, you will also often end up with
warning output from rpmbuild before the prompt, so we can no longer
assume the prompt will be the first output.

This issue was discovered while trying to build rpm packages for
OpenSUSE 15 which has the newer rpm.

Signed-off-by: DJ Mountney <david@twkie.net>
@twk3
Copy link
Contributor Author

twk3 commented Jun 15, 2019

This isn't ready yet. Works for the new rpmsign, but not the old. Probably because the older one does the prompt without a newline. Will test that now.

@twk3
Copy link
Contributor Author

twk3 commented Jun 15, 2019

Updated, confirmed this is working for centos 6, centos 7, opensuse 42.3, and opensuse 15

@tas50 tas50 merged commit eecac7e into chef:master Dec 31, 2019
@tas50
Copy link
Contributor

tas50 commented Dec 31, 2019

@twk3 Thanks for the detailed explanation and confirming this doesn't break existing systems.

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.

2 participants