Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove application specific properties from Robot CRD #43

Open
oliver-goetz opened this issue Apr 28, 2020 · 2 comments
Open

Remove application specific properties from Robot CRD #43

oliver-goetz opened this issue Apr 28, 2020 · 2 comments

Comments

@oliver-goetz
Copy link
Contributor

Adding application specific properties to Robot CRD could overload this CRD over the time.
We should find a more flexible way to transport those data in the future

Follow up to our discussion in pull request #41

@ensonic
Copy link
Contributor

ensonic commented Apr 29, 2020

We are also discussing to maybe rename the crd to onprem to highlight that the construct is there to support the hub-spoke pattern of a master cluster that had n children associated. Which of the onprem clusters is a robot or eg an onprem cache node can be established by using labels.

Not sure if it makes most sense to also have the robot crd to support your use cases in parallel or what would be a better way to provide the status information instead.

@oliver-goetz
Copy link
Contributor Author

It sounds reasonable to rename the Robot CRD. I have the same understanding of the use case pattern as you have.
I would prefer naming it "edge". This sounds a bit industrial IoT alike. Edge could be in a plant next to a machine, on a robot, in a server room or maybe on am offshore platform.
In our terms "onprem" means something like "not in the cloud but still in an (own) datacenter".

Concerning the question of how to proceed with the robot CRD. I like the idea of sharing information between the clusters. A property like trolleyAttached is probably too specific for a use case, but defining common properties of a robot like status and maybe batteryPercentage could be useful. If there is a new CRD to represent the children it would be safe opening robot CRD for that, because you don't risk screwing up platform functions like AppRollout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants