From 4bee1fe0c89e22900505510b7d324da0ba38d467 Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 15:21:31 -0700 Subject: [PATCH 01/12] [CICD] Ensure proper handling of permission check failures in CI workflow --- .github/workflows/continuous_integration.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index 48865542..3997cb7f 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -28,6 +28,9 @@ jobs: with: github_token: ${{ github.token }} permission: write + - name: Set Default Permission Output + if: ${{ failure() }} + run: echo "::set-output name=permitted::false" - name: Check PR Safe to Run if: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }} run: exit 1 From daaa050e36b5ecc5a94a6fa74b670b28d4634d49 Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 15:58:28 -0700 Subject: [PATCH 02/12] test --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index 03be94d5..de14f200 100644 --- a/README.md +++ b/README.md @@ -38,4 +38,5 @@ result = cloud_predictor.predict_real_time(test_data) cloud_predictor.cleanup_deployment() # Batch inference result = cloud_predictor.predict(test_data) + ``` From c128b15f74dc4decbfa36afdb2535a8fcc5aaba2 Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 16:05:04 -0700 Subject: [PATCH 03/12] test --- README.md | 1 - 1 file changed, 1 deletion(-) diff --git a/README.md b/README.md index de14f200..03be94d5 100644 --- a/README.md +++ b/README.md @@ -38,5 +38,4 @@ result = cloud_predictor.predict_real_time(test_data) cloud_predictor.cleanup_deployment() # Batch inference result = cloud_predictor.predict(test_data) - ``` From 249ac5c55b17feab9dffd09966a1620640acd3b6 Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 16:08:11 -0700 Subject: [PATCH 04/12] test --- .github/workflows/continuous_integration.yml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index 3997cb7f..b5ee21f0 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -31,6 +31,13 @@ jobs: - name: Set Default Permission Output if: ${{ failure() }} run: echo "::set-output name=permitted::false" + - name: Debug Information + run: | + echo "Event Name: ${{ github.event_name }}" + echo "Labels: ${{ toJson(github.event.pull_request.labels) }}" + echo "Permitted: ${{ steps.check.outputs.permitted }}" + echo "Safe to Test Label Present: ${{ contains(github.event.pull_request.labels.*.name, 'safe to test') }}" + if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run if: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }} run: exit 1 From 34d73050716b27326b3f2792810d2398a65e3afc Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 16:08:32 -0700 Subject: [PATCH 05/12] test --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index 03be94d5..de14f200 100644 --- a/README.md +++ b/README.md @@ -38,4 +38,5 @@ result = cloud_predictor.predict_real_time(test_data) cloud_predictor.cleanup_deployment() # Batch inference result = cloud_predictor.predict(test_data) + ``` From 73260c1b4908920d408d4c84ac8ea57cac9af89c Mon Sep 17 00:00:00 2001 From: Tony Hu Date: Mon, 14 Oct 2024 16:55:19 -0700 Subject: [PATCH 06/12] test --- .github/workflows/continuous_integration.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index b5ee21f0..3006c2ce 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -37,6 +37,7 @@ jobs: echo "Labels: ${{ toJson(github.event.pull_request.labels) }}" echo "Permitted: ${{ steps.check.outputs.permitted }}" echo "Safe to Test Label Present: ${{ contains(github.event.pull_request.labels.*.name, 'safe to test') }}" + echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }}" if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run if: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }} @@ -345,4 +346,4 @@ jobs: issue-number: ${{ github.event.number }} body: | Job PR-${{ github.event.number }}-${{ env.SHORT_SHA }} is done. - Docs are uploaded to https://d12sc05jpx1wj5.cloudfront.net/PR-${{ github.event.number }}/${{ env.SHORT_SHA }}/index.html \ No newline at end of file + Docs are uploaded to https://d12sc05jpx1wj5.cloudfront.net/PR-${{ github.event.number }}/${{ env.SHORT_SHA }}/index.html From 94ef1abb69587fca20c4c7a0704d4a4d1307571e Mon Sep 17 00:00:00 2001 From: tonyhu Date: Mon, 14 Oct 2024 17:18:47 -0700 Subject: [PATCH 07/12] Update continuous_integration.yml --- .github/workflows/continuous_integration.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index 3006c2ce..7d24afdf 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -40,7 +40,7 @@ jobs: echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }}" if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run - if: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }} + if: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }} run: exit 1 - name: Remove Safe to Test Label # One commit is safe doesn't mean the next commit is safe. if: ${{ github.event_name == 'pull_request_target' }} From 64f6ef29bda8c480abbcb1bcb354fbee302854c5 Mon Sep 17 00:00:00 2001 From: tonyhu Date: Mon, 14 Oct 2024 17:20:10 -0700 Subject: [PATCH 08/12] Update continuous_integration.yml --- .github/workflows/continuous_integration.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index 7d24afdf..d43c75e5 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -37,7 +37,7 @@ jobs: echo "Labels: ${{ toJson(github.event.pull_request.labels) }}" echo "Permitted: ${{ steps.check.outputs.permitted }}" echo "Safe to Test Label Present: ${{ contains(github.event.pull_request.labels.*.name, 'safe to test') }}" - echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && (contains(github.event.pull_request.labels.*.name, 'safe to test') == 'false') && steps.check.outputs.permitted == 'false' }}" + echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }}" if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run if: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }} From b4edfa9e8585ea96133972d075c50437f85560fd Mon Sep 17 00:00:00 2001 From: tonyhu Date: Mon, 14 Oct 2024 17:30:13 -0700 Subject: [PATCH 09/12] Update continuous_integration.yml --- .github/workflows/continuous_integration.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index d43c75e5..f7d5c5f1 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -36,6 +36,9 @@ jobs: echo "Event Name: ${{ github.event_name }}" echo "Labels: ${{ toJson(github.event.pull_request.labels) }}" echo "Permitted: ${{ steps.check.outputs.permitted }}" + echo "${{ github.event_name == 'pull_request_target' }}" + echo "${{ !contains(github.event.pull_request.labels.*.name, 'safe to test') }}" + echo "${{ !steps.check.outputs.permitted }}" echo "Safe to Test Label Present: ${{ contains(github.event.pull_request.labels.*.name, 'safe to test') }}" echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }}" if: ${{ github.event_name == 'pull_request_target' }} From de1e874d3b4a5f1daf88d46162165baa06469add Mon Sep 17 00:00:00 2001 From: tonyhu Date: Mon, 14 Oct 2024 17:33:36 -0700 Subject: [PATCH 10/12] Update continuous_integration.yml --- .github/workflows/continuous_integration.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index f7d5c5f1..ed8bad2d 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -38,9 +38,9 @@ jobs: echo "Permitted: ${{ steps.check.outputs.permitted }}" echo "${{ github.event_name == 'pull_request_target' }}" echo "${{ !contains(github.event.pull_request.labels.*.name, 'safe to test') }}" - echo "${{ !steps.check.outputs.permitted }}" + echo "${{ steps.check.outputs.permitted == 'false' }}" echo "Safe to Test Label Present: ${{ contains(github.event.pull_request.labels.*.name, 'safe to test') }}" - echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }}" + echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && steps.check.outputs.permitted=='false' }}" if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run if: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }} From 5dfa18adf09d266f0393b4fae4484a45d7750d9b Mon Sep 17 00:00:00 2001 From: tonyhu Date: Mon, 14 Oct 2024 17:35:16 -0700 Subject: [PATCH 11/12] Update continuous_integration.yml --- .github/workflows/continuous_integration.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/continuous_integration.yml b/.github/workflows/continuous_integration.yml index ed8bad2d..9aedca62 100644 --- a/.github/workflows/continuous_integration.yml +++ b/.github/workflows/continuous_integration.yml @@ -43,7 +43,7 @@ jobs: echo "Condition Evaluation: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && steps.check.outputs.permitted=='false' }}" if: ${{ github.event_name == 'pull_request_target' }} - name: Check PR Safe to Run - if: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && !steps.check.outputs.permitted }} + if: ${{ github.event_name == 'pull_request_target' && !contains(github.event.pull_request.labels.*.name, 'safe to test') && steps.check.outputs.permitted=='false' }} run: exit 1 - name: Remove Safe to Test Label # One commit is safe doesn't mean the next commit is safe. if: ${{ github.event_name == 'pull_request_target' }} From 45d59b56dcd666c4b5ef0bb9d02a663767ba9830 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 11 Nov 2024 10:19:03 +0000 Subject: [PATCH 12/12] Update packaging requirement from <24.0,>=23.0 to >=23.0,<25.0 Updates the requirements on [packaging](https://github.com/pypa/packaging) to permit the latest version. - [Release notes](https://github.com/pypa/packaging/releases) - [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst) - [Commits](https://github.com/pypa/packaging/compare/23.0...24.2) --- updated-dependencies: - dependency-name: packaging dependency-type: direct:production ... Signed-off-by: dependabot[bot] --- setup.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/setup.py b/setup.py index 46411388..37af7d04 100644 --- a/setup.py +++ b/setup.py @@ -112,7 +112,7 @@ def default_setup_args(*, version): "autogluon.common>=0.7", # <2 because unlikely to introduce breaking changes in minor releases. >=1.10 because 1.10 is 3 years old, no need to support older "boto3>=1.10,<2.0", - "packaging>=23.0,<24.0", + "packaging>=23.0,<25.0", # updated sagemaker is required to fetch latest container info, so we don't want to cap the version too strict # otherwise cloud module needs to be released to support new container "sagemaker>=2.126.0,<3.0",