Skip to content
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

create workflow for calling cointainers-mirror workflows #250

Closed

Conversation

Gregory-Pereira
Copy link
Collaborator

/cc @lmilbaum @sallyom This should allow us to call workflows in the other repo however it requires a secrets.PAT to be created for a personal access token that has permission to create workflow_dispatches. Maybe we do this for a bot user?

@Gregory-Pereira
Copy link
Collaborator Author

Gregory-Pereira commented Apr 11, 2024

although for running the actual workflows like model_server and chatbot we will need a PAT with more permissions:

{
  "message": "Must have admin rights to Repository.",
  "documentation_url": "https://docs.github.com/rest/actions/workflows#create-a-workflow-dispatch-event"
}

What I can do:

  • list workflows in both repos
  • run mirror_repos workflow in containers/ai-lab-recipes

what I cannot do:

  • run chatbot workflow in containers-mirror/ai-lab-recipes
  • run model_server workflow in containers-mirror/ai-lab-recipes

@Gregory-Pereira Gregory-Pereira added the hold Do not merge label Apr 13, 2024
@Gregory-Pereira
Copy link
Collaborator Author

/hold until I can get @lmilbaum to TAL and approve if we can move forward with PATs or if I should construct a Github application for it

@Gregory-Pereira
Copy link
Collaborator Author

We have decided to move away from this approach, instead experimenting with self hosted runners leveraging the https://github.com/philips-labs/terraform-aws-github-runner repository. More context around this decision can be found in the thread starting here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant