minor #39745 Fix CI: Use full path to binaries in dockerized steps (jderusse)

derrabus · derrabus · commit 92ccb0a9dffd · 2021-01-07T01:31:17.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- Fix CI: Use full path to binaries in dockerized steps | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - ~4 hours ago Github automatically adds the `PATH` env variable into docker steps. Which is an issue for the openldap, because the image stores the binaries in a vendor-prefixed folder. This PR use full path to binaries instead of relying on PATH Commits ------- 9a3fc5a Fix bitnami env PATH
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -75,7 +75,7 @@ jobs:
                 uses: docker://bitnami/openldap
                 with:
                   entrypoint: /bin/bash
-                  args: -c "(ldapwhoami -h localhost:3389 -D cn=admin,dc=symfony,dc=com -w symfony||sleep 5) && ldapadd -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony -f src/Symfony/Component/Ldap/Tests/Fixtures/data/fixtures.ldif && ldapdelete -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony cn=a,ou=users,dc=symfony,dc=com"
+                  args: -c "(/opt/bitnami/openldap/bin/ldapwhoami -h localhost:3389 -D cn=admin,dc=symfony,dc=com -w symfony||sleep 5) && /opt/bitnami/openldap/bin/ldapadd -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony -f src/Symfony/Component/Ldap/Tests/Fixtures/data/fixtures.ldif && /opt/bitnami/openldap/bin/ldapdelete -h ldap:3389 -D cn=admin,dc=symfony,dc=com -w symfony cn=a,ou=users,dc=symfony,dc=com"
 
             -   name: Configure composer
                 run: |
