Bug #6942

[Documentation] Improve compute node setup script

Added by Brett Smith about 4 years ago. Updated about 4 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

Task #7060: Review 6942-improve-compute-node-setup-scriptResolvedWard Vandewege

Associated revisions

Revision 21f2d53a (diff)
Added by Ward Vandewege about 4 years ago

Improve compute node setup script.

closes #6942

Revision c62a4ff1 (diff)
Added by Ward Vandewege about 4 years ago

Some more improvements based on review feedback.

closes #6942

Revision ba0bc2f7
Added by Ward Vandewege about 4 years ago

Merge branch '6942-improve-compute-node-setup-script'

closes #6942

History

#1 Updated by Ward Vandewege about 4 years ago

  • Assigned To set to Ward Vandewege

#2 Updated by Ward Vandewege about 4 years ago

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

Applied in changeset arvados|commit:21f2d53a70dbf6ae04db07e163f4485e3af47478.

#3 Updated by Brett Smith about 4 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.

#4 Updated by Ward Vandewege about 4 years ago

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

Also available in: Atom PDF