1. 15 Jun, 2008 3 commits
    • Guido Trotter's avatar
      FailoverInstance: change AddInstance with Update · b6102dab
      Guido Trotter authored
      We're not adding a new instance, just making configuration changes to
      the one we're working on.
      
      Reviewed-by: imsnah
      
      b6102dab
    • 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
    • Iustin Pop's avatar
      Fix an error message in instance add · 3e91897b
      Iustin Pop authored
      There is a mistake in the error message generated when we can't reach a
      node for checking for available disk space. Without it, the error
      message is:
      Failure: prerequisites not met for this operation:
      Cannot get current information from node '{u'gnte2.lab.k1024.org':
      {'cpu_total': 1, 'memory_free': 480, 'vg_size': 131068, 'memory_total':
      504, 'bootid': '2176dd3b-2f96-42f0-8b6e-2873ecaf5f9c', 'memory_dom0':
      134, 'vg_free': 130172}, u'gnte1.lab.k1024.org': False}'
      
      instead of the expected:
      Failure: prerequisites not met for this operation:
      Cannot get current information from node 'gnte2.lab.k1024.org'
      
      Reviewed-by: imsnah
      3e91897b
  2. 13 Jun, 2008 4 commits
  3. 12 Jun, 2008 7 commits
  4. 11 Jun, 2008 1 commit
    • Guido Trotter's avatar
      Remove SimpleStore cache · c9673d92
      Guido Trotter authored
      SimpleStore is instantiated anew most of the times it's used, so having
      a cache inside it serves no purpose. Removing it.
      
      Reviewed-by: iustinp
      c9673d92
  5. 06 Jun, 2008 9 commits
  6. 31 May, 2008 8 commits
  7. 30 May, 2008 2 commits
    • Iustin Pop's avatar
      Complete removal of md/drbd 0.7 code · abdf0113
      Iustin Pop authored
      This patch removes the last of the md and drbd 0.7 code. Cluster which
      have the old device types will be broken if they have this applied.
      
      Reviewed-by: imsnah
      abdf0113
    • 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
  8. 29 May, 2008 1 commit
  9. 24 May, 2008 1 commit
  10. 15 May, 2008 4 commits
    • Michael Hanselmann's avatar
      Update command line help and manpages with mandatory options · bdb7d4e8
      Michael Hanselmann authored
      Reviewed-by: ultrotter
      bdb7d4e8
    • Guido Trotter's avatar
      document cluster verify --no-nsplus1-mem option · 3cf7c9fa
      Guido Trotter authored
      Add this recently added option to the gnt-cluster man page before
      releasing 1.2.4.
      
      Reviewed-by: imsnah
      
      3cf7c9fa
    • Guido Trotter's avatar
      Fix drbd show parser to handle valueless keywords · 63012024
      Guido Trotter authored
      It turns out in some cases there can exist keywords without an
      associated value exported by drbdsetup show. This patch makes the value
      part optional in our parser, so that if it's not present the parsing
      result will contain an array with just the keyword in it. This is not a
      problem since we check all keyword names before accessing their values,
      so we won't mistakenly try to access the value of a valueless keyword.
      
      Reviewed-by: iustinp
      
      63012024
    • Guido Trotter's avatar
      Split drbd command creation and execution · 333411a7
      Guido Trotter authored
      Make _AssembleDisk more similar to _AssembleNet by splitting the
      generation of the drbdsetup command and its execution. While not
      changing anything this makes it easier to manipulate the command just in
      certain cases, which in the future we'll need to do.
      
      Reviewed-by: iustinp
      
      333411a7