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

[Bug] Need to port Publish to PyPl workflow to ARC cluster #457

Closed
vadiklyutiy opened this issue Jul 24, 2024 · 7 comments
Closed

[Bug] Need to port Publish to PyPl workflow to ARC cluster #457

vadiklyutiy opened this issue Jul 24, 2024 · 7 comments
Assignees
Labels
bug Something isn't working Task Or "Story" in JIRA's term.

Comments

@vadiklyutiy
Copy link
Collaborator

After merging from CentML/hidet Tests workflow runs on ARC cluster. But there is one more workflow that counts on the previously used instance - "Publish to PyPl" workflow.
We need to port it to running on ARC cluster.

@yaoyaoding @wangshangsam @c-fteixeira

@vadiklyutiy vadiklyutiy added the bug Something isn't working label Jul 24, 2024
@yaoyaoding
Copy link
Member

In the Publish to PyPI workflow, there are two steps:

  1. Perform release-level tests
  2. Build the package wheel and submit to PyPI

The first step requires a GPU while the second step does not. It's okay to only port for the first step to run on the arc cluster. @c-fteixeira

@vadiklyutiy
Copy link
Collaborator Author

vadiklyutiy commented Jul 25, 2024

if so, we can instead of 1. just call Test workflow. Publish workflow move to GitHub hosted server.
@yaoyaoding Is it correct that Publish use the same tests set as Test?

@yaoyaoding
Copy link
Member

if so, we can instead of 1. just call Test workflow. Publish workflow move to GitHub hosted server. @yaoyaoding Is it correct that Publish use the same tests set as Test?

The current release tests are the same as our daily tests. But we planned to enable some slow tests in the release tests.

@vadiklyutiy
Copy link
Collaborator Author

The current release tests are the same as our daily tests. But we planned to enable some slow tests in the release tests.

I think it might be just a arg of workflow that enables slow tests. No need to copy-paste a lot

@yaoyaoding
Copy link
Member

Yeah, it's better if we can resue some workflow code by using arg to specify whether to enable more tests.

@vadiklyutiy
Copy link
Collaborator Author

@c-fteixeira is it done? yes?

@wangshangsam wangshangsam added the Task Or "Story" in JIRA's term. label Aug 1, 2024
@c-fteixeira
Copy link
Collaborator

closed by #461 and #463

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Task Or "Story" in JIRA's term.
Projects
None yet
Development

No branches or pull requests

4 participants