Skip to content

Include metadefs data files in glance container image - #161

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
fmount:glance-metadefs
Sep 2, 2026
Merged

Include metadefs data files in glance container image#161
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
fmount:glance-metadefs

Conversation

@fmount

@fmount fmount commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

The metadefs JSON files from the glance source tree were not being copied into the container image, causing glance-manage db_load_metadefs to load nothing even when invoked correctly. This resulted in the metadefs resource_types API returning an empty list [0].
Even if they are served from the db, they are initialized from the json files provided in the path [1] (which is the default since we do not customize it, see [2]).

As usual, update sources as part of the patch.

[0] https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/logs//c56/rdoproject.org/c56192dd51ff40b89d7f981030d8a718/controller/ci-framework-data/tests/test_operator/tempest-tests-tempest/tempest_results.html

[1] https://opendev.org/openstack/glance/src/branch/master/glance/db/sqlalchemy/metadata.py#L40
[2] https://opendev.org/openstack/glance/src/branch/master/glance/db/sqlalchemy/metadata.py#L222

The metadefs JSON files from the glance source tree were not being
copied into the container image, causing `glance-manage db_load_metadefs`
to load nothing even when invoked correctly. This resulted in the
metadefs resource_types API returning an empty list.

Signed-off-by: Francesco Pantano <fpantano@redhat.com>

@konan-abhi konan-abhi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
/approve

@openshift-ci

openshift-ci Bot commented Sep 2, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: konan-abhi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Sep 2, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit 8d871b8 into openstack-k8s-operators:main Sep 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants