resty: default appstore_service_name to helx-appstore

[joshua-seals]

Full disclosure, I'm back and forth on whether a default is worth having - this addresses an inconsistency I observed in deploying the de-ambassador changes though.

The de-Ambassador resty config resolves static upstreams (proxy_pass without an nginx variable: /private-route, /auth, /) at startup and refuses to boot if one is unresolvable. The umbrella deploys with Release.Name=helx, so the appstore Service is "helx-appstore", but the default was plain "appstore" -> "host not found in upstream" CrashLoopBackOff. Match the ui_service_name: helx-ui convention so a stock helm install works without a --set override.

Chart version 4.5.5 -> 4.5.6.

[frostyfan109]

It's still a somewhat common pattern to deploy each chart required for helx individually. I typically prefer this since it allows you to easily install local versions of the chart without having to package them into a local version of the helx chart. So you could do something like if Chart.IsRoot then appstore else .Release.Name-appstore

[joshua-seals]

It's still a somewhat common pattern to deploy each chart required for helx individually. I typically prefer this since it allows you to easily install local versions of the chart without having to package them into a local version of the helx chart. So you could do something like if Chart.IsRoot then appstore else .Release.Name-appstore

I also prefer that individual deployment pattern - so this is not super critical unless using the helx-chart. All that to say, this should be very different with the monorepo project. All these changes will eventually change again in that overarching helm.