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

run compatibility in parallel using multiple workflows #215

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

wilsonwang371
Copy link
Collaborator

Why are these changes needed?

Run compatibility tests in parallel to save time

Related issue number

#159

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@wilsonwang371 wilsonwang371 force-pushed the wilson/parallel-workflow branch 10 times, most recently from a6dc8b0 to ee83eef Compare March 28, 2022 23:10
@wilsonwang371
Copy link
Collaborator Author

@Jeffwan please take a look at this patch. It is ready for review now.

Copy link
Collaborator

@Jeffwan Jeffwan left a comment

Choose a reason for hiding this comment

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

I have some minor comments. @wilsonwang371

- name: Build Docker Image - Apiserver
run: docker build -t kuberay/apiserver:${{ steps.vars.outputs.sha_short }} -f apiserver/Dockerfile .
- name: Upload Artifact Operator
uses: actions/upload-artifact@v2
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am new to this plugin, How does upload-artifact work?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It will upload a file that we can use later in other jobs. I want to reuse the images built here in other following jobs.

uses: actions/upload-artifact@v2
with:
name: operator_img
path: /tmp/operator.tar
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the lifecycle of /tmp/operator.tar? Do we need to rename it with git hash etc?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no need, later when we load it again, it will have the same tag when we save it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it should be only availlable on ths same workflow.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Got it. Make sense

@Jeffwan Jeffwan merged commit 7e088b9 into ray-project:master Mar 29, 2022
@wilsonwang371 wilsonwang371 deleted the wilson/parallel-workflow branch March 30, 2022 00:00
lowang-bh pushed a commit to lowang-bh/kuberay that referenced this pull request Sep 24, 2023
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.

None yet

2 participants