Skip to content
This repository has been archived by the owner on Oct 26, 2023. It is now read-only.

feat/audit large repos #92

Merged
merged 1 commit into from
Apr 29, 2022
Merged

feat/audit large repos #92

merged 1 commit into from
Apr 29, 2022

Conversation

scott-es
Copy link
Contributor

@scott-es scott-es commented Apr 22, 2022

enable feature to audit large repos, to detect whether the API git tree response from github is truncated or not. This indicates that a given repo will need to be cloned locally in order to detect its files and determine which projects need to be refreshed into Snyk.

can test against a single repo with:

./snyk_scm_refresh.py --org-id=<org_id> --audit-large-repos --repo-name=<owner/repo_name>

the kibana repo is large enough test on if you clone it and import into snyk

after running, the results of all repos processed are written to snyk-scm-refresh_large-repos-audit-results.csv

image

@scott-es scott-es added the WIP Work in progress label Apr 22, 2022
@scott-es scott-es requested a review from a team as a code owner April 22, 2022 04:36
@scott-es scott-es force-pushed the feat/audit-large-repos branch from 788f94d to 18f59a7 Compare April 28, 2022 21:32
@scott-es scott-es removed the WIP Work in progress label Apr 28, 2022
@scott-es scott-es force-pushed the feat/audit-large-repos branch from 8bc1e2f to 41f62d7 Compare April 28, 2022 21:43
@huytquach-snyk
Copy link
Contributor

This worked and ran it locally - apparently I am not in cs-engineers to approve it.

@scott-es scott-es requested review from huytquach-snyk and removed request for huytquach-snyk April 29, 2022 02:00
Copy link
Contributor

@huytquach-snyk huytquach-snyk left a comment

Choose a reason for hiding this comment

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

This worked and tested with Kibana repo

@huytquach-snyk huytquach-snyk merged commit 5d5fde7 into develop Apr 29, 2022
@huytquach-snyk huytquach-snyk deleted the feat/audit-large-repos branch April 29, 2022 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants