Project

General

Profile

Actions

Bug #17600

closed

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

Added by Javier Bértoli over 1 year ago. Updated over 1 year 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:
-
Release relationship:
Auto

Description

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


Subtasks 1 (0 open1 closed)

Task #17646: Review #17600 arvados-formula, branch 17600-fix-shellinabox-pam-configResolvedJavier Bértoli05/10/2021

Actions

Related issues

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

Actions
Actions #1

Updated by Peter Amstutz over 1 year ago

Actions #2

Updated by Javier Bértoli over 1 year ago

  • Status changed from New to In Progress
Actions #3

Updated by Javier Bértoli over 1 year 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

Actions #4

Updated by Javier Bértoli over 1 year 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.

Actions #5

Updated by Javier Bértoli over 1 year ago

  • Status changed from In Progress to Feedback
Actions #6

Updated by Lucas Di Pentima over 1 year ago

This LGTM, thanks!

Actions #7

Updated by Javier Bértoli over 1 year ago

  • Status changed from Feedback to Resolved

Merged

Actions #8

Updated by Peter Amstutz over 1 year ago

  • Release set to 38
Actions

Also available in: Atom PDF