Bug #351
To API implementation δεν επιστρέφει buildInProgress (409)
Status: | Closed | Start date: | 03/31/2011 | |
---|---|---|---|---|
Priority: | Medium | Due date: | 04/08/2011 | |
Assignee: | Giorgos Verigakis | % Done: | 0% |
|
Category: | Cyclades API | Spent time: | - | |
Target version: | v0.2 |
Description
Το api/v1.1redux πρέπει να επιστρέφει HTTP 409 [buildInProgress] όταν το μηχάνημα είναι BUILD και του ζητήσω κάτι, όπως προβλέπεται από το spec. Το κάνει o [verigak], μαζί με το υπόλοιπο API implementation, δεν επείγει αλλά πρέπει να γίνει.
Associated revisions
Raise if vm is deleted or being built
Fixes #351
History
#1 Updated by Giorgos Verigakis over 13 years ago
Το αν ένα μηχάνημα είναι σε BUILD state είναι ένας έλεγχος που πρέπει να κάνει το API και να επιστρέφει κατευθείαν error χωρίς να μιλήσει με το logic; Ή πρέπει να στέλνει όλα τα requests στο logic το οποίο θα κάνει raise κάποιο Exception για να το πιάσει το API;
Αν ισχύει το δεύτερο χρειάζομαι από το logic να μου πει ποιο θα είναι αυτό το exception.
#2 Updated by Vangelis Koukis over 13 years ago
Θα πρότεινα, όπως και με τις υπόλοιπες ενέργειες, ο έλεγχος να γίνει μέσα στην start_action(), που αποτελεί κομμάτι του logic.
Μπορείς να την επεκτείνεις ώστε να γυρίζει exception αντίστοιχο αυτών που επιστρέφει τώρα; τώρα επιστρέφει γενικό
InvalidActionError.
Ίσως έχει νόημα αυτό να μείνει, αν η ζητούμενη ενέργεια είναι ανύπαρκτη, και να προστεθούν δύο ακόμη: ένα MachineIsDeletedError κι ένα MachineBeingBuiltError.
Μπορείς να κάνεις ανάλογη αλλαγή στην start_action() και να πιάσεις το MachineBeingBuiltError στην υλοποίηση του API, επιστρέφοντας το ανάλογο cloudFault;
#3 Updated by Vangelis Koukis about 13 years ago
- Category set to Cyclades API
- Status changed from Assigned to Closed
- Target version set to v0.2
Αυτό έγινε, 5231a38a6, επιστρέφει κανονικά buildInProgress cloudfault, κλείνει.