Synnefo

Επίσης έχει γίνει η βασική διασύνδεση του ui με το RS API. Η jQuery.ajax στέλνει πλεον POST και DELETE methods τα οποία πιάνουν οι

• api.handlers.ServerHandler.create για create machine
• api.handlers.ServerActionHandler.create για shutdown machine
• api.handlers.ServerActionHandler.create για reboot machine
• api.handlers.ServerHandler.delete για destroy machine

(στο list view των machines μονο για την ωρα)

Ο κώδικας που υπάρχει σχετικά με τους handlers (api/handlers.py) έχει πολλά περιττά if όσον αφορά το rapi και τα Mock objects, που καλό θα ήταν να σβήσουμε ώστε να μην μας μπερδεύουν, και να τα αντικαταστήσουμε με μηνύματα λάθους (πχ "η σύνδεση με το rapi έχει πέσει").

Έτσι και δεν θα συντηρούμε κώδικα που δεν χρειάζεται αλλά και θα ξεχωρίσουμε την κάθε περίπτωση (που επιστρέφει δεδομένα το rapi και που η ΒΔ του Django.

Τι λέτε?

Markos Gogoulos wrote:

Ο κώδικας που υπάρχει σχετικά με τους handlers (api/handlers.py) έχει πολλά περιττά if όσον αφορά το rapi και τα Mock objects, που καλό θα ήταν να σβήσουμε ώστε να μην μας μπερδεύουν, και να τα αντικαταστήσουμε με μηνύματα λάθους (πχ "η σύνδεση με το rapi έχει πέσει").

Έτσι και δεν θα συντηρούμε κώδικα που δεν χρειάζεται αλλά και θα ξεχωρίσουμε την κάθε περίπτωση (που επιστρέφει δεδομένα το rapi και που η ΒΔ του Django.

Τι λέτε?

Δίνεις ένα παράδειγμα τι εννοείς; Ή στείλε ένα patch όπως θα ήθελες να είναι, για να καταλάβουμε ακριβώς το θέμα;

πχ στο class ServerHandler, στη def read_all έχει το εξής:

if not rapi: # No ganeti backend. Return mock objects
....
else: # ganeti backend. return objects
....

προτεινω να ασχοληθούμε με το if rapi, και το if not rapi να βγάζει κάποιο μήνυμα οτι δεν υπάρχει σύνδεση με το ganeti ή κάτι τέτοιο...

Markos Gogoulos wrote:

πχ στο class ServerHandler, στη def read_all έχει το εξής:

if not rapi:

1. No ganeti backend. Return mock objects
   ....
   else:
2. ganeti backend. return objects
   ....

προτεινω να ασχοληθούμε με το if rapi, και το if not rapi να βγάζει κάποιο μήνυμα οτι δεν υπάρχει σύνδεση με το ganeti ή κάτι τέτοιο...

Με αυτό απαντάς και στην απαίτηση για unit tests.

Συγκεκριμένα:

Κρατάς μόνο τον κώδικα για rapi.

Τον κώδικα χωρίς rapi τον μεταφέρεις σε unit tests τα οποία φτιάχνεις σύμφωνα με τις σχετικές οδηγίες.

Προσοχή, θέλουμε unit tests, όχι doc tests.

Ενδεικτικά κάποια πράγματα που πρέπει κατ'ελάχιστο να γίνουν. Νομίζω υπάρχουν κι άλλα που θα βγουν μετά από μια πρώτη υλοποίηση:

α) Η υλοποίηση του GET για /servers να κοιτάει τη βάση, τώρα υπάρχει ακόμη κώδικας που κάνει RAPI calls εκεί
β) να μπουν όλα τα RAPI calls για reboot/shutdown/restart [εκεί υπάρχει θέμα και στο GUI]
γ) να παίξει το POST για create [εκεί θέλει αρκετά: τυχαίο password, λήψη metadata για το νέο server (βλ. API documentation), το GUI να του περνάει metadata αυτά που διαβάζει από το image)]
δ) Πλήρες documentation στον κώδικα για τα (α), (β), (γ)
ε) Χειρισμός προβληματικών καταστάσεων, επιστροφή λογικών κωδικών λαθών HTTP [βλ. #211]

Πλέον των παραπάνω, υπάρχουν ακόμη τα εξής bugs:

• GET /servers/detail
  • το hostId δεν μπαίνει μέσα στα metadata αλλά απ'έξω
  • υπάρχει ένα "Server_Label" μέσα στα metadata
  • υπάρχει ένα "item" tag το οποίο πρέπει να φύγει
• GET /servers/id
  • επιστρέφει τα πεδία του VirtualMachine model χωρίς καμία μετατροπή σε RS API

Markos Gogoulos wrote:

Για το GET /servers/detail

το rackspace api επιστρεφει
<addresses>
<public>
<ip addr="67.23.10.132"/>
<ip addr="67.23.10.131"/>
</public>
<private>
<ip addr="10.176.42.16"/>
</private>
</addresses>

ενω εμεις επιστρεφουμε
<addresses>
<ip_six>
<ip addr="::1"/>
</ip_six>
<ip_four>
<ip addr="192.168.2.1"/>
</ip_four>
</addresses>

μιας και δεν εχουμε public/private αλλα ip_four και ip_six. Ειμαστε οκ με αυτο?

Ας κρατήσουμε ακριβώς τα δικά τους tags και να προσθέσουμε ό,τι νέο χρειάζεται.
Ένας client που ρωτάει, να μπορεί να βρει αυτά που περιμένει. Οπότε, κάνε την απάντηση:

<addresses>
  <public>
    <ip addr="192.168.2.1"/>    
    <ip6 addr="::1"/>
  </public>
  <private>
  </private>
</addresses>

Ώστε να υπάρχει κανονικά η προβλεπόμενη διάκριση public/private, να υπάρχει public/ip και να υπάρχει και public/ip6 tag.

Όσον αφορα το metadata θελει περισσοτερη δουλεια, γιατι δεν καλυπτεται το σεναριο απο τον OSXMLEmitter emitter και θα χρειαστει επεκταση

(το api επιστρεφει κατι του στυλ)
<metadata>
<meta key="Server Label">Web Head 1</meta>
<meta key="Image Version">2.1</meta>
</metadata>

Αυτά πρέπει να προκύπτουν από το μοντέλο για τα metadata, ένα <meta key=KEY> tag για κάθε γραμμή του που αφορά στο συγκεκριμένο VM.

Επίσης, να προστεθεί ως attribute (και όχι ως metadata) ένα πεδίο "description" στο server (όπως γίνεται και στα images)

να επιστρέφεται και το attribute "progress" όπως ορίζεται στο RS API. Αρχικά, θα παίρνει τιμές μόνο "0" κατά το ξεκίνημα και "100" όταν τελειώσει το creation

έχουν μπει τα progress και description. Μένει το metadata

To GET servers/detail και servers/id πρέπει να επιστρέφει τα πεδία:
flavorRef (αντι του flavorId)
imageRef (αντί του imageId)
+ created
+ updated

και γενικά όσα λείπουν από το OpenStack Compute API:
http://docs.openstack.org/openstack-compute/developer/content/ch04s01.html#d5e880

το flavorId έγινε flavorRef και το imageId έγινε imageRef. Επίσης μπήκαν τα created και updated. Πέρα απο το metadata το οποίο δεν εμφανιζεται σωστά, λείπει κάτι άλλο?