heat: generate config for heat.conf - #137
Conversation
For the heat-engine image, heat-manage db_sync requires the default heat.conf, otherwise it fails with the error: ERROR: No 'script_location' key found in configuration during heat db sync which comes from alembic. For the heat-api iamge, the keystone certificate can't be verified without the default heat.conf in place. Signed-off-by: James Slagle <jslagle@redhat.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Are we sure it's not some packaging issue, as I'm pretty convinced we don't need default heat.conf?
That key is in the package bundled ini https://github.com/openstack/heat/blob/master/heat/db/alembic.ini#L3 and it's looked from there https://github.com/openstack/heat/blob/master/heat/db/migration.py#L59-L62 not from heat.conf AFAIK. Also db_sync runs with explicit --config-dir https://github.com/openstack-k8s-operators/heat-operator/blob/main/internal/heat/dbsync.go#L32, so there won't be any need for default heat.conf.
heat-api can always read configuration from heat.conf.d https://github.com/openstack/heat/blob/81fc6092fee459fbfe659f2b8f70dcc0c4ba09b9/heat/api/openstack/wsgi.py#L4. Default generated heat.conf would not have any certificates etc. Do you've the built images somewhere that I can check? |
|
I did build the images from s2i main and the deployed with openstack-k8s-operators/openstack-operator#2042 and did not see any issues. Am I missing something? Note: I tagged heat-api image as heat-api-cfn after build though it was not required |
For the heat-engine image, heat-manage db_sync requires the default
heat.conf, otherwise it fails with the error: ERROR: No
'script_location' key found in configuration during heat db sync which
comes from alembic.
For the heat-api iamge, the keystone certificate can't be verified
without the default heat.conf in place.
Signed-off-by: James Slagle jslagle@redhat.com