Customize query in gerrit
Details
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | jeena | T224935 Move local-charts helm charts to a chart repository | |||
| Resolved | jeena | T228910 Move restbase chart from local-charts to deployment-charts repository |
Event Timeline
Comment Actions
As I wrote on the patch, I wonder if it's a good idea to put RESTBase in deployment-charts given that it will never be put into k8s, and given that we already have restrouter in here I find it confusing. @akosiaris @Joe thoughts?
Comment Actions
Basically, once RESTRouter is fully functional, RESTBase itself will represent only the storage API layer, so to speak. We will have two options at that point: (i) keep using it; or (ii) use kask instead. In the first case, IMHO it's better to keep the service as close to the storage as possible, which would mean keep RB where it is, while in the latter case RB goes away completely.
Comment Actions
Will RESTBase be developed and deployed from the same repo as RESTRouter? If that's the case, then (i) AND RB staying on the cassandra nodes will mean diverging methods of deployment, possibly divergent nodejs versions and base OS, with all the consequences of that (like having to upgrade cassandra nodes in tandem). Granted it will be possible to keep them in sync, but it will come at a cost which it might or might not make sense to pay. The 2 alternatives, i.e. (ii) and keeping RESTBase but moving it to kubernetes as well, don't suffer from the above, albeit they have their own costs (e.g. making sure kask is able functionally to perform RB's role, or some increased latency, although probably in the single digit ms range). It's a number of things that will have to be weighed I guess.
Comment Actions
Change 517557 had a related patch set uploaded (by Jeena Huneidi; owner: Jeena Huneidi):
[operations/deployment-charts@master] Add restbase chart (port from local-charts)
Comment Actions
Change 517557 abandoned by Jeena Huneidi:
Add restbase chart (port from local-charts)
Reason:
Abandoning in favor of making some small changes to the RESTRouter chart to support dev usage.
Comment Actions
Change 545421 had a related patch set uploaded (by Jeena Huneidi; owner: Jeena Huneidi):
[operations/deployment-charts@master] Modify Restrouter chart to allow for minikube development
Comment Actions
Change 545421 abandoned by Jeena Huneidi:
Modify Restrouter chart to allow for minikube development
Reason:
Abandoning due to https://phabricator.wikimedia.org/T242461
Comment Actions
Restrouter is no longer in k8s in production and according to https://phabricator.wikimedia.org/T242461 might go away. Until that happens, we decided to keep the restbase chart in local-charts and update it to restrouter due to being able to merge and iterate more quickly.
Comment Actions
Change 570236 had a related patch set uploaded (by Jeena Huneidi; owner: Jeena Huneidi):
[releng/local-charts@master] Remove restbase chart and add restrouter chart
Comment Actions
Change 571601 had a related patch set uploaded (by Brennen Bearnes; owner: Brennen Bearnes):
[releng/dev-images@master] new RESTBase image & improved build instructions
Comment Actions
Change 571601 merged by Jeena Huneidi:
[releng/dev-images@master] new RESTBase image & improved build instructions
Comment Actions
Mentioned in SAL (#wikimedia-releng) [2020-02-12T01:24:36Z] <brennen> Updating dev-images docker-pkg files on contint1001 for T228910
Comment Actions
Change 570236 merged by Jeena Huneidi:
[releng/local-charts@master] Remove restbase chart and add restrouter chart