Skip to content

Add new checkbox for optional data in service ping

What does this MR do and why?

This MR adds new checkbox Include Optional data in Service Ping under Metrics and profiling.

More information - #428622 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Without Service Ping :

Before After
Screenshot_2024-02-06_at_9.29.17_AM Screenshot_2024-02-06_at_9.26.50_AM
Screenshot_2024-02-06_at_9.29.45_AM Screenshot_2024-02-06_at_9.27.24_AM
Screenshot_2024-02-06_at_9.27.45_AM

With Service Ping :

Before After
Screenshot_2024-02-06_at_9.31.36_AM Screenshot_2024-02-06_at_9.32.32_AM

How to set up and validate locally

There are many permutations needs to be verified before it gets merged. Here is the sheet to reference all the use cases.

Verify below things:

  • This change is only for EE. It should not affect to CE in any way. So, we will not be showing this new checkbox in CE.

  • If operational license is true, Enable Service Ping checkbox will be disabled.

  • If operational license if false, Enable Service Ping checkbox will be shown. Also Include Optional data in Service Ping will be disabled, If Enable Service Ping is unchecked.

  • Another checkbox Enable Registration Features which used to be dependent on Enable Service Ping, It will now be dependent on this new checkbox. Reference - #428622 (comment 1720672567)

Related to #428622 (closed)

Edited by Ankit Panchal

Merge request reports