Skip to content

Use PULP3_* env vars for Hosted Pulp credentials [RHELDST-45461] - #720

Open
crungehottman wants to merge 1 commit into
release-engineering:masterfrom
crungehottman:duplicate-auth-creds
Open

crungehottman wants to merge 1 commit into
release-engineering:masterfrom
crungehottman:duplicate-auth-creds

Conversation

@crungehottman

Copy link
Copy Markdown
Member

Update KonfluxSource to read hosted pulp credentials from PULP3_* environment variables instead of PUSHSOURCE_KONFLUX_PULP_* variants.

This aligns with pubtools-pulp's Pulp3ClientService, which reads from the same PULP3_* env vars. Since both pushsource and pubtools-pulp connect to the same hosted pulp instance with the same credentials, sharing one set of env vars eliminates duplication in the Tekton task configuration.

Update KonfluxSource to read hosted pulp credentials from PULP3_*
environment variables instead of PUSHSOURCE_KONFLUX_PULP_* variants.

This aligns with pubtools-pulp's Pulp3ClientService, which reads from
the same PULP3_* env vars. Since both pushsource and pubtools-pulp
connect to the same hosted pulp instance with the same credentials,
sharing one set of env vars eliminates duplication in the Tekton task
configuration.
@codecov

codecov Bot commented Sep 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (9105a74) to head (a6bd7ff).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #720   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           59        59           
  Lines         2736      2736           
=========================================
  Hits          2736      2736           
Flag Coverage Δ
unit-tests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

No deployments
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.

1 participant