Project

General

Profile

Actions
Copy link

Idea #8066

closed

[Deployment] API server package postinst sets up git_internal_dir

Added by Brett Smith over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Nico César
Category:
-
Target version:
2016-01-20 Sprint
Story points:
1.0

Description

The API server has code to try to set this up itself, but it frequently fails because the parent directory doesn't exist, or the API server can't write to it. Doing it from the package is much less error-prone.

When crunch-dispatch inits the repository, it frequently has wrong ownerships (root:root instead of www-data:www-data).

  • Take the git init code out of crunch-dispatch.
  • After config:check succeeds, grab the value of git_internal_dir. If it doesn't exist, set up all directories as necessary as www-data:root 0770, then su -c "git init …" www-data. Make sure to follow symlinks all the way through, from the existence check to the final init.
  • Make sure Puppet isn't going to thrash on these changes.
  • Test on a test cluster to make sure it doesn't make any changes.

Subtasks 1 (0 open — 1 closed)

Task #8134: Review arvados-dev branchResolvedWard Vandewege01/22/2016Actions

Related issues

Blocked by Arvados - Idea #8014: [Deployment] /var/www/arvados-RAILSPKG/config includes symlinks to /etc/arvados/RAILSPKG as appropriateResolvedBrett Smith01/04/2016Actions
Actions
Copy link
 #1

Updated by Brett Smith over 8 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Brett Smith over 8 years ago

  • Subject changed from [Deployment] arvados-api-server-upgrade.sh sets up git_internal_dir to [Deployment] API server package postinst sets up git_internal_dir
Actions
Copy link
 #3

Updated by Brett Smith over 8 years ago

  • Target version set to Arvados Future Sprints
Actions
Copy link
 #4

Updated by Brett Smith over 8 years ago

  • Description updated (diff)
  • Story points set to 1.0
Actions
Copy link
 #5

Updated by Brett Smith over 8 years ago

  • Target version changed from Arvados Future Sprints to 2016-01-20 Sprint
Actions
Copy link
 #6

Updated by Brett Smith over 8 years ago

  • Assigned To set to Nico César
Actions
Copy link
 #7

Updated by Nico César over 8 years ago

  • Description updated (diff)
Actions
Copy link
 #8

Updated by Brett Smith over 8 years ago

Please wait for #8014 to merge before you start writing code. Or maybe base your initial arvados-dev development branch off my 8014 branch.

Actions
Copy link
 #9

Updated by Nico César about 8 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #10

Updated by Nico César about 8 years ago

the testing was done the following way:

docker run -ti -v arvados/packages/ubuntu1404/arvados-api-server_0.1.20160104193447.3344f5b-3_amd64.deb:/root/a.deb ubuntu bash

apt-get install curl
sudo gpg --keyserver hkp://keys.gnupg.net --recv-keys 409B6B1796C275462A1703113804BB82D39DC0E3
\curl -sSL https://get.rvm.io | sudo bash -s stable --ruby=2.1
source /usr/local/rvm/scripts/rvm
export WEB_SERVICE=nginx
apt-get install postgresql-server-dev-9.3 nginx libcurl4-gnutls-dev
gem install curb -v '0.8.8'
gem install pg -v '0.17.1'
dpkg -i /root/a.deb
Actions
Copy link
 #11

Updated by Ward Vandewege about 8 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link

Also available in: Atom PDF