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

Include option to manually run GHA aws_test, initialize .gitignore #1

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

aaronkanzer
Copy link
Collaborator

No description provided.

@hvgazula
Copy link
Collaborator

Do you prefer both or either?

@hvgazula hvgazula merged commit e723688 into main Feb 14, 2024
2 of 3 checks passed
@aaronkanzer
Copy link
Collaborator Author

@hvgazula Just having either as an option seems to be a nice convenience.

We might want to limit running this command on every push in the future (perhaps we constrain with more rules) just to limit costs -- not sure how costly each run is on a GPU-related ec2 instance type.

@hvgazula
Copy link
Collaborator

spot instances are cheaper than on-demand..but I remember some issues with requesting those instances.. 🤷‍♂️

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.

2 participants