VIP use in OIM HA environment #2628
Conversation
|
Can one of the admins verify this patch? |
| hosts: oim | ||
| connection: ssh | ||
| gather_facts: false | ||
| tags: deploy_containers |
There was a problem hiding this comment.
why this flow has been changed? if this is moved resources will create early and containers will be restarted? are we splitting resources and pcs container deployment separately?
There was a problem hiding this comment.
The containers are already deployed at this point, so deploying the pcs container before the pulp config allows us to use the VIP with the pulp configuration. As of right now, I kept all resources together since I think with this flow we don't need to split.
| @@ -18,12 +18,13 @@ nodelist { | |||
| } | |||
| {% if hostvars['localhost']['oim_ha_data'].ansible_facts.oim_ha.enable_oim_ha %} | |||
| {%- for node_details in hostvars['localhost']['oim_ha_data'].ansible_facts.oim_ha.passive_nodes.node_details %} | |||
| {% if node_details.ADMIN_IP is defined %} | |||
There was a problem hiding this comment.
why admin_ip is capital here?
There was a problem hiding this comment.
The definition in the high_availability_config is capitalized:
node_details:
- {SERVICE_TAG : ",HOSTNAME: "",ADMIN_MAC: "",ADMIN_IP: "",BMC_IP: ""}
| images_directory_path: "{{ oim_shared_path }}/omnia/images" | ||
| read_write_mode: "0644" | ||
| container_tag: "latest" | ||
| container_tar_file: "omnia_images.tar" |
There was a problem hiding this comment.
all container images will be stored in single tar file? we need to check omnia_kubespray based on k8s check in software_config.json
There was a problem hiding this comment.
Yes, with podman 2.2 they support "-m" for multi-image saves. Will add the check for kubespray.
| - name: Set OIM admin IP to virtual IP if OIM HA is enabled | ||
| when: enable_oim_ha | ||
| ansible.builtin.set_fact: | ||
| network_data: "{{ network_data | combine(update_data, recursive=true) }}" |
There was a problem hiding this comment.
instead of overriding the input value override admin ip and update the place using the fqdn path with admin_ip
|
|
||
| # Usage: omnia_pcs_setup.js | ||
| container_tar_file: "omnia_images.tar" | ||
| container_images_path: "{{ oim_shared_path }}/omnia/images/{{ container_tar_file }}" |
There was a problem hiding this comment.
Update images folder in omnia_startup.sh cleanup flow
| state: absent | ||
|
|
||
| - name: Save container images | ||
| ansible.builtin.command: "podman save -m -o {{ images_directory_path }}/{{ container_tar_file }} {{ container_images }}" |
There was a problem hiding this comment.
Description of the Solution
Utilizing the VIP as the admin IP when OIM HA is enabled.
Suggested Reviewers
If you wish to suggest specific reviewers for this solution, please include them in this section. Be sure to include the @ before the GitHub username.
@priti-parate
@abhishek-sa1