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

Add common workflow to be consumed by all components #8

Conversation

ChristianZaccaria
Copy link
Contributor

@ChristianZaccaria ChristianZaccaria commented Oct 24, 2023

Issue link

Resolves #7

What changes have been made

Added common-workflow to add created issues from our repos to Sprint Board

All repos will be able to run this common workflow once they are each updated to call the mentioned workflow.

See issue for more context

Verification steps

I think the only way of testing this would be by:

  • Merging this PR
  • Merging PR to call this workflow from distributed-workloads repo
  • Create an issue from distributed-workloads repo
  • See if workflow is called and issue gets added to Sprint Board

If successful, we can proceed for all other repos.

@ChristianZaccaria
Copy link
Contributor Author

Reviewers: @astefanutti @sutaakar

name: Common Workflow

on:
workflow_call:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you considered to make this a composite action, like https://github.com/project-codeflare/codeflare-operator/blob/main/.github/actions/kind/action.yml ?
Such action can be then called using https://github.com/project-codeflare/codeflare-operator/blob/main/.github/workflows/e2e_tests.yaml#L48-L49 .

However such approach still need to have workflow file in every repository.
Thinking if the application approach (like OpenShift CI) would have sense or whether it would be an overkill.

@ChristianZaccaria
Copy link
Contributor Author

This PR is no longer needed. The workflow has been removed.

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.

Add common-workflow to add issues to Sprint Board
2 participants