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

Copy in #270

Open
wants to merge 5 commits into
base: ec2-new-implementation
Choose a base branch
from
Open

Copy in #270

wants to merge 5 commits into from

Conversation

KesterTan
Copy link

@KesterTan KesterTan commented Jan 26, 2025

Attempts to fix the copy-in issue that we're still facing even after retries.

  1. Captures invalid inputfiles structure
  2. Adds -p flag and permissions when creating autolab directory. This ensures that the specified directory and any necessary parent directories are created. If the directory already exists, the command does not raise an error, it would originally throw an error.
  3. Added thread pool to limit scp commands so that instance is not overloaded
  4. Added log statements with job id for more in-depth logging.

Ran a thousand jobs on https://dev.autolabproject.com/courses/test-course/jobs?id=500 without failure.

@KesterTan KesterTan marked this pull request as ready for review February 2, 2025 17:04
@KesterTan KesterTan requested a review from evanyeyeye February 2, 2025 17:04
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.

1 participant