Feature request: Include restore error details in VirtualMachineFileRestore phase-wait failures

Offen
  • Stimmt etwas nicht?
#2,847 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
67/100
Issue-Typ
Feature
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
backend

Rechercherichtung

Read ocp_resources/virtual_machine_file_restore.py and inspect the existing wait behavior in ocp_resources/resource.py to choose the helper interface. Add focused tests for reaching the requested phase, failure with status.errorMessage, and failure without it. Done means the helper reports useful failure details while inherited wait_for_status() behavior remains unchanged for other callers and phases.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Summary

Add a restore-specific phase-wait helper to VirtualMachineFileRestore in a follow-up PR.

When a restore reaches Failed while a caller waits for a successful phase, include the operator-provided status.errorMessage in the failure details. This will make restore failures easier to diagnose without a separate status lookup.

Required changes

  • Add the helper in ocp_resources/virtual_machine_file_restore.py, outside the generated-code markers.
  • Detect the Failed phase while waiting for the requested successful phase and report status.errorMessage when available.
  • Handle an absent or empty error message without introducing a secondary exception.
  • Preserve the inherited wait_for_status() behavior for other callers and phases.
  • Document the helper and add focused tests.

The implementation can inspect ocp_resources/resource.py for the existing wait behavior before selecting the helper's interface.

Acceptance criteria

  • A successful restore returns as documented when it reaches the requested phase.
  • A failed restore reports the failed phase and the operator error message when available.
  • Missing error details still produce a useful failure diagnostic.
  • Existing wait behavior remains unchanged outside the new helper.
  • Tests cover success, failure with an error message, and failure without an error message.

Context

Requested by @ema-aka-young for a separate PR, rather than expanding PR #2846.

Vorherrschende Sprache
Python
Sterne
28
Forks
85
Ø Merge
1 T. 3 Min.
Gemergte PRs (30 T.)
30

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus RedHatQE/openshift-python-wrapper

Alle Issues in RedHatQE/openshift-python-wrapper

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.