Skip to content

fix: CI Updates #18987

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.

Sign up for GitHub

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

Jump to bottom
Merged
AztecBot merged 1 commit into from
Jan 5, 2026
Merged

fix: CI Updates #18987

AztecBot merged 1 commit into from
Jan 5, 2026

Conversation

@charlielye
Copy link
Contributor

@charlielye charlielye commented Dec 11, 2025 •
edited
Loading

  • Brings the ci dashboard code from iac repo to ci3/dashboard.
  • Attempt to fix cleanup on makefile build path. We used setsid but think correct approach is to do a background subshell () &
  • Fix PR's not getting RUN_ID's.
  • We explicitly name the dashboard CI runs are shown on with CI_DASHBOARD.
    • <merge queue target>
    • prs
    • nightly
    • network
  • Add ci dash command so you can see your CI runs direct in terminal.

ludamad
ludamad reviewed Dec 11, 2025
View reviewed changes
ci3/dashboard/rk.py

def section_view(section: str) -> str:
offset = int(request.args.get('offset', 0))
limit = int(request.args.get('limit', 100))
Copy link
Collaborator

@ludamad ludamad Dec 11, 2025

Choose a reason for hiding this comment

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

does this support http://ci.aztec-labs.com/chonk-breakdowns?

@socket-security
Copy link

socket-security bot commented Dec 12, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addedpypi/​gunicorn@​23.0.098100100100100
Addedpypi/​ansi2html@​1.9.210010010010070
Addedpypi/​flask@​3.1.298100100100100
Addedpypi/​redis@​7.1.099100100100100
Addedpypi/​requests@​2.32.599100100100100
Addedpypi/​flask-httpauth@​4.8.0100100100100100
Addedpypi/​flask-compress@​1.23100100100100100

View full report

@socket-security
Copy link

socket-security bot commented Dec 12, 2025 •
edited
Loading

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Medium
Native binaries present: pypi packaging

Location: Package overview

From: ? → pypi/[email protected] → pypi/[email protected]

ℹ Read more on: This package | This alert | Why is native code a concern?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at [email protected].

Suggestion: Verify that the inclusion of native code is expected and necessary for this package's functionality. If it is unnecessary or unexpected, consider using alternative packages without native code to mitigate potential risks.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore pypi/[email protected]. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@charlielye charlielye requested a review from ludamad January 2, 2026 12:58
@AztecBot AztecBot enabled auto-merge January 2, 2026 13:02
@charlielye
fix: CI Updates
ac31bd5 
* Brings the ci dashboard code from `iac` repo to `ci3/dashboard`.
* Attempt to fix cleanup on makefile build path. We used setsid but think correct approach is to do a background subshell `() &`
* Fix PR's not getting RUN_ID's.
* We explicitly name the dashboard CI runs are shown on with CI_DASHBOARD.
  * `<merge queue target>`
  * `prs`
  * `nightly`
  * `network`
*  Add `ci dash` command so you can see your CI runs direct in terminal.
@AztecBot AztecBot added this pull request to the merge queue Jan 5, 2026
@AztecBot
Copy link
Collaborator

AztecBot commented Jan 5, 2026 •
edited
Loading

Flakey Tests

🤖 says: This CI run detected 2 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (\0338;;http://ci.aztec-labs.com/3baa740e16e1228a�3baa740e16e1228a8;;�\033): barretenberg/acir_tests/scripts/browser_prove.sh a_1_mul webkit (25s) (code: 1) (\033Aztec Bot\033: fix: CI Updates (#18987))
\033FLAKED\033 (\0338;;http://ci.aztec-labs.com/da9ae4f3d566f6a8�da9ae4f3d566f6a88;;�\033):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/add_rollup.test.ts (192s) (code: 1) group:e2e-p2p-epoch-flakes (\033Aztec Bot\033: fix: CI Updates (#18987))

Merged via the queue into next with commit 5dd5cf6 Jan 5, 2026
16 checks passed
@AztecBot AztecBot deleted the cl/ci_updates branch January 5, 2026 17:19
johnathan79717 added a commit that referenced this pull request Jan 7, 2026
@johnathan79717
fix: add missing json import to chonk-breakdowns dashboard
fef66cf 
PR #18987 moved the CI dashboard from iac repo to aztec-packages but
the json import was missing, causing 500 errors on the flows API.
@johnathan79717 johnathan79717 mentioned this pull request Jan 7, 2026
Merged
johnathan79717 added a commit that referenced this pull request Jan 7, 2026
@johnathan79717
fix: add missing json import to chonk-breakdowns dashboard
187c6ac 
PR #18987 moved the CI dashboard from iac repo to aztec-packages but
the json import was missing, causing 500 errors on the flows API.
johnathan79717 added a commit that referenced this pull request Jan 7, 2026
@johnathan79717
fix: add missing json import to chonk-breakdowns dashboard (#19390)
9d0c168 
## Summary
- Adds missing `import json` to the chonk-breakdowns dashboard

PR #18987 moved the CI dashboard from iac repo to aztec-packages but the
json import was missing, causing 500 errors on the
`/api/breakdown/flows` endpoint.
github-actions bot pushed a commit that referenced this pull request Jan 8, 2026
@johnathan79717
fix: add missing json import to chonk-breakdowns dashboard (#19390)
b749f25 
## Summary
- Adds missing `import json` to the chonk-breakdowns dashboard

PR #18987 moved the CI dashboard from iac repo to aztec-packages but the
json import was missing, causing 500 errors on the
`/api/breakdown/flows` endpoint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ludamad ludamad ludamad approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@charlielye @AztecBot @ludamad