sentry icon indicating copy to clipboard operation
sentry copied to clipboard

chore(cells) Switch ReactView and GenericReactView to control silo

Open markstory opened this issue 1 month ago • 2 comments

We render the bulk of our HTML via ReactPage. Right now that endpoint is marked as all_silo to make integration with tests simpler. Switch these endpoints to control as we're trying to have a minimal number of endpoints marked as all_silo and I want to better understand the scope of test failures from this change.

Refs INFRENG-238

markstory avatar Dec 08 '25 22:12 markstory

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104552      +/-   ##
===========================================
+ Coverage   80.52%    80.54%   +0.02%     
===========================================
  Files        9352      9354       +2     
  Lines      400460    400945     +485     
  Branches    25696     25696              
===========================================
+ Hits       322477    322952     +475     
- Misses      77533     77543      +10     
  Partials      450       450              

codecov[bot] avatar Dec 08 '25 22:12 codecov[bot]

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

github-actions[bot] avatar Dec 09 '25 22:12 github-actions[bot]