-
Notifications
You must be signed in to change notification settings - Fork 341
feat: migrate 50% of agentic workflows from upload-asset to upload-artifact #25342
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
Changes from 3 commits
f89cefc
13f0943
dd17133
79e559c
e462f5e
016cbca
cace937
3f57dd5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,7 +15,9 @@ tools: | |
| agentic-workflows: | ||
| timeout: 300 | ||
| safe-outputs: | ||
| upload-asset: | ||
| upload-artifact: | ||
| default-retention-days: 30 | ||
| max-retention-days: 30 | ||
|
||
| timeout-minutes: 45 | ||
| imports: | ||
| - uses: shared/daily-audit-discussion.md | ||
|
|
@@ -296,9 +298,9 @@ Use `sns.set_theme(style="darkgrid")` for a professional dark-grid look and `plt | |
|
|
||
| --- | ||
|
|
||
| ## Step 5 — Upload Charts as Assets | ||
| ## Step 5 — Upload Charts as Artifacts | ||
|
|
||
| For each successfully generated chart in `/tmp/gh-aw/python/charts/*.png`, use the `upload asset` safe-output tool to publish it. Collect the returned URL for each chart. | ||
| Stage each successfully generated chart from `/tmp/gh-aw/python/charts/*.png` into `/tmp/gh-aw/safeoutputs/upload-artifacts/`, then call the `upload_artifact` safe-output tool for each chart with `retention_days: 30`. Collect and record the returned `aw_*` ID for each chart. | ||
|
|
||
| --- | ||
|
|
||
|
|
@@ -333,39 +335,39 @@ Create a discussion with the following structure. Replace placeholders with real | |
|
|
||
| ### 🔗 GitHub API Calls Trend (90 days) | ||
|
|
||
|  | ||
| 📎 **Chart: GitHub API Calls Trend** — artifact `{api_calls_trend_aw_id}` available in the [workflow run artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) | ||
|
||
|
|
||
| {2–3 sentences: highlight the trend direction, peak days, and any notable spikes in total REST API consumption} | ||
|
|
||
| --- | ||
|
|
||
| ### 🔗 GitHub API Calls by Workflow Trend (30 days) | ||
|
|
||
|  | ||
| 📎 **Chart: GitHub API Calls by Workflow Trend** — artifact `{workflow_api_trend_aw_id}` available in the [workflow run artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) | ||
|
|
||
| {2–3 sentences: note which workflows consistently consume the most API quota and any emerging patterns over the last 30 days} | ||
|
|
||
| --- | ||
|
|
||
| ### 🔗 GitHub REST API Calls Heatmap (90 days) | ||
|
|
||
|  | ||
| 📎 **Chart: GitHub REST API Calls Heatmap** — artifact `{api_heatmap_aw_id}` available in the [workflow run artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) | ||
|
|
||
| {2–3 sentences: describe weekly patterns, busiest days, and any anomalies in REST API consumption} | ||
|
|
||
| --- | ||
|
|
||
| ### 🍩 Top API Burners (24h) | ||
|
|
||
|  | ||
| 📎 **Chart: Top API Burners** — artifact `{api_burners_donut_aw_id}` available in the [workflow run artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) | ||
|
|
||
| {2–3 sentences: describe which workflows dominate API consumption, their share of the total, and any concentration risk} | ||
|
|
||
| --- | ||
|
|
||
| ### 🔗 GitHub REST API Consumption by Workflow (last 24h) | ||
|
|
||
|  | ||
| 📎 **Chart: GitHub REST API Consumption by Workflow** — artifact `{api_by_workflow_aw_id}` available in the [workflow run artifacts](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) | ||
|
|
||
| {2–3 sentences: identify the top REST API consumers, note any workflows near the 15k/hr limit, and suggest optimisation opportunities} | ||
|
|
||
|
|
||
Large diffs are not rendered by default.
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot skip archive true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 79e559c — added
allow.skip-archive: trueto theupload-artifactconfig.