Merge ~kissiel/checkbox/+git/support:provider-diff into ~checkbox-dev/checkbox/+git/support:master

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Jonathan Cave
Approved revision: 8476c7525e3265c87f8a98f38c6bd57e69d14f66
Merged at revision: 5b95a8818db6ae326f9562277176cfd865d48d2b
Proposed branch: ~kissiel/checkbox/+git/support:provider-diff
Merge into: ~checkbox-dev/checkbox/+git/support:master
Diff against target: 167 lines (+155/-0)
2 files modified
bootstrap-checkbox.sh (+10/-0)
provider_diff.py (+145/-0)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Maciej Kisielewski (community) Needs Fixing
Review via email: mp+320785@code.qastaging.launchpad.net

Description of the change

Add script for diffing providers. The diff prints out jobs that have been added or deleted between two revisions.

To test it, you have to provide a configuration file that will tell the script what to diff, and what's needed.

Scenario 1:
Show me what changed in the last commit for P-P-S (taking my local repository provider)

before:
    source: /home/kissiel/snappy/plainbox-provider-snappy
    source-subdir: plainbox-provider-snappy
    source-commit: HEAD^
after:
    source: /home/kissiel/snappy/plainbox-provider-snappy
    source-subdir: plainbox-provider-snappy
    source-commit: HEAD

additional_providers:
    - p-p-c:
        source: git+ssh://git.launchpad.net/plainbox-provider-checkbox

Scenario 2:
Show me what changed since "v1.0" tag:

before:
    source: https://git.launchpad.net/plainbox-provider-snappy
    source-subdir: plainbox-provider-snappy
    source-commit: v1.0
after:
    source: /home/kissiel/snappy/plainbox-provider-snappy
    source-subdir: plainbox-provider-snappy
    source-commit: HEAD

additional_providers:
    - p-p-c:
        source: git+ssh://git.launchpad.net/plainbox-provider-checkbox

To post a comment you must log in.
Revision history for this message
Maciej Kisielewski (kissiel) wrote :

11:32 < joc> maybe just usage note / instruction note worthy
11:32 < kissiel> joc: well, I just opened the file and there's no info at all :D
11:33 < kissiel> bad!

review: Needs Fixing
Revision history for this message
Jonathan Cave (jocave) wrote :

Used the script to test the differences on a provider being modified for SRU work. All worked as expected and was very helpful.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches