-
Notifications
You must be signed in to change notification settings - Fork 0
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
Failure on forked repo - Token secret not supplied #3
Comments
work in preparation for solution 1 . #3
Test implemented - Solution 1
On merge
|
Continued Test implemented - Solution 1Workflows:
Notes
Mergeworkflows: 🔴 |
Continued Test implemented - Solution 1workflows:
|
Continued Test implemented - Solution 1
Notes
|
workflow is erroring. test to see if set as string fixes the issue. . #3 #4 nofusscomputing/ansible_collection_centurion#24 nofusscomputing/ansible_collection_centurion#25
Solution 1 - Outcome
moving forward
|
🟢 Solution 1 - Forked repo test
Success
|
On a forked repo the secret is not available. figure out a way to have this workflow run. Test should have already been done in the scratchpad, however they weren't. this was the purpose of nofusscomputing/scratchpad#8
Debugging
🔴 Action: opened by collaborator
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10381144059
🔴 Action: assigned by collaborator
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10381144478
🔴 Action: edited by collaborator_
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10382453884
🟢 Action: milestoned by repo owner
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10381380285
🟢 Action: assigned by repo owner
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10382250836
🔴 Action: edited by repo owner
https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10382256327
Proposed Solution 1
Split workflow into two parts.
Triage
runs in the context of the issue/PR and on completion triggers (workflow_run
) the actual workflow which should run in the context of the repo. Idea is the same as this actionfor this to work the details of what was triaged will be required to be passed to triggered workflow.
Solution 1 Problem
There does not appear to be a way to specify the item url for action
actions/add-to-project
ref: Add 'payload' field to allow for Repository Dispatch actions/add-to-project#341
Adds feature that allows specifying via payload
Links
Blocks: feat(playbooks): migration of centurion playbooks to github ansible_collection_centurion#17
Blocked By: Add 'payload' field to allow for Repository Dispatch actions/add-to-project#341
Related: Add CI to the Repo ansible_collection_centurion#15
Related: new line scratchpad#8 namely comment
Related: Action does not work on PRs from forks actions/add-to-project#163
Ref: https://github.com/nofusscomputing/ansible_collection_centurion/actions/runs/10381144478/job/28742085387
Tasks
🚧 Does "Proposed solution 1" work
Update workflow to use proposed
TRIAGE_
variablesAll Conditional statements updated for
TRIAGE_
variablesAdd to project
action receives specifieditem_number
update field
actions receives specifieditem_number
🟢 Partial success. see Failure on forked repo - Token secret not supplied #3 (comment)
🟢 forked repo test was a success for secrets access. Failure on forked repo - Token secret not supplied #3 (comment)
/cc @jasonpagetas
The text was updated successfully, but these errors were encountered: