Skip to content

Commit 2675c7b

Browse files
authored
Merge branch 'master' into fabric
2 parents ad60f75 + c85660c commit 2675c7b

File tree

6 files changed

+33
-20
lines changed

6 files changed

+33
-20
lines changed

.github/checkgroup.yml

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -176,14 +176,15 @@ subprojects:
176176
- "lightning-fabric (GPUs) (testing Fabric | latest)"
177177
- "lightning-fabric (GPUs) (testing Lightning | latest)"
178178

179-
- id: "lightning_fabric: TPU workflow"
180-
paths:
181-
# tpu CI availability is very limited, so we only require tpu tests
182-
# to pass when their configurations are modified
183-
- ".github/workflows/tpu-tests.yml"
184-
- "tests/tests_fabric/run_tpu_tests.sh"
185-
checks:
186-
- "test-on-tpus (pytorch, pjrt, v4-8)"
179+
# Temporarily disabled
180+
# - id: "lightning_fabric: TPU workflow"
181+
# paths:
182+
# # tpu CI availability is very limited, so we only require tpu tests
183+
# # to pass when their configurations are modified
184+
# - ".github/workflows/tpu-tests.yml"
185+
# - "tests/tests_fabric/run_tpu_tests.sh"
186+
# checks:
187+
# - "test-on-tpus (pytorch, pjrt, v4-8)"
187188

188189
# SECTION: common
189190

.github/workflows/_legacy-checkpoints.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,13 @@ jobs:
111111
retention-days: ${{ env.KEEP_DAYS }}
112112
include-hidden-files: true
113113

114+
- name: access secrets
115+
# export to env bool if secrets.AWS_REGION is not empty
116+
run: echo "WITH_SECRETS=$([ -n '${{ secrets.AWS_REGION }}' ] && echo 1 || echo 0)" >> $GITHUB_ENV
117+
114118
- run: pip install -r requirements/ci.txt
115119
- name: Upload checkpoints to S3
116-
if: ${{ secrets[AWS_REGION] != '' }}
120+
if: ${{ env.WITH_SECRETS == '1' }}
117121
working-directory: ${{ env.LEGACY_FOLDER }}
118122
env:
119123
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY }}
File renamed without changes.

.github/workflows/ci-tests-fabric.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,16 +116,24 @@ jobs:
116116
mkdir -p $PYPI_CACHE_DIR
117117
ls -lh $PYPI_CACHE_DIR
118118
119-
- name: Env. variables
119+
- name: Expand Env. variables
120120
run: |
121121
# Switch PyTorch URL between stable and test/future
122122
python -c "print('TORCH_URL=' + str('${{env.TORCH_URL_TEST}}' if '${{ matrix.pytorch-version }}' == '2.7' else '${{env.TORCH_URL_STABLE}}'))" >> $GITHUB_ENV
123123
# Switch coverage scope
124124
python -c "print('COVERAGE_SCOPE=' + str('lightning' if '${{matrix.pkg-name}}' == 'lightning' else 'lightning_fabric'))" >> $GITHUB_ENV
125125
# if you install mono-package set dependency only for this subpackage
126126
python -c "print('EXTRA_PREFIX=' + str('' if '${{matrix.pkg-name}}' != 'lightning' else 'fabric-'))" >> $GITHUB_ENV
127+
- name: Append Env. vars for MacOS
128+
if: ${{ runner.os == 'macOS' }}
129+
run: |
130+
# trying to avoid "gloo" issue with SIGABRT
131+
echo "GLOO_SOCKET_IFNAME=lo0" >> $GITHUB_ENV
132+
- name: Append Env. vars for Windows
133+
if: ${{ runner.os == 'windows' }}
134+
run: |
127135
# Avoid issue on Windows with PyTorch 2.4: "RuntimeError: use_libuv was requested but PyTorch was build without libuv support"
128-
python -c "print('USE_LIBUV=0' if '${{matrix.os}}' == 'windows-2022' and '${{matrix.pytorch-version}}' == '2.4' else '')" >> $GITHUB_ENV
136+
echo "USE_LIBUV=0" >> $GITHUB_ENV
129137
130138
- name: Install package & dependencies
131139
timeout-minutes: 20

.github/workflows/release-pkg.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -179,12 +179,12 @@ jobs:
179179
with:
180180
pkg-folder: dist/${{ steps.folder.outputs.pkg }}
181181
pypi-token: ${{ secrets[format('PYPI_TOKEN_{0}', matrix.name)] }}
182-
# FIXME: this is not working suddenly, Unrecognized named-value: 'secrets'
183-
# legacy-checkpoints:
184-
# needs: [build-packages]
185-
# uses: ./.github/workflows/_legacy-checkpoints.yml
186-
# with:
187-
# push_to_s3: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
188-
# upload_local: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
189-
# create_pr: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
190-
# secrets: inherit
182+
183+
legacy-checkpoints:
184+
needs: [build-packages]
185+
uses: ./.github/workflows/_legacy-checkpoints.yml
186+
with:
187+
push_to_s3: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
188+
upload_local: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
189+
create_pr: ${{ startsWith(github.event.ref, 'refs/tags') || github.event_name == 'release' }}
190+
secrets: inherit

0 commit comments

Comments
 (0)