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 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Javier Bértoli
Category:
Deployment
Target version:
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/2021Actions

Related issues

Blocks Arvados - Idea #17512: Release Arvados 2.2ResolvedPeter Amstutz05/03/2021Actions
Actions #1

Updated by Peter Amstutz over 3 years ago

Actions #2

Updated by Javier Bértoli over 3 years ago

  • Status changed from New to In Progress
Actions #3

Updated by Javier Bértoli over 3 years 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 3 years 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 3 years ago

  • Status changed from In Progress to Feedback
Actions #6

Updated by Lucas Di Pentima over 3 years ago

This LGTM, thanks!

Actions #7

Updated by Javier Bértoli over 3 years ago

  • Status changed from Feedback to Resolved

Merged

Actions #8

Updated by Peter Amstutz over 3 years ago

  • Release set to 38
Actions

Also available in: Atom PDF