1. 14 Oct, 2008 1 commit
    • Iustin Pop's avatar
      Change over to beparams · 338e51e8
      Iustin Pop authored
      This big patch changes the master code to use the beparams. Errors might
      have crept in, but it passes a small burnin.
      
      Reviewed-by: ultrotter
      338e51e8
  2. 06 Oct, 2008 1 commit
    • Iustin Pop's avatar
      Start implementation of parallel burnin · ec5c88dc
      Iustin Pop authored
      This patch introduces a simple framework for executing jobs in parallel
      in burnin (the ExecJobSet function) and the "--parallel" command line
      flag.
      
      The patch also changes the instance creation to run in parallel when the
      above flag is given. Error handling/instance removal is currently flacky
      with this options if there are errors in the instance creation.
      
      We also modify burnin to reuse a single client.
      
      Reviewed-by: imsnah
      ec5c88dc
  3. 18 Sep, 2008 1 commit
  4. 12 Sep, 2008 1 commit
  5. 27 Aug, 2008 1 commit
    • Iustin Pop's avatar
      Another burnin fix · e17188f4
      Iustin Pop authored
      This is a result of the log timestamp changes.
      
      Reviewed-by: imsnah
      e17188f4
  6. 30 Jul, 2008 1 commit
    • Iustin Pop's avatar
      Unify SetupDaemon/SetupLogging · 59f187eb
      Iustin Pop authored
      The 'old-style' info, error, debug logs do not make much sense. This
      patch unifies the SetupLogging and SetupDaemon functions. As a result,
      all the commands logs to a 'commands.log' file.
      
      The patch also changes the log setup to keep going if there's an error
      in setting up the file logging but we're logging to stderr.
      
      Also, burnin now logs to its own file (burnin.log).
      
      Reviewed-by: ultrotter
      59f187eb
  7. 14 Jul, 2008 1 commit
    • Iustin Pop's avatar
      Improve cli.SubmitOpCode · e2212007
      Iustin Pop authored
      Currently, the feedback_fn argument to SubmitOpCode is no longer used.
      We still need it in burnin, so we re-enable it by making the code call
      that function with the msg argument in case feedback_fn is callable. The
      patch also modifies burnin to accept the new argument format (msg is not
      a triple instead of a string).
      
      The patch also removes the ‘proc’ argument as it's obsolete; instead we
      can accept a luxi.Client instance (noone uses this right now).
      
      Reviewed-by: ultrotter
      e2212007
  8. 01 Jul, 2008 1 commit
    • Guido Trotter's avatar
      Burnin doesn't need a Processor · 2da55f7c
      Guido Trotter authored
      In 2.0 burnin submits job to the master daemon, so it doesn't need to
      create an internal Processor anymore. Even if the processor is not used
      anywhere in the burnin code it was still initialized as a leftover of
      how burnin used to work. Fixing this.
      
      Reviewed-by: iustinp
      2da55f7c
  9. 15 Jun, 2008 1 commit
    • Guido Trotter's avatar
      Burnin: Use iallocator in import/export · f9af35c8
      Guido Trotter authored
      Currently the iallocator option is ignored by burnin at import/export
      time even if passed in. With this patch it becomes used. The log message
      used by the importer is also changed to reflect this.
      
      This patch also improves import/export on the non-iallocator case:
        - The secondary node is not passed anymore on non-mirrored templates
        - On mirrored templates the secondary node is logged
      
      Reviewed-by: imsnah
      
      f9af35c8
  10. 13 Jun, 2008 1 commit
  11. 30 May, 2008 1 commit
    • Iustin Pop's avatar
      LURemoveInstance: fix op.ignore_failures usage · 5c54b832
      Iustin Pop authored
      Currently: the LURemoveInstance.Exec() method uses the ignore_failures
      attribute of the OpRemoveInstance opcode, but it doesn't check for its
      existence. The patch adds this attribute to _OP_REQP and to all the
      places where this opcode was created.
      
      This attributes is always passed by gnt-instance, but burnin didn't pass
      it so it can fail if it enters the 'fail to remove disks' branch of the
      method (which is why it was not triggered until now).
      
      Reviewed-by: ultrotter, imsnah
      5c54b832
  12. 28 Apr, 2008 2 commits
    • Iustin Pop's avatar
      Move the 'cmd' lock from cli.py to ganeti-masterd · a4af651e
      Iustin Pop authored
      This patch removes the lock and the lock options from cli.py and moves
      them to the master.
      
      Later during development we can remove it completely, but for now it's
      good to protect any other tool that uses the lock directly.
      
      Reviewed-by: imsnah,ultrotter
      a4af651e
    • Iustin Pop's avatar
      Convert cli.SubmitOpCode to use the master · 685ee993
      Iustin Pop authored
      This patch converts the cli.py SubmitOpCode method to use the unix
      protocol and thus execute the opcodes via the master.
      
      The patch allows a partial burnin to work with the master. Currently the
      query opcodes, since they are executed via the SubmitOpCode, are
      executed inside a job too, which is suboptimal, but they work fine.
      
      The cmd lock has been removed from the master, but the cli.py still
      takes the lock. This is ok for this in-progress patch (since the master
      still has only one executor thread). This will be fixed in a future
      patch.
      
      Reviewed-by: ultrotter
      685ee993
  13. 24 Apr, 2008 2 commits
    • Iustin Pop's avatar
      Implement replace secondary via the iallocator · b6e82a65
      Iustin Pop authored
      This patch implements secondary replace via the iallocator. The new
      opcode parameter 'iallocator' behaves like this: if passed, it will
      always compute and assign a new secondary, behaving in effect as if the
      secondary node has been passed. It conflicts with actually giving the
      secondary too.
      
      [Note: not tested with remote_raid1, but the code should behave the
      same, we only touch CheckPrereq and we assign a node.]
      
      The patch also adds burnin support for the replace secondary operation;
      with this in place, burnin can fully work with auto-assigned nodes.
      
      Reviewed-by: ultrotter
      b6e82a65
    • Iustin Pop's avatar
      Implement memory size option for burnin · 5e767b34
      Iustin Pop authored
      This simple patch adds memory size for the burnin instances, which helps
      testing allocator algorithm response based on the available node memory
      and instance memory size values.
      
      Reviewed-by: ultrotter
      5e767b34
  14. 23 Apr, 2008 5 commits
    • Manuel Franceschini's avatar
      Fix burnin ImportExport CreateInstance opcode · 96bb2f71
      Manuel Franceschini authored
      This patch adds the two attributes file_storage_dir and file_driver
      to the CreateInstance opcode in the ImportExport method, since
      otherwise it will fail with the file storage patches.
      
      Reviewed-by: iustinp
      96bb2f71
    • Guido Trotter's avatar
      burnin: improve instance create message · 6d54548e
      Guido Trotter authored
      When the allocator or a non-mirrored template are used the message shown by
      burnin at instance creation time was wrong. Fixing it.
      
      Reviewed-by: iustinp
      
      6d54548e
    • 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
  15. 15 Apr, 2008 1 commit
  16. 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
  17. 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
  18. 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
  19. 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
  20. 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
  21. 11 Jan, 2008 1 commit
  22. 08 Jan, 2008 1 commit
  23. 21 Nov, 2007 3 commits
  24. 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
  25. 04 Nov, 2007 1 commit
  26. 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
  27. 31 Oct, 2007 1 commit
  28. 29 Oct, 2007 1 commit
  29. 21 Sep, 2007 1 commit