newrelic / nr1-pathpoint

Pathpoint is an enterprise platform tracker that models system health in relation to actual user-impacting business stages.
Apache License 2.0
8 stars 19 forks source link

Don't Store Demo Config in Nerstore #58

Open JimHagan opened 1 year ago

JimHagan commented 1 year ago

When pathpoint is unable to retrieve a config from Nerdstore (or obtain the "last save version") it reverts to the "demo" config that is actually bundled with the nerdlet code. This is okay, BUT we don't need to save that as a stored config, we should just keep loading it from the bundle. When a user actually does a forced save of the stages, steps, or touchpoint editors it will then update the nerdstore.

Why do this? There can be some cases (or it is surmized) that Pathpoint could have a transient issue reading the config from Nerstore and it could cause pathpoint to wipeout a good config with a demo config.

@Sandeep10parmar

rsamanez commented 1 year ago

I believe that by correcting the 59 issue, this would no longer be a problem, since it is easy to recover the configuration from the last saved configuration. I will check this condition making the 59 issue correction.