Skip to content

Commit

Permalink
Merge pull request pmem#5816 from szczepax/configure-runner-update
Browse files Browse the repository at this point in the history
common: Update configure runner and opensuse setup scripts
  • Loading branch information
janekmi authored Aug 23, 2023
2 parents bcaab45 + b067ebb commit 6e4f445
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 5 deletions.
15 changes: 13 additions & 2 deletions utils/ansible/configure-self-hosted-runner.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
# connection: local
vars:
testUser: pmdkuser
package_url: https://github.com/actions/runner/releases/download/v2.304.0/actions-runner-linux-x64-2.304.0.tar.gz
package_url: https://github.com/actions/runner/releases/download/v2.306.0/actions-runner-linux-x64-2.306.0.tar.gz
runner_folder: /home/{{ testUser }}/actions-runner
repo_url: https://github.com/pmem/pmdk
vars_list: "{{ vars_gha.split(',') }}"
Expand All @@ -59,7 +59,18 @@
remote_src: yes

- name: "Change owner to {{ testUser }}"
shell: chown -R {{ testUser }} {{ runner_folder }}
shell: chown -R $(id -u {{ testUser }}):$(id -g {{ testUser }}) {{ runner_folder }}

# Make sure the following environment variables are present in the env
# to ensure propagation to the actions-runner's environment.
- name: "Add env variables into env.sh checklist"
lineinfile:
path: "{{ runner_folder }}/env.sh"
line: " '{{ item.line }}'"
insertafter: "^varCheckList=\\("
loop:
- line: PKG_CONFIG_PATH
- line: HOME

- name: "Add runner to GHA"
shell: |
Expand Down
3 changes: 0 additions & 3 deletions utils/ansible/opensuse-setup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -145,9 +145,6 @@
- name: "Install valgrind from source"
script: ../docker/images/install-valgrind.sh

- name: "Run the install-libndctl script with arguments"
script: ../docker/images/install-libndctl.sh tags/v70.1

# Disable AppArmor.
# AppArmor may block proper GHA runner installation and startup.
# ==`Suse` condition is inherited from the original version of
Expand Down

0 comments on commit 6e4f445

Please sign in to comment.