Docker multi arch #1
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a Docker multi-architecture inspector tool to help analyze Docker images stored in Cloudsmith repositories. The tool provides hierarchical breakdown of images by tag, showing manifest digests, platform information, sync status, and download counts.
- Added a shell script that inspects Docker multi-arch images via Cloudsmith APIs
- Provides download count aggregation across all manifests for complete image statistics
- Updates repository README to reflect its new purpose as a support engineering resource collection
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| README.md | Updated repository description from template to support engineering resources |
| Docker/Multi-Arch-Inspector/run.sh | Main shell script for inspecting Docker multi-arch images |
| Docker/Multi-Arch-Inspector/README.md | Documentation explaining the multi-arch inspector tool usage |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| CLOUDSMITH_URL="https://docker.cloudsmith.io" | ||
| fi | ||
|
|
||
| # uthorization header |
There was a problem hiding this comment.
Typo in comment: 'uthorization' should be 'Authorization'
| # uthorization header | |
| # Authorization header |
Copilot uses AI. Check for mistakes.
| case ${LC_ALL:-${LC_CTYPE:-$LANG}} in *UTF-8*|*utf8*) : ;; *) CHECK='OK'; CROSS='X' ;; esac | ||
|
|
||
| completed() { printf '%s%s%s %s\n' "$GREEN" "$CHECK" "$RESET" "$*"; } | ||
| progress() { printf '%s%s%s %s\n' "$YELLOW" "$TIMER" "$RESET" "$*"; } |
There was a problem hiding this comment.
Variable YELLOW is referenced but never defined. This will cause the progress function to output empty color codes.
Copilot uses AI. Check for mistakes.
|
|
||
| completed() { printf '%s%s%s %s\n' "$GREEN" "$CHECK" "$RESET" "$*"; } | ||
| progress() { printf '%s%s%s %s\n' "$YELLOW" "$TIMER" "$RESET" "$*"; } | ||
| quarantined() { printf '%s%s%s %s\n' "$ORANGE" "$VULN" "$RESET" "$*"; } |
There was a problem hiding this comment.
Variable ORANGE is referenced but never defined. This will cause the quarantined function to output empty color codes.
Copilot uses AI. Check for mistakes.
| ;; | ||
|
|
||
| Failed) | ||
| echo " |____ Status: ${STATUS[0]} ${FAIL}" |
There was a problem hiding this comment.
Variable FAIL is referenced but never defined. This should be CROSS based on the other status display patterns.
Copilot uses AI. Check for mistakes.
| ;; | ||
|
|
||
| Failed) | ||
| echo " |____ Status: ${STATUS[1]} ${FAIL}" |
There was a problem hiding this comment.
Variable FAIL is referenced but never defined. This should be CROSS based on the other status display patterns.
Copilot uses AI. Check for mistakes.
| getDockerDigests () { | ||
|
|
||
| local nTAG="$1" | ||
| local totalDownloads=0 |
There was a problem hiding this comment.
The totalDownloads variable is declared as local inside getDockerDigests function but is being incremented inside the nested getPackageData function where it's not accessible. This will always show 0 downloads.
Copilot uses AI. Check for mistakes.
| getDigestData "${DIGESTS[i]}" | ||
| echo | ||
| done | ||
| echo " |___ Total Downloads: ${totalDownloads}" |
There was a problem hiding this comment.
The totalDownloads variable is declared as local inside getDockerDigests function but is being incremented inside the nested getPackageData function where it's not accessible. This will always show 0 downloads.
Copilot uses AI. Check for mistakes.
|
|
||
| <img src="example.gif" width=50%> | ||
|
|
||
| ## Prequisities |
There was a problem hiding this comment.
Typo in heading: 'Prequisities' should be 'Prerequisites'
| ## Prequisities | |
| ## Prerequisites |
Copilot uses AI. Check for mistakes.
|
|
||
| ## How to use | ||
|
|
||
| Execute run.sh and pass in 4 arguements ( domain, org, repo and image name). |
There was a problem hiding this comment.
Typo: 'arguements' should be 'arguments'
| Execute run.sh and pass in 4 arguements ( domain, org, repo and image name). | |
| Execute run.sh and pass in 4 arguments ( domain, org, repo and image name). |
Copilot uses AI. Check for mistakes.
| # Docker Multi-Arch images overview script | ||
|
|
||
| The script will return an overview of your Docker multi-arch images stored in your Cloudsmith repository. | ||
| It provides a hierarchial breakdown of each image by tag, showing the index digest and it's associated manifest digests with their platform, cloudsmith sync status and downloads count. |
There was a problem hiding this comment.
Typo: 'hierarchial' should be 'hierarchical'
| It provides a hierarchial breakdown of each image by tag, showing the index digest and it's associated manifest digests with their platform, cloudsmith sync status and downloads count. | |
| It provides a hierarchical breakdown of each image by tag, showing the index digest and it's associated manifest digests with their platform, cloudsmith sync status and downloads count. |
Copilot uses AI. Check for mistakes.
1 participant
📄 Summary
Added Docker multi-arch script.