Project

General

Profile

Actions

Bug #6942

closed

[Documentation] Improve compute node setup script

Added by Brett Smith over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Documentation
Target version:
Start date:
08/10/2015
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

Near the bottom of the compute node install guide, there's a shell script that includes a Python script to create the initial compute node record in Arvados. This script needs the following improvements:

  • socket.gethostname() may return the shortname or FQDN depending on exactly how /etc/hosts is written, which we don't want to fix. Improvement: make sure to always get the FQDN, then split it up into shortname and domain parts.
  • The body of the create request should include the address and domain fields.
  • The shell should should $ARVADOS_API_HOST in URLs, rather than the api shortname, to work in a wider variety of installations.

Subtasks 1 (0 open1 closed)

Task #7060: Review 6942-improve-compute-node-setup-scriptResolvedWard Vandewege08/10/2015

Actions
Actions #1

Updated by Ward Vandewege over 9 years ago

  • Assigned To set to Ward Vandewege
Actions #2

Updated by Ward Vandewege over 9 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:21f2d53a70dbf6ae04db07e163f4485e3af47478.

Actions #3

Updated by Brett Smith over 9 years ago

Reviewing 21f2d53

  • The curl command at the very end of the script also needs its URL changed to use $ARVADOS_API_HOST.
  • Would it maybe be a little nicer to pass fqdn to socket.gethostbyname()? I think that would help be even more sure that the results are internally consistent.

This is good to merge with at least the first change. Thanks.

Actions #4

Updated by Ward Vandewege over 9 years ago

Excellent, thank you. I applied both changes and merged.

Actions

Also available in: Atom PDF