| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
This commit makes lorry-controller understand YAML lorry files. If
the file cannot be parsed as YAML then we fall back to attempting
to load it as JSON, before giving up on loading it completely.
The test for broken JSON is modified to use a string which is
invalid for both YAML and JSON.
Change-Id: If83e2e44b38e6fb63dbf0b857e143fdcabab78ac
|
|
|
|
| |
Change-Id: Iad9139bb6bbce07926ca800c0525601a43341c69
|
|
|
|
| |
Change-Id: I3a95880618218da00991f8ac58dee54d06a0da41
|
|
|
|
| |
Change-Id: Id79d4d05a5cabd7f7a634dd0decdee6bd4005f29
|
|
|
|
| |
Change-Id: I6d06353462badac114c685a2f655bc41c6820aff
|
|
|
|
| |
Change-Id: I5d958457db3b4867962f6f77eab2201cfd6d0be7
|
|
|
|
| |
Change-Id: I23c23886b5389d75032c64bf4e78db7edca1af63
|
|
|
|
| |
Change-Id: Ic1ef8b983cc84b0ce75dc57f9e05b5cdda343dc6
|
|
|
|
| |
Change-Id: I2bb0aaf428e331a0bcd5a1e3111d4c7bca4afede
|
|
|
|
| |
Change-Id: I74dc0265fb3c92259101317d655eb55ccb62c119
|
|
|
|
| |
Change-Id: I9f5733ad493db4aafa775109def44316e9ae34c9
|
|
|
|
|
|
|
|
|
|
|
| |
We made this change more than a year ago in git.baserock.org, to
test if that would make it consume less resources, without breaking
anything.
That change seemed to solve the problems we were having, and we never
had any related issue.
Change-Id: Iabb34bbe8df4c329361c2b446851b10e3c43e4de
|
|
|
|
|
|
|
|
|
|
|
|
| |
Most of us thought this was the default configuration,
but it wasn't.
This change means that the webapp service will only be accessible
from the same host that is running it, making it more secure.
Thanks to Ben Brown for finding this error.
Change-Id: I27f6604a0ebc7a093b63b11345b9c8963ed91c18
|
|
|
|
|
|
| |
Should have done it this way in 2e3195f.
Change-Id: I015a3c1241a59277801cf39865f87e8c3ac96747
|
|
|
|
|
|
| |
While we're here, seeing as Adam mentioned it.
Change-Id: I5ddb86c70d76a84cf12fbd4eb91f3802e490d745
|
|
|
|
|
|
|
| |
* Does not explicitely test 'globs' validation.
* Also verifies a missing 'ignore' field does not result in error.
Change-Id: I8140185a485cccdf7086533d3afcc6b7fc5f121b
|
|
|
|
| |
Change-Id: I42fbb8a2f2150cfbc48e07340ecedea76f41639a
|
|
|
|
|
|
|
|
| |
Due to an unwillingness to add another IMPLEMENTS that copypasta'd
the same "MATCH_n = os.environ['MATCH_n']", add this small library
in an attempt to reduce the amount of repitition.
Change-Id: I64dc67ad7bd4c0d7572906168c72b0628a7574db
|
|
|
|
| |
Change-Id: I363b73c897b6728d9938b474f352a11d8554d669
|
|
|
|
| |
Change-Id: I218d4b23fb27526674f96b2f2566bb9ff526f688
|
|
|
|
|
|
|
| |
The 'ignore' and 'globs' fields expect values to be string lists,
add checks for this in config validation.
Change-Id: I96fc8fa217a6f62b6b67f604bd50b2fbea723db5
|
|
|
|
|
|
|
|
| |
The README lists 'ignore' in the group of optional keys for
trove specifications, yet any attempt to read the configuration
without the key being present would result in a KeyError.
Change-Id: I05121535b970c6d7382def46ffa720209f794633
|
|
|
|
| |
Change-Id: I0833f43b97a26532767996040104ffb2f44a3558
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
From a disk space point of view, 1 year's worth of logs,
while potentially excessive, isn't too much of an ask,
since disk space is cheap enough.
However our queries on the database run slower when it is large,
so we need a shorter log retention policy,
and it's best when the defaults do the right thing.
1 day's worth of logs was found to be 87MB,
which means 3 days is roughly 250MB, which is acceptable.
Change-Id: If3dd58fa01f785bc7d7029a45b6a0fc35c2c2b1d
|
|\
| |
| |
| |
| | |
Reviewed-By: Pedro Alvarez <pedro.alvarez@codethink.co.uk>
Reviewed-By: Adam Coldrick <adam.coldrick@codethink.co.uk>
|
|/
|
|
|
| |
These had meaning in the previous implementation of lorry-controller,
but are ignored by the current version.
|
|\
| |
| |
| |
| | |
Reviewed-By: Pedro Alvarez <pedro.alvarez@codethink.co.uk>
Reviewed-By: Adam Coldrick <adam.coldrick@codethink.co.uk>
|
| |
| |
| |
| |
| |
| |
| | |
The documentation on wiki.baserock.org described the 'ignore' field as
"glob patterns on repository pathnames which to not mirror (trove only)"
but it was actually being treated as a simple list of repos to ignore.
Globs are much more useful.
|
|\ \
| |/
| |
| |
| | |
Reviewed-By: Pedro Alvarez <pedro.alvarez@codethink.co.uk>
Reviewed-By: Adam Coldrick <adam.coldrick@codethink.co.uk>
|
|/ |
|
|\
| |
| |
| |
| | |
Reviewed-By: Richard Maw <richard.maw@codethink.co.uk>
Reviewed-By: Francisco Redondo Marchena <francisco.marchena@codethink.co.uk>
|
|/
|
|
|
|
|
|
|
| |
urllib2 doesn't honour proxy settings despite our best efforts. It turns
out that lorry-controller currently ignores all proxy settings.
Requests, by comparison, does whatever the 'http_proxy' and
'https_proxy' environment variables tell it to do. This makes
lorry-controller actually work behind a proxy.
|
|\
| |
| |
| |
| | |
Reviewed-By: Adam Coldrick <adam.coldrick@codethink.co.uk>
Reviewed-By: Emmet Hikory <emmet.hikory@codethink.co.uk>
|
|/
|
|
|
| |
Previously the proxy.conf file had to contain 'username' and 'password'
fields, which meant that it didn't work for some HTTP proxy setups.
|
|\
| |
| |
| |
| | |
Reviewed-By: Francisco Redondo Marchena <francisco.marchena@codethink.co.uk>
Reviewed-By: Sam Thursfield <sam.thursfield@codethink.co.uk>
|
| |
| |
| |
| | |
This adds them to both the JSON and HTML outputs.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When getting information about a lorry spec from WEBAPP (/1.0/lorry/PATH),
we now need it to return two lists of job ids: one for all jobs for that
lorry spec, one with failed jobs only.
We test here the version of the request that returns JSON. The version that
returns HTML is similar (or so we assume), but it's harder to test the HTML
output so we're happy with this.
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Reviewed-by: Richard Maw
Reviewed-by: Pedro Alvarez
|
| | | |
|
| | | |
|