Merge lp:~1chb1n/ubuntu-openstack-ci/ubuntu-openstack-ci-dev into lp:ubuntu-openstack-ci

Proposed by Ryan Beisner
Status: Merged
Approved by: Ryan Beisner
Approved revision: 5
Merged at revision: 5
Proposed branch: lp:~1chb1n/ubuntu-openstack-ci/ubuntu-openstack-ci-dev
Merge into: lp:ubuntu-openstack-ci
Diff against target: 39 lines (+8/-2)
2 files modified
populate/mappings.yaml (+1/-1)
scan_branches.py (+7/-1)
To merge this branch: bzr merge lp:~1chb1n/ubuntu-openstack-ci/ubuntu-openstack-ci-dev
Reviewer Review Type Date Requested Status
Ryan Beisner (community) Approve
Review via email: mp+236937@code.launchpad.net

Description of the change

update mappings, adjust base name logic in branch scanner

To post a comment you must log in.
Revision history for this message
Ryan Beisner (1chb1n) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'populate/mappings.yaml'
2--- populate/mappings.yaml 2014-10-02 15:18:00 +0000
3+++ populate/mappings.yaml 2014-10-02 16:56:57 +0000
4@@ -55,7 +55,7 @@
5 - lp:~openstack-charmers/charms/trusty/quantum-gateway/next
6 - lp:~openstack-charmers/charms/trusty/swift-proxy/next
7 - lp:~openstack-charmers/charms/trusty/swift-storage/next
8- - lp:~uosci/ubuntu-openstack-ci/trunk/
9+ - lp:~uosci/ubuntu-openstack-ci/trunk
10 - lp:~openstack-charmers/+junk/openstack-charm-testing
11
12 # in relation to openstack-charm-testing branch
13
14=== modified file 'scan_branches.py'
15--- scan_branches.py 2014-10-02 15:18:00 +0000
16+++ scan_branches.py 2014-10-02 16:56:57 +0000
17@@ -215,6 +215,11 @@
18 return contexts
19
20
21+def strip_chars(str):
22+ '''Strip characters from string'''
23+ return str.translate(None,'+~/\\')
24+
25+
26 def create_properties_dict(changed):
27 '''Generate properties data for non-charm, which can be
28 passed to a make lint-properties, unit-properties files
29@@ -227,8 +232,9 @@
30 _base_name = _branch_str.split('/')[-2]
31 _display_name = '{}-next'.format(_base_name)
32 else:
33- _base_name = _branch_str.split('/')[-1]
34+ _base_name = strip_chars('-'.join(_branch_str.split('/')[-2:]))
35 _display_name = _base_name
36+ _base_name.translate(None,'+~')
37
38 contexts.append({
39 'branch': _branch_str,

Subscribers

People subscribed via source and target branches