mila-iqia / milatools

Tools to connect to and interact with the Mila cluster
MIT License
62 stars 12 forks source link

Change ControlPersist from 600 to 'yes' #106

Closed lebrice closed 6 months ago

lebrice commented 7 months ago

This makes it much more convenient to connect to clusters where 2FA is enabled, as the procedure needs to be done only one per machine boot (instead of once every 10 minutes).

codecov-commenter commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.31%. Comparing base (db9b7bc) to head (a8e7929). Report is 13 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #106 +/- ## ======================================= Coverage 59.31% 59.31% ======================================= Files 12 12 Lines 1777 1777 ======================================= Hits 1054 1054 Misses 723 723 ``` | [Flag](https://app.codecov.io/gh/mila-iqia/milatools/pull/106/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia) | Coverage Δ | | |---|---|---| | [integrationtests](https://app.codecov.io/gh/mila-iqia/milatools/pull/106/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia) | `27.51% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/mila-iqia/milatools/pull/106/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia) | `59.20% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.