SUSE / rmt

RPM repository mirroring tool and registration proxy for SUSE Customer Center.
Other
38 stars 45 forks source link

Fix: Set correct user for service and cli #1040

Closed ngetahun closed 10 months ago

ngetahun commented 11 months ago

Description

RMT server service uses non-root user which causes access permissions issues. This PR fixes this issue.

Fixes bsc#1215176

Change Type

Please select the correct option.

Checklist

Please check off each item if the requirement is met.

Other Notes

Please use this space to provide notes or thoughts to the team, such as tips on how to review/demo your changes.

How to test

Pre-requisite: Setup a vm with nginx and ssl for client script setup. Also, please mirror the client system product on your test rmt.

  1. Pull the rmt-client-setup on your client system:

curl <ip>/tools/rmt/client/setup

  1. Register your client:

./rmt-client-setup https://<ip>

This should work without any issue,

ngetahun commented 11 months ago

Fix: RMT server install creates secrets.yml.key with root owner

ngetahun commented 10 months ago

@jsegitz Please review the new changes. I have incorporated your suggestions.

jsegitz commented 10 months ago

I think it's fine now

suse-tests-pass commented 10 months ago

Well Done! Your tests are still passing. https://ci.suse.de/job/scc-RMT-integration-tests/346622/console If the given link has expired,you can force a Prophet rerun by just deleting this comment. (Merged d4d748562ea13b0c5e8cb236633fd0ad088f9cc9 into f8da43c2e93aaa039461449d7c4b90128c33405b)