Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add armv7lhf to list of valid archsrichardipsum/armv7lhf_port | Richard Ipsum | 2013-11-11 | 1 | -1/+1 |
| | |||||
* | Add armhf detection | Richard Ipsum | 2013-11-04 | 1 | -0/+9 |
| | |||||
* | Added scenario test to prevent morph edit from being used without a system ↵ | Daniel Firth | 2013-11-01 | 3 | -1/+17 |
| | | | | argument | ||||
* | Added scenario test for preventing building uncommitted systems | Daniel Firth | 2013-11-01 | 3 | -1/+29 |
| | |||||
* | Merge remote-tracking branch 'origin/pedroalvarez/S9428' | Lars Wirzenius | 2013-11-01 | 4 | -12/+39 |
|\ | |||||
| * | Add regression test, morph branch doesn't remove existing branch dir | Pedro Alvarez | 2013-10-31 | 1 | -0/+20 |
| | | |||||
| * | Some tests refactorized for better understanding. | Pedro Alvarez | 2013-10-31 | 2 | -12/+16 |
| | | |||||
| * | Don't remove an existing branch in 'morph branch' | Pedro Alvarez | 2013-10-31 | 1 | -0/+3 |
|/ | |||||
* | Merge branch 'pedroalvarez/cmdtest-definitive2' | Pedro Alvarez | 2013-10-31 | 2 | -0/+20 |
|\ | | | | | | | | | | | Reviewed-by: Dan Firth: +1 Richard Ipsum: +1 | ||||
| * | Modify morph test suite to use the new cmdtest | Pedro Alvarez | 2013-10-30 | 2 | -0/+20 |
|/ | |||||
* | Deploying a non-cluster morphology now displays an error message | Daniel Firth | 2013-10-30 | 3 | -1/+22 |
| | |||||
* | Added scenario test for preventing cross-building | Daniel Firth | 2013-10-30 | 5 | -2/+52 |
| | |||||
* | Merge remote-tracking branch 'origin/liw/cache-server-settings' | Lars Wirzenius | 2013-10-29 | 3 | -4/+37 |
|\ | | | | | | | Reviewed-by: Daniel Silverstone | ||||
| * | Add --artifact-cache-server and --git-resolve-cache-server | Lars Wirzenius | 2013-10-29 | 3 | -4/+37 |
|/ | |||||
* | Merge remote-tracking branch 'origin/pedroalvarez/S9240' | Lars Wirzenius | 2013-10-28 | 3 | -3/+39 |
|\ | | | | | | | | | Reviewed-by: Lars Wirzenius Reviewed-by: Dan Firth | ||||
| * | Adding test | Pedro Alvarez | 2013-10-28 | 2 | -1/+36 |
| | | |||||
| * | Building with references to tags will now instead use the underlying commit | Daniel Firth | 2013-10-28 | 1 | -2/+3 |
|/ | |||||
* | Exlcuding yarn files when checking the lenght of lines | Pedro Alvarez | 2013-10-28 | 1 | -0/+3 |
| | |||||
* | Tidy up debug output of morph build | Daniel Firth | 2013-10-28 | 4 | -13/+11 |
| | |||||
* | Merge remote-tracking branch 'origin/liw/fstab.configure' | Lars Wirzenius | 2013-10-24 | 3 | -0/+179 |
|\ | | | | | | | | | | | | | Reviewed-by: Richard Maw At his suggestion, fixed the call to sorted() to be a call to asciibetical(). | ||||
| * | Add fstab.configure | Lars Wirzenius | 2013-10-24 | 3 | -0/+179 |
| | | | | | | | | | | This will allow the user to append text to /etc/fstab during a deployment, without having to write custom configuration extensions. | ||||
* | | Merge remote-tracking branch 'origin/liw/cachedrepo-unit-test-fix-v2-rebase' | Lars Wirzenius | 2013-10-24 | 3 | -7/+17 |
|\ \ | |/ |/| | | | Reviewed-by: Richard Maw | ||||
| * | Fix app.status call in log_dict_diff | Lars Wirzenius | 2013-10-24 | 1 | -2/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This bug was triggered by the fix to CachedRepo.resolve_ref and without this fix, the resolve_ref fix will break the test suite. The bug is that log_dict_diff calls the status method with an msg keyword argument that may contain percentage characters. status interprets the value of msg as a format string, and the percentage characters trigger formatting to happen. The fix for that is to not interpolate the value of key and dictA[key] and dictB[key] into msg before calling status, but letting status do that. Thus the msg values are changed to reference %(key)s instead and passing in a value for key as a separate argument. Ditto for dictA[key] and dictB[key]. | ||||
| * | Make CachedRepo.resolve_ref handle non-existent SHA1 | Lars Wirzenius | 2013-10-24 | 1 | -4/+9 |
| | | | | | | | | | | This changes how CachedRepo runs git to get the SHA1 information it needs, based on a suggestion by Richard Maw. | ||||
| * | Fix CachedRepo.resolve_ref unit test for missing SHA1 | Lars Wirzenius | 2013-10-23 | 1 | -1/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CachedRepo.resolve_ref does, effectively, this: absref = git rev-parse --verify $ref git log -1 --format=format:%T $absref Roughly, "git rev-parse" takes any ref and returns the corresponding SHA1. If the ref looks like a SHA1 (40 hex digits), it is returned as-is, and despite --verify is not checked for existence. "git log" then takes the SHA1 and returns the **tree** SHA1, as opposed to the commit one, and if the commit doesn't exist, barfs. The unit test for resolve_ref with an invalid SHA1 currently succeeds for the wrong reason. The mocked _rev_parse fails for an unknown SHA1 (raising cliapp.AppException), which causes resolve_ref to raise InvalidReferenceError, which the unit test expects. However, the real implementation of _rev_parse wouldn't fail in that way, and so the unit test doesn't test the thing it's meant to test: that resolve_ref actually works the expected way for an unknown SHA1. What actually happens is that resolve_ref calls _show_tree_hash, which raises cliapp.AppException for an unknown SHA1, resulting in horror and despair, instead of resolve_ref raising InvalidReferenceError. This commit fixes the unit test so that it causes the right code path in resolve_ref to be executed. This makes the unit test suite to fail. | ||||
* | | Merge branch 'baserock/richardmaw/bugfix/null-build-without-push' of ↵ | Richard Maw | 2013-10-17 | 1 | -2/+2 |
|\ \ | |/ |/| | | | | | | | | | git://git.baserock.org/baserock/baserock/morph Reviewed-by: Lars Wirzenius Reviewed-by: Richard Ipsum | ||||
| * | build-without-push: Handle null repo/ref | Richard Maw | 2013-10-14 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This wasn't working, since the build-without-push code works by including morphologies that have had their ref changed to the branch's name. This is not done by edit if it is null, since the extra ref resolution can cause interesting issues. Instead of changing edit to continue to alter the ref, build will now allow null meaning this current repo/ref. This has the side-effect of also checking morphologies referred to by a null ref, even if they aren't altered, but it still works correctly. | ||||
* | | Merge branch 'danielfirth/S9121' | Daniel Firth | 2013-10-15 | 3 | -3/+10 |
|\ \ | |/ |/| | | | | | Reviewed by Lars Wirzenius Reviewed by Richard Maw | ||||
| * | Deployment failures will now remove the disk image | Dan Firth | 2013-10-14 | 3 | -3/+10 |
| | | |||||
* | | Reduced debug output for environment properties | Dan Firth | 2013-10-14 | 1 | -1/+2 |
| | | |||||
* | | Merge branch 'danielfirth/RT193' | Daniel Firth | 2013-10-11 | 6 | -14/+21 |
|\ \ | | | | | | | | | | | | | Reviewed by: Lars Wirzenius Reviewed by: Pedro Alvarezwq | ||||
| * | | Added chunkname prefix to some elements of the debug log. | Dan Firth | 2013-10-11 | 6 | -14/+21 |
| |/ | |||||
* | | Merge branch 'danielfirth/RT219' | Daniel Firth | 2013-10-11 | 3 | -8/+10 |
|\ \ | | | | | | | | | | | | | Reviewed by Lars Wirzenius Reviewd by Richard Ipsum | ||||
| * | | Morph now executes extensions in the repository containing the morphologies | Daniel Firth | 2013-09-26 | 3 | -8/+10 |
| | | | |||||
* | | | Merge remote-tracking branch 'remotes/origin/danielfirth/RT174' | Lars Wirzenius | 2013-10-10 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | | | | | | | | | Changed the message according to suggestion by Rob Kendrick, supported by Daniel Silverstone. | ||||
| * | | | Changed 'figuring out the right build order' to 'Computing artifact graph' | Daniel Firth | 2013-10-07 | 1 | -1/+1 |
| | |/ | |/| | |||||
* | | | Remove TAB characters | Lars Wirzenius | 2013-10-10 | 1 | -2/+2 |
| | | | |||||
* | | | Merge branch 'danielfirth/RT240' | Daniel Firth | 2013-10-10 | 1 | -0/+14 |
|\ \ \ | | | | | | | | | | | | | | | | | Reviewed by: Lars Wirzenius Reviewed by: Pedro Alvarez | ||||
| * | | | Morph now validates the 'tarball-server' option as a url | Daniel Firth | 2013-10-10 | 1 | -0/+14 |
| |/ / | |||||
* | | | Merge remote-tracking branch 'origin/baserock/richardipsum' | Lars Wirzenius | 2013-10-08 | 3 | -2/+48 |
|\ \ \ | | | | | | | | | | | | | | | | | Reviewed-by: Lars Wirzenius Reviewed-by: Richard Maw | ||||
| * | | | Fix bug in autotools version | Richard Ipsum | 2013-10-08 | 1 | -2/+3 |
| | | | | |||||
| * | | | Add yarn test for autotools version guesser | Richard Ipsum | 2013-10-08 | 2 | -0/+43 |
| |/ / | |||||
* | | | Merge remote-tracking branch 'origin/danielfirth/RT241' | Lars Wirzenius | 2013-10-08 | 2 | -2/+6 |
|\ \ \ | | | | | | | | | | | | | | | | | Reviewed-by: Lars Wirzenius Reviewed-by: Richard Maw | ||||
| * | | | Added 'description' field to static defaults for cluster morphologies | Dan Firth | 2013-10-07 | 2 | -2/+6 |
| |/ / | |||||
* | | | Merge remote-tracking branch 'origin/danielfirth/RT189' | Lars Wirzenius | 2013-10-08 | 29 | -207/+64 |
|\ \ \ | | | | | | | | | | | | | Changed the error (exception) to list all obsolete fields. | ||||
| * | | | Re-enabled validation for morph edit | Dan Firth | 2013-10-01 | 1 | -10/+0 |
| | | | | |||||
| * | | | 'system-kind' and 'disk-size' system morphology fields now raise ↵ | Dan Firth | 2013-10-01 | 2 | -0/+39 |
| | | | | | | | | | | | | | | | | ObsoleteFieldError | ||||
| * | | | Dropped support for 'system-kind' from system morphology files | Dan Firth | 2013-09-27 | 27 | -124/+20 |
| | | | | |||||
| * | | | Dropped support for 'disk-size' from system morphology files' | Dan Firth | 2013-09-27 | 15 | -79/+10 |
| | |/ | |/| | |||||
* | | | Fix test suite to work with current git | Lars Wirzenius | 2013-10-08 | 2 | -5/+5 |
| |/ |/| | | | | | | | Reviewed-by: Dan Firth Reviewed-by: Richard Maw |