Activity
From 01/13/2010 to 02/11/2010
02/11/2010
- 06:42 pm Revision 650fd0c8: Skip line-length warnings in man
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 06:42 pm Revision 96b9071c: Revert "Workaround man page output for long PREFIX dirs"
- This reverts commit 83d9f4366f3aa9ae360e27bfe6619402793e9eb5.
man is still unable to wrap some long lines, so we sim... - 06:00 pm Revision abd8e836: Run instance hooks on more nodes
- This should fix issue 68: some hooks should be run on more nodes than
currently. GrowDisk runs on both nodes, remove ... - 06:00 pm Revision 08eec276: Add {NEW,OLD}_{PRIMARY,SECONDARY} vars to hooks
- Per issue 71, the migrate and failover need special variables for
keeping the nodes consistent during instance migrat... - 03:29 pm Revision 83d9f436: Workaround man page output for long PREFIX dirs
- A long PREFIX variable (to configure) will result in very long
LOCALSTATEDIR, which when concatenated with lib/ganeti... - 01:50 pm Revision cb573a31: Convert scripts to pass options to the JobExecutor
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:50 pm Revision dd713605: Pass debug mode to noded for OS-related calls
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:50 pm Revision cff5fa7f: Add the options attribute to cli.JobExecutor
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:49 pm Revision 400ca2f7: Update scripts to pass 'opts' to SubmitOpCode
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:49 pm Revision 265e6244: Add debug mode to burnin
- There are two entry points to job execution in burnin, ExecOp and
ExecOrQueue, and these are modified to call the new... - 01:49 pm Revision 293ba2d8: Implement generic CLI options->opcode updates
- This patch changes SubmitOpCode and SubmitOrSend such that we have a
single function that does generic CLI options to... - 01:46 pm Revision ea34193f: Change the debug CLI option to integer/count
- This changes from boolean to integer/count (for a future differentiation
based on the actual debug level). All the us... - 01:46 pm Revision ee844e20: Add a generic 'debug_level' attribute to opcodes
- Also automatically fix opcodes which have this missing in the LU init
routine.
Signed-off-by: Iustin Pop <iustin@goo...
02/10/2010
- 03:54 pm Revision b44b0141: Fix bug introduced in commit 413b747
- While commit 413b747 fixed the issue of poll(2) returning too
soon, it didn't work when the poll(2) call should've be... - 03:13 pm Revision adf385c7: Fix dumpers/loaders after __slots__ cleanup
- Commit 154b958 changed (correctly) the __slots__ usage, but this broke
dumpers/loaders since we relied directly on th... - 02:49 pm Revision 413b7472: Fix locking bug causing high CPU usage
- Iustin Pop noticed unusually high CPU usage with 2.1's master
daemon, even with very simple opcodes like OP_TEST_DELA... - 12:49 pm Revision a4f12da4: Check built manpages for errors
- One fix is necessary in gnt-cluster.sgml. Also adding “DELETE_ON_ERROR”
target to remove output file if an error occu... - 11:32 am Revision f3448a3c: Fix confd procotol design description
- The protocol design for confd was missing a description of the fourcc
code which we use to distinguish between differ...
02/09/2010
- 01:34 pm Revision 18337ca9: Implement instance rename QA tests
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 12:04 pm Revision 7ea7bcf6: Add an early release lock/storage for disk replace
- This patch adds an early_release parameter in the OpReplaceDisks and
OpEvacuateNode opcodes, allowing earlier release... - 11:31 am Revision c4a73785: Fix "gnt-instance rename" functionality
- Commit 91e0748c (Unify the “--no-ip-check” option) broke the options
variable name for ‘--no-ip-check’ but since we d...
02/08/2010
- 04:04 pm Revision 4c06a2d0: Merge branch 'stable-2.1' into devel-2.1
- * stable-2.1:
TLReplaceDisks: Delay iallocator when evacuating node
Implement debug level across OS-related RPC c... - 01:35 pm Revision 94a1b377: TLReplaceDisks: Delay iallocator when evacuating node
- When evacuating nodes, the iallocator was run for all
instances without taking planned changes into consideration.
Th...
02/04/2010
- 12:47 pm Revision e8cd4796: Update NEWS file and bump version to 2.0.6
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 12:30 pm Revision 2b7a98ae: Update documentation for the text backend
- 11:58 am Revision 62eb3aa2: Update NEWS file for the 0.2.3 release
02/03/2010
- 07:04 pm Revision 4a0e011f: Implement debug level across OS-related RPC calls
- This doesn't implement the full functionality, we need to add the debug
level to the opcodes too, but at least won't ... - 03:55 pm Revision dd9e9f9c: Second try to fix LUVerifyCluster
- My previous patch, commit 785d142, fixed the case where a node is marked
offline. With this patch it'll also handle o... - 03:01 pm Revision 785d142e: LUVerifyCluster: Fix bug with offline nodes
- […]
* Other Notes
- NOTICE: 1 offline node(s) found.
* Hooks Results
Failure: command execution error:
iteration... - 03:01 pm Revision 26751075: utils: Fix retry delay calculator
- Before this patch, it would always sleep for at least
the time specified as the upper limit. Now it actually
limits t... - 01:29 pm Revision 146b37eb: Fix secondary node selection for existing N+1
- In case a secondary node is already N+1 failed, currently the node
selection will accept a node that cannot start (at... - 10:24 am Revision a4a6e623: Rewrite the node add checks for simpler layout
- This will make it clearer than many if…then choices.
02/02/2010
- 04:39 pm Revision e18def2a: ganeti-cleaner: does 'echo 0' instead of 'exit 0'
- This is from Debian Bug#568105.
ganeti-cleaner in one case erroneously calls "echo 0" instead of "exit
0". As ganeti...
02/01/2010
- 04:55 pm Revision 72456eb2: Bump RPC protocol version to 30
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:42 pm Revision 82ad3594: Find OpenSSL program at configure time
- Otherwise we depend on PATH at runtime.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Guido Tro... - 04:42 pm Revision a55474c7: Move function generating SSL certs into utils
- Also add unittest.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Guido Trotter <ultrotter@googl... - 03:20 pm Revision 5b2069a9: Initial design for inter-cluster instance moves
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:01 pm Revision e7307f08: Merge branch 'devel-2.1'
- * devel-2.1:
Bump version to 2.1.0~rc5
Makefile.am: Targets used directly should depend on BUILT_SOURCES
Make t... - 02:55 pm Revision 9f53d9ce: Merge remote branch 'origin/stable-2.1' into devel-2.1
- * origin/stable-2.1:
Bump version to 2.1.0~rc5
Fix missing bridge for xen instances
Fix flipping MC flag bug
... - 02:32 pm Revision 8432c491: Bump version to 2.1.0~rc5
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 02:08 pm Revision eb732fb5: Makefile.am: Targets used directly should depend on BUILT_SOURCES
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 12:44 pm Revision 9b3b6845: Merge branch 'devel-2.0' into devel-2.1
- * devel-2.0:
Three small typos in man pages
Conflicts:
man/gnt-cluster.sgml (trivial)
Signed-off-by: Iustin Pop ... - 12:37 pm Revision 433c63aa: Make the snapshot decision based on disk type
- … instead of disk size, which is not as reliable. This actually
simplifies the code; but it still leaves the possibil... - 12:10 pm Revision ba0b05fe: Three small typos in man pages
- The credit goes again to Lintian.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <han...
01/29/2010
- 01:58 pm Revision 0183a697: Fix missing bridge for xen instances
- Xen instances nic definitions miss the target bridge.
This bug was introduced in commit 503b97a9.
Signed-off-by: Al...
01/28/2010
- 04:46 pm Revision cea0534a: Fix flipping MC flag bug
- Currently unofflining or undraining an already functional master
candidate node, can cause it to demote itself. In or... - 01:44 pm Revision 7369e826: ganeti-watcher: ensure confd is running as well
- Ganeti-confd should be running on all 2.1 nodes.
Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iu... - 11:43 am Revision 551b6283: Add capability to use syslog for logging
- This patch adds a configure-time parameter that will set the defaults
used by all programs, and command-line paramete...
01/27/2010
- 06:26 pm Revision 8388e9ff: Initial KVM security improvement design for 2.2
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:35 pm Revision 81198f6e: node daemon: allow working with broken queue dir
- In case the queue dir cannot be create/initialized, currently
ganeti-noded exits. This means that a read-only filesys... - 03:35 pm Revision aac3fbf0: utils.FileLock: handle init errors properly
- If the open of the lock file fails (due to whatever reason), 'self'
won't have the 'fd' attribute, and thus we fail i... - 03:35 pm Revision 8645ff83: daemon-utils: remove usage of here-docs
- In some versions of bash, here-docs and here-strings use temporary
files, which means daemon-util needs a writable te... - 02:17 pm Revision 7a8da378: Fix typo in ganeti-os-interface(7)
- The variable is called “IMPORT_INDEX”, not “IMPORT_IDX”.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Revie... - 12:19 pm Revision ec44d893: locking: add/fix @type information
- This patch missing @type information for all public methods, modifies
one to conform to the rest, and removes some in... - 11:17 am Revision 154b9580: Fix __slots__ definitions
- According to http://docs.python.org/reference/datamodel.html#slots
* The action of a __slots__ declaration is limite...
01/26/2010
- 02:58 pm Revision 4c1a504b: Merge branch 'devel-2.0' into devel-2.1
- * devel-2.0:
Fix the mocks.py for 2.0 unittests
LURemoveNode safety in face of wrong node list
Fix an unsafe fo... - 11:53 am Revision 825e13df: Fix the mocks.py for 2.0 unittests
- The recent change to use LogWarning with multiple arguments in
mpcu.py/HooksMaster broke the (simple) mock we have in...
01/25/2010
- 06:23 pm Revision 89b70f39: Add a crude disable for DRBD barriers
- Ideally we want to/will have per-device DRBD controls of disk/metadata
flushes. In the meantime, we want at least a d... - 06:17 pm Revision 9bb31ea8: LURemoveNode safety in face of wrong node list
- LURemoveNode runs under the BGL, which means we're guaranteed that the
list of nodes as retrieved in CheckPrereq is s... - 06:16 pm Revision 62579388: Fix an unsafe formatting bug
- This might fix issue 84; in any case, the current situation is that we
have a potentially unsafe formatting, which sh... - 06:16 pm Revision 691744c4: Ensure all int/float conversions are handled right
- int()/float() can raise either ValueError (in case of int("a")), or
TypeError (in case of int(None)). We had many bug...
01/22/2010
- 06:38 pm Revision b9f8d18f: Merge branch 'devel-2.1'
- * devel-2.1: (38 commits)
Bump version to 2.1.0~rc4
Update man page of gnt-instance
KVM: fix pylint warning
K... - 06:38 pm Revision d80cb8c4: Merge branch 'stable-2.1' into devel-2.1
- * stable-2.1:
Bump version to 2.1.0~rc4
KVM: fix pylint warning
KVM: be more resilient on broken migration answ... - 06:21 pm Revision 0ea0ce20: Bump version to 2.1.0~rc4
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:08 pm Revision 2feecf12: Fix lint warnings after my last changes
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:36 pm Revision 16b037a9: Remove http.HttpJsonConverter
- With the move of the content-type handling to the various users of the HTTP
layer, this class isn't really useful any... - 04:36 pm Revision 627ad739: rapi: Don't access request's private object directly
- This way the request handlers don't need as much knowledge
about the inside of the request and context objects and it... - 04:17 pm Revision ab221ddf: http.server: No longer enfore JSON encoding for body
- The HTTP layer shouldn't care about the contents of the request data or
responses. This requires further changes in t... - 04:17 pm Revision 200e38ac: http.server: Refuse HTTP/1.1 request without Host header
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:17 pm Revision c10e2751: ganeti-rapi: Remove serial number/ETag code
- It was never used as intended.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin... - 04:17 pm Revision 580558a3: http: Add two new exceptions, one constant
- These will be useful in the future in case we don't enfore JSON encoding
anymore in the http.server module. The HTTP ... - 04:17 pm Revision 231db3a5: Factorize LUXI parsing and handling code
- Also fix a typo in http/__init__.py and add unittests
for the LUXI parsing and formatting functions.
Signed-off-by: ... - 04:16 pm Revision 797506fc: Simplify LUXI exceptions
- Having only one exception hierarchy makes catching them simpler. Before
ProtocolError would derive directly from Exce... - 03:55 pm Revision c7c568b4: Update man page of gnt-instance
- This patch updates the man page of gnt-instance to include the newly
added tags filtering.
Signed-off-by: René Nussb... - 03:45 pm Revision e4dd2299: KVM: fix pylint warning
- Specify string format arguments as logging function parameters
Signed-off-by: Guido Trotter <ultrotter@google.com> - 03:05 pm Revision c4e388a5: KVM: be more resilient on broken migration answers
- Before, when doing kvm live migrations we use to accept an "unknown
status" but to reject anything that didn't match ...
01/21/2010
- 05:38 pm Revision 39dfd93e: Allow filtering for (node-)tags
- This change introduces startup, shutdown, reboot, reinstall by using
instance respectively node tags as a selection c...
01/20/2010
- 03:47 pm Revision 2ebf1568: Add unittests for cli.GenerateTable
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:47 pm Revision ec39d63c: cli: Fix bug when not using headers
- Commit 9fe72672 added code to not write spaces at the end of each line.
Unfortunately it didn't work properly when no...
01/19/2010
- 06:18 pm Revision 03956d39: daemon-util: Fix quoting issue
- This patch fixes a quoting issue in daemon-util:
$ EXTRA_MASTERD_ARGS=--no-voting /etc/init.d/ganeti restart
[…]
* g...
01/15/2010
- 02:17 pm Revision 0175fd8c: Bump version to 2.1.0~rc3
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: ... - 01:23 pm Revision f9d0783b: Merge branch 'stable-2.1' into devel-2.1
- * stable-2.1: (22 commits)
Fix long-standing race condition bug in locking unittest
confd client: copy the peers ... - 01:22 pm Revision 1b3a7656: Switch the SplitKeyVal function to accept escapes
- This tiny patch switches the SplitKeyVal function (and thus the command
line options like -H, -B, etc.) to UnescapeAn... - 12:16 pm Revision 51e3bb92: Fix long-standing race condition bug in locking unittest
- Every once in a while we saw failures in the locking unittest, but
weren't able to determine the cause. This patch fi...
01/14/2010
- 07:07 pm Revision db169865: confd client: copy the peers in UpdatePeerList
- Since the peer list is shuffled by the client, we don't keep a reference
to the list which was passed in, but copy it... - 06:38 pm Revision a804261a: Move instance relocation test upper in the chain
- Currently we test each instance for relocation in checkMove; however, it
is a little more clear if we pass only the r... - 06:05 pm Revision 5ad86777: Split the balancing function in two parts
- Currently in the balancing function we do two thing:
- take the decision where to do a new balancing round or not
- ... - 04:15 pm Revision 210ed839: Small update to the Makefile
- 04:09 pm Revision 0d478754: Makefile: Switch from subshell to $(MAKE) -C
- It seems that set -e does not affect subshell (only simple commands),
and thus we don't actually get failures from ma... - 02:17 pm Revision 43983a88: testutils: Print name of test program before running it
- While the name can be looked up in Makefile.am, this
is useful when an automated test just hangs and needs
to be kill... - 02:17 pm Revision 6d7cc5ff: Don't use hardcoded name for pylint
- Look it up at configure time instead.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop ... - 02:16 pm Revision f60d352e: Partially revert "Makefile.am: Run pylint on all Python code"
- This partially reverts commit e4e7c7df3a2356103c2442a91d628ca0470fb267,
which made newer pylint versions fail.
Signe... - 02:16 pm Revision b4086ce8: build-bash-completion: Take care of pylint warnings
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 11:26 am Revision 5b69bc7c: Add an UnescapeAndSplit function
- In many cases, where we accept (usually from the command line) a list of
parameters, we remove the use of the separat...
01/13/2010
- 07:09 pm Revision 5548de18: Small improvements for release script
- - Print usage if no tree-ish (tag, branch or commit) was
specified.
- Use “gntrelease.XXXXXXXXXX” as temporary dire... - 07:09 pm Revision e4e7c7df: Makefile.am: Run pylint on all Python code
- That is, all Python code except unittests. In particular,
autotool/build-bash-completion was missing from the
previou... - 07:08 pm Revision f59418db: build-bash-completion: Fix a few pylint warnings
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 07:08 pm Revision bdba4757: check-python-code: Use “set -e” to abort on errors
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:45 pm Revision e2e92ea0: Generate hmac file with a newline at the end
- This makes it slightly easier to cut&paste its content.
Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed... - 05:47 pm Revision d2c8afb1: jqueue: Don't return negative number for unchecked jobs when archiving
- When the queue was empty, the calculation for unchecked jobs while
archiving would return -1. ``last_touched`` is set... - 03:03 pm Revision 9fe72672: cli.GenerateTable: Don't write EOL spaces
- With this change, there won't be unnecessary space characters
at the end of lines.
Signed-off-by: Michael Hanselmann... - 03:03 pm Revision 9fa2e150: Improve logging for workerpool tasks by providing __repr__
- Before it would log something like “starting task
(<ganeti.http.client._HttpClientPendingRequest object at 0x2aaaad17... - 03:03 pm Revision 02fc74da: workerpool: Simplify log messages
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:03 pm Revision d16e6fd9: workerpool: Use worker name as thread name
- This way it shows up in debug logs.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <i... - 03:02 pm Revision 89e2b4d2: workerpool: Make worker ID alphanumeric
- Having a proper name instead of just a number makes debugging
easier.
Signed-off-by: Michael Hanselmann <hansmi@goog... - 03:02 pm Revision 71e1863e: locking: Fix race condition in LockSet
- This patch fixes a race condition when acquiring all locks in
a LockSet instance. The list of lock names needs to be ... - 03:02 pm Revision 4776e022: mcpu: Log lock status with sorted names
- Reading and comparing sorted lists is easier when debugging locking problems.
Signed-off-by: Michael Hanselmann <han... - 03:02 pm Revision 9b154270: locking: Append to list outside error handling block
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 03:02 pm Revision 56452af7: locking: Don't fail in error handling if lock isn't owned
- In case an exception was thrown while acquiring the lock, not necessarily all
owned locks are also really acquired. B...
Also available in: Atom