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.

Sign up for GitHub

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

Jump to bottom

Fix integrity test #58

Closed
TheBits opened this issue Apr 9, 2024 · 2 comments · Fixed by #59 or #62
Closed

Fix integrity test #58

TheBits opened this issue Apr 9, 2024 · 2 comments · Fixed by #59 or #62
Assignees
@TheBits

Comments

@TheBits
Copy link
Contributor

TheBits commented Apr 9, 2024

The workflow Collect and Publish catalog broke down today.
@TheBits TheBits self-assigned this Apr 9, 2024
@TheBits TheBits linked a pull request Apr 9, 2024 that will close this issue
[runpod] Hide A100 #59
Merged
@TheBits TheBits mentioned this issue Apr 9, 2024
Merged
@peterschmidt85
Copy link
Contributor

@TheBits Do I get it right that your fix not only removes the check from the integration tests but also removes these GPUs entirely from the offers? If so, then this is not the right fix.

@TheBits TheBits mentioned this issue Apr 11, 2024
Closed
@TheBits
Copy link
Contributor Author

TheBits commented Apr 11, 2024

@peterschmidt85 Yes it's removes these GPUs entirely from the offers.
I will rework this test. I will try to get all GPUs name from API insted GPU_MAP variable.

@TheBits TheBits linked a pull request Apr 11, 2024 that will close this issue
Runpod: made the test_gpu_present trigger threshold easier #62
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheBits TheBits

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@TheBits @peterschmidt85