Skip to content

Commit 7624630

Browse files
committed
Make config directories consistent across services
Some services used edpm_container_standalone and others directly use edpm_container_manage and the way we generate the config files and use them is spread across multiple locations which is confusing and difficult to troubleshoot. This sanitizes the locations. This is required to identify all the services deployed vs to be deployed and remove orphaned containers/services. jira: https://issues.redhat.com/browse/OSPRH-19243 Signed-off-by: rabi <[email protected]>
1 parent e305edf commit 7624630

File tree

206 files changed

+677
-819
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

206 files changed

+677
-819
lines changed

docs/source/roles/role-edpm_container_manage.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ This Ansible role allows to do the following tasks:
6969
include_role:
7070
name: edpm_container_manage
7171
vars:
72-
edpm_container_manage_config: "/var/lib/edpm-config/container-startup-config/step_1"
72+
edpm_container_manage_config: "/var/lib/openstack/container-startup-config/step_1"
7373
edpm_container_manage_config_id: "edpm_step1"
7474
7575
@@ -141,7 +141,7 @@ overrides the image setting in one-off.
141141
name: edpm_container_manage
142142
vars:
143143
edpm_container_manage_config_patterns: 'haproxy.json'
144-
edpm_container_manage_config: "/var/lib/edpm-config/container-startup-config/step_1"
144+
edpm_container_manage_config: "/var/lib/openstack/container-startup-config/step_1"
145145
edpm_container_manage_config_id: "edpm_step1"
146146
edpm_container_manage_clean_orphans: false
147147
edpm_container_manage_config_overrides:

plugins/modules/container_config_data.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,10 @@
7272
EXAMPLES = """
7373
- name: Generate containers configs data
7474
container_config_data:
75-
config_path: /var/lib/edpm-config/container-startup-config/step_1
75+
config_path: /var/lib/openstack/container-startup-config/step_1
7676
- name: Generate containers configs data for HAproxy and override image
7777
container_config_data:
78-
config_path: /var/lib/edpm-config/container-startup-config/step_1
78+
config_path: /var/lib/openstack/container-startup-config/step_1
7979
config_pattern: 'haproxy.json'
8080
config_overrides:
8181
haproxy:

plugins/modules/container_config_hash.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,15 +55,15 @@
5555
description:
5656
- Config volume prefix
5757
type: str
58-
default: '/var/lib/config-data'
58+
default: '/var/lib/openstack'
5959
"""
6060

6161
EXAMPLES = """
6262
- name: Update config hashes for container startup configs
6363
container_config_hash:
6464
"""
6565

66-
CONTAINER_STARTUP_CONFIG = '/var/lib/edpm-config/container-startup-config'
66+
CONTAINER_STARTUP_CONFIG = '/var/lib/openstack/container-startup-config'
6767
BUF_SIZE = 65536
6868

6969

plugins/modules/edpm_container_manage.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@
9292
- name: Run containers
9393
edpm_container_manage:
9494
config_id: edpm_step1
95-
config_dir: /var/lib/edpm-config/container-startup-config/step_1
95+
config_dir: /var/lib/openstack/container-startup-config/step_1
9696
"""
9797

9898

plugins/modules/edpm_nftables_from_files.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
- name: Get nftables rules
5151
register: edpm_nftables_rules
5252
edpm_nftables_from_files:
53-
src: /var/lib/edpm-config/firewall
53+
src: /var/lib/openstack/firewall
5454
"""
5555

5656
RETURN = """

plugins/modules/edpm_nftables_snippet.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
EXAMPLES = """
5959
- name: Inject snippet for CI
6060
edpm_nftables_snippet:
61-
dest: /var/lib/edpm-config/firewall/ci-rules.yaml
61+
dest: /var/lib/openstack/firewall/ci-rules.yaml
6262
content: |
6363
- rule_name: 010 Allow SSH from everywhere
6464
rule:

roles/edpm_ceph_hci_pre/defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
edpm_ceph_hci_pre_configure_firewall: true
2424

2525
# Name of file to be created on EDPM nodes with ceph firewall rules
26-
edpm_ceph_hci_pre_firewall_yaml_file: /var/lib/edpm-config/firewall/ceph-networks.yaml
26+
edpm_ceph_hci_pre_firewall_yaml_file: /var/lib/openstack/firewall/ceph-networks.yaml
2727

2828
# List of IP ranges which can connect to Ceph RBD/Mon ports
2929
# If list is empty, the firewall rule will not specify a source address

roles/edpm_ceph_hci_pre/meta/argument_specs.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,4 +146,4 @@ argument_specs:
146146
The absolute path to a YAML file on the EDPM node which will be
147147
created by the role. The file will be parsed by the edpm_nftables
148148
role which creates the actual firewall rule.
149-
default: /var/lib/edpm-config/firewall/ceph-networks.yaml
149+
default: /var/lib/openstack/firewall/ceph-networks.yaml

roles/edpm_ceph_hci_pre/molecule/default/converge.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
hosts: all
2626
gather_facts: false
2727
vars:
28-
edpm_ceph_hci_pre_firewall_yaml_file: /var/lib/edpm-config/firewall/ceph-networks-overrides.yaml
28+
edpm_ceph_hci_pre_firewall_yaml_file: /var/lib/openstack/firewall/ceph-networks-overrides.yaml
2929
edpm_ceph_hci_pre_storage_ranges: [172.18.0.0/24]
3030
edpm_ceph_hci_pre_rgw_frontend_ranges: [172.18.0.0/24]
3131
edpm_ceph_hci_pre_nfs_frontend_ranges: [172.18.0.0/24]

roles/edpm_ceph_hci_pre/molecule/default/verify.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
gather_facts: false
2020
tasks:
2121
- name: Read generated file from defaults
22-
command: cat /var/lib/edpm-config/firewall/ceph-networks.yaml
22+
command: cat /var/lib/openstack/firewall/ceph-networks.yaml
2323
register: ceph_networks
2424

2525
- name: Assert that default generated file is correct
@@ -32,7 +32,7 @@
3232
loop: "{{ ceph_networks.stdout | from_yaml }}"
3333

3434
- name: Read generated file from overrides
35-
command: cat /var/lib/edpm-config/firewall/ceph-networks-overrides.yaml
35+
command: cat /var/lib/openstack/firewall/ceph-networks-overrides.yaml
3636
register: ceph_networks_overrides
3737

3838
- name: Assert that overridden generated file is correct

0 commit comments

Comments
 (0)