Skip to content

Add DASH/DPU/SmartSwitch ecosystem review system with multi-repo tracking#694

Draft
Copilot wants to merge 29 commits intomainfrom
copilot/update-project-view-summary
Draft

Add DASH/DPU/SmartSwitch ecosystem review system with multi-repo tracking#694
Copilot wants to merge 29 commits intomainfrom
copilot/update-project-view-summary

Conversation

Copy link

Copilot AI commented Jan 29, 2026

Implements comprehensive review system for tracking DASH/DPU/SmartSwitch activity across 11+ SONiC repositories. Includes automated report generation, PR activity analysis, and trend visualization.

Review Documents

Latest Report (Jan 29 - Feb 18, 2026)

  • DASH_DPU_SMARTSWITCH_FILTERED_REVIEW_2026-01-29_to_2026-02-18.md - Complete analysis: 41 PRs created, 23 merged
  • REVIEW_SUMMARY_2026-01-29_to_2026-02-18.md - Executive summary with key metrics
  • Keyword filtering: DPU OR DASH OR SmartSwitch OR Smart in title/body
  • Excludes: [action] and submodule tags

Additional Reports

  • Original Jan 15-27 review for comparison
  • DASH ecosystem-only view (3 core repos)
  • Full Azure/SONiC project view (13+ repos)

Automation

Scripts (/scripts)

  • generate_weekly_review.sh / .py - Core generation logic
  • generate_weekly_review_enhanced.sh - With PR activity metrics
  • generate_trend_chart.sh - Week-over-week merge analysis

GitHub Actions (.github/workflows/weekly-review.yml)

  • Schedule: Every Tuesday 2 PM UTC (0 14 * * 2)
  • Auto-generates filtered reviews
  • Creates PR with updated reports

Configuration

DATE_START="2026-01-29"
DATE_END="2026-02-18"
KEYWORDS="DPU OR DASH OR SmartSwitch OR Smart"
EXCLUDE="NOT [action] NOT submodule in:title"

Report Features

PR Analysis

  • Separate sections for Created vs Merged PRs
  • Repository distribution with activity levels
  • Complete PR tables with titles, dates, keywords matched

Activity Metrics (when enabled)

  • Comment counts per PR
  • Review status and approvals
  • Label additions/changes
  • High-engagement vs quick-merge patterns

Trend Analysis (example data)

  • Week-over-week merge velocity
  • ASCII charts for visualization
  • Statistical summaries (avg, peak, trend direction)

Repository Health

  • 11 repositories tracked: DASH, sonic-buildimage, sonic-mgmt, sonic-swss, etc.
  • Consistent "Relevant Items" column showing actual PR counts
  • Activity level indicators (🟢 High, 🟡 Moderate, 🔴 Low)

Navigation

REVIEW_DOCUMENTS_INDEX.md - Central index with quick access to all reports
EXPANDED_REVIEW_README.md - Getting started guide for latest expanded review
WEEKLY_REVIEW_GUIDE.md - How to run automation scripts

Query Pattern

All data sourced from GitHub Search API matching user's query pattern:

org:sonic-net is:pr created:2026-01-29..2026-02-18 
(DPU OR DASH OR SmartSwitch OR Smart) in:title,body 
NOT [action] NOT submodule in:title

Trend chart data marked as example until real historical data is collected.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh auth status (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Can you do a review of this project view and provide a summary update pertaining to the changes I made this week, and an overall view of progress?


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Copilot AI changed the title [WIP] Update project view summary with recent changes Add comprehensive project review and progress analysis Jan 29, 2026
Copilot AI requested a review from KrisNey-MSFT January 29, 2026 20:38
Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Copilot AI changed the title Add comprehensive project review and progress analysis Add multi-repository ecosystem analysis for DASH project Jan 29, 2026
Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Copilot AI changed the title Add cross-repo DASH/DPU/SmartSwitch activity tracking with PR engagement metrics Add week-over-week PR merge velocity trend visualization Jan 30, 2026
Copilot stopped work on behalf of KrisNey-MSFT due to an error February 2, 2026 23:03
…'Smart'

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

…fication steps

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Copilot AI changed the title Add week-over-week PR merge velocity trend visualization Fix trend chart: replace sample data with generation instructions and correct keywords Feb 2, 2026
Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Co-authored-by: KrisNey-MSFT <22226040+KrisNey-MSFT@users.noreply.github.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 694 in repo sonic-net/DASH

Copilot AI changed the title Fix trend chart: replace sample data with generation instructions and correct keywords Add DASH/DPU/SmartSwitch ecosystem review system with multi-repo tracking Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments