Skip to content

ci: pass Harbor secrets explicitly to reusable dockerbuild workflow so other organizations can re-use them#9

Closed
arnemileswinter wants to merge 1 commit into
eclipse-xfsc:mainfrom
arnemileswinter:main
Closed

ci: pass Harbor secrets explicitly to reusable dockerbuild workflow so other organizations can re-use them#9
arnemileswinter wants to merge 1 commit into
eclipse-xfsc:mainfrom
arnemileswinter:main

Conversation

@arnemileswinter

Copy link
Copy Markdown

This makes the same docker workflow work from forks. Before, the workflow was not able to access secrets from other organizations. We need to be explicit about secrets.

Before, the workflow would complain, as per this example run:

[](https://github.com/arnemileswinter/facis-dcs/actions/runs/28652511704/job/84973741194#step:4:1)Run git clone [https://github.com/eclipse-xfsc/dev-ops.git](https://github.com/eclipse-xfsc/dev-ops.git) devops

[](https://github.com/arnemileswinter/facis-dcs/actions/runs/28652511704/job/84973741194#step:4:49)Cloning into 'devops'...

[](https://github.com/arnemileswinter/facis-dcs/actions/runs/28652511704/job/84973741194#step:4:50)clone successfull

[](https://github.com/arnemileswinter/facis-dcs/actions/runs/28652511704/job/84973741194#step:4:51)Fehler: Umgebungsvariable HARBOR_CREDENTIALS ist nicht gesetzt.

[](https://github.com/arnemileswinter/facis-dcs/actions/runs/28652511704/job/84973741194#step:4:52)Error: Process completed with exit code 1.

GitHub Actions secrets:inherit does not propagate across different owners.
Add explicit HARBOR_HOST and HARBOR_CREDENTIALS mapping to both dockerbuild jobs so secrets are forwarded once upstream declares them in on.workflow_call.secrets.

Signed-off-by: Arne Winter <a.winter@neusta.de>
@sonarqubecloud

sonarqubecloud Bot commented Jul 3, 2026

Copy link
Copy Markdown

@SchulzeStTSI

SchulzeStTSI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

I can't merge this, because it's required then to add in hundreds of workflows the secrets first:) propose an optional solution please

@SchulzeStTSI

Copy link
Copy Markdown
Contributor

try this please: #10

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