Fix trigger and variables

facebookresearch · Oct 23, 2024 · 0d9b953 · 0d9b953
1 parent 65002e4
commit 0d9b953
Show file tree

Hide file tree

Showing 6 changed files with 55 additions and 85 deletions.
diff --git a/.github/actions/early_return_for_forked_pull_requests/action.yml b/.github/actions/early_return_for_forked_pull_requests/action.yml
diff --git a/.github/actions/linux/action.yml b/.github/actions/linux/action.yml
@@ -5,7 +5,6 @@ inputs:
 runs:
   using: composite
   steps:
-  - uses: actions/checkout@v4.1.0
   - name: Preparing environment - Conda
     run: |-
       curl -o Miniconda3-py38_4.8.3-Linux-x86_64.sh https://repo.anaconda.com/miniconda/Miniconda3-py38_4.8.3-Linux-x86_64.sh
@@ -20,4 +19,4 @@ runs:
     run: |-
       ~/miniconda3/bin/conda init bash
       ~/miniconda3/bin/conda create -n hydra python=${{ inputs.py_version }} -yqc conda-forge
-    shell: bash
+    shell: bash
diff --git a/.github/actions/macos/action.yml b/.github/actions/macos/action.yml
@@ -5,12 +5,16 @@ inputs:
 runs:
   using: composite
   steps:
-  - uses: actions/checkout@v4.1.0
   - name: restore_cache
     uses: actions/cache@v3.3.2
     with:
       key: "-${{ inputs.cache_key_version }}-macos-sys-{{ .Branch }}-${{ inputs.py_version }}"
-      path: UPDATE_ME
+      path: |-
+        ~/miniconda3
+        ~/Library/Caches/Homebrew
+  - uses: maxim-lobanov/setup-xcode@v1.6.0
+    with:
+      xcode-version: latest-stable
   - name: Preparing environment - Conda
     run: |-
       if [[ -f ~/miniconda3/LICENSE.txt ]] ; then
@@ -31,6 +35,8 @@ runs:
     shell: bash
   - name: Preparing environment - Hydra
     run: |-
+      source $HOME/.bash_profile
+      echo $PATH
       conda create -n hydra python=${{ inputs.py_version }} -yqc conda-forge
       conda run -n hydra pip install nox --progress-bar off
     shell: bash
@@ -40,4 +46,4 @@ runs:
       path: |-
         ~/miniconda3
         ~/Library/Caches/Homebrew
-      key: "-${{ inputs.cache_key_version }}-macos-sys-{{ .Branch }}-${{ inputs.py_version }}"
+      key: "-${{ inputs.cache_key_version }}-macos-sys-{{ .Branch }}-${{ inputs.py_version }}"
diff --git a/.github/actions/windows/action.yml b/.github/actions/windows/action.yml
@@ -0,0 +1,16 @@
+name: windows
+inputs:
+  py_version:
+    required: true
+runs:
+  using: composite
+  steps:
+  - uses: conda-incubator/setup-miniconda@v3
+    with:
+      miniconda-version: "latest"
+      python-version: ${{ inputs.py_version }}
+      activate-environment: hydra
+  - uses: actions/setup-java@v4
+    with:
+      distribution: 'microsoft'
+      java-version: '21'
diff --git a/.github/workflows/core_tests.yml b/.github/workflows/core_tests.yml
@@ -1,35 +1,8 @@
 name: facebookresearch/hydra/core_tests
 on:
-  workflow_dispatch:
-    inputs:
-      plugin_test:
-        required: true
-      test_plugins:
-        required: true
-      cache_key_version:
-        required: false
-        default: v1
-env:
-  AWS_ACCESS_KEY_ID: xxxxXT3Z
-  AWS_DEFAULT_REGION: xxxxst-2
-  AWS_SECRET_ACCESS_KEY: xxxxGsxB
-  CIRCLECI_TOKEN: xxxxed79
-  INSTANCE_ROLE_ARN: xxxxr-v1
+  push
 jobs:
-  trigger_plugin_pipelines:
-    if: ${{ !(${{ inputs.plugin_test }}) }}
-    runs-on: ubuntu-latest
-    container:
-      image: python:3.8
-    steps:
-    - uses: "./.github/actions/early_return_for_forked_pull_requests"
-    - uses: actions/checkout@v4.1.0
-    - name: Kick off Plugin tests
-      run: |-
-        python tools/ci/circleci_pipeline.py
-        echo "Done kicking off plugin tests."
   test_macos:
-    if: ${{ !(${{ inputs.plugin_test }}) }}
     runs-on: macos-latest
     strategy:
       matrix:
@@ -40,14 +13,12 @@ jobs:
         - '3.11'
     steps:
     - uses: actions/checkout@v4.1.0
-    - uses: maxim-lobanov/setup-xcode@v1.6.0
-      with:
-        xcode-version: 13.4.1
     - uses: "./.github/actions/macos"
       with:
         py_version: "${{ matrix.py_version }}"
     - name: Testing Hydra
       run: |-
+        source $HOME/.bash_profile
         export NOX_PYTHON_VERSIONS=${{ matrix.py_version }}
         conda activate hydra
         pip install nox dataclasses --progress-bar off
