1. 23 Apr, 2008 3 commits
    • Guido Trotter's avatar
      burnin: remove created exports · 4a7ff493
      Guido Trotter authored
      Make burnin remove exports after importing the instance back.
      
      Reviewed-by: iustinp
      
      4a7ff493
    • Manuel Franceschini's avatar
      Add instance rename to burnin · 054a8696
      Manuel Franceschini authored
      This patch adds the possibility to execute instance renaming when
      doing a cluster burnin. The optional parameter --rename takes
      one 'spare' instance name.
      Given this option it renames all given instances to the spare one
      and back to their original name.
      
      Reviewed-by: iustinp
      054a8696
    • Iustin Pop's avatar
      Allow burnin to use an iallocator in instance creation · b91bde14
      Iustin Pop authored
      This patch adds iallocator support to burnin, currently only in instance
      creation. This means that, depending on the external algorithm, not all
      nodes might get instances.
      
      The patch also restricts node selection and iallocator options together.
      
      Reviewed-by: ultrotter
      b91bde14
  2. 15 Apr, 2008 1 commit
  3. 10 Apr, 2008 1 commit
    • Iustin Pop's avatar
      Rework the results of OpDiagnoseOS opcode · 1f9430d6
      Iustin Pop authored
      Currently, the opcode DiagnoseOS is the only opcode that return a
      structure of objects.OS (which is a custom class, and not a simple
      python object) and furthermore all the processing of OS validity across
      nodes is left to the clients of this opcode.
      
      It would be more logical to have this opcode be similar to list
      instances/nodes, in the sense that:
        - it should return a table of results
        - the fields in the table should be selectable
      
      This patch does the above. The possible fields are:
        - name (os name)
        - valid (bool representing validity across all nodes)
        - node_status, which is a complicated structure required for ‘gnt-os
          diagnose’
      
      With this patch, gnt-os list becomes a very simple iteration over the
      list of results, filtering out non-valid ones. gnt-os diagnose is still
      complicated, but no more than before.
      
      The burnin tool has also been modified to work with the modified
      results, and is simpler because of this (it only needs to know if an OS
      is valid or not, not the per-node details).
      
      Reviewed-by: imsnah
      1f9430d6
  4. 25 Mar, 2008 1 commit
    • Iustin Pop's avatar
      Remove the option to create md/drbd7 instances · f9193417
      Iustin Pop authored
      This patch removes the options that allow to create local_raid1 or
      remote_raid1 instances. It also modifies the documentation and removes
      these disk templates from burnin and from qa.
      
      Reviewed-by: imsnah
      f9193417
  5. 14 Feb, 2008 1 commit
    • Iustin Pop's avatar
      Two small improvements to burnin · d7b47a77
      Iustin Pop authored
      This tiny patch fixes the verbose option to actually work, and also when
      creating instances it logs the secondary node too (even if this doesn't
      apply for plain templates, it doesn't create an error).
      
      Reviewed-by: imsnah
      d7b47a77
  6. 20 Jan, 2008 1 commit
    • Iustin Pop's avatar
      Allow use of 'diskless' disk template in burnin · bd249e2f
      Iustin Pop authored
      Even if this doesn't have any practical use for actually creating
      instances, it can be used for very fast burnin and testing just the
      add/start/stop/remove functionality.
      
      This has also revealed a bug in export/import related to diskless
      instances, so it's educational value is proved.
      
      Reviewed-by: ultrotter
      bd249e2f
  7. 14 Jan, 2008 3 commits
    • Guido Trotter's avatar
      Make instance start/stop skippable at burnin time · d4844f0f
      Guido Trotter authored
      Even though burnin was born just to do that test it now contains a lot more
      things one might try, so it makes sense to make instance start/stop optional
      too.
      
      This creates a burnin that at the bare minimum tests instance create and
      remove, if all the --no options are specified, but usually does a lot more.
      
      Reviewed-by: iustinp
      
      d4844f0f
    • Guido Trotter's avatar
      Do instance export and import during burnin · bd5e77f9
      Guido Trotter authored
      Instances get exported to a remote node, then removed and imported back to
      their original nodes. This should be an idempotent option from the instance
      point of view, and help making sure ImportExport is kept up to date.
      
      It will also help making burnin take a lot longer, which is nice to take a nap.
      "...but I'm doing a cluster burnin...". Unfortunately this subfeature is a bit
      jeopardized by the fact that the new code can be skipped with the
      --no-importexport option, but nobody needs to know that, do they?
      
      Reviewed-by: iustinp
      bd5e77f9
    • Iustin Pop's avatar
      Allow burnin to take "-t plain" as an option · 4aa036ab
      Iustin Pop authored
      The burnin code deals with "-t plain", but the command line parser
      doesn't allow that as an option. This patch fixes this issue.
      
      Reviewed-by: ultrotter
      4aa036ab
  8. 11 Jan, 2008 1 commit
  9. 08 Jan, 2008 1 commit
  10. 21 Nov, 2007 3 commits
  11. 08 Nov, 2007 2 commits
    • Iustin Pop's avatar
      Simplify burnin output · 21546b1c
      Iustin Pop authored
      Currently, the burnin output is very verbose (we log the entire output
      from LU feedback) and duplicate in some cases (as we enable debugging).
      
      This patch disables debugging and only logs the burnin output when an
      error occurs.
      
      Reviewed-by: imsnah
      21546b1c
    • Iustin Pop's avatar
      Refactor burnin to improve disk replacement · 175f44c2
      Iustin Pop authored
      This patch changes the burnin:
        - split the code into functions
        - add replace2 (replace the secondary) for remote_raid1
        - add replace1/2 for drbd8
      
      Reviewed-by: imsnah
      175f44c2
  12. 04 Nov, 2007 1 commit
  13. 02 Nov, 2007 1 commit
    • Iustin Pop's avatar
      Improve burnin with regard to drbd8 · 79f87a76
      Iustin Pop authored
      Allow burnin to use the new drbd8 template (for which case one needs to
      disable replacement of disks, burnin does yet support that with drbd8).
      
      The patch also changes do-replace[12] to no-replace[12] as that is what
      they actually do.
      
      Reviewed-by: imsnah
      79f87a76
  14. 31 Oct, 2007 1 commit
  15. 29 Oct, 2007 1 commit
  16. 21 Sep, 2007 1 commit
  17. 14 Sep, 2007 1 commit
  18. 14 Aug, 2007 1 commit
    • Iustin Pop's avatar
      Style changes for pep-8 and python-3000 compliance. · 3ecf6786
      Iustin Pop authored
      This changes the raising of exceptions from:
        raise Exception, value
      to
        raise Exception(value)
      
      as the first form will be removed in python-3000 and the second form is
      preferred now.
      
      The changes also involve a few cases of changing from raising standard
      exceptions and use our own ones.
      
      The new version also fixes many pylint-generated warnings, especially in
      ganeti-noded where I changed many methods to @staticmethod.
      
      There is no functionality changed (barring any bugs).
      3ecf6786
  19. 18 Jul, 2007 1 commit
  20. 16 Jul, 2007 1 commit