Add entries for DRAC cluster during mila init [MT-61]

[lebrice]

Asks the user if they also have access to the DRAC cluster. Adds the following block to the ssh config:

# Compute Canada
Host beluga cedar graham narval niagara
  Hostname %h.alliancecan.ca
  User {user}
Host mist
  Hostname mist.scinet.utoronto.ca
  User {user}
Host !beluga  bc????? bg????? bl?????
  ProxyJump beluga
  User {user}
Host !cedar   cdr? cdr?? cdr??? cdr????
  ProxyJump cedar
  User {user}
Host !graham  gra??? gra????
  ProxyJump graham
  User {user}
Host !narval  nc????? ng?????
  ProxyJump narval
  User {user}
Host !niagara nia????
  ProxyJump niagara
  User {user}

TODOs:

• Checks for passwordless SSH aren't working correctly with DRAC clusters
• There are too many empty lines in the generated config

[codecov-commenter]

Codecov Report

Attention: 46 lines in your changes are missing coverage. Please review.

Comparison is base (987a516) 55.74% compared to head (1202630) 58.99%.
Report is 3 commits behind head on master.

Files	Patch %	Lines
milatools/cli/init_command.py	71.53%	39 Missing ⚠️
milatools/cli/local.py	82.60%	4 Missing ⚠️
milatools/cli/commands.py	25.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #54      +/-   ##
==========================================
+ Coverage   55.74%   58.99%   +3.25%     
==========================================
  Files           9        9              
  Lines        1202     1268      +66     
==========================================
+ Hits          670      748      +78     
+ Misses        532      520      -12     

Flag	Coverage Δ
unittests	58.99% <71.95%> (+3.25%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[satyaog]

Noticing that there's a lot of test files. Are all of the tests are necessary of if some of them could be removed? If so, no need to do this in this PR and an issue could be created. I was also wondering if a diff could encompass more info in less lines? For example, the with no initial ssh config file or with this initial content could become implicit in the formatting. This too could become an issue if it is though to bring value to the readability

[satyaog]

Should we delete this method and only have the new display function?

[lebrice]

Yes, I'm thinking of removing the Local and Remote classes in the longer term.