Warnings when using PHP 8.3 on self-hosted runners about setup-php

[shivammathur]

@Sn0wCrack

Please provide the workflow you are using.
Also please ensure that you are specifying runner env as self-hosted.
https://github.com/shivammathur/setup-php?tab=readme-ov-file#self-hosted-setup

[Sn0wCrack]

It looks like I had some workflows that didn't have the self-hosted env, which is probably why this occurred.

I'll make this change and see if this resolves the issue by itself.

[shivammathur]

@Sn0wCrack
Following up on this, were you able to fix it on your end.

[Sn0wCrack]

Sorry for not updating, had not had a change to test the changes after implementing them, took some time to go through all of our repositories as well, only a handful of workflows were missing the environment variable.

I've tested it again now after adding the runner environment variable and I still receive these warnings on those servers.

I'm assuming this fix won't be retroactive and I might have to manually remove the installed PHP 8.3 to have the action pull down the correct files again?

Here's my action workflow file for reference as well:

name: Continuous Integration

# Controls when the workflow will run
on:
  workflow_dispatch:
  push:
    branches: [ master ]
    paths:
      - '**.php'
      - '**.js'
      - '**.vue'
      - '**.ts'
      - '**.json'
      - '**.lock'
    tags-ignore:
      - 'v*'
  pull_request:
    branches: [ master ]
    paths:
      - '**.php'
      - '**.js'
      - '**.vue'
      - '**.ts'
      - '**.json'
      - '**.lock'

jobs:
  lint:
    runs-on: [ self-hosted, Linux, ci ]

    strategy:
      matrix:
        php-version: ['8.3']

    steps:
      - uses: actions/checkout@v4

      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: ${{ matrix.php-version }}
          extensions: bcmath, ctype, fileinfo, json, mbstring, openssl, pdo, pdo_mysql tokenizer
          tools: composer
        env:
          runner: self-hosted

      - name: Setup Node
        uses: actions/setup-node@v3
        with:
          cache: 'yarn'
          node-version: 'lts/iron'

      - name: Install JavaScript Dependencies
        run: yarn install --frozen-lockfile

      - name: Install PHP Dependencies
        uses: ramsey/composer-install@v3

      - name: Lint JavaScript
        run: yarn lint

      - name: Lint PHP
        run: ./vendor/bin/pint --test

  test:
    runs-on: [ self-hosted, Linux, ci ]

    services:
      database:
        image: mariadb:10
        env:
          MYSQL_DATABASE: database
          MYSQL_ALLOW_EMPTY_PASSWORD: false
          MYSQL_ROOT_PASSWORD: password
        ports:
          - 3306
        options: >-
          --health-cmd="mysqladmin ping"
          --health-interval=10s
          --health-timeout=5s
          --health-retries=3

      cache:
        image: redis:7
        ports:
          - 6379

    strategy:
      matrix:
        php-version: ['8.3']

    steps:
      - uses: actions/checkout@v4

      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: ${{ matrix.php-version }}
          extensions: bcmath, ctype, fileinfo, json, mbstring, openssl, pdo, pdo_mysql tokenizer, redis, igbinary, gd, imagick
          tools: composer
          coverage: pcov
        env:
          runner: self-hosted

      - name: Setup Node
        uses: actions/setup-node@v3
        with:
          cache: 'yarn'
          node-version: 'lts/iron'

      - name: Install JavaScript Dependencies
        run: yarn install --frozen-lockfile

      - name: Install PHP Dependencies
        uses: ramsey/composer-install@v3

      - name: Build JavaScript
        run: yarn dev

      - name: Setup tasks
        run: |
          touch .env
          php artisan storage:link

      - name: Run Tests
        run: php artisan test --coverage --parallel

[shivammathur]

Yes, it will require manually removing the previously installed version.