Skip to content

Commit

Permalink
ci: drop useless chown call
Browse files Browse the repository at this point in the history
AFAIK we anyway run self-hosted runners from root due to all those
problems like [1]. No reason to include the hack into every workflow
file. If we'll going to start runners from a non-root user, it is better
to pass `--user $PROPER_UID` to all docker jobs instead, see [2]: at
least it does not require to place a boilerplate into all workflow
files.

Didn't touch perf_* jobs: don't want to dig inside this part of the
infrastructure ATM.

It reverts PR #5953.

[1]: actions/checkout#211
[2]: actions/runner#691
  • Loading branch information
Totktonada authored and locker committed Dec 13, 2021
1 parent 474d1cc commit a0a2e8b
Show file tree
Hide file tree
Showing 39 changed files with 0 additions and 232 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/centos_7.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/centos_7_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/centos_8.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/centos_8_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/coverity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,6 @@ jobs:
options: '--init'

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v1
- uses: ./.github/actions/environment
- name: test
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debian_10.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debian_10_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debian_11.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debian_11_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debian_9.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/debug_coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
# Finds an associated PR (PR can be detected only on push and never on pull_request).
# WARNING !!! use in these ways only:
# on push: steps.findPr.outputs.pr
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/default_gcc_centos_7.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_30.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_31.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_32.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_33.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_33_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_34.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/fedora_34_aarch64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: graviton

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/luacheck.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/opensuse_15_1.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/opensuse_15_2.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/out_of_source.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release_asan_clang11.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release_clang.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release_lto.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release_lto_clang11.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/reusable_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,6 @@ jobs:
OS: ${{ inputs.os }}
DIST: ${{ inputs.dist }}
steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: 'Correct file permissions in the workspace'
run: sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2
with:
ref: ${{ inputs.ref }}
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/source.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,6 @@ jobs:
runs-on: ubuntu-20.04-self-hosted

steps:
# Permissions correction is needed for self-host runners,
# where work path is saved between different workflows runs.
- name: correct permissions in working directory
shell: bash
run: |
sudo chown -R $(id -u):$(id -g) .
- uses: actions/checkout@v2.3.4
with:
fetch-depth: 0
Expand Down
Loading

0 comments on commit a0a2e8b

Please sign in to comment.