Activity
From 03/15/2010 to 04/13/2010
04/13/2010
- 02:21 pm Revision 691e9563: Mark cluster-merge as executable
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: René Nussbaumer <rn@google.com>
04/12/2010
- 05:53 pm Revision 725ec2f1: QA: Make the rapi credential handling less involving
- Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 05:15 pm Revision 30198f04: Merge remote branch 'devel-2.1'
- * devel-2.1:
Update import documentation for the recent changes
Add a identify-defaults options for import
Fix ... - 04:57 pm Revision 33ea43b6: Update import documentation for the recent changes
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 04:57 pm Revision e588764d: Add a identify-defaults options for import
- When importing an instance, all the saved valued will be used as
explicitly specified values, overriding the cluster ... - 04:57 pm Revision b6cd72b2: Fix create/import verification of hvparams
- Currently the instance creation checks the cluster hv defaults + the new
parameters for validity, ignoring the os-spe... - 04:57 pm Revision d63479b5: objects.Cluster: add method to get hv defaults
- Currently the FillHV method is the one that does the cluster hvparams +
os hvparams merger. However, in some cases we... - 04:57 pm Revision 0af0f641: Reuse NIC information from export
- If the user doesn't pass any nics in import, do not use a default
one-nic, but instead read the nics from the export ... - 04:57 pm Revision cc0d88e9: Reuse backend parameters from export
- Similar to the previous patches, if we're missing some parameters and
the export has them (either in the new style or... - 04:57 pm Revision 9b12ed0f: Reuse disk information from export
- If the user doesn't pass the disk information on import, automatically
reuse the number and size of disks. This loses... - 04:56 pm Revision 9f88b0e8: Reuse hypervisor parameters in import
- If available, we reuse the parameters from the export info.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-b... - 04:56 pm Revision f276c4b5: Read disk template from export info
- This patch changes the instance import to read the disk template
automatically from the export info, if the opcode do... - 04:56 pm Revision c1c31426: CreateInstance: separate the reading of the export
- We move the reading of the export to a separate function, to simplify
CheckPrepreq and also read it earlier. This wil... - 04:56 pm Revision 22f50b1d: Move code from ExpandNames to CheckPrereq
- This is needed since only in CheckPrereq we have the nodes locked, and
future import enhancements will need to have a... - 04:56 pm Revision 417eabe2: CreateInstance: Move some code to CheckArguments
- ExpandNames holds too much non-locking code (first LU to be converted to
ExpandNames, and we didn't have CheckArgumen... - 04:56 pm Revision 3c8954ad: Export more instance parameters in instance export
- Currently the backend parameters are not exported automatically, but
only a few directly in the '[instance]' section.... - 04:56 pm Revision 6801eb5c: Export the nicparams too during instance export
- The patch tries to export all params (based on the dict defined in
constants), using None for missing keys.
Signed-o... - 04:56 pm Revision 2a518543: Handle errors better for wrong nic_count in export
- This fixes an old 'FIXME' entry.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hans... - 03:53 pm Revision 06fcbaf1: QA: Make sure RAPI credentials are setup on cluster init
- This patch makes sure that the Ganeti RAPI credentials are setup,
if any, on cluster init time.
Signed-off-by: René ... - 12:36 pm Revision e7a25b08: Merge branch 'devel-2.1'
- * devel-2.1:
burnin: only remove instances we actually added
burnin.ExecOrQueue: add post-process function
burn...
04/09/2010
- 05:54 pm Revision fe7c59d5: Fix new pylint errors
- Under squeeze pylint reports the following errors:
************* Module ganeti.serializer
E1103:155:LoadSignedJson: I... - 05:54 pm Revision b05f29a6: burnin.ExecOrQueue: remove variable argument list
- In order to later add an optional parameter we transform the variable
ops argument list in an explicit list.
Signed-... - 05:54 pm Revision 2c035435: burnin.ExecOrQueue: add post-process function
- If a post-process function is passed to ExecOrQueue it is executed if
and only if the job is successful. This happens... - 05:54 pm Revision 1e82a86b: burnin: only remove instances we actually added
- Currently burnin, if proceding in parallel, will remove all instances
which were passed, even if they failed to add. ... - 11:48 am Revision edf7d66e: Rename the confd_client unittest (to confd.client)
- This is to keep same naming across all tests (modules separate with dot,
followed by _unittest.py).
Signed-off-by: I... - 10:57 am Revision ebacb943: Make watcher request the max coverage
- Since the actions are potentially destructive, we should try to get a
consistent view of the cluster, so it's better ... - 10:57 am Revision cc6484c4: ConfdClient.SendRequest: allow max coverage
- This patch changes the coverage parameter to allow specification of max
coverage (via -1), versus auto-computation (d...
04/08/2010
- 06:50 pm Revision 6328fea3: Document the watcher node maintenance feature
- The patch changes significantly the watcher man page, as it was very
simplistic.
Signed-off-by: Iustin Pop <iustin@g... - 06:50 pm Revision 50273051: Watcher: automatic shutdown of orphan resources
- This patch changes the watcher so that it maintains (on all nodes) the
list of instances and DRBD devices by shutting... - 06:50 pm Revision 5c465a95: Export the maintain_node_health option in ssconf
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 06:50 pm Revision 3953242f: Add a new cluster parameter maintain_node_health
- This will be used to conditionally enable the watcher node maintenance
feature.
Signed-off-by: Iustin Pop <iustin@go... - 06:50 pm Revision aa2efc52: Add a new confd callback (StoreResultCallback)
- This new callback simply stores (without calling any lower-level
callback) the last result; coupled with the filterin... - 06:50 pm Revision bfbbc223: ConfdClient: add synchronous wait for replies mode
- Currently, there is no way for a user of the confd client library to
know how many replies there should be, whether a... - 06:50 pm Revision 71e114da: ConfdClient: unify some internal variables
- Currently the requests are tracked in _request and in _expire_requests.
This is conventient, but it restricts the abi...
04/07/2010
- 06:47 pm Revision cf6fee17: Merge branch 'devel-2.1'
- * devel-2.1:
Fix consistency checks in ConfdFilterCallback
Fix utils.WaitForFdCondition inner retry loop
Fix bu... - 04:57 pm Revision 39292d3a: Fix consistency checks in ConfdFilterCallback
- Commit 49b3fda added consistency checks, but these are wrongly triggered
for old responses - we need to make sure to ... - 04:55 pm Revision 1b429e2a: Fix utils.WaitForFdCondition inner retry loop
- Commit dfdc4060 added WaitForFdCondition which uses utils.Retry without
handling timeout exceptions. This breaks any ... - 04:31 pm Revision cc2f004d: Fix bug introduced in 76e5f8b54: mkdir mode
- After commit 76e5f8b54, mkdir_mode in utils.RenameFile is
no longer passed to Makedirs. This is fixed by this patch.
... - 04:15 pm Revision 76e5f8b5: utils: Move wrapper code around os.makedirs into separate function
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 11:55 am Revision 2004e673: Fix unittest for the rapi client library
- Wrong escape, so we make sure to use proper escapes (we want the
backslashes to be embedded, not interpreted). Also c...
04/06/2010
- 05:37 pm Revision 95ab4de9: Adding RAPI client library.
- Signed-off-by: David Knowles <dknowles@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
Signed-off-by: Iustin ... - 03:32 pm Revision 921efbb3: Merge remote branch 'devel-2.1'
- * devel-2.1:
Extend ConfdFilterCallback with consistency checks
Abstract the confd client creation
Remove unuse... - 12:51 pm Revision 49b3fdac: Extend ConfdFilterCallback with consistency checks
- Note that users of the callback will have to manually check the
attribute.
Signed-off-by: Iustin Pop <iustin@google.... - 12:27 pm Revision 5b349fd1: Abstract the confd client creation
- Most creation of confd clients will do the same steps: read MC file,
parse it, read HMAC key, etc. We abstract this f...
03/31/2010
- 06:51 pm Revision e065714c: Remove unused import from test file
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 06:51 pm Revision 7ba594c0: kvm_flag hypervisor parameter
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 05:59 pm Revision cef34868: Move the runas user at execution time
- Everything still works the same way, but the user is calculated each
time we start kvm, rather than stored in the con... - 12:54 pm Revision 57587760: Fix iallocator crash when no solutions exist
- Commit 5436576 added an un-guarded `head' call, which crashes with
“Prelude.head: empty list” when no results exists ... - 12:51 pm Revision 934c62dc: Fix IAllocator multi-evacuate message
- Since Ganeti passes full host names (not common-suffix-stripped), we
need to remove the suffix from the evac_nodes ke...
03/30/2010
- 05:21 pm Revision 33664046: Send "501 Not Implemented" back when method not found
- Before this was "400 Bad Request" and thus it didn't reflect
the reality.
Signed-off-by: René Nussbaumer <rn@google....
03/26/2010
- 03:56 pm Revision e6ce18ac: Adding QA RAPI tests for activate-disks and deactivate-disks calls
- * This also adds support for authenticated RAPI calls
* Other HTTP methods than GET/POST
Signed-off-by: René Nussbau...
03/25/2010
- 02:47 pm Revision b39bf4bb: SerializableConfigParser: Make Loads class indep
- Currently SerializableConfigParser.Loads is a static method that returns
a SerializableConfigParser. With this patch ...
03/23/2010
- 06:14 pm Revision 71834b2a: Unbreak command line job submission
- A change introduced in 5299e61f modified the contents of
JobExecutor.jobs, missing a place where this tuple was decon... - 01:04 pm Revision 91e2d9ec: Write grow support for file storage
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 01:04 pm Revision 2c42c5df: Allow file storage to be grown
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 12:21 pm Revision 4f7a6a10: Distribute list of enabled hypervisors in ssconf
- This can be used by nodes to know which hypervisors they are supposed to
support.
Signed-off-by: Iustin Pop <iustin@... - 12:21 pm Revision bbcf7ad0: Extend the hypervisor API with name-only shutdown
- Currently the ShutdownInstance method of the hypervisors takes a full
instance object. However, when doing instance s... - 12:21 pm Revision 10e689d4: Watcher: do not warn for missing hooks dir
- If the hooks dir does not exist, do not warn needlessly. This is similar
to commit a9b7e346 (for backend.py).
Signed... - 12:21 pm Revision 55c85950: Watcher: fix some doc typos
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: René Nussbaumer <rn@google.com> - 11:56 am Revision 24f40363: Merge branch 'devel-2.1'
- * devel-2.1:
ganeti-confd: Call pyinotify flags correctly
Fix burnin error when trying to grow a file volume
So... - 11:31 am Revision 675bf1b7: ganeti-confd: Call pyinotify flags correctly
- The "apparently pylint was right" commit.
Although the pyinotify constants work on old distributions, they fail on
n... - 11:31 am Revision 728489a3: Fix burnin error when trying to grow a file volume
- Abstract the growable disk types in a ganeti constants, and only run
disk grow, from burnin, on them.
Signed-off-by:... - 10:35 am Revision ed904904: Fix backend.VerifyNode behaviour for VG problems
- In case LVM is broken, backend.GetVolumeList will raise an RPC exception
(as expected since it's a function exposed o... - 10:35 am Revision 3a488770: Some epydoc fixes
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 10:35 am Revision 02c521e4: A rewrite of LUClusterVerify
- Per issue 90, current cluster verify is very very brittle. It's one of
the oldest pieces of code, with only additions... - 10:35 am Revision e7b61bb0: Introduce a bool CLI option type
- This option type enforces its value to either True or False, relieving
the scripts from manually parsing the values i... - 10:33 am Revision 2263aec2: Adding missing documentation to make the docs better
- Also fixed a typo I noticed.
Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
03/22/2010
- 08:06 pm Revision cdeefd9b: Remove race condition in FileStorage.Create
- Rather than checking that the file doesn't exist, and then creating it,
we create it with O_CREAT | O_EXCL, making su... - 08:06 pm Revision 263b8de6: KVM: Check instances for actual liveness
- Currently if we find a live process with the pid we saved we assume kvm
is alive. What could happen, though, is that ... - 08:06 pm Revision 4fbb3c60: KVM: improve GetInstanceInfo docstring
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 08:06 pm Revision 284c69f0: Revert "Only override any and all if not defined"
- This reverts commit bd5617020a50bcd08269330638d64078c1b30b71.
Turns out our and python's any/all are not compatible.... - 05:25 pm Revision 0a37de5f: Adding RAPI call to deactivate-disks for an instance
- Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 05:25 pm Revision 2197b66f: Adding RAPI call for activate-disks on an instance
- Signed-off-by: René Nussbaumer <rn@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>
03/18/2010
- 05:28 pm Revision d8f5a37d: Add a hint to masterd for inconsistent clusters
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 04:29 pm Revision 9d099698: SimpleConfigReader: add docstrings
- All non-oneliner functions, after this patch, have their docstring.
Signed-off-by: Guido Trotter <ultrotter@google.c... - 03:56 pm Revision 791f317d: Merge branch 'devel-2.1'
- * devel-2.1:
burnin: implement basic confd testing
AsyncUDPSocket.process_next_packet
WaitForSocketCondition: r... - 01:39 pm Revision 8496d93c: ConfdClient: add synchronous features
- By sending requests with async=False, and receiving replies with
ReceiveReply we can more easily use confd from a syn... - 01:39 pm Revision d63997b3: Replace @keyword with @param in confd client
- @keyword was used inappropriately.
Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Michael Hanselma... - 01:39 pm Revision 04cdf663: ConfdCountingCallback
- This new confd callback counts received replies for the registered
queries.
Signed-off-by: Guido Trotter <ultrotter@... - 01:39 pm Revision a619a1dd: burnin: implement basic confd testing
- Just a few queries are checked, but this should give us confidence that
at least the basic confd framework is working... - 01:39 pm Revision dfdc4060: WaitForSocketCondition: rename, handle EINTR
- - Rename WaitForSocketCondition to SingleWaitForFdCondition
- Avoid potentially infinite loop, if we continue to ge... - 01:39 pm Revision 6ddf5c8f: AsyncUDPSocket.process_next_packet
- This function allows receiving socket data synchronously.
Signed-off-by: Guido Trotter <ultrotter@google.com>
Review... - 01:39 pm Revision dcd511c8: move http.WaitForSocketCondition to utils
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 01:38 pm Revision 8216580d: AsyncUDPSocket: abstract do_read function
- This basically implements read handling, without catching all
exceptions. When using the socket in synchronous mode, ... - 01:38 pm Revision be0636e3: Burnin: don't add/remove routed nics
- Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Iustin Pop <iustin@google.com> - 01:38 pm Revision bd561702: Only override any and all if not defined
- If any or all are already defined (because we're using a new version of
python) just link them inside "utils" rather ...
03/17/2010
- 07:19 pm Revision f942a838: Add RPC calls to create and remove X509 certificates
- Certificates and keys generated using these functions will be used for
inter-cluster instance moves. As per design, t... - 07:05 pm Revision bc85bc75: Merge remote branch 'origin/devel-2.1'
- * origin/devel-2.1:
backend: Two small style fixes
Allow cluster copy file over the replication net
Enhance cli... - 06:58 pm Revision 56569f4e: backend: Two small style fixes
- - Pass keyword parameter as such
- Replace “not x == y” with “x != y”
Signed-off-by: Michael Hanselmann <hansmi@goog... - 06:22 pm Revision 74adc100: Allow cluster copy file over the replication net
- This patch introduces the option “--use-replication-network” for the
cluster copyfile functionality, which is useful ... - 06:22 pm Revision e9e26bb3: Enhance cli.GetOnlineNodes query/filtering
- This patch allows GetOnlineNodes to return the secondary IPs instead of
the node names, and to provide filtering of t... - 05:18 pm Revision 68857643: utils: Add functions to sign and verify X509 certs using HMAC
- Certificates exchanged via an untrusted third party should be
signed to ensure they haven't been modified.
Signed-of... - 05:18 pm Revision 3db3eb2a: Add cluster domain secret
- Information exchanged between different clusters via untrusted
third parties (e.g. for remote instance import/export)... - 04:11 pm Revision 25a8792c: Instance creation: implement --no-install mode
- This is a simple patch that adds the no-install mode for instance
creation, allowing import from foreign source of th... - 04:10 pm Revision 96b39bcc: Allow OS changes without reinstallation
- This patch modifies LUSetInstanceParms to allow OS name changes, without
reinstallation, in case an OS gets renamed o... - 04:10 pm Revision 231cd901: cmdlib: Abstract OS checks
- This patch moves the node-has-os checks to a separate function.
Signed-off-by: Iustin Pop <iustin@google.com>
Review...
03/16/2010
- 04:14 pm Revision c89e6fdf: Merge remote branch 'origin/devel-2.1'
- * origin/devel-2.1:
Improve “gnt-cluster renew-crypto”
Fix behaviour of gnt-node modify -C no
Signed-off-by: Mic... - 04:05 pm Revision d512e84b: Improve “gnt-cluster renew-crypto”
- - Report exception text immediately instead of just logging it
- Remove leftover assertion from when it still used “g... - 03:29 pm Revision 8fe9239e: Fix behaviour of gnt-node modify -C no
- The current check on whether we require auto_promote or not is wrong, as
we check whether we will have exactly the co...
03/15/2010
- 07:00 pm Revision 084aba47: Merge remote branch 'origin/devel-2.1'
- * origin/devel-2.1:
Rightname confd's HMAC key
Rename SSL_CERT_FILE to NODED_CERT_FILE
Clarify the error messag... - 06:40 pm Revision 168c1de2: Rename SSL_CERT_FILE to NODED_CERT_FILE
- To be consistent with RAPI_CERT_FILE, the rather generic named
“SSL_CERT_FILE” constant is renamed to “NODED_CERT_FIL... - 06:40 pm Revision 6b7d5878: Rightname confd's HMAC key
- Currently, the ganeti-confd's HMAC key is called “cluster HMAC key” or
simply “HMAC key” everywhere. With the impleme... - 06:18 pm Revision 01b6558a: Clarify the error message for ':' in PV names
- As described in issue 93, just saying ':' is not a valid char can be
confusing.
Signed-off-by: Iustin Pop <iustin@go... - 05:50 pm Revision b701a5df: Merge remote branch 'origin/devel-2.1'
- * origin/devel-2.1:
Implement QA tests for disk template changes
Update instance modify documentation
Implement... - 05:46 pm Revision 7f69aabb: Implement QA tests for disk template changes
- The new test depends on the drbd type tests being enabled, and test
conversion to plain and back to drbd.
Signed-off... - 05:45 pm Revision bbf74a76: Update instance modify documentation
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Michael Hanselmann <hansmi@google.com> - 05:38 pm Revision 2f414c48: Implement conversion from drbd to plain
- This is much simpler than the opposite, with fewer possibilities of
failures.
Signed-off-by: Iustin Pop <iustin@goog... - 05:38 pm Revision e29e9550: Implement conversion from plain to drbd
- This patch adds a new mode to instance modify, the changing of the disk
template. For now only plain to drbd conversi... - 05:29 pm Revision 31624382: Abstract check that an instance is down
- Multiple LUs require that an instance is not running while they operate
on the instance (reinstall, rename, modify, r... - 05:29 pm Revision 701384a9: Abstract node free disk space check
- Both create instance and grow disk check the free disk space on nodes
using the same, duplicate code. Since we'll nee... - 05:28 pm Revision 5d55819e: Abstract disk template verification
- This is a simple check, but we'll need it in multiple places.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed... - 05:17 pm Revision 17227cd1: Update documentation for disk adoption
- Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 05:17 pm Revision 5029db65: Implement disk adoption mode in gnt-instance
- This patch modifies the parsing of the “--disk” argument to instance
create to accept “adopt” as a valid key, which b... - 05:17 pm Revision c3589cf8: LUCreateInstance: implement disk adoption mode
- This new mode, valid only for the plain template disk, allows creation
of an instance based on existing logical volum... - 05:16 pm Revision df4272e5: LUCreateInstance: Move parameter init earlier
- This way, the parameters are available in CheckArguments too.
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed... - 05:16 pm Revision d8aee57e: ConfigWriter: add an LV reservation manager
- This patch adds an LV reservation manager to be used for LV names. Since
we now have four such managers, we create a ... - 04:55 pm Revision 5fa16422: Fix two issues related to check-man
- First, we don't need to check man pages at sed time, because this means
everyone building the package will do - we on... - 04:39 pm Revision 0babc371: utils.RunCmd: Test case with reset_env set and setting variables
- Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Guido Trotter <ultrotter@google.com> - 04:25 pm Revision bb3776b4: Merge remote branch 'origin/devel-2.1'
- * origin/devel-2.1: (116 commits)
Implement replacing cluster certs and keys via “gnt-cluster renew-crypto”
cli: ...
Also available in: Atom