1. 25 Jul, 2012 1 commit
  2. 05 Jul, 2012 1 commit
  3. 27 Jun, 2012 1 commit
  4. 26 Jun, 2012 2 commits
  5. 15 Jun, 2012 1 commit
  6. 12 Jun, 2012 1 commit
  7. 08 Jun, 2012 1 commit
  8. 07 Jun, 2012 1 commit
  9. 14 May, 2012 1 commit
  10. 10 May, 2012 1 commit
  11. 26 Apr, 2012 1 commit
  12. 19 Apr, 2012 1 commit
  13. 01 Feb, 2012 1 commit
  14. 31 Jan, 2012 1 commit
  15. 19 Jan, 2012 1 commit
  16. 17 Jan, 2012 1 commit
    • Iustin Pop's avatar
      Rework instance rename QA tests · 4c1a464b
      Iustin Pop authored
      After looking again at the QA speeds, it turned out that instance
      rename was run 12 times via the command line and 12 times via RAPI;
      outside of start/stop which we need for maintenance during QA, this is
      the most used test.
      
      This patch does a few changes with regard to instance rename:
      
      - it changes rename functions to rename the instance back to the
        original name
      - which allows us not to run said functions twice, A→B and then B→A
      
      This brings down the number of rename tests from 12 to 8 (for both
      cmdline and RAPI).
      
      Furthermore, the CLI rename test was doing too many actual instance
      renames (at Ganeti level), so we rework it to be simpler (brings down
      the average duration from ~27s to ~21s), and we reorganise the run of
      various tests such that we do fewer instances start/stop tests (10
      instead of 14).
      
      Overall we only gain about 3 minutes (~7%), but still it's a speedup.
      Signed-off-by: default avatarIustin Pop <iustin@google.com>
      Reviewed-by: default avatarRené Nussbaumer <rn@google.com>
      4c1a464b
  17. 08 Dec, 2011 1 commit
  18. 22 Nov, 2011 1 commit
  19. 30 Aug, 2011 1 commit
  20. 25 Jul, 2011 1 commit
  21. 06 Apr, 2011 1 commit
  22. 15 Mar, 2011 1 commit
  23. 03 Mar, 2011 1 commit
    • René Nussbaumer's avatar
      LUInstanceRename: Fail if renamed hostname mismatch · 31fe5102
      René Nussbaumer authored
      There's a problem if you run gnt-instance rename with a non FQDN and the
      renamed LU tries to resolve the hostname to make it FQDN. It could be
      that this resolved hostname was just a CNAME to another name which leads
      to wrongly renamed instances name.
      
      Given a name of "foobar" is a CNAME to "moohost" now if you gonna rewrite an
      instance named "foobar1" to "foobar" it will pickup the host "moohost".
      This is not obvious behaviour. If we detect this case we abort with a
      prereq error. The user then however can continue with a FQDN and
      --no-name-check if he intends to force the operation.
      Signed-off-by: default avatarRené Nussbaumer <rn@google.com>
      Reviewed-by: default avatarIustin Pop <iustin@google.com>
      31fe5102
  24. 25 Feb, 2011 1 commit
  25. 12 Jan, 2011 1 commit
    • Iustin Pop's avatar
      Run pylint over QA code too · 3582eef6
      Iustin Pop authored
      Right now, the QA code is not covered by pylint, and this shows at
      least one low-impact bug.
      
      This patch does the necessary changes to make QA pylint-clean, and the
      changes the makefile to run pylint for it.
      
      Notable changes:
      
      - qa_utils.GenericQueryTest: randfields was not used at all, and my
        belief is that it was indented to be used in order not to modify the
        input list; so I replaced randfields with fields, so we only shuffle
        the our local copy
      - qa_node.TestOutOfBand was using it's own copy of AcquireNode(), so I
        replaced it with the existing version
      - qa_os: was using 'dir' in a couple of places, replaced with dirname
      Signed-off-by: default avatarIustin Pop <iustin@google.com>
      Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
      3582eef6
  26. 14 Dec, 2010 1 commit
  27. 10 Dec, 2010 1 commit
  28. 09 Dec, 2010 1 commit
    • Guido Trotter's avatar
      Simplify instance rename qa test · e5c2accd
      Guido Trotter authored
      The current instance rename qa testing function can only perform
      back-and-forth renames, both for command line and rapi. In order to be
      able to perform same-name rename tests we change it to be able to
      perform simple renames, and then we change qa to call it to perform both
      sides of the renaming.
      
      The same change is applied both to the local and the rapi test.
      Signed-off-by: default avatarGuido Trotter <ultrotter@google.com>
      Reviewed-by: default avatarIustin Pop <iustin@google.com>
      e5c2accd
  29. 29 Nov, 2010 1 commit
    • Iustin Pop's avatar
      Simplify QA commands · 2f4b4f78
      Iustin Pop authored
      Currently, 95% of the QA commands are executed in the same way: on the
      master, based on a command list and with expectancies for succes:
      
          AssertEqual(StartSSH(master['primary'],
                               utils.ShellQuoteArgs(cmd)).wait(), 0)
      
      The rest 5% are variations on this theme (maybe the command needs to
      fail, or the node is different, etc.). Based on this, we can simplify
      the code significantly if we abstract the common theme into a new
      AssertCommand() function. This saves ~250 lines of code in the QA suite,
      around 8% of the entire QA code size.
      
      Additionally, the output was very cryptic before (the famous "QA error:
      1 != 0" messages), whereas now we show a clear error message (node,
      command, exit code and failure mode).
      
      The patch replaces single quotes with double quotes in all the parts of
      the code that I touch; let me know if that's not OK…
      Signed-off-by: default avatarIustin Pop <iustin@google.com>
      Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
      2f4b4f78
  30. 01 Nov, 2010 2 commits
  31. 10 Aug, 2010 2 commits
  32. 29 Jul, 2010 1 commit
  33. 07 Jul, 2010 1 commit
  34. 01 Jul, 2010 1 commit
  35. 22 Apr, 2010 1 commit
  36. 15 Mar, 2010 1 commit
  37. 12 Feb, 2010 1 commit