Skip to content

Introduce job timeouts, speed up builds#9

Merged
arjkesh merged 10 commits intomainfrom
cd/ci-improvements
Dec 17, 2025
Merged

Introduce job timeouts, speed up builds#9
arjkesh merged 10 commits intomainfrom
cd/ci-improvements

Conversation

@camille-004
Copy link
Copy Markdown
Collaborator

@camille-004 camille-004 commented Dec 16, 2025 •
edited by arjkesh
Loading

Description

Incorporated small changes to follow best practices:

  • Added timeouts to GPU jobs to prevent hung tests from wasting GPU resources for the default 6-hour timeout
  • Switch from gzip to zstd compression for faster container initialization, can lead to significant improvement in pull/decompress time

Testing

build pass: 851c27c
ops/benchmark/unit tests: a6e60a4

CI Configuration

config:
  build: false
  # valid options are "ops" and "benchmark"
  test: []

Checklist

  • Documentation updated (if needed)
  • CI configuration reviewed

@camille-004 camille-004 requested a review from arjkesh December 16, 2025 00:56
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Dec 16, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@camille-004 camille-004 changed the title Cd/ci improvements Small CI Improvements Dec 16, 2025
arjkesh
arjkesh reviewed Dec 16, 2025
View reviewed changes
Comment thread .github/workflows/tilegym-ci.yml Outdated
Comment thread .github/workflows/tilegym-ci.yml
Comment thread .github/workflows/tilegym-ci.yml
Comment thread .github/workflows/tilegym-ci.yml Outdated
@arjkesh arjkesh changed the title Small CI Improvements Introduce job timeouts, speed up builds Dec 16, 2025
@arjkesh
Copy link
Copy Markdown
Collaborator

arjkesh commented Dec 16, 2025

/ok to test a6ec380

@arjkesh
Copy link
Copy Markdown
Collaborator

arjkesh commented Dec 17, 2025

/ok to test 851c27c

@arjkesh
Copy link
Copy Markdown
Collaborator

arjkesh commented Dec 17, 2025

/ok to test a6e60a4

@arjkesh
Copy link
Copy Markdown
Collaborator

arjkesh commented Dec 17, 2025

/ok to test d1bf66f

@arjkesh arjkesh enabled auto-merge (squash) December 17, 2025 01:45
@arjkesh arjkesh merged commit c4aa20f into main Dec 17, 2025
8 checks passed
@arjkesh arjkesh deleted the cd/ci-improvements branch December 17, 2025 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arjkesh arjkesh arjkesh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@camille-004 @arjkesh