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

add better handling of snapshot restores, better debug logging #629

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

nhudson
Copy link
Collaborator

@nhudson nhudson commented Mar 7, 2024

This PR is an attempt to fix a few issues surrounding restoring using VolumeSnapshots

  • Fix issue where we can't searching using metadata fields via the k8s API
2024-03-07T17:05:45.437283Z ERROR reconciling object:reconcile:reconcile:reconcile_cnpg: controller::snapshots::volumesnapshots:441: No ready VolumeSnapshots found for org-coredb-inst-test-isnt-403 object.ref=CoreDB.v1alpha1.coredb.io/org-coredb-inst-test-isnt-403-240304.org-coredb-inst-test-isnt-403-240304 object.reason=reconciler requested retry trace_id=00000000000000000000000000000000 instance_name=org-coredb-inst-test-isnt-403-240304
  • Add better logging in conductor to help troubleshoot where sometimes spec.restore.volumeSnapshot is null instead of a boolean.

issue: TEM-3281

@nhudson nhudson merged commit b24dbea into main Mar 7, 2024
14 checks passed
@nhudson nhudson deleted the nhudson/TEM-3281 branch March 7, 2024 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants