yugabyte / yugabyte-db

YugabyteDB - the cloud native distributed SQL database for mission-critical applications.
https://www.yugabyte.com
Other
8.92k stars 1.06k forks source link

[Docs] Deploy YugabyteDB in Google Cloud Platform with GCP Deployment Manager #19760

Open yogendra opened 11 months ago

yogendra commented 11 months ago

Description

This is broken. It does not deploy the database and fails in downloading step.

Main cause of failure is usage/invocation of the install_script.sh

Line with issue - https://github.com/yugabyte/gcp-deployment-manager/blob/master/script/startup-script.sh#L11

This common_scripts/install_software.sh script expects a YB_RELEASE value, i.e. YB_VERSION+ '-b' (Example: 2.19.1.0-b10). The other script scripts/install_software.sh expects YB_VERSION and finds the latest corresponding YB_RELEASE using docker hub image tags

There are 3 options to resolve this:

  1. Enhance or replace common_scripts/install_software.sh with scripts/install_software.sh. I would recommend this, as this script is referred in multiple automations (gcp deployment manager/terraform, aws cloud formation/terraform, etc.)
  2. Replace usage of common_scripts/install_software.sh with scripts/install_software.sh
  3. Replace parameter to be YB_RELEASE

Warning: Please confirm that this issue does not contain any sensitive information

yogendra commented 11 months ago

Upon additional inspection here is what I have observed.

Working Cloud Method Scripts Parameter Value Format Link
AWS TF scripts/install_software.sh YB_VERSION link
AWS Cloud Formation common_scripts/install_software.sh YB_RELEASE link
GCP TF scripts/install_software.sh YB_VERSION link
🚫 GCP DM common_scripts/install_software.sh YB_VERSION link
Azure TF scripts/install_software.sh YB_VERSION link
Azure ARM common_scripts/install_software.sh YB_RELEASE link
ddhodge commented 8 months ago

@iSignal Not entirely a docs issue, who can TAL?