[nrf noup] ci: Fix the path for running compliance

fixup![nrf noup] ci: Fix the paths The compliance should be run from hostap repo and not sdk-zephyr. And also disable few tests that won't work reliably. Signed-off-by: Chaitanya Tata <[email protected]>
nrfconnect · Feb 28, 2024 · b4a54f5 · b4a54f5
1 parent b2ff7e0
commit b4a54f5
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/.github/workflows/compliance.yml b/.github/workflows/compliance.yml
@@ -59,14 +59,13 @@ jobs:
       id: compliance
       env:
         BASE_REF: ${{ github.base_ref }}
-      working-directory: sdk-zephyr
       if: contains(github.event.pull_request.user.login, 'dependabot[bot]') != true
       run: |
-        export ZEPHYR_BASE="$(dirname "$(pwd)")/sdk-zephyr"
+        export ZEPHYR_BASE="$(dirname "$(pwd)")/sdk-hostap/sdk-zephyr"
         # debug
         ls -la
         git log  --pretty=oneline | head -n 10
-        ./scripts/ci/check_compliance.py --annotate -e gitlint -e KconfigBasic -e Kconfig \
+        $ZEPHYR_BASE/scripts/ci/check_compliance.py --annotate -e ModulesMaintainers -e KconfigBasicNoModules -e KconfigBasic -e Kconfig \
         -c origin/${BASE_REF}..
 
     - name: upload-results
@@ -75,14 +74,16 @@ jobs:
       if: contains(github.event.pull_request.user.login, 'dependabot[bot]') != true
       with:
         name: compliance.xml
-        path: sdk-zephyr/compliance.xml
+        path: sdk-hostap
 
     - name: check-warns
       if: contains(github.event.pull_request.user.login, 'dependabot[bot]') != true
       run: |
+        echo $(pwd)
+        ls -la
         export ZEPHYR_BASE="$(dirname "$(pwd)")/sdk-hostap/sdk-zephyr"
-        if [[ ! -s $ZEPHYR_BASE/"compliance.xml" ]]; then
-          echo "::error ::compliance.xml not found in $ZEPHYR_BASE"
+        if [[ ! -s $(dirname "$(pwd)")/sdk-hostap/compliance.xml ]]; then
+          echo "::error ::compliance.xml not found in $(dirname "$(pwd)")/sdk-hostap/compliance.xml"
           exit 1;
         fi