Skip to content

Commit

Permalink
config diff old/new fix
Browse files Browse the repository at this point in the history
Summary: I wrongly assumed we store new config in a dice. But it is not true. Just swapping two args to make diff calculation correct

Reviewed By: blackm00n

Differential Revision: D58668343

fbshipit-source-id: ab6f05aa8dc088b861057011ee7f23a4fd3195f3
  • Loading branch information
perehonchuk authored and facebook-github-bot committed Jun 17, 2024
1 parent 8c997e2 commit 9528c3d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/buck2_server/src/ctx.rs
Original file line number Diff line number Diff line change
Expand Up @@ -521,7 +521,7 @@ impl CellConfigLoader {
let (new_configs, config_metrics) = if dice_ctx.is_injected_legacy_configs_key_set().await? {
let injected_legacy_configs = dice_ctx.get_injected_legacy_configs().await?;
let root_cell = cells_and_configs.cell_resolver.root_cell();
let diff_data = ConfigDiffMetrics::new(root_cell, &injected_legacy_configs, &cells_and_configs.configs_by_name);
let diff_data = ConfigDiffMetrics::new(root_cell, &cells_and_configs.configs_by_name, &injected_legacy_configs);
(diff_data.has_changed(), Some(diff_data))
} else {
// first invocation of a daemon
Expand Down

0 comments on commit 9528c3d

Please sign in to comment.