Services detail page now shows the ingressess related to it

[marcosdiez]

requires #6664

[codecov]

Codecov Report

Merging #6658 (913b034) into master (31ccdcb) will increase coverage by 0.19%.
The diff coverage is 0.00%.

❗ Current head 913b034 differs from pull request most recent head 15a4804. Consider uploading reports for the commit 15a4804 to get more accurate results

@@            Coverage Diff             @@
##           master    #6658      +/-   ##
==========================================
+ Coverage   41.85%   42.05%   +0.19%     
==========================================
  Files          44       44              
  Lines        1240     1227      -13     
  Branches      179      161      -18     
==========================================
- Hits          519      516       -3     
- Misses        682      711      +29     
+ Partials       39        0      -39     

[marcosdiez]

All changes made. Thank you for the code review, @floreks !

[floreks]

I guess we'll have to wait first for the API version migration PR.

[marcosdiez]

It's here: #6664 @floreks

[floreks]

@marcosdiez check my comment there 🙂

[floreks]

If you would just fix the static check issue we can merge this.

[floreks]

Thanks!

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: floreks, marcosdiez

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [floreks]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment