Activity
From 04/09/2011 to 05/08/2011
05/06/2011
- 06:47 pm Revision e3303a4e: Fix instance failover/migration w.r.t TLMigrateInstance
- Commit 1c6e5787 removed the iallocator and target_node keyword
parameters from TLMigrateInstance, but I didn't update... - 05:35 pm Revision 931413f6: Rename instance: only show new name when different
- It makes not sense to show messages like:
Fri May 6 02:04:01 2011 - INFO: Resolved given name 'instance18' to
'inst... - 01:46 pm Revision 9d5b1371: Fix race condition in LUGroupAssignNodes
- The original code would get all node information and their groups
without before acquiring the necessary locks. With ...
05/05/2011
- 02:25 pm Revision 1c6e5787: Fix DTS_EXT_MIRROR migration
- Commit faaabe3c fixed failover behaviour for DTS_INT_MIRROR instances, however
it broke migration for DTS_EXT_MIRROR ... - 02:09 pm Revision 8b0273a5: Use node group locking for replacing disks
- This is one of the first opcodes to make use of node group locking. To
get an instance's node groups, the instance's ... - 02:09 pm Revision 2674690b: config: Add function to determine instance's groups
- This will be used for locking only the necessary node group(s)
for per-instance operations.
Signed-off-by: Michael H... - 12:41 pm Revision 54bd7d76: TLMigrateInstance: Fix live migration breakage
- Commit 77fcff4 unintentionally incorporated code from
TLMigrateInstance.CheckPrereq into TLMigrateInstance._RunAlloca...
05/04/2011
- 05:19 pm Revision 46c936d6: cmdlib: Update error messages, remove some punctuation
- - Clarify some error messages
- Remove unnecessary punctuation
- Merge two if conditions in one place
Signed-off-by:... - 03:13 pm Revision 9d0671ba: Re-wrap and fix formatting issues in gnt-instance.rst
- This is mostly rewrapping plus fixing a few small issues in
gnt-instance.rst.
Signed-off-by: Iustin Pop <iustin@goog... - 12:25 pm Revision 81dd9bd1: Documentation for the new parameters for KVM
- Options added/updated are: cdrom2_image_path, floppy_image_path,
cdrom_disk_type and boot_order.
Signed-off-by: Iust...
05/03/2011
- 06:53 pm Revision 958d01f8: cmdlib: Fix typo, s/nick/NIC/
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 12:25 pm Revision 8dddc5bc: A small optimisation in cluster verify
- This removes (count of instances + count of nodes) lock
acquires/releases.
Signed-off-by: Iustin Pop <iustin@google....
05/02/2011
- 04:08 pm Revision 72740756: A few docstring fixes
- At least one generates an epydoc error :)
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselm... - 03:41 pm Revision d143f2c6: luxi: do not handle KeyboardInterrupt
- With the current code, it's possible to mistake a ^C for a protocol
error:
node1# gnt-job info 221691
[press ^C]
Unh... - 03:41 pm Revision 225e2544: Handle EPIPE errors while writing to the terminal
- This handles EPIPE errors in two places: ToStream (to catch logging
done in GenericMain itself) and in GenericMain (t... - 03:41 pm Revision 20d317d4: Cluster verify: check for missing bridges
- Currently cluster verify doesn't check for bridge information; the
only checks are done at instance create and failov... - 12:16 pm Revision cc130cc7: Use floppy disk and a second CDROM on KVM
- Hi all,
this patch will add 3 new KVM parameters and a new option.
New Parameters:
- floppy_image_path = "" -> Spec...
04/29/2011
- 05:15 pm Revision 52f33103: cmdlib: Factorize lock releasing
- There will be more lock releasing with upcoming changes, so this will
centralize the logic behind it (what locks to k... - 04:01 pm Revision 3de8301d: Merge branch 'devel-2.4'
- * devel-2.4:
TLReplaceDisks: Use implicit loop for dictionary
Release unneeded locks while replacing disks
lock... - 03:56 pm Revision 29b8eaee: TLReplaceDisks: Use implicit loop for dictionary
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:55 pm Revision 1bee66f3: Release unneeded locks while replacing disks
- If an iallocator is used, “gnt-instance replace-disks” would acquire the
locks of all nodes (only the allocator will ... - 03:41 pm Revision 07cba1bc: locking: Export “list_owned” from lock manager
- This is analog to “is_owned” and will be used for assertions.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
... - 03:41 pm Revision d8d838cb: gnt-instance: Fix typo in error message
- The iallocator parameter is “-I”, not “-i”.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iusti... - 03:38 pm Revision 580c971e: Merge branch 'devel-2.4'
- * devel-2.4: (24 commits)
mlock: fail gracefully if libc.so.6 cannot be loaded
Allow creating the DRBD metadev in... - 03:05 pm Revision adc523ab: mlock: fail gracefully if libc.so.6 cannot be loaded
- This allows noded to continue instead of blowing up if the libc major
number changes.
Signed-off-by: Iustin Pop <ius... - 02:48 pm Revision 661515f6: cmdlib: Drop SSH runner from LU base class
- It is no longer used.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
04/28/2011
- 07:02 pm Revision 0b606e54: cmdlib.py: fix indentation in _VerifyNode
- Signed-off-by: Adeodato Simo <dato@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 05:26 pm Revision c6627031: TLMigrateInstance: Fix confusing text
- Commit d5cafd31 changed this error message, swapping the
text parts in the process.
Signed-off-by: Michael Hanselman... - 05:25 pm Revision a2e885ee: LUInstanceRename: Amend comment about lock
- Also add an assertion.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.... - 05:25 pm Revision 342f9172: iallocator: Relocation nodes must be in same group
- Quoting from iallocator.rst: “[…] ``relocate`` request is used when an
existing instance needs to be moved within its... - 03:24 pm Revision 55f67cc9: Fix 'unused import' lint error
- Sorry!
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:06 pm Revision 8f9a87c5: SetEtcHostsEntry: maintain existing ordering
- Currently RemoveEtcHostsEntry keeps the ordering, but SetEtcHostsEntry
not, as it will always write the new entry at ... - 01:06 pm Revision 24c855a7: Convert utils.nodesetup to utils.WriteFile(data=…)
- It makes no sense to iteratively write the new etc/hosts file, as we
can pre-compute the desired contents (neither th... - 01:03 pm Revision 87001920: Allow creating the DRBD metadev in a different VG
- This is a simple change to allow specifying a different VG for the
meta device during the creation of instances and a... - 01:03 pm Revision c260fa25: Make _GenerateDRBD8Branch accept different VG names
- This is a small change to make this function take a list of VG names,
instead of a single one.
Signed-off-by: Iustin... - 01:02 pm Revision 1d39e245: Fix WriteFile with unicode data
- Unicode is fun, indeed:
>>> len(buffer("abc"))
3
>>> len(buffer(u"abc"))
12
So we can't pass unicode data to buffer...
04/27/2011
- 06:26 pm Revision 88aa7f66: Fix for multiple VGs - PlainToDrbd and replace-disks
- Converting an instance from 'plain' to 'drbd'. The old code would
create the drbd volumes in the default VG and then... - 06:26 pm Revision fd09d178: Replace disks: keep the meta device in the same VG
- This patch enhances the multi-VG support in replace disks, by keeping
the meta device in the same VG, as opposed to m... - 06:26 pm Revision d833acc6: Prevent readding of the master node
- This breaks Ganeti in multiple ways. If we don't make the check in
gnt-node itself, then bootstrap.SetupNodeDaemon wi... - 06:26 pm Revision cce6f357: Fix punctuation in an error message
- IIRC we don't use punctuation at the end of error messages.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-b... - 06:26 pm Revision 2db04578: Improve error messages in cluster verify/OS
- A few issues in the clarity of the error messages are fixed:
- "ERROR: node node3: OS API version lenny-image": no p... - 06:26 pm Revision 437c3e77: Fix potential data-loss in utils.WriteFile
- os.write can do incomplete writes, as long as at least some bytes have
been written (like write(2)):
>>> os.write(fd...
04/21/2011
- 04:08 pm Revision fe508a9d: QA: Add tests for node group tags
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:08 pm Revision 414ebaf1: RAPI: Add support for tagging node groups
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:08 pm Revision 1ffd2673: masterd: Add support for tagging node groups
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 04:08 pm Revision 819cbfe5: gnt-group: Add commands for tagging groups
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:41 pm Revision dadf6b7d: cli: Fix wrong argument kind for groups
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:41 pm Revision b5a418aa: Quote filename in gnt-instance.8
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 02:53 pm Revision 24fed61e: QA: Adding a config option to disable cluster epo
- Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
04/20/2011
- 04:54 pm Revision 0f7098f3: TLMigrateInstance: remove 10s sleeps
- TLMigrateInstance._ExecMigration contains two 10-second sleeps between
individual migration steps.
Apart from prolon... - 03:37 pm Revision 97b40f39: Fix typo in LUGroupAssignNodes
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 02:54 pm Revision 5c097318: gnt-instance info: automatically request locking
- Commit dae661a4 added support for controlling the locking, but it
didn't modify the gnt-instance info code, which lea... - 01:35 pm Revision bee8c465: Document the dependency on OOB for gnt-node power
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com>
04/19/2011
- 07:41 pm Revision 675e2bf5: Fix master IP activation in failover with no-voting
- Thanks to net.for.hub@gmail.com for reporting this. The logic in
masterd.CheckMasterd did an early return in case of ... - 06:41 pm Revision 6e7f0cd9: disk wiping: fix bug in chunk size computation
- The current wipe_chunk_size computation is doing min(int_value,
float_value). For small disks (below 10GiB), the actu... - 04:11 pm Revision 9ff4f2c0: Update manpages and other documents with editor settings
- No rewrapping is done in this patch, just updates to the settings.
Signed-off-by: Michael Hanselmann <hansmi@google.... - 03:56 pm Revision 1b1a08e8: gnt-group list: Query filter support
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 03:56 pm Revision 2afd577f: gnt-node list: Query filter support
- Update manpage, quote field names.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iu... - 03:54 pm Revision 87e87959: gnt-instance list: Query filter support
- Update manpage, quote field names.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iu... - 03:38 pm Revision a0aa6b49: Fix bug in watcher
- If “utils.RunParts” were to raise an exception, a log message was
written and the code continued to run. Due to the e...
04/18/2011
- 06:55 pm Revision b107fe05: opcodes: Change parameter type definition for query filter
- The old definition wouldn't accept integers.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iust... - 06:55 pm Revision 442587bf: cli: Add option to force names to be treated as filter
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:55 pm Revision 2928de47: cli: Add support for parsing query filters
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:55 pm Revision 3f2f55bb: qlang: Add function to distinguish filters from names
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:55 pm Revision cb1e6c3c: cli: Error reporting for query filter parsing
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:55 pm Revision 7578ab0a: qlang: Add parser for query filter language
- With this parser, command line utilities will be able to provide filters
through query2 in a simplistic language. Exa... - 06:55 pm Revision 3802f3cf: Update ganeti.7 manpage for query filter language
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 12:58 pm Revision 6a062ff9: htools: make some error messages more explicit
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 11:56 am Revision 7c670076: Add instance query field for OS parameters
- These were not available as a query field before. Update unittests
and description text for the other “..params” fiel...
04/14/2011
- 07:35 pm Revision 65a884ef: QA: also run gnt-cluster repair-disk-sizes
- So that we don't happen again to break this forever without realising
it.
The patch also replaces one ' with ".
Sig... - 01:33 pm Revision 4a2c0db0: Release locks before wiping disks during instance creation
- Ganeti 2.3 introduced an optional feature to overwrite an instance's
disks on creation. Unfortunately the code kept a... - 11:37 am Revision e8f472d1: Fix shared_file_storage_dir on upgrades
- If the cluster was upgraded from 2.4 or earlier, this key won't exist
(it's only set to a correct value on cluster in... - 11:37 am Revision 3e8b5a9c: QA: run the redist-conf command
- This was (AFAICS) completely missing from the QA suite.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: G... - 11:36 am Revision 2afc9238: Prevent ssconf values from having non-string values
- For whatever reason, my test cluster managed to acquire
shared_file_storage_dir with a None value, instead of empty
s... - 11:35 am Revision 9cbc1edb: Add some tests for the auto_balance attribute
- It tests node add/remove secondary, rather than cluster-level N+1
checks, but it's better than nothing.
Signed-off-b... - 11:35 am Revision 929b60d8: Node operations: take into account auto_balance
- This patch changes the add to secondary/remove from secondary code to
not deduct/add the instance's memory if the ins... - 11:35 am Revision bc782180: Read/write auto_balance via Text
- This also means _another_ change in the text format; we really should
move to json…
The unittests are also update fo... - 11:34 am Revision a041ebb5: Read auto_balance via Rapi
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Adeodato Simo <dato@google.com> - 11:34 am Revision 6880526c: Read auto_balance via Luxi
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Adeodato Simo <dato@google.com> - 11:33 am Revision 93439b1c: Show the auto_balance flag in the instance listing
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Adeodato Simo <dato@google.com>
04/13/2011
- 02:22 pm Revision ccfa86ba: cli: Replace hardcoded strings with constants
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 01:01 pm Revision a9d68e40: utils.WriteFile: Close file before renaming
- Issue 154 (http://code.google.com/p/ganeti/issues/detail?id=154)
reported an “Operation not supported” error when wri... - 01:01 pm Revision 154d7ba5: Fix distcheck
- README is not copied to the build tree.
Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Po... - 01:01 pm Revision accbf5e3: Nicer formatting for group query error
- Before this patc the message would look like “Some groups do not exist:
[u'foo', u'bar']”, now it's “Some groups do n... - 01:00 pm Revision 69d1b79d: gnt-instance.8: Fix wrongly formatted title
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
04/12/2011
- 11:55 am Revision 49e11c49: Some more changes to Makefile.am for htools
- I duplicate the BINARY= rule in the ghc invocation in order to be able
to silence the if, which was confusing.
Addit... - 11:55 am Revision c352b0a9: Add a new attribute to Instance.Instance
- This will mirror Ganeti's be/auto_balance one, which we need to use to
properly match N+1 computations.
Signed-off-b...
Also available in: Atom