Bug #17600

[deployment][arvados-formula] webshell's shellinabox pam file must be named /etc/pam.d/shellinabox

Added by Javier Bértoli 14 days ago. Updated about 17 hours ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Deployment
Target version:
Start date:
05/10/2021
Due date:
% Done:

100%

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

Description

The formula installs the pam file as arvados, which is wrong. Needs to be renamed


Subtasks

Task #17646: Review #17600 arvados-formula, branch 17600-fix-shellinabox-pam-configResolvedJavier Bértoli


Related issues

Blocks Arvados - Story #17512: Release Arvados 2.2New05/03/2021

History

#1 Updated by Peter Amstutz 8 days ago

#2 Updated by Javier Bértoli 8 days ago

  • Status changed from New to In Progress

#3 Updated by Javier Bértoli 7 days ago

I was confusing the shellinabox pam file with the libpam-arvados-go pam configuration file.

Removed the libpam-arvados-go references in the formula (as they're not functional as they are setup there) and fixed all the shellinabox references and tests.

Should be fixed in commit 675eac6@arvados-formula, branch 17600-fix-shellinabox-pam-config. Need to test it in a cluster

#4 Updated by Javier Bértoli 3 days ago

Side note: Moved the formula to our own repository (git://git.arvados.org/arvados-formula.git) and github mirrors it the same way as the arvados formula.

Added a few more changes and run a vagrant test. The file is correct. Commit ea03991@arvados-formula, branch 17600-fix-shellinabox-pam-config.

#5 Updated by Javier Bértoli 3 days ago

  • Status changed from In Progress to Feedback

#6 Updated by Lucas Di Pentima 3 days ago

This LGTM, thanks!

#7 Updated by Javier Bértoli 2 days ago

  • Status changed from Feedback to Resolved

Merged

Also available in: Atom PDF