Skip to content
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

Compute GlanceAPI storage configuration hash #648

Merged

Conversation

fmount
Copy link
Contributor

@fmount fmount commented Nov 5, 2024

Add .Spec.Storage and .Spec.ImageCache to the Hash based on the enabled_backends.
This will trigger an API refresh not only when the enabled_backends string changes, but also if the Storage interface or the ImageCache config (e.g. storageClass, storageRequest, externalStorage) is updated as part of any day2 operation.

Jira: https://issues.redhat.com/browse/OSPRH-11211

Copy link
Contributor

@abays abays left a comment

Choose a reason for hiding this comment

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

/lgtm

Add .Spec.Storage configuration and .Spec.ImageCache to the Hash based on the
enabled_backends.
This will trigger an API refresh not only when the enabled_backends string
changes, but also if the Storage interface or imageCache (e.g. storageClass,
storageRequest, externalStorage) changes as part of any day2 ops.

Signed-off-by: Francesco Pantano <fpantano@redhat.com>
@fmount
Copy link
Contributor Author

fmount commented Nov 8, 2024

/test glance-operator-build-deploy-tempest

Copy link
Contributor

@konan-abhi konan-abhi left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

Copy link
Contributor

openshift-ci bot commented Nov 11, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, fmount, konan-abhi

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:
  • OWNERS [abays,fmount,konan-abhi]

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

@fmount
Copy link
Contributor Author

fmount commented Nov 11, 2024

/test functional

@fmount
Copy link
Contributor Author

fmount commented Nov 11, 2024

/test glance-operator-build-deploy-kuttl

1 similar comment
@fmount
Copy link
Contributor Author

fmount commented Nov 11, 2024

/test glance-operator-build-deploy-kuttl

@openshift-merge-bot openshift-merge-bot bot merged commit 125104b into openstack-k8s-operators:main Nov 11, 2024
7 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.

3 participants