Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SCB support for ssh operations #108

Merged
merged 4 commits into from Oct 14, 2021
Merged

Add SCB support for ssh operations #108

merged 4 commits into from Oct 14, 2021

Conversation

asalajan
Copy link
Contributor

@asalajan asalajan commented Oct 5, 2021

Description

Shell Control Box (SCB) is an activity monitoring appliance from Balabit (now One Identity) that controls privileged access to remote servers.
Added support for using ops with SCB for the following operations: ssh, tunnel, proxy, ansible play, run and sync

Motivation and Context

In my organisation using SCB will be mandatory for security reasons and ops-cli is already being used.

How Has This Been Tested?

Used ops-cli with an aws cluster for the following operations:

  • ssh
  • ssh tunnel
  • ssh proxy
  • run ansible playbook - ops play command
  • ops run command
  • ops sync command

All operations were tested in each of the following scenario:

  • connection via bastion host
  • connection via SCB and then bastion host

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@amuraru
Copy link
Contributor

amuraru commented Oct 5, 2021

Fix code scanning results and then +1

@@ -109,5 +109,7 @@ def configure_common_ansible_args(parser):
help='Ask sudo pass for commands that need sudo')
parser.add_argument('--limit', type=str,
help='Limit run to a specific server subgroup. Eg: --limit newton-dcs')
parser.add_argument('--noscb', action='store_false', dest='use_scb',
help='Disable use of Shell Control Box (SCB) even it is enabled in the cluster config')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/it/if/g

'--noscb',
action='store_false',
dest='use_scb',
help='Disable use of Shell Control Box (SCB) even it is enabled in the cluster config')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/it/if/g

@@ -26,6 +28,8 @@ def configure(self, parser):
help='Value for remote user that will be used for ssh')
parser.add_argument('src', type=str, help='Source dir')
parser.add_argument('dest', type=str, help='Dest dir')
parser.add_argument('--noscb', action='store_false', dest='use_scb',
help='Disable use of Shell Control Box (SCB) even it is enabled in the cluster config')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

Shell Control Box (SCB) is an activity monitoring appliance from Balabit (now One Identity) that controls privileged access to remote servers.
Added support for using ops with SCB for the following operations: ssh, tunnel, proxy, ansible play, run and sync
@pep8speaks
Copy link

pep8speaks commented Oct 5, 2021

Hello @asalajan! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-10-13 09:13:39 UTC

@asalajan asalajan merged commit 181e570 into adobe:master Oct 14, 2021
@asalajan asalajan deleted the scb branch October 14, 2021 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants