release-engineering / ubi-population-tool

A tool for populating UBI repositories.
GNU General Public License v3.0
3 stars 14 forks source link

Authenticate session with kerberos #234

Closed drepelov closed 4 months ago

drepelov commented 4 months ago

To access ubi-manifest server, requests session now needs to be authenticated using kerberos. The requests-kerberos library used for the authentication needs some base packages to be installed in order to work properly. Therefore this commit adds an Install base packages step in tox-test.yml otherwise the tests would fail.

Note: on Jenikins the tool runs on fedora-jenkins-minimal image, where those packages are already installed so I think no changes there should be necessary.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.77%. Comparing base (55f454d) to head (3a56f30).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #234 +/- ## ========================================== + Coverage 98.75% 98.77% +0.01% ========================================== Files 7 7 Lines 724 732 +8 ========================================== + Hits 715 723 +8 Misses 9 9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

drepelov commented 4 months ago

run tests

rbikar commented 4 months ago

run tests

rbikar commented 4 months ago

@drepelov

As agreed, taking over this PR. I've added a couple of things that should good enough for further deployment with kerb. auth.

I don't expect jenkins tests to pass.