Synnefo

στο http://code.grnet.gr/projects/synnefo/repository/revisions/d19719cdb21417ecbd00a968d0ab5bbe5f495573 μπήκε η GET μεταβλητή changes_since η οποία αρχικά είναι κενή, ενώ μετά από κάθε κλήση GET /api/server/details αλλάζει στην ώρα που έτρεξε το request ώστε να σταλεί την επόμενη φορά.

Ο ServerHandler διαβάζει τη μεταβλητή αυτή και επιστρέφει τα machines τα οποία έχουν γίνει update την τελευταία ώρα (θα αλλάξει να παίρνει το changes_since). Αν ένα μηχάνημα έχει γίνει DESTROY κατά τη διάρκεια αυτή, το ui το καταλαβαίνει και το βγάζει απο τη λίστα

OK. Πρέπει να δεις επίσης τα εξής:

α) Η συνθήκη είναι updated__gt, πρέπει να είναι μεγαλύτερο του χρόνου changes-since, όχι ίσο, ο πελάτης ενδιαφέρεται για ό,τι άλλαξε μετά από τη στιγμή changes-since. Από το spec του OpenStack Compute 1.1: "your ReST client may use the changes-since parameter to check for changes since a previous request", οπότε ο χρόνος changes-since θα είναι πιθανότατα το Date: της απάντησης.

β) αν δεν έχει αλλάξει τίποτε, πρέπει να επιστρέφεται με τη μία HTTP 304 NOT MODIFIED: "if nothing has changed since the changes-since time, a 304 (Not Modified) response will be returned."

γ) Πρέπει να υποστηρίζεται και για το μοντέλο Image, αργότερα και για τα Disks.

δ) Όλες οι λεπτομέρειες εδώ:

http://docs.openstack.org/openstack-compute/developer/content/ch03s08.html

ΥΓ. Δεν χρειάζεται αναφορά στο πλήρες url του revision. Αρκεί μια αναφορά του στυλ "Refs #291" μέσα στο commit message, και το commit συνδέεται αυτόματα με το ticket, δείτε εδώ: http://www.redmine.org/projects/redmine/wiki/RedmineSettings#Referencing-issues-in-commit-messages και στο μέηλ που είχα στείλει προχθές.

Σε γενικές γραμμές λειτουργεί ωραία, βλέπω ένα πλήρες GET /servers στην αρχή, και μετά GETs με ?changes-since.
Ωστόσο, υπάρχουν τα εξής προβλήματα:
α) πρέπει να υποστηρίζεται και για το GET /images
β) δεν συμβαδίζει με το OpenStack v1.1 spec: http://docs.openstack.org/openstack-compute/developer/content/ch03s08.html, το οποίο προδιαγράφει ότι η τιμή στο ?changes-since είναι ISO 8601 datetime, ή σκέτη ημερομηνία χωρίς timestamp. Δείτε εκεί για όλες τις λεπτομέρειες κι αλλάξτε το αναλόγως.
γ) δεν υπάρχει πρόβλεψη για χειρισμό λανθασμένης εισόδου από [κακόβουλο] χρήστη. Ο κώδικας κάνει χύμα int(changes-since), το οποίο στην καλύτερη μπορεί να πετάξει ValueError. ΓΕΝΙΚΑ, οποτεδήποτε γίνεται χειρισμός πραγμάτων από το χρήστη πρέπει να γράφεται αμυντικός κώδικας. Στη συγκεκριμένη περίπτωση, το Exception το πιάνει γενικά κι επιστρέφει HTTP 500, αλλά το σωστό είναι Bad Request. ΠΡΕΠΕΙ να υπάρχει ρητός χειρισμός αναμενόμενων exceptions.
δ) χρειάζεται προσθήκη tests για το ?changes-since στα unit tests. Έφτιαξα έναν κόμβο, τον βλέπω να εμφανίζεται-εξαφανίζεται κλπ στις απαντήσεις, αν παίξω με πλήρη ή partial GETs λίγο πριν φτιαχτεί και λίγο αφού φτιαχτεί;

Πάντως φαίνεται να κάνει ποοοολύ ελαφρύτερη την κοινή περίπτωση όπου κάποιος το έχει αφήσει. Απλώς παίρνει HTTP 304 NOT MODIFIED κάθε 5 sec. Μια χαρά.

στο GET /images υπάρχει λόγος που θέλουμε να υποστηρίζεται?

Ναι.

α) το λέει το API. Είναι λογικό να το περιμένουν οι clients.
β) η υλοποίησή του είναι αντιγραφή του κώδικα για τους servers, και δεν χρειάζεται πάνω από 20λεπτο.

Υπενθυμίζω και τα υπόλοιπα σημεία, (β)-(δ).

Οκ τα α-β-γ, μένει το δ

Το changes_since δεν πρέπει να τσεκάρει οτι η ημερομηνία της τελευταίας φοράς που ρώτησε δεν είναι πολύ παλιά?

(πχ now - last_update < 1 minute)

δεδομενου οτι οταν υπαρχει changes_since, κανουμε query για ολα τα virtual machines (και τα deleted)