Project

General

Profile

Review #510

Review branch specify_FQDN_and_remote_port

Added by Pierre-Louis Bonicoli over 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Marc Dequènes
Category:
-
Start date:
2017-02-06
Branch:
specify_FQDN_and_remote_port

Description

Please, could you review the branch called specify_FQDN_and_remote_port (admin repository) ?

Notes:

These modifications allow to execute ansible/ansible-playbooks without requiring unversioned SSH settings.

Related issues

Copied from DuckCorp Infrastructure - Bug #509: Review branch add_iface_korutopiResolved2017-02-06

Actions

History

#1

Updated by Pierre-Louis Bonicoli over 6 years ago

  • Copied from Bug #509: Review branch add_iface_korutopi added
#2

Updated by Marc Dequènes over 6 years ago

Would'nt ansible-specific parameters like this be better in the 'hosts' file? like all grouped in the 'ungrouped' group?
Except from this question, the content is fine.

#3

Updated by Pierre-Louis Bonicoli about 6 years ago

Usage of YAML for inventory file isn't documented and there were some related issues fixed recently, for example ensure all hosts in all.

I would prefer not to use an undocumented feature.

#4

Updated by Pierre-Louis Bonicoli about 6 years ago

Branch updated: *_vars are split.

#5

Updated by Pierre-Louis Bonicoli about 6 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 50
#6

Updated by Pierre-Louis Bonicoli about 6 years ago

  • Tracker changed from Bug to Review
#7

Updated by Pierre-Louis Bonicoli about 6 years ago

  • Branch set to specify_FQDN_and_remote_port
#8

Updated by Pierre-Louis Bonicoli about 6 years ago

  • Status changed from In Progress to Resolved

Merged, thank you for the review !

Also available in: Atom PDF