From 652e41d93456b52bff533ef2602cbe87faad9a50 Mon Sep 17 00:00:00 2001 From: Bartek Grzybowski Date: Mon, 4 Mar 2019 15:34:19 +0100 Subject: Refine application_pre_install_role variable usage As 'application_pre_install_role' variable set in config/application_configuration.yml must stay in sync with patch role directory name defined in package.conf under APP_CONFIGURATION variable, it's name shall stay under complete user control without any variable as part of it's name. App_helm_build_targets list and app_helm_chart_name should default to 'onap' in application_configuration.yml as customizing them isn't supported at the moment. Also fixed some wording in comments/docs in relevant files. Change-Id: I82b6dd519fbc3426739a11fac1e02eb495a668c0 Issue-ID: OOM-1697 Signed-off-by: Bartek Grzybowski --- ansible/.gitignore | 1 + ansible/application/README.md | 15 +++++++++------ ansible/group_vars/all.yml | 4 ++-- 3 files changed, 12 insertions(+), 8 deletions(-) (limited to 'ansible') diff --git a/ansible/.gitignore b/ansible/.gitignore index 521beaa8..e5505c9e 100644 --- a/ansible/.gitignore +++ b/ansible/.gitignore @@ -1,2 +1,3 @@ ansible_chroot application/* +certs/ diff --git a/ansible/application/README.md b/ansible/application/README.md index d260b3cb..36f69bd5 100644 --- a/ansible/application/README.md +++ b/ansible/application/README.md @@ -6,9 +6,10 @@ also on target server where package is installed. ## Application configuration -All application related configuration variables are defined in file -`application_configuration.yml` in this folder. The name of configuration file -does not matter but it must be given to ansible run as command line variable file. +All application related configuration variables are defined in the +`application_configuration.yml` file in this folder. The name of the configuration +file can be altered but it must be passed to ansible run as command line +variable file nevertheless. Example: ``` @@ -26,12 +27,14 @@ Helm charts transfer from packaging up to the target infra server. ## Application specific roles Installer supports optional custom pre and post install roles. Custom roles' code folders -are placed to this directory at packaging time and name of those folders are configured in -application.yml with variable `application_pre_install_role` and `application_post_install_role`. +are placed into this directory at packaging time and names of those folders shall be configured in +application_configuration.yml with variable `application_pre_install_role` and `application_post_install_role`. +Note that these directory names must correspond to those configured in APP_CONFIGURATION inside package.conf +during package build time. Example: ``` -application_pre_install_role: "{{ app_name }}-patch-role" +application_pre_install_role: "my-pre-install-role" ``` ## Inventory hosts diff --git a/ansible/group_vars/all.yml b/ansible/group_vars/all.yml index dc40238c..85363951 100755 --- a/ansible/group_vars/all.yml +++ b/ansible/group_vars/all.yml @@ -113,7 +113,7 @@ app_helm_charts_infra_directory: "{{ app_data_path }}/helm_charts" # Main Helm chart to install # e.g. app_helm_chart_name: onap -app_helm_chart_name: "{{ app_name }}" +app_helm_chart_name: # Targets for helm charts repository build # e.g. for ONAP Casablanca @@ -137,7 +137,7 @@ app_kubernetes_namespace: "{{ app_name }}" # Optional application custom Ansible roles name for pre and post install logic. # Location of additional custom roles is defined in ansible.cfg with roles_path. -# e.g. application_pre_install_role: "{{ app_name }}-patch-role" +# e.g. application_pre_install_role: "my-pre-install-role" application_pre_install_role: application_post_install_role: -- cgit 1.2.3-korg