@@ -58,10 +29,7 @@ jobs:
           nox -s lint test_tools test_core test_jupyter_notebooks -ts
         fi
   test_linux:
-    if: ${{ !(${{ inputs.plugin_test }}) }}
     runs-on: ubuntu-latest
-    container:
-      image: ubuntu
     strategy:
       matrix:
         py_version:
@@ -86,8 +54,7 @@ jobs:
           nox -s lint test_tools test_core test_jupyter_notebooks -ts
         fi
   test_win:
-    if: ${{ !(${{ inputs.plugin_test }}) }}
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     strategy:
       matrix:
         py_version:
@@ -96,25 +63,22 @@ jobs:
         - '3.10'
         - '3.11'
     steps:
-#     # This item has no matching transformer
-#     - circleci_windows_:
+    - uses: actions/checkout@v4.1.0
+    - uses: "./.github/actions/windows"
     - name: Testing Hydra
       run: |-
-        $env:NOX_PYTHON_VERSIONS="${{ matrix.py_version }}"
-        $env:ConEmuDefaultCp=65001
-        $env:PYTHONIOENCODING="utf_8"
-        conda activate hydra
-        If ($env:${{ github.event.number }}) {
+        conda run -n hydra pip install nox --progress-bar off
+        set NOX_PYTHON_VERSIONS="${{ matrix.py_version }}"
+        set ConEmuDefaultCp=65001
+        set PYTHONIOENCODING="utf_8"
+        If ("${{ github.event.number }}") {
           nox -s lint test_tools test_core test_jupyter_notebooks lint_plugins test_plugins test_plugins_vs_core -ts
         } else {
           nox -s lint test_tools test_core test_jupyter_notebooks -ts
         }
         exit $LASTEXITCODE
   test_linux_omc_dev:
-    if: ${{ !(${{ inputs.plugin_test }}) }}
     runs-on: ubuntu-latest
-    container:
-      image: ubuntu
     strategy:
       matrix:
         py_version:

diff --git a/.github/workflows/plugin_tests.yml b/.github/workflows/plugin_tests.yml
@@ -1,26 +1,24 @@
 name: facebookresearch/hydra/plugin_tests
 on:
-  workflow_dispatch:
+  push:
     inputs:
-      plugin_test:
-        required: true
       test_plugins:
         required: true
-      cache_key_version:
-        required: false
-        default: v1
-env:
-  AWS_ACCESS_KEY_ID: xxxxXT3Z
-  AWS_DEFAULT_REGION: xxxxst-2
-  AWS_SECRET_ACCESS_KEY: xxxxGsxB
-  CIRCLECI_TOKEN: xxxxed79
-  INSTANCE_ROLE_ARN: xxxxr-v1
+        default: hydra_rq_launcher,hydra_ax_sweeper,hydra_submitit_launcher,hydra_nevergrad_sweeper
 jobs:
+  echo_event_number:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4.1.0
+    - uses: "./.github/actions/linux"
+      with:
+        py_version: '3.11'
+    - name: "echo event number"
+      run: |-
+        echo ${{ github.event.number }}
   test_plugin_linux:
-    if: ${{ inputs.plugin_test }}
+    if: github.event.number
     runs-on: ubuntu-latest
-    container:
-      image: ubuntu
     strategy:
       matrix:
         py_version:
@@ -43,7 +41,7 @@ jobs:
         pip install nox dataclasses --progress-bar off
         nox -s lint_plugins test_plugins test_plugins_vs_core -ts
   test_plugin_macos:
-    if: ${{ inputs.plugin_test }}
+    if: github.event.number
     runs-on: macos-latest
     strategy:
       matrix:
@@ -56,22 +54,20 @@ jobs:
         - "${{ inputs.test_plugins }}"
     steps:
     - uses: actions/checkout@v4.1.0
-    - uses: maxim-lobanov/setup-xcode@v1.6.0
-      with:
-        xcode-version: 13.4.1
     - uses: "./.github/actions/macos"
       with:
         py_version: "${{ matrix.py_version }}"
     - name: "${{ matrix.test_plugin }}"
       run: |-
+        source $HOME/.bash_profile
         export NOX_PYTHON_VERSIONS=${{ matrix.py_version }}
         export PLUGINS=${{ matrix.test_plugin }}
         conda activate hydra
         pip install nox dataclasses --progress-bar off
         nox -s lint_plugins test_plugins test_plugins_vs_core -ts
   test_plugin_win:
-    if: ${{ inputs.plugin_test }}
-    runs-on: ubuntu-latest
+    if: github.event.number
+    runs-on: windows-latest
     strategy:
       matrix:
         py_version:
@@ -82,8 +78,8 @@ jobs:
         test_plugin:
         - "${{ inputs.test_plugins }}"
     steps:
-#     # This item has no matching transformer
-#     - circleci_windows_:
+    - uses: actions/checkout@v4.1.0
+    - uses: "./.github/actions/windows"
     - name: "${{ matrix.test_plugin }}"
       run: |-
         $env:NOX_PYTHON_VERSIONS="${{ matrix.py_version }}"