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

Importing JoshiaSiegel GitHub Actions #16023

Draft
wants to merge 25 commits into
base: master
Choose a base branch
from
Draft

Importing JoshiaSiegel GitHub Actions #16023

wants to merge 25 commits into from

Conversation

emvaldes
Copy link
Collaborator

@emvaldes emvaldes commented Sep 27, 2024

This Pull Request (PR) is intended only as a placeholder for importing the remote GitHub Actions developed by Mr. Josiah Siegel. We will work on completing this import aiming to do not case any impact on the existing workflows.

Note: The objective is to remove any dependencies on external repositories outside the DevSecOps team.

@emvaldes emvaldes added the DevSecOps Team Aq DevSecOps work label label Sep 27, 2024
@emvaldes emvaldes added this to the in progress milestone Sep 27, 2024
@emvaldes emvaldes requested a review from a team as a code owner September 27, 2024 21:45
@emvaldes emvaldes requested review from devopsmatt and removed request for devopsmatt September 27, 2024 21:46
Copy link
Collaborator

@devopsmatt devopsmatt left a comment

Choose a reason for hiding this comment

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

some minor items to address

uses: ./
id: connect_vpn
with:
PING_URL: '${{ secrets.PING_URL }}'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Secret

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We do not have this specific secret in this repo. I will continue to review this workflow.


- name: Connect VPN
env:
TLS_KEY: ${{ inputs.TLS_KEY }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

validate inputs and env vars are set

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We have this input defined in the Environment Secrets section.

repos:
description: 'Repos to scan:'
required: true
default: '[\"josiahsiegel\/runleaks\",\"josiahsiegel\/AzViz-action\"]'
Copy link
Collaborator

Choose a reason for hiding this comment

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

update repo


- name: Fetch random repo name
if: github.event_name == 'schedule' || github.event_name == 'push'
uses: JosiahSiegel/randomrepo@v1.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs update

@@ -0,0 +1,163 @@
# runleaks

[![Scan Action Logs](https://github.com/JosiahSiegel/runleaks/actions/workflows/main.yml/badge.svg?branch=main)](https://github.com/JosiahSiegel/runleaks/actions/workflows/main.yml)
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs update

- name: Checkout
uses: actions/checkout@v3
- name: Scan run logs
uses: josiahsiegel/runleaks@v1
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs upddate

## Local testing
* Registers default patterns
```sh
git clone https://github.com/JosiahSiegel/runleaks.git
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs update

@@ -0,0 +1,110 @@
# Terraform Stats

[![Test action](https://github.com/JosiahSiegel/terraform-stats/actions/workflows/test_action.yml/badge.svg)](https://github.com/JosiahSiegel/terraform-stats/actions/workflows/test_action.yml)
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs update


```yml
- name: Terraform stats
uses: josiahsiegel/terraform-stats@<latest-version>
Copy link
Collaborator

Choose a reason for hiding this comment

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

needs update

@devopsmatt devopsmatt marked this pull request as draft September 27, 2024 22:19
@scott-aquia
Copy link
Collaborator

Inserting a comment here to address the request from the bot

Copy link

github-actions bot commented Oct 8, 2024

Test Results

1 241 tests  ±0   1 237 ✅ ±0   7m 38s ⏱️ -12s
  162 suites ±0       4 💤 ±0 
  162 files   ±0       0 ❌ ±0 

Results for commit 5bd7df7. ± Comparison against base commit ffbe8be.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Oct 8, 2024

Integration Test Results

 53 files  ±0   53 suites  ±0   27m 38s ⏱️ -6s
410 tests ±0  401 ✅ ±0  9 💤 ±0  0 ❌ ±0 
413 runs  ±0  404 ✅ ±0  9 💤 ±0  0 ❌ ±0 

Results for commit 5bd7df7. ± Comparison against base commit ffbe8be.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DevSecOps Team Aq DevSecOps work label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants