Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Jean-Marc Eurin <jmeurin@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@5256 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: Dale Curtis <dalecurtis@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@5214 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: Jean-Marc Eurin <jmeurin@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4909 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Once again I went through the autotest tree with my automated script to
check all autotest source, and discovered several places that needed
permission and indentation (including trailing whitespace) fixes. I have
checked the generated patch and indeed it looks OK. Will commit this
shortly, as preparation for autotest 0.13.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4907 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Adding some logging in the drones.
Signed-off-by: Jean-Marc Eurin <jmeurin@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4900 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Bugfix for my previous change. The unittest unfortunately hid the problem
because our self.god.stub_with() stubs non-existant things out with a new
value without checking to see if the thing being stubbed out existed in the
first place.
Signed-off-by: Gregory Smith <gps@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4898 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Allow limiting of the number of hosts that go through the scheduler's
periodic Repair Failed -> Verifying cleanup cycle at once. When you've
got an infrastructure issue that kicks all your hosts into repair
failed, launching reverifies on all of them at once can overwhelm the
server.
Signed-off-by: Gregory Smith <gps@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4897 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Add some site-specific parsing utilities.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4871 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
In some occasions, the job end time might not be set (job stuck that
had to be aborted, for example). If that happens, the operation
t_end - t_begin used to get the job execution time will fail with an
IndexError, since t_end is None. Handle that case properly.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4859 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Ordering by reason makes it easy to group failures and
easily identifying them.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4858 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Make changes on the scheduler notification code to allow
for shorter code. Also, introduce a new common_lib.utils
function that generates a pretty print representation of
any matrix, that will adjust the length of all columns
automagically, making the resulting e-mail report much
prettier to see, even with very large test names. Example:
header = ("Word1", "Word2", "Word3")
rows = [["Heeeeeeeeeeeeeey", "Hey", "Eh"],
["Orange", "Nonononono", "Pineapple"],
["Egg", "Spam", "Bacon"]]
result = matrix_to_string(rows, header)
Gives:
Word1 Word2 Word3
Heeeeeeeeeeeeeey Hey Eh
Orange Nonononono Pineapple
Egg Spam Bacon
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4806 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
(,) within labels properly.
Chromium OS autotest project had assigned labels with comma to test hosts
and this situation makes the current host_keyvals file impossible to parse
out correct label values. This change will fix/enhance that.
git-svn-id: svn://test.kernel.org/autotest/trunk@4793 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Fix scheduler email notifications when the job is not run at all
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4779 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4768 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Changes from v2:
* Make sure results from SERVER_JOB and CLIENT_JOB.* will not
disturb the user test stats in case of a job failure
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4767 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
readily available, without having to open the web interface.
This patch adds important info to the regular status e-mails:
* Number of tests executed
* Success rate
* Details about test failures
So people can decide whether they actually want to inspect
the results more carefully.
Changes from v1:
* Refactored the original code, that repeated a lot of logic to
generate the subject and the body of the status e-mail. By
refactoring it becomes a lot easier to make changes on the
e-mail body if needed.
* Fixed a bug, where job failures were being treated incorrectly.
* Made floats to be displayed only with 2 decimals.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4766 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: Gregory Smith <gps@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4747 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
This will allow test developers to specify certain parameters that a test
control file may take, so that users can then easily set those parameters
on job create. Enabling this feature removes the ability to edit the
control file directly on job creation.
Feature is currently INCOMPLETE. Do not attempt to use. This feature will
be committed in small pieces for the sake of having smaller code reviews.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4720 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4623 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
GATHERING state. Also modify a logging string to be more precise.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4591 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
necessary jobs in bulk. This avoids a scheduler crash on an assertion.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4585 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
scheduler is first started.
Signed-off-by: Scott Zawalski <scottz@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4488 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
not all queued verify tasks.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4487 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4449 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
lose a slot for a process permanently if the paired results were lost
before the process started.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4447 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
running on a specified set of drones.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4439 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
I've noticed that monitor_db.py issues a SyntaxWarning
as soon as it is started:
19:56:55 INFO | Killing monitor_db
19:56:55 INFO | STARTING monitor_db with log file
/usr/local/autotest/logs/scheduler.log.2010-04-15-19.56.55
/usr/local/autotest/scheduler/monitor_db.py:1779: SyntaxWarning:
assertion is always true, perhaps remove parentheses?
assert (self.TASK_TYPE is not None,
I noticed that the whole statement fits under 80 chars,
so the parenthesis can be removed safely, getting rid of
the warning.
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4421 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4399 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Fixed bug where a drone that a user is not allowed to access may get a
job scheduled on it anyways if all drones are over capacity.
Modified capacity computation to handle drones with the same ratio of
running-to-max processes, but different total max processes.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4345 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
fixes an issue where the scheduler created multiple HostlessQueueTask
objects for a single hostless queue entry, causing several autoserv
processes to be launched when the agents are run.
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4304 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
control where in the results directory autoserv will store the server control
file.
This also changes the archving stage in the scheduler to make use of this
argument, so that the control file from the archving stage is written to
control.archive and does not overwrite the control.srv from the job itself.
Signed-off-by: John Admanski <jadmanski@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4294 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
is not required to interact with non Django portions of the code.
This includes the enums RebootBefore, RebootAfter and Test.Type
git-svn-id: svn://test.kernel.org/autotest/trunk@4280 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: James Ren <jamesren@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4279 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
repository. Trying again.
git-svn-id: svn://test.kernel.org/autotest/trunk@4278 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
a tick every cycle, even if there are no queued metahost jobs.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4274 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
do anything about this and it is just extra spam for us at this point in time.
This will only affect admins who use the SCHEDULER.notify_email variable
Signed-off-by: Scott Zawalski <scottz@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4258 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
SiteMonitorProc as a base class for MonitorProc so that site code can
override the monitor_db arguments and/or setup environment variables
needed for the scheduler to run properly at the given site.
Signed-off-by: Gregory Smith <gps@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4253 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
module doesn't depend on monitor_db, only the other way around. The separation and isolation of dependencies should help us organize the scheduler code a bit better.
This was made possible largely by the many changes we made late last year to improve statelessness of the scheduler. It was motivated here by my work on pluggable metahost handlers, which will need to depend on scheduler models. Without this separation, we'd end up with circular dependencies.
Also includes some fixes for metahost schedulers.
Signed-off-by: Steve Howard <showard@google.com>
Property changes on: scheduler/scheduler_models.py
git-svn-id: svn://test.kernel.org/autotest/trunk@4252 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
framework and moves the default, label-based metahost assignment code into a handler. It includes some refactorings to the basic scheduling code to make things a bit cleaner.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4232 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
gets correctly handled as a directory.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4196 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
call to register_pidfile() from get_pidfile_info() in my previous change, but now I realize the purpose of it was to reset the pidfile age.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4164 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4163 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
The scheduler is now registering and unregistering pidfiles correctly on its own, and this was causing files to get accidentally re-registered after being unregistered, causing pidfile leaks.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4162 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
explicitly recorded by a server-side control file or code that it calls. This CL just adds the record_summary() method to the server_job object. It lacks any special parser support or TKO DB changes, those will come later.
This also includes a couple of minor changes to support conitnuous parsing and final reparsing for hostless jobs. Since hostless jobs are a common intended use case for summary results, they'll need full parsing support to be useful.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4161 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
default to the new autotest_system user.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4156 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
* added ExtendedManager.join_custom_field(), which uses the introspection magic from populate_relationships (now factored out) to infer the type of relationship between two models and construct the correct join. join_custom_field() presents a much simpler, more Django-y interface for doing this sort of thing -- compare with add_join() above it.
* changed TKO custom fields code to use join_custom_field()
* added some cases to AFE rpc_interface_unittest to ensure populate_relationships() usage didn't break
* simplified _CustomQuery and got rid of _CustomSqlQ. _CustomQuery can do the work itself and its cleaner this way.
* added add_where(), an alternative to extra(where=...) that fits more into Django's normal representation of WHERE clauses, and therefore supports & and | operators later
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4155 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
.archiver_failed file.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4130 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
* can be passed as an argument to create_job, stored in AFE DB
* scheduler reads them from the AFE DB and writes them to the job-level keyval file before the job starts
* parser reads them from the keyval file and writes them to the TKO DB in a new table
Since the field name "key" happens to be a MySQL keyword, I went ahead and made db.py support proper quoting of field names. Evetually it'd be really nice to deprecate db.py and use Django models exclusively, but that is a far-off dream.
Still lacking support in the AFE and TKO web clients and CLIs, at least the TKO part will be coming soon
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4123 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
* display allowed users in the status server
* ensure drone users reparse correctly
* fix a bug in max_runnable_processes introduced by user restrictions (could pass empty set to max())
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4121 592f7852-d20e-0410-864c-8624ca9c26a4
|
|
object creation. This is the right path to use for creating tasks -- it include duplication avoidance and automatic owner tagging.
Signed-off-by: Steve Howard <showard@google.com>
git-svn-id: svn://test.kernel.org/autotest/trunk@4120 592f7852-d20e-0410-864c-8624ca9c26a4
|