Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Commit

Permalink
Merge pull request #3546 from edx/benp/remove-node-jenkins-worker
Browse files Browse the repository at this point in the history
Remove node ppa from jenkins-worker.
  • Loading branch information
Ben Patterson committed Dec 12, 2016
2 parents 4b465ce + c3d09f2 commit 62c6ca1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
4 changes: 0 additions & 4 deletions playbooks/roles/jenkins_worker/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,10 @@ jenkins_user: "jenkins"
jenkins_group: "jenkins"
jenkins_home: /home/jenkins

# repo for nodejs
jenkins_chrislea_ppa: "ppa:chris-lea/node.js"

jenkins_edx_platform_version: master

# System packages
jenkins_debian_pkgs:
- nodejs
- pkg-config
- libffi-dev
- python-dev
Expand Down
4 changes: 0 additions & 4 deletions playbooks/roles/jenkins_worker/tasks/system.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,6 @@
owner={{ jenkins_user }} group={{ jenkins_group }} mode=400
ignore_errors: yes

# adding chris-lea nodejs repo
- name: add ppas for current versions of nodejs
apt_repository: repo="{{ jenkins_chrislea_ppa }}"

- name: Install system packages
apt: pkg={{','.join(jenkins_debian_pkgs)}}
state=present update_cache=yes
Expand Down

0 comments on commit 62c6ca1

Please sign in to comment.