[Access] Fixed public network execution data service component

[UlyanaAndrukhiv]

Issue

The current state of the ANs contains an issue related to the creation of the "public network execution data service" component.
Error: panic: component may only be started once

Context

This pull request addresses the problem by implementing necessary fixes to the component creation process.

[peterargue]

what issue did you see with the existing code? Can you add a test to catch it?

[UlyanaAndrukhiv]

what issue did you see with the existing code? Can you add a test to catch it?

I added "public-network-execution-data-sync-enabled" flag to integration test to catch it and I see panic: component may only be started once error for public network execution data service component. It happens cause RegisterBlobService already starts the component, see here

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ae8ab05) 55.95% compared to head (6cabb8a) 55.76%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5375      +/-   ##
==========================================
- Coverage   55.95%   55.76%   -0.20%     
==========================================
  Files        1023     1012      -11     
  Lines       98930    98219     -711     
==========================================
- Hits        55356    54770     -586     
+ Misses      39368    39268     -100     
+ Partials     4206     4181      -25     

Flag	Coverage Δ
unittests	55.76% <ø> (-0.20%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[peterargue]

I added "public-network-execution-data-sync-enabled" flag to integration test to catch it

I don't see this change in your PR? Can you add a test validating that execution sync works as expected between an AN/ON with this setting is enabled? for instance, you could create an ON version of execution_state_sync_test.go or extend it to include observers as well

[janezpodhostnik]

This bug also affects the automated load testing.
The access note fails on startup due to "component may only be started once".
I have confirmed its related to "-public-network-execution-data-sync-enabled=true". If i disable it everything works as expected.

[Guitarheroua]

Looks good for me