Activity
From 12/14/2008 to 01/12/2009
01/12/2009
- 06:06 pm Revision 4040a784: Skip offline nodes in gnt-cluster commands
- This patch makes gnt-cluster copyfile and command skip the offline
nodes.
Reviwed-by: ultrotter, imsnah - 03:25 pm Revision 5c22d16e: burnin: Add tests for add/remove disks and NICs
- This patch adds testing of add/remove disks and NICs to the burnin.
Reviewed-by: imsnah - 02:42 pm Revision 19708787: Heavy redo of gnt-instance info output
- In 2.0, we have more parameters in drbd's logical_id, and passing the
results over json makes them unicode which look... - 02:42 pm Revision 4cfb9426: Fix some errors in instance modify --disk remove
- The RpcResult introduction still left some bugs (after multiple patches):
- we don't correctly check the result typ... - 12:27 pm Revision f57c76e4: Fix an error handling case in instance info
- The checking for invalid instance names in LUQueryInstanceData is broken
since commit 1642.
Reviewed-by: imsnah - 11:14 am Revision afee0879: Introduce a very simple LU to force config updates
- This LU can be used to force a push of the config in case it's needed,
for example after an upgrade to update the ssc...
01/09/2009
- 06:24 pm Revision 8a113c7a: Add a new ssconf file with the ganeti version
- The patch adds a new ssconf file containing the ganeti version.
Reviewed-by: imsnah - 05:34 pm Revision 7d585316: Work around a DRBD sync speed race condition
- This is modified forward-port of commit 1544 on the 1.2 branch:
When DRBD is doing its dance to establish a connec... - 04:58 pm Revision 90e722d1: burnin: Add activate/deactivate disks
- Reviewed-by: imsnah
- 04:58 pm Revision cfacfd6e: burnin: use the new replace_disks constants
- This patch updates burnin to the latest replace disks constant, and
changes the constants' values to be more accurate... - 04:26 pm Revision 729164d0: burnin: do not use offline nodes
- This patch makes burnin skip the offline nodes in it's builtin node
selection. It also removes an extra line.
Review... - 04:26 pm Revision 94a02bb5: Fix gnt-os for offline nodes
- We shouldn't query offline nodes in gnt-os. This patch adds an utility
function to ConfigWriter that returns the name... - 02:52 pm Revision 186ec53c: Silence warning on node list for offline nodes
- The warning in node list is meant for nodes that return wrong
information, but for offline nodes this case is normal.... - 02:52 pm Revision 7d88772a: Rework the daemonization sequence
- The current fork+close fds sequence has deficiencies which are hard to
work around:
- logging can start logging bef... - 02:22 pm Revision 7e9366f7: Cleanup replace-disks modes and options
- In 1.2, due to the md+drbd7 legacy, we had a complex choice of replace
modes, and the new drbd8 modes where forced in...
01/08/2009
- 06:39 pm Revision 82e37788: Fix cluster verify/node net test for offline nodes
- For offline nodes, we shouldn't add them to the NV_NODELIST and
NV_NODENETTEST tests since they most likely won't suc... - 06:05 pm Revision 3247bbac: rpc: Add a method for easy check of remote results
- The patch adds a new method to the rpc.RpcResult class called
"RemoteFailMsg" which is useful for the RPC calls which... - 04:16 pm Revision 56e7640c: Add an instance_migratable rpc call
- This is a forward-port of commit 1194 on the 1.2 branch:
This call will check whether an instance is up on its pri... - 02:03 pm Revision cf8df3f3: bdev: forward-port ReAttachNet/DisconnectNet
- This is plain copy of the 1.2 ReAttachNet and DisconnectNet methods on
the DRBD8 device, with the logger to logging m...
01/07/2009
- 07:02 pm Revision 5282084b: backend: Remove symlinks by disk name
- This is a modified forward-port of commit 1184 on the 1.2 branch:
backend: Remove symlinks by disk name, not using... - 07:02 pm Revision b2e7666a: Pass instance name to rpc call blockdev_close
- This is an extract of commit 1166 on the 1.2 branch (Add a rpc call for
drbd network reconfiguration), but only the b... - 07:02 pm Revision 03dfa658: Fix the _RemoveBlockDevLinks() function
- This is a forward-port of commit 1163 on the 1.2 branch:
This fixes the removal of the instance symlinks (probably ... - 07:01 pm Revision 3c9c571d: Remove instance's symlinks
- This is a forward-port of commits 1150 and 1151 on the 1.2 branch:
Add _RemoveBlockDevLinks auxiliary function, cal... - 07:01 pm Revision ec596c24: Catch BlockDeviceError when starting instance
- This is a forward-port of commit 1149 on the 1.2 branch:
_GatherAndLinkBlockDevs used to raise the errors.BlockDevi... - 07:01 pm Revision 9332fd8a: Create symlinks to intances' block devices
- This is a forward-port of commit 1148 on the 1.2 branch:
Change the _GatherBlockDevs private function, called only ... - 07:01 pm Revision 069cfbf1: Simplify hypervisor block_devices structure
- This is a partial forward-port of commit 1136 on the 1.2 branch:
The hypervisor doesn't need to be passed the whol... - 04:38 pm Revision 2b17c3c4: _AssembleInstanceDisks: fix rpcresult handling
- Commit 2117 changed _AssembleInstanceDisks to correctly parse the
failure status of the new RpcResult structure, but ...
01/06/2009
- 11:57 am Revision e09fdcfa: Fix some pylint-detected issues
- Two bad indentation cases and a missing variable.
Reviewed-by: imsnah
12/19/2008
- 09:31 pm Revision 5b099da9: ganeti.bootstrap: Set permissions on newly uploaded files
- Reviewed-by: amishchenko
- 09:31 pm Revision 699777f2: ganeti.cmdlib: Check remote API certificate on "gnt-cluster verify"
- Reviewed-by: amishchenko
- 09:30 pm Revision 2438c157: ganeti.bootstrap: Upload remote API certificate to new nodes
- Reviewed-by: amishchenko
- 09:30 pm Revision 5557b04c: ganeti.bootstrap: Prepare for remote API certificate
- Reviewed-by: amishchenko
- 09:30 pm Revision c4415fd5: ganeti.bootstrap: Write SSL key to temporary file and set permissions
- Previously, we set the permissions only after writing the key. This
gave other users on the system a small window dur... - 09:30 pm Revision 61a08fa3: ganeti.bootstrap: Generate SSL certificate for remote API
- Reviewed-by: amishchenko
- 09:29 pm Revision 40a97d80: ganeti.bootstrap: Move SSL certificate generation into separate function
- Reviewed-by: amishchenko
- 02:58 pm Revision b5b67ef9: ganeti-rapi: Implement HTTP authentication
- Passwords are stored in "$localstatedir/lib/ganeti/rapi_users". User
options specify the access permissions of a user... - 02:58 pm Revision 7e9760c3: ganeti-rapi: Introduce per-request context
- This will be used to evaluate access permissions to resources.
Reviewed-by: amishchenko - 02:57 pm Revision e6e94655: ganeti.http: Function to read password file
- Lines in the password file are of the following format:
<username> <password> [options]
Fields are separated by w... - 02:57 pm Revision 68fa9caf: ganeti.http: Add support for private data in HTTP requests
- Reviewed-by: amishchenko
- 02:57 pm Revision be500c29: ganeti.http: Add support for basic HTTP authentication
- As per RFC2617.
Reviewed-by: amishchenko - 02:57 pm Revision f8bd7df3: ganeti.http: Prepare authentication for HTTP server
- The authentication class will override PreHandleRequest.
Reviewed-by: amishchenko
12/18/2008
- 06:39 pm Revision dd875d32: Job queue: Allow more than one file rename per RPC call
- Reviewed-by: ultrotter
- 06:38 pm Revision d7fd1f28: ganeti.jqueue: Group job archivals to reduce number of RPC calls
- Reducing the actual number of RPC calls will come in another patch.
Reviewed-by: ultrotter - 06:38 pm Revision f8ad5591: Prevent RPC timeout on auto-archiving jobs
- With a large job queue, auto-archiving jobs can take a very long time,
causing timeouts on the luxi RPC layer. With t... - 06:38 pm Revision 78d12585: jqueue: When auto-archiving jobs, calculate job status only once
- This is done by passing the job object to _ArchiveJobUnlocked instead
of only the job ID. Also return whether job was... - 06:23 pm Revision 58b22b6e: Use subdirectories for job queue archive
- As it turned out, having many files in a single directory can be
very painful. With this patch, only 10'000 files are... - 06:23 pm Revision 6e797216: Add rename function automatically creating directories if needed
- Unfortunately, os.makedirs in Python 2.4 is not safe against multiple
processes creating the same directory tree at t... - 06:21 pm Revision aea0ed67: ganeti.http: Don't pass poller object around
- They're cheap to instantiate and doing this changes makes the code
a bit simpler.
Reviewed-by: ultrotter - 03:45 pm Revision 79589f25: Rename http.HttpInternalError to HttpInternalServerError
- All other exceptions are named after the error name in RFC2616 (HTTP/1.1).
Reviewed-by: amishchenko - 03:45 pm Revision b3660886: ganeti.http: Add more constants and errors
- Reviewed-by: amishchenko
- 03:45 pm Revision 45eac583: ganeti.http: Ignore ENOTCONN when shutting down the connection
- Reviewed-by: amishchenko
- 03:44 pm Revision a8e01e9f: Implement support for additional headers with HTTP errors
- Reviewed-by: amishchenko
12/17/2008
- 04:30 pm Revision f30ca1e6: Add simple unittests for ganeti.http
- More complex unittests will need some refactoring in the HTTP code.
Reviewed-by: amishchenko - 04:09 pm Revision e38220e4: ganeti.bootstrap: Whitespace fix
- Reviewed-by: iustinp
- 03:18 pm Revision f87b405e: Add job queue size limit
- A job queue with too many jobs can increase memory usage and/or make
the master daemon slow. The current limit is jus... - 01:24 pm Revision 7167159a: utils.KillProcess: Use waitpid() to wait for child processes
- Sometimes the proc filesystem doesn't reflect the current status of
a process. By calling waitpid(), we make sure to ...
12/16/2008
- 06:24 pm Revision 26de0cf1: Release ganeti 2.0~alpha1
- Reviewed-by: iustinp
- 06:24 pm Revision 513e896d: LUConnectConsole: fix primary_node online check
- The primary node is part of the instance, not of the opcode.
Reviewed-by: iustinp - 06:24 pm Revision bf988c29: _RunCmdPipe: handle EINTR in poller.poll()
- poll() can be interrupted. rather than failing we retry until it
returns.
Reviewed-by: iustinp - 06:24 pm Revision 14aa53cb: KVM: improve socat interface
- Call socat with a full path specified at configure time, rather than
just by its name, and check for the binary to ex... - 06:23 pm Revision dd71f05b: KVM: use a different default kernel path
- It makes sense for the default kvm kernel not to be called "xenU".
Reviewed-by: iustinp
12/15/2008
- 12:06 pm Revision f088165d: ganeti.http: Add three TODOs for improvements
- Reviewed-by: iustinp
- 11:48 am Revision f2e13d55: ganeti.http: Explicitly initiate handshake
- Otherwise it would be done on the first read/write operation, making
error handling more difficult (such as EOF durin... - 11:40 am Revision d7fa9982: ganeti.http: Implement handshake socket operation
- Reviewed-by: iustinp
- 11:40 am Revision 39cfc25b: ganeti.http: Handle SSL_ERROR_ZERO_RETURN
- Also add a comment next to the place where the SSL connection is shut
down.
Reviewed-by: iustinp
12/14/2008
- 02:05 pm Revision 3c7f6c44: cleanup: ConfigWriter, initialize all attributes
- We should initialized the _last_cluster_serial in the constructor too (just to
be consistent).
Reviewed-by: amishchenko - 02:05 pm Revision f6f91001: cleanup: rapi v2 instance tags wrong attribute
- This was changed in the past, but it seems this class was forgotten.
Reviewed-by: amishchenko - 02:05 pm Revision 3f3dfc15: cleanup: http server, line too long
- Reviewed-by: amishchenko
- 02:05 pm Revision 5a9c3f46: cleanup: http client, line too long
- Reviewed-by: amishchenko
- 02:04 pm Revision 5661b908: cleanup: xen hypervisor
- Wrong indentation and uniformize one method signature.
Reviewed-by: amishchenko - 02:04 pm Revision 08137f9e: cleanup: kvm code likes to redefine names
- Reviewed-by: amishchenko
- 02:04 pm Revision 9c034cbe: lib/ssh.py: import the logging module
- This only means most of our error paths in this module were not working
(and generating exceptions).
Reviewed-by: am... - 02:04 pm Revision bf75f132: SshRunner: add docstring for _BuildSshOptions
- Reviewed-by: amishchenko
- 02:04 pm Revision 67047322: Improve _autoconf.py comments
- This adds a docstring to the _autoconf.py file detailing how it's
generated (the other comment it's not visible in py... - 02:04 pm Revision ad1bf20c: cleanup: use _ for unused loop counter
- Reviewed-by: amishchenko
- 02:04 pm Revision 805f0c07: cleanup: WorkerPool, wrong variable name
- Quoting Michael: "why is this even working?"
Reviewed-by: imsnah,amishchenko - 02:03 pm Revision 0b5ad33e: cleanup: TcpPing, wrong variable name
- The default value of 'False' wasn't initialized properly. It doesn't
require initialization, but it's cleaner this wa... - 02:03 pm Revision cd34e7bf: cleanup: SetEtcHostsEntry unused var
- Reviewed-by: amishchenko
- 02:03 pm Revision a0add446: cleanup: fix IAllocator hypervisor usage
- Two problems: the iallocator.hypervisor wasn't initialized to None in
the constructor, so pylint doesn't realize it's... - 02:03 pm Revision 4504c3d6: cleanup: LUReplaceDisks unused vars
- And a small whitespace fix.
Reviewed-by: amishchenko - 02:03 pm Revision 9ca87a96: cleanup: do not hide upper-scope name
- hypervisor is a module, so we shouldn't use it as an argument.
Reviewed-by: amishchenko - 02:03 pm Revision 43017d26: cleanup: fix use of _CheckNodeOnline
- A few cases of wrong variable name.
Reviewed-by: amishchenko - 02:03 pm Revision 8153e7e3: cleanup: LUAddNode, LUSetNodeParams unused variable
- This is a leftover from the abstraction of AdjustCandidatePool, and it
also requires the config lock, so it's better ... - 02:02 pm Revision d1dc3548: cleanup: LURenameCluster wrong variable name
- Reviewed-by: amishchenko
- 02:02 pm Revision 95268cc3: cleanup: fix export NIC count the same way as disk
- For safety, we use the same algorithm as in disk count.
Reviewed-by: amishchenko - 02:02 pm Revision bca2e7f4: cleanup: fix backend._RecursiveFindBD
- _RecursiveFindBD takes a parameter that isn't used; moreover, nowhere in
the SVN history can I find a case that it ha... - 02:02 pm Revision 9a4e6f41: cleanup: more unused vars
- Reviewed-by: amishchenko
- 02:02 pm Revision 3bc6be5c: cleanup: sanitize a default parameter
- Instead of relying that the usage of the parameter is ok with mutable
default parameters, let's just make it safer..
... - 02:02 pm Revision 9728ae5d: cleanup: exceptions should derive from Exception
- Reviewed-by: amishchenko
- 02:02 pm Revision 7bc9e58f: cleanup: fix GatherMasterVotes
- Remove unused vars
Reviewed-by: amishchenko - 02:01 pm Revision 531baf8e: cleanup: _InitSSHSetup doesn't need its argument
- Reviewed-by: imsnah
- 02:01 pm Revision fab1e3a4: cleanup: fix 'variable unused' warning
- In the iteration we don't care about the node names, so we change the
for loop to be over the values (and not iterval...
Also available in: Atom