Updated HTTPRoute for ToysStore with new label.

[david-martin]

• Added a "deployment" label to the HTTPRoute resource so it can be tied to istio metrics

i.e. the destination_workload and destination_service_name labels on the istio_requests_total metric

[codecov]

Codecov Report

Merging #524 (7c8e80c) into main (ece13e8) will increase coverage by 0.30%.
Report is 24 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #524      +/-   ##
==========================================
+ Coverage   80.20%   80.51%   +0.30%     
==========================================
  Files          64       70       +6     
  Lines        4492     4969     +477     
==========================================
+ Hits         3603     4001     +398     
- Misses        600      651      +51     
- Partials      289      317      +28     

Flag	Coverage Δ
integration	71.38% <ø> (+0.10%)	⬆️
unit	33.43% <ø> (+3.40%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
api/v1beta1 (u)	71.42% <ø> (ø)
api/v1beta2 (u)	88.51% <100.00%> (-2.92%)	⬇️
pkg/common (u)	83.01% <ø> (-5.81%)	⬇️
pkg/istio (u)	75.14% <100.00%> (+1.23%)	⬆️
pkg/log (u)	94.73% <ø> (ø)
pkg/reconcilers (u)	∅ <ø> (∅)
pkg/rlptools (u)	79.77% <100.00%> (+0.31%)	⬆️
controllers (i)	76.31% <82.44%> (-0.49%)	⬇️

see 10 files with indirect coverage changes

[jasonmadigan]

LGTM

[david-martin]

@jasonmadigan added a service label as well for matching with destination_service_name so aggregating can be done per httproute (which isn't possible with destination_workload in cases of 429s where it's set to unknown)