SAP / ewm-cloud-robotics

Source code, containers & Helm charts enabling users to leverage the core of Google Cloud Robotics to automate fulfilment warehouse orders & tasks commissioned by SAP EWM
Apache License 2.0
32 stars 18 forks source link

Install process dummy robots #37

Open JS2103 opened 2 years ago

JS2103 commented 2 years ago

Hey, I'm currently working on my Cloud Robotics-Cluster and have problems to install the dummy robots since the RobotType is removed from the Registry API. (https://github.com/googlecloudrobotics/core/commit/fc578d673fbe758fa943c8f8fc71cee6cb09ef5d) Is there an easy way to fix the problem by myself, e.g. change the helm template?

Best regards and thanks, Jan

drigz commented 2 years ago

Hi Jan, sorry for the breakage here! I wasn't aware that the RobotType was being used as it's spec still said something like "TODO: add useful features". I've reverted that change, perhaps it's enough for you to redeploy Cloud Robotics.

Are you following the https://googlecloudrobotics.github.io/core/how-to/deploy-from-sources flow or are you using binary releases?

JS2103 commented 2 years ago

Thank you for the fast reply and solution. I used the Quickstart instructions, which uses the binary releases. Please correct me if I'm wrong. I will give it a try tomorrow.

drigz commented 2 years ago

OK, good to know. If you encounter similar problems in future, you can try the deploy-from-sources approach to work around it (the ewm-cloud-robotics docs link to this) until it's properly fixed, as that lets you either deploy an older version or make changes before deploying.

oliver-goetz commented 2 years ago

@JS2103 I will remove the references to RobotType soon. They are not used anyhow.

In the meantime you could remove the robot type from dummy robot's helm chart: here and here