Code

Helm install script#27

Code

Closed

Fortune-Ndlovu wants to merge 29 commits intoredhat-developer:mainredhat-developer/rhdh-chart:mainfrom Fortune-Ndlovu:helm-install-scriptCopy head branch name to clipboard

+155

Commits

Commits on Jun 18, 2024

• Added helm installation script.

  

  Fortune Ndlovu

  committed
  790122e

  Copy full SHA for 790122e

• Copyright statement and information about the relevant Apache Licence.

  

  Fortune Ndlovu

  committed
  60003cd

  Copy full SHA for 60003cd

• Refractored the function to detect cluster router base: Instead of using a combination of commands get route or get ingress followed by grep to search for a line containing console or default and t…

  Show description for 62317f8
  

  Fortune Ndlovu

  committed
  62317f8

  Copy full SHA for 62317f8

• Added clear instructions and a description of the script, the options available, and examples of usage all using the usage function, this helps keep docs information centralized reducing cognitive …

  Show description for 59965c6
  

  Fortune Ndlovu

  committed
  59965c6

  Copy full SHA for 59965c6

Commits on Jun 19, 2024

• Added --help option to display usage information.

  

  Fortune Ndlovu

  committed
  6e1a1a1

  Copy full SHA for 6e1a1a1

• .sh file name change as noted that the install-helm.sh file makes it sound like it stalls he Helm executable.

  

  Fortune Ndlovu

  committed
  733f5f8

  Copy full SHA for 733f5f8

• Added --cli <oc|kubectl> option to specify which CLI tool to use.

  Show description for 45c1e1f
  

  Fortune Ndlovu

  committed
  45c1e1f

  Copy full SHA for 45c1e1f

• Placed the checks for essential information at the top of the file ensuring that the script exists early if the necessary components are not available. This prevents unnecessary execution of other …

  Show description for 40aa740
  

  Fortune Ndlovu

  committed
  40aa740

  Copy full SHA for 40aa740

• Support for --release-name and --generate-name options: Allows users to specify a custom release name or generate one automatically.

  

  Fortune Ndlovu

  committed
  d4a5a5c

  Copy full SHA for d4a5a5c

Commits on Jun 24, 2024

• Merge branch 'redhat-developer:main' into helm-install-script

  Fortune-Ndlovu

  authored
  aa66059

  Copy full SHA for aa66059

• Using dirname to determine the base directory of the script ensuring that it can be executed from any location on the filesystem. In addition capturing additional arguments and appending extra argu…

  Show description for 8996e31
  Fortune-Ndlovu

  committed
  8996e31

  Copy full SHA for 8996e31

• Removed login check because we already provide feedback if the user is not logged in through Helm and other commands.

  Fortune-Ndlovu

  committed
  7673f4b

  Copy full SHA for 7673f4b

• Updated the detect_cluster_router_base function to use oc get ingress.config.openshift.io/cluster to fetch the correct router base domain on OpenShift.

  Fortune-Ndlovu

  committed
  77042ae

  Copy full SHA for 77042ae

Commits on Jun 25, 2024

• Refactored the script to only handle the CLI Helm installation for OCP.

  Fortune-Ndlovu

  committed
  a990059

  Copy full SHA for a990059

Commits on Jun 28, 2024

• Added install.md file.

  Fortune-Ndlovu

  committed
  bc57eed

  Copy full SHA for bc57eed

• Update install.md

  Fortune-Ndlovu

  authored
  8ba23d9

  Copy full SHA for 8ba23d9

• Update install.md

  Fortune-Ndlovu

  authored
  1b5f427

  Copy full SHA for 1b5f427

Commits on Jul 1, 2024

• Add suggestions

  Fortune-Ndlovu

  committed
  ca84b36

  Copy full SHA for ca84b36

• Update install.md

  Fortune-Ndlovu

  authored
  8b01512

  Copy full SHA for 8b01512

Commits on Jul 8, 2024

• Improve error handling and better practices in setting configuration values

  Fortune-Ndlovu

  committed
  cafbd47

  Copy full SHA for cafbd47

Commits on Jul 16, 2024

• If detect_cluster_router_base fails, we could just exit with the right message.

  Fortune-Ndlovu

  committed
  813fa8b

  Copy full SHA for 813fa8b

• since the ROUTER_BASE is always required and will either be provided by the user or auto-detected, there's no need to add the router base to the Helm arguments conditionally.

  Fortune-Ndlovu

  committed
  a67d111

  Copy full SHA for a67d111

• removing case

  Fortune-Ndlovu

  committed
  2317626

  Copy full SHA for 2317626

• adding space

  Fortune-Ndlovu

  committed
  905e1ae

  Copy full SHA for 905e1ae

• Not needed to specify the path to the Chart default values.yaml file.

  Fortune-Ndlovu

  committed
  401589f

  Copy full SHA for 401589f

• Trapping for each condition separately, and then providing a call to action for the user so they know where to get oc or helm to install it.

  Fortune-Ndlovu

  committed
  41bf45b

  Copy full SHA for 41bf45b

Commits on Jul 18, 2024

• List all Helm releases in the specified name space

  Fortune-Ndlovu

  committed
  dd8c73e

  Copy full SHA for dd8c73e

Commits on Jul 19, 2024

• Adding the -i flag to install the chart if it doesnt exists, or upgrade it if it does. And removing the md file because the instructions are present in the usage function of the script.

  Fortune-Ndlovu

  committed
  cfa136f

  Copy full SHA for cfa136f

Commits on Jul 22, 2024

• Testing script

  Fortune-Ndlovu

  committed
  595db25

  Copy full SHA for 595db25