Skip to content

feat(demo): Add an ANALYZER=tango option to the local demo - #919

Open
manjari25 wants to merge 1 commit into
manjari/tango-schema-cifrom
manjari/tango-demo-analyzer
Open

manjari25 wants to merge 1 commit into
manjari/tango-schema-cifrom
manjari/tango-demo-analyzer

Conversation

@manjari25

Copy link
Copy Markdown
Contributor

Why?

The Tango analyzer, its fake server and its schema are in place, but nothing in the local demo uses them. This adds an option to run the demo with it, so conflicts between batches can be seen following a build-target graph instead of directory overlap.

What?

  • make local-submitqueue-start PROVIDER=git ANALYZER=tango swaps demo-queue's analyzer for Tango and runs the fake Tango server beside the stack. Stop with the same variables. Only PROVIDER=git supports it, because the fake reads targets from that provider's repository.
  • The sandbox gets a TARGET file per demo/NN folder, committed on top of whatever is there. demo/00 and demo/01 are libraries, and even folders depend on demo/00 and odd folders on demo/01. Changes in different folders can now conflict.
  • make demo-requests refuses a FOLDERS above 10 against a sandbox seeded this way, since only demo/00 to demo/09 have targets and changes elsewhere would never conflict.
  • QUICKSTART.md and the provider README describe the option

Test Plan

  • make local-submitqueue-start PROVIDER=git ANALYZER=tango, then make demo-requests PROVIDER=git COUNT=8 FOLDERS=10.
  • The target_signature table in the application database has a row per batch.
  • batch_dependent links changes in different folders that depend on the same library.
  • FOLDERS=12 fails right away with an error.
  • make local-submitqueue-stop PROVIDER=git ANALYZER=tango removes the stack, including the fake server.
  • Unit tests
  • CI

Issue

@manjari25
manjari25 added this pull request to stack #912 October 22, 2026 06:07
@manjari25
manjari25 marked this pull request as ready for review October 22, 2026 06:10
@manjari25
manjari25 requested review from a team, behinddwalls and sbalabanov as code owners October 22, 2026 06:10
@manjari25
manjari25 force-pushed the manjari/tango-demo-analyzer branch from 6617bd1 to 9b7ea47 Compare October 22, 2026 19:04

This branch has not been deployed

No deployments
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.

1 participant