-
Notifications
You must be signed in to change notification settings - Fork 86
Make config directories consistent across services #1032
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rabi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
46e49a3 to
b508988
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/780a3e986886479b8546e1c65d23c9f4 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 01m 03s |
75fc1e3 to
9d512d3
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/b04ad20a2c7d4387900cdf2a162f1bde ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 56m 50s |
|
recheck |
1 similar comment
|
recheck |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f4d24fd67b354ca591fc3022420fa533 ❌ openstack-k8s-operators-content-provider FAILURE in 8m 51s |
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/e35489636030495892495512fe857d33 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 42m 22s |
f4bc0dd to
595f9ac
Compare
|
recheck |
15010fa to
9988203
Compare
|
recheck |
1 similar comment
|
recheck |
7624630 to
c371f02
Compare
|
recheck |
d711948 to
df6cb90
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/556e0a58c79b43b2aa4e8dee7246beba ❌ openstack-k8s-operators-content-provider NODE_FAILURE Node request 100-0008070840 failed in 0s |
b6b831c to
e2d8570
Compare
32846a8 to
5405363
Compare
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]>
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ae575bd1c5214c64b93df75021bed925 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 37m 50s |
|
recheck |
Some services used
edpm_container_standaloneand others directly useedpm_container_manageand 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.
This PR changes the following:
Converts nova and telemetry services to use edpm_container_standalone (generates the startup configs in the same place /var/lib/edpm-config/container-startup-config/
Places all service config files in /var/lib/openstack/ and changes the path to find bind mounts to generate EDPM_CONFIG_HASH.
Moves container_defs/kolla_config/config templates/files to separate folders in the role so that they can be easily differeniated. Also changes some json configs to yaml to be consistent with other services.
jira: https://issues.redhat.com/browse/OSPRH-19243