Activity
From 08/25/2008 to 09/23/2008
09/23/2008
- 03:10 pm Revision a1578d63: Switch to static minors for DRBD
- With some todos remaining, this patch switches the DRBD devices to use
the passed minors, and the cmdlib code (add in... - 03:10 pm Revision a81c53c9: Implement config support for drbd static minors
- This patch adds support for allocating static minors.
Like for the LVM uuids, we add a new cache for the temporarily... - 03:10 pm Revision 468b46f9: Fix disk replace secondary with static minors
- The code in 'updating instance configuration' section of the replace
disks with change secondary node was setting a w...
09/22/2008
- 02:32 pm Revision ffa1c0dc: Extend DRBD disks with minors attribute
- This patch converts the DRBD disks to contain also a minor (per each
node) attribute. This minor is not yet used and ...
09/18/2008
- 02:13 pm Revision 3fa93523: Apply filter properly in LUQuery{Nodes, Instances}
- Currently when not locking all nodes/instances are returned, regardless
if the user asked only for some of them. With... - 02:12 pm Revision c2c2a903: Remove auto_balance from burnin/cmdlib
- There is no such feature in trunk yet.
Reviewed-by: iustinp
09/17/2008
- 07:07 pm Revision ca0aa6d0: Add utils.ReadFile function
- It abstracts exception handling and is like a complement to
utils.WriteFile.
Reviewed-by: iustinp - 05:51 pm Revision b5b22be2: doc/locking.txt: Job queue functions can be called from LUs
- The previous description was unclear. Reported by Guido Trotter.
Reviewed-by: ultrotter
09/12/2008
- 11:23 am Revision 6e2dc934: Make burnin aware of hvm device type flags.
- Merged from branches/ganeti/ganeti-1.2 r1648
Use static values for new hvm instance flags
Reviewed-by: iustinp
09/11/2008
- 08:45 pm Revision 64d3bd52: GetAllInstancesInfo, change internal iterator name
- GetAllInstancesInfo used "node" as an iterator name. Change it to
instance to make it less confusing.
Reviewed-by: i... - 08:45 pm Revision 746ea1da: gnt-instance: fix tags commands online help
- It used to refer to "nodes", which was confusing.
Reviewed-by: iustinp - 08:45 pm Revision 8646adce: Parallelize Tag operations
- For now we lock the instance/node for adding/deleting tags from it, but
we could probably in the future do without, w... - 08:44 pm Revision c53279cf: Parallelize LUSetClusterParams (and add a FIXME)
- Reviewed-by: imsnah
- 12:46 pm Revision 87765cb0: Fix a typo in gnt-instance.sgml
- Reviewed-by: imsnah
- 12:44 pm Revision 3656b3af: Parallelize LURemoveExport
- Reviewed-by: imsnah
- 12:44 pm Revision cf472233: Parallelize LURemoveInstance
- Using the new add/remove infrastructure this becomes pretty easy! :)
Reviewed-by: imsnah - 12:44 pm Revision 7baf741d: Parallelize LUCreateInstance
- Finally, instance create on different node, without iallocator, can run
in parallel. Iallocator usage still needs all... - 12:44 pm Revision ca2a79e1: Implement adding/removal of locks by declaration
- With this patch LUs can declare locks to be added when they start and/or
removed after they finish. For now locks can... - 12:44 pm Revision d2aff862: LockSet: forbid add() on a partially owned set
- This patch bans add() on a half-acquired set. This behavior was
previously possible, but created a deadlock if someon... - 12:43 pm Revision ab62526c: Fix typo in a locking.py comment
- Reviewed-by: imsnah
- 12:43 pm Revision 80ee04a4: Use is_owned to determine whether to unlock
- Now that is_owned is public we don't need to play games at the end of an
LU. If we're still owning anything we just r... - 12:43 pm Revision d4f4b3e7: Add GanetiLockManager.is_owned function
- This is a public version of the private function we already had.
We don't just change the previous version because it... - 12:43 pm Revision d4803c24: Fix LockSet._names() to work with the set-lock
- If the set-lock is acquired, currently, the _names function will fail on
a double acquire of a non-recursive lock. Th... - 11:25 am Revision aefef4f4: Add gnt-instance (start|stop) --submit
- Finish the --submit changes with these two, which (because they are
multi-opcode commands) require special handling.
...
09/10/2008
- 08:46 pm Revision e74798c1: jqueue: Add common RPC error handling function
- We didn't decide yet what exactly it should do with failed nodes.
Reviewed-by: ultrotter - 08:07 pm Revision 57a2fb91: Remove locking of instances in certain queries
- This patch is similar to the node patch (rev 1650). We disable locking
of instance (and nodes) if we only query stati... - 08:07 pm Revision 0b2de758: Add an atomic ConfigWrite.GetAllInstanceInfo()
- In order to be able to query instance without locking them, we need the
same atomic query of multiple instances as fo... - 08:06 pm Revision 94bbfece: Add ConfigWriter._UnlockedGetInstanceList/Info()
- This patch splits the GetInstanceInfo and GetInstanceList methods into
two parts, one locked one _Unlocked similar to... - 08:06 pm Revision b7329c9c: Do not use jobs in gnt-instance _ExpandNames()
- In the gnt-instance script, _ExpandNames() uses jobs to query instance
names. This is not optimal, so we change it to... - 08:06 pm Revision 6340bb0a: Implement "--submit" on gnt-instance
- This patch adds support for the “--submit” parameter in the gnt-instance
script, for the commands where it makes sens... - 06:43 pm Revision e9d741b6: Rewrite the 'only submit job' handling in scripts
- The "sys.exit(0)" was not nice as you couldn't differentiate it from
other exit codes. We change this to a specially ... - 02:03 pm Revision c8d8b4c8: Optimize the OpQueryNodes for names only
- Currently, OpQueryNodes is locking all nodes (in shared mode), which
will also block the special case of querying onl... - 02:02 pm Revision d65e5776: Add a way to export all node information at once
- The patch adds a new function to export all node information at once
(i.e. atomically with respect to the configurati...
09/09/2008
- 03:57 pm Revision 8785cb30: ganeti-noded: Add constant for queue lock timeout
- Reviewed-by: iustinp
- 03:47 pm Revision 1bc59f76: Never remove job queue lock in node daemon
- Otherwise, corruption could occur in some corner cases. E.g. when
LeaveNode is running in a child and is in the proce... - 03:25 pm Revision 36205981: Implement master startup safety check
- This is an initial version of the master startup checks. It's a very
rudimentary change, however in normal usage (an ... - 03:24 pm Revision 4e071d3b: Export backend.GetMasterInfo over the rpc layer
- We create a multi-node call so that querying all nodes for agreement
will be fast.
Reviewed-by: imsnah - 03:24 pm Revision bd1e4562: Change backend._GetMasterInfo to return more data
- The _GetMasterInfo() function needs to export the master name too to be
useful in master safety checks. This patch ma... - 01:42 pm Revision a987fa48: Parallelize LUQueryInstanceData
- Reviewed-by: iustinp
- 01:42 pm Revision d4b9d97f: Parallelize LUVerify{Cluster,Disks}
- These are two easy querying LUs which require shared access to all
nodes/instances.
Reviewed-by: iustinp - 01:41 pm Revision efd990e4: Parallelize LUReplaceDisks
- This is the most complex parallelization so far. We have to lock one
instance (and its nodes) plus one more node if d... - 01:41 pm Revision 9513b6ab: _LockInstancesNodes: support append mode
- This will be used to lock the instance's nodes in addition to some more.
Reviewed-by: iustinp - 01:41 pm Revision b2751b57: Processor: remove ChainOpCode
- This function was incompatible with the new locking system, and its
usage has been removed from the code. For now LUs... - 01:41 pm Revision f22a8ba3: Parallelize LU{A,Dea}ctivateInstanceDisks
- Now that they are not used in other opcodes by chaining,
this can easily be done.
Reviewed-by: iustinp - 01:40 pm Revision 023e3296: LUReplaceDisks: remove use of ChainOpCode
- The calls to OpActivateInstanceDisks and OpDeactivateInstanceDisks has
been replaced by _StartInstanceDisks and _Safe... - 01:40 pm Revision 155d6c75: Create new _SafeShutdownInstanceDisks function
- This new function checks whether an instance is running, before shutting
down its disks. This is what the Exec() of L... - 01:40 pm Revision 3a5d7305: Fix a typo in LogicalUnit.ExpandNames docstring
- s/locking.LEVEL_INSTANCES/locking.LEVEL_INSTANCE/
Reviewed-by: iustinp - 01:40 pm Revision f6d9a522: Use constants.LOCKS_REPLACE instead of hardcoding
- This constant replaces what we used to write in recalculate_locks, and
represents the lock recalculation mode. It liv... - 12:39 pm Revision de8c7666: Fix LUReplaceDisks with iallocator
- self._RunAllocator() sets self.op.remote_node, but doesn't return the
new remote node. If we set it to the return val... - 12:01 pm Revision 506cff12: Use lock timeout for queue updates in ganeti-noded
- This helps to prevent complete deadlocks.
Reviewed-by: iustinp
09/08/2008
- 06:54 pm Revision 86de84dd: Fix LUGrowDisk
- The rpc library returns a list, not a tuple, so we'll accept both.
Reviewed-by: iustinp - 06:53 pm Revision 43f5ea7a: Fix iallocator run
- The rpc library returns a list, not a tuple, so we'll accept both.
Reviewed-by: iustinp - 06:53 pm Revision dcde0241: OpVerifyDisks returns a list, not a tuple
- Fixing the check in gnt-cluster, or gnt-cluster verify-disks is broken.
Since the version in 1.2 used to return a tup... - 04:44 pm Revision 6657590e: Parallelize LUExportInstance
- Unfortunately for the first version we need to lock all nodes. The patch
discusses why this is and discuss ways to im... - 04:44 pm Revision 31e63dbf: Parallelize LUGrowDisk
- Reviewed-by: iustinp
- 04:43 pm Revision 849da276: LURebootInstance: lock only primary when possible
- When rebooting an instance and we're not changing it's disks status (all
the cases except in a "full" reboot) we can ... - 04:43 pm Revision a82ce292: Add primary_only flag to _LockInstancesNodes
- As the name says when the flag is on (the default is off) only the
primary nodes are locked, as opposed to all of the...
09/05/2008
- 06:38 pm Revision aa74b828: utils.FileLock: Implement timeout
- The timeout can be used in ganeti-noded to be more robust against
deadlocks.
Reviewed-by: iustinp - 04:49 pm Revision a25c1b2a: Add lock documentation for job queue and ganeti-noded
- Also change title formatting to match client-api.txt.
Reviewed-by: iustinp - 03:29 pm Revision f1f3f45c: noded: Get job queue lock while purging queue content
- Only one process should modify the queue at the same time.
Reviewed-by: iustinp - 03:19 pm Revision 26a61f87: QA: Remove dry run mode
- It didn't work as planned because some commands depend on the return
value or output of some operations.
Reviewed-by... - 02:00 pm Revision e310b019: Add locking.ALL_SET constant and use it
- Rather than specifying None in needed_locks every time, with a nice
comment saying to read what we mean rather than w... - 01:57 pm Revision 45bc5e4a: utils.SplitTime: More rounding fixes
- SplitTime didn't round the same on different platforms. This patch changes
it to use microseconds and not care about ... - 01:40 pm Revision e0458af5: Remove bom-byte
- This is not nice, removing it :)
Please use 'set nobomb' in your vi init file.
Reviewed-by: ultrotter
09/04/2008
- 06:12 pm Revision ea47808a: Prevent mistakes using _GetWantedNodes
- All the users of _GetWantedNodes have been converted to be concurrent
LUs, and thus cannot call this function with an... - 06:12 pm Revision 21a15682: Paralleliza LUQueryNodeVolumes and LUQueryExports
- Reviewed-by: iustinp
- 06:12 pm Revision 6bf01bbb: Parallelize LUDiagnoseOS
- Reviewed-by: iustinp
- 06:12 pm Revision 895ecd9c: LUQueryExports: make 'node' field mandatory
- It turns out this fields was already mandatory. If it hadn't beed valid,
in fact, a value of None would have been pas... - 06:11 pm Revision 204f2086: s/Chain(OpQueryExports)/rpc.call_export_list(...)/
- Parallel opcodes are not (yet?) supported for chaining. Turns out
though that chaining is used only four times in the... - 06:11 pm Revision b91a34a5: Fix wrong indentation in LUQueryNodes
- Reviewed-by: iustinp
- 05:53 pm Revision d0c11cf7: Merge r1607 from branches/ganeti/ganeti-1.2
- Use a default vnc_bind_address if None is specified
Reviewed-by: iustinp
09/02/2008
- 07:23 pm Revision e1d2aa39: merge r1569 from branches/ganeti/ganeti-1.2
- Implement more options for gnt-backup import
Reviewed-by: ultrotter - 03:57 pm Revision 3fb1e1c5: merge r1568 from branches/ganeti/ganeti-1.2
- Add more fields to gnt-instance list
Reviewed-by: imsnah - 03:15 pm Revision 6291574d: merge r1548 from branches/ganeti/ganeti-1.2
- Fix wrong wording of instance rename error message.
Reviewed-by: imsnah - 03:12 pm Revision 16034f1f: merge r1547 from branches/ganeti/ganeti-1.2
- Document behaviour of gnt-instance console for HVM
Reviewed-by: imsnah - 02:46 pm Revision 20e23543: merge r1542, r1543, r1573 from branches/ganeti/ganeti-1.2
- Implement interactive instance OS reinstall.
Reviewed-by: ultrotter - 12:09 pm Revision a4273aba: merge r1541 from branches/ganeti/ganeti-1.2
- more information for VNC console port
Reviewed-by: ultrotter - 11:42 am Revision 04c4330c: merge r1540 from branches/ganeti/ganeti-1.2
- Allow access to HVM serial console
Reviewed-by: imsnah
09/01/2008
- 07:05 pm Revision 34b6ab97: merge r1539 from branches/ganeti/ganeti-1.2
- Display VNC console port in gnt-instance info.
Reviewed-by: iustinp - 05:12 pm Revision 5bc84f33: merge r1538 from branches/ganeti/ganeti-1.2
- Check HVM device type on instance modify as well.
Reviewed-by: imsnah - 02:37 pm Revision cfefe007: Check memory size before setting it
- With this change when a user asks for a new memory size for an instance,
the number is checked instead of just applie... - 02:37 pm Revision 4300c4b6: Pass the force param to SetInstanceParms
- It was already allowed in gnt-instance modify, but ignored.
It will be used to force skipping parameter checks.
This...
08/29/2008
- 07:57 pm Revision 02d2adf4: Merge r1534 from branches/ganeti/ganeti-1.2
- Add HVM device type flag 4/4
Reviewed-by: ultrotter - 07:30 pm Revision ad2023a3: Merge r1537 from branches/ganeti/ganeti-1.2
- Add HVM device type flags 3/4
Reviewed-by: ultrotter - 07:17 pm Revision 5397e0b7: Merge r1536 from branches/ganeti/ganeti-1.2
- Add HVM device type flags 2/3
Reviewed-by: ultrotter - 06:04 pm Revision b77ba978: utils.SplitTime: Fix rounding of milliseconds
- Reported by Iustin.
It used to return this:
>>> utils.SplitTime(1234.999999999999)
(1234, 1000)
while it should've ... - 06:01 pm Revision b894f5a8: merge r1535 from branches/ganeti/ganeti-1.2
- Add HVM device type flags 1/4
Reviewed-by: ultrotter - 05:41 pm Revision c6520286: Merge r1296 from branches/ganeti/ganeti-1.2
- doc fix: Describe default values for HVM instance options & cleanup.
Reviewed-by: iustinp - 04:57 pm Revision 56db8a84: Merge r1295 from branches/ganeti/ganeti-1.2
- Clarify cluster IP requirement.
Reviewed-by: iustinp - 04:42 pm Revision 5c735209: Make WaitForJobChanges deal with long jobs
- This patch alters the WaitForJobChanges luxi-RPC call to have a
configurable timeout, so that the call behaves nicely... - 03:47 pm Revision 3fc175f0: merge r997 from branches/ganeti/ganeti-1.2
- Fix gnt-instance modify for HVM parameters
This patch makes gnt-instance modify work again for the advanced
HVM para... - 03:45 pm Revision 0f933d15: Add doc/locking.txt, documenting locking order
- Reviewed-by: imsnah
08/28/2008
- 06:35 pm Revision 082c5adb: Fix error message when masterd is not listening
- Reported by Iustin.
Reviewed-by: iustinp - 01:29 pm Revision 6683bba2: Fix issue when acquiring empty lock sets
- By design if an empty list of locks is acquired from a set, no locks are
acquired, and thus release() cannot be calle...
08/27/2008
- 05:52 pm Revision 5685c1a5: jqueue: Replace normal cache dict with weakref dict
- A job should only exist once in memory. After the cache is cleaned,
there can still be references to a job somewhere ... - 05:52 pm Revision 70552c46: jqueue: Keep timestamp of opcode start and end
- Reviewed-by: ultrotter
- 05:48 pm Revision 65548ed5: jqueue: Reset run_op_idx after job is done
- It can be confusing otherwise.
Reviewed-by: ultrotter - 01:05 pm Revision e17188f4: Another burnin fix
- This is a result of the log timestamp changes.
Reviewed-by: imsnah - 12:55 pm Revision 6abe9194: Fix a small typo in a constant
- Seems noone ran a burnin lately :)
Reviwed-by: amischenko,ultrotter - 11:34 am Revision 6c5a7090: Make sure that client programs get all messages
- This is a large patch, but I can't figure out how to split it without
breaking stuff. The old way of getting messages...
08/26/2008
- 06:53 pm Revision 305cb9bb: QA: Use pseudo-tty via SSH
- This gives continous output instead it being buffered.
Reviewed-by: ultrotter - 06:44 pm Revision e67bd559: Add simple lock debug output
- Currently it can only be enabled by modifying utils.py, but we can
add a command line parameter later if needed.
Rev...
08/25/2008
- 05:57 pm Revision e50c3a9e: Use python2.4 when developing
- Reviewed-by: ultrotter
- 05:56 pm Revision 39e0d19e: Remove references to YAML
- I forgot to remove these when converting the QA configuration from YAML
to JSON.
Reviewed-by: ultrotter
Also available in: Atom