Project

General

Profile

Actions

Feature #14712

closed

[git] arv-git-httpd uses cluster config file

Added by Peter Amstutz almost 4 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Eric Biagiotti
Category:
-
Target version:
Start date:
08/26/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
2.0
Release relationship:
Auto

Description

See GitServer section of https://dev.arvados.org/projects/arvados/wiki/Config_migration_key_mapping for the canonical list of keys.

Does not include the git sync component which is part of the API Server.

Includes building the framework for config loading which knows how manage the fallback hierarchy of new vs old configuration data as well as how to warn on mismatches if both are present.


Subtasks 1 (0 open1 closed)

Task #15479: Review 14712-git-httpd-configResolvedEric Biagiotti08/26/2019

Actions

Related issues

Related to Arvados - Story #13648: [Epic] Use one cluster configuration file for all componentsResolved

Actions
Actions #1

Updated by Peter Amstutz almost 4 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Peter Amstutz almost 4 years ago

  • Status changed from In Progress to New
Actions #3

Updated by Peter Amstutz almost 4 years ago

  • Related to Story #13648: [Epic] Use one cluster configuration file for all components added
Actions #4

Updated by Peter Amstutz almost 4 years ago

  • Tracker changed from Bug to Feature
Actions #5

Updated by Lucas Di Pentima almost 4 years ago

services/api/script/arvados-git-sync.rb config file: arvados-clients.yml

production:
  gitolite_url: /var/lib/arvados/git/repositories/gitolite-admin.git
  gitolite_tmp: /var/lib/arvados/git
  arvados_api_host: uuid_prefix.example.com
  arvados_api_token: "zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz" 
  arvados_api_host_insecure: false
  gitolite_arvados_git_user_key: "ssh-rsa AAAAB[...]KztXLNLSbp git@gitserver" 

MAPPINGS

Repositories:
  Gitolite:
    AdminURL: /var/lib/arvados/git/repositories/gitolite-admin.git
    TmpDir: /var/lib/arvados/git
    SSHUserKey: ssh-rsa AAAAB[...]KztXLNLSbp git@gitserver

arv-git-httpd config file: /etc/arvados/git-httpd/git-httpd.yml

Client:
  APIHost: uuid_prefix.your.domain
  Insecure: false
GitCommand: /var/lib/arvados/git/gitolite/src/gitolite-shell
GitoliteHome: /var/lib/arvados/git
Listen: :9001
RepoRoot: /var/lib/arvados/git/repositories

MAPPINGS

Repositories:
  Gitolite:
    HomeDir: /var/lib/arvados/git
    RootDir: /var/lib/arvados/git/repositories
    HTTPAccess: :9001
Actions #6

Updated by Lucas Di Pentima almost 4 years ago

Corrected version:

Clusters:
  xyzzy:
    SuperuserToken: zzzzzz
    GitServer:
      [...]
      AdminRepoURL: /var/lib/[...]/gitolite-admin.git              # arvados-git-sync.rb/gitolite_url
      SSHUserKey: ssh-rsa AAAAB[...]KztXLNLSbp git@gitserver       # arvados-git-sync.rb/gitolite_arvados_git_user_key
      TmpDir: /var/lib/arvados/git                                 # arvados-git-sync.rb/gitolite_tmp
      GitCommand: /var/lib/arvados/git/gitolite/src/gitolite-shell # arv-git-httpd/GitCommand
      GitoliteHomeDir: /var/lib/arvados/git                        # arv-git-httpd/GitoliteHome
Actions #7

Updated by Lucas Di Pentima almost 4 years ago

TODO: Check SSHUserKey's real usage to see if that's an appropriate naming

Actions #8

Updated by Tom Morris almost 4 years ago

  • Description updated (diff)
  • Target version changed from To Be Groomed to Arvados Future Sprints
  • Story points set to 2.0
Actions #9

Updated by Tom Morris over 3 years ago

  • Assigned To set to Eric Biagiotti
  • Target version changed from Arvados Future Sprints to 2019-07-31 Sprint
Actions #10

Updated by Eric Biagiotti over 3 years ago

  • Target version changed from 2019-07-31 Sprint to 2019-08-14 Sprint
Actions #11

Updated by Eric Biagiotti over 3 years ago

  • Target version changed from 2019-08-14 Sprint to 2019-08-28 Sprint
Actions #12

Updated by Eric Biagiotti over 3 years ago

  • Status changed from New to In Progress
Actions #13

Updated by Eric Biagiotti over 3 years ago

Latest at 44a871434f648052a410f158fc8e09ec17c11339
Tests at https://ci.curoverse.com/view/Developer/job/developer-run-tests/1489/.

Manually tested using arvbox:
- Created in account in WB, added my ssh key, and ran git clone git@172.17.0.2:arvados.git from my terminal.
- Ran git clone https://172.17.0.2:9000/arvados.git from my terminal. Username is whateverYouWant and the password is the user token.

Actions #14

Updated by Lucas Di Pentima over 3 years ago

Just 2 minor comments:

  • At config.default.yml line 429, the comment seems to have a missing double.
  • At the install guide, file doc/install/install-arv-git-httpd.html.textile.liquid lines 280 & 291, I believe the services’ section should be GitHTTP instead of GitHttp.

Other than that it LGTM, thanks!

Actions #15

Updated by Eric Biagiotti about 3 years ago

Fixed in

Lucas Di Pentima wrote:

Just 2 minor comments:

  • At config.default.yml line 429, the comment seems to have a missing double.
  • At the install guide, file doc/install/install-arv-git-httpd.html.textile.liquid lines 280 & 291, I believe the services’ section should be GitHTTP instead of GitHttp.

Other than that it LGTM, thanks!

Fixed in 3c0e4dc35b5f2fc34e050fef304cdec0cebe51ae. Thanks!

Actions #16

Updated by Eric Biagiotti about 3 years ago

  • Status changed from In Progress to Resolved
Actions #17

Updated by Peter Amstutz almost 3 years ago

  • Release set to 22
Actions

Also available in: Atom PDF