arkade icon indicating copy to clipboard operation
arkade copied to clipboard

Published 20 hours ago • verified publisher icon alexellis

Add terrascan CLI

Open yankeexe opened this issue 3 years ago • 11 comments

Signed-off-by: Yankee Maharjan [email protected]

Description

Add Terrascan CLI. It detects compliance and security violations across Infrastructure as Code to mitigate risk before provisioning cloud-native infrastructure.

Motivation and Context

  • [x] I have raised an issue to propose this change (required)

How Has This Been Tested?

Added Test Case Test_DownloadTerrascan.

Tested on Linux amd64 and Linux arm64.

Are you a GitHub Sponsor (Yes/No?)

  • [ ] Yes
  • [x] No

Types of changes

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

Documentation

  • [x] I have updated the list of tools in README.md if (required) with ./arkade get -o markdown
  • [ ] I have updated the list of apps in README.md if (required) with ./arkade install --help

Checklist:

  • [x] My code follows the code style of this project.
  • [x] My change requires a change to the documentation.
  • [x] I have updated the documentation accordingly.
  • [x] I've read the CONTRIBUTION guide
  • [x] I have signed-off my commits with git commit -s
  • [x] I have added tests to cover my changes.
  • [x] All new and existing tests passed.
  • [ ] I have tested this on arm, or have added code to prevent deployment

yankeexe avatar Aug 10 '21 14:08 yankeexe

@yankeexe we need a quick rebase on this please

@dirien I also noticed that you didn't regenerate the README in your PR (unless I missed it) #499

alexellis avatar Aug 11 '21 07:08 alexellis

Approved, but please rebase so that I can merge.

@alexellis done! We can merge this.

yankeexe avatar Aug 11 '21 07:08 yankeexe

@alexellis , not sure what you mean with regenerate the README? I added the tool into the readme -> #498

dirien avatar Aug 11 '21 07:08 dirien

We have another conflict on this one, sorry, but you will need to rebase.

alexellis avatar Sep 15 '21 08:09 alexellis

FYI this is still pending

alexellis avatar Oct 03 '21 16:10 alexellis

@alexellis Ready for merge!

yankeexe avatar Oct 20 '21 02:10 yankeexe

@yankeexe can you rebase again please?

Shikachuu avatar Jan 27 '22 09:01 Shikachuu

@yankeexe we migrated terrascan repo ! https://github.com/tenable/terrascan

sangam14 avatar May 13 '22 20:05 sangam14

@yankeexe are you still intrested to finish this PR using the new repository and a rebase?

Shikachuu avatar Jul 01 '22 17:07 Shikachuu

@Shikachuu ready for merge!

yankeexe avatar Jul 07 '22 15:07 yankeexe

This requires a rebase due to the age, and other PRs having being merged recently.

alexellis avatar Jul 12 '22 08:07 alexellis

Closed in a1f31ad83c82e8501269ddbd7f90c454095aa709

alexellis avatar Aug 08 '22 09:08 alexellis