Merge lp:~sinzui/juju-release-tools/publish-beta12 into lp:juju-release-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 320
Proposed branch: lp:~sinzui/juju-release-tools/publish-beta12
Merge into: lp:juju-release-tools
Diff against target: 178 lines (+33/-39)
5 files modified
build_package.py (+1/-1)
generate_agents.py (+1/-1)
make_release_notes.py (+21/-18)
tests/test_build_package.py (+1/-1)
tests/test_make_release_notes.py (+9/-18)
To merge this branch: bzr merge lp:~sinzui/juju-release-tools/publish-beta12
Reviewer Review Type Date Requested Status
Juju Release Engineering Pending
Review via email: mp+300249@code.launchpad.net

Description of the change

We need to publish beta12 which has daily versions.

1. Update the rules to parse the version to strip the '~' portion or the -ubuntu portion
2. Stop creating source packages for wily. We accidentally did this for 1.25.6.
   We are not certain we can release without deleting manual intervention
3. I had a dirty branch with template changes per issue I saw with gz's annoucement.
   I the changes are included here, though I don't consider them final.

To post a comment you must log in.
323. By Curtis Hovey

Use a single re.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'build_package.py'
2--- build_package.py 2016-05-20 14:42:31 +0000
3+++ build_package.py 2016-07-15 22:07:12 +0000
4@@ -30,7 +30,7 @@
5 14.04 trusty LTS
6 14.10 utopic HISTORIC
7 15.04 vivid HISTORIC
8-15.10 wily SUPPORTED
9+15.10 wily HITORIC
10 16.04 xenial LTS
11 16.10 yakkety DEVEL
12 """
13
14=== modified file 'generate_agents.py'
15--- generate_agents.py 2016-06-17 21:06:59 +0000
16+++ generate_agents.py 2016-07-15 22:07:12 +0000
17@@ -119,7 +119,7 @@
18 'control'])
19 control = deb822.Deb822(control_str)
20 control_version = control['Version']
21- base_version = re.sub('-0ubuntu.*$', '', control_version)
22+ base_version = re.sub('(~|-0ubuntu).*$', '', control_version)
23 major_minor = '.'.join(base_version.split('-')[0].split('.')[0:2])
24 series = juju_series.get_name_from_package_version(control_version)
25 architecture = control['Architecture']
26
27=== modified file 'make_release_notes.py'
28--- make_release_notes.py 2016-07-15 13:50:50 +0000
29+++ make_release_notes.py 2016-07-15 22:07:12 +0000
30@@ -18,29 +18,33 @@
31 PROPOSED = 'proposed'
32
33 DEVEL_TEMPLATE = """\
34-A new development release of Juju, {version} is here!
35+A new development release of Juju, {version}, is here!
36+
37
38 ## What's new?
39
40 {notable}
41
42+
43 ## How do I get it?
44
45 If you are running ubuntu, you can get it from the juju devel ppa:
46
47-sudo apt-add-repository ppa/devel
48-sudo apt update; sudo apt install juju-2.0
49+ sudo apt-add-repository ppa/devel
50+ sudo apt update; sudo apt install juju-2.0
51
52 Windows, Centos, and OS X users can get a corresponding installer at:
53
54 https://launchpad.net/juju-core/+milestone/{version}
55
56+
57 ## Feedback Appreciated!
58
59 We encourage everyone to subscribe the mailing list at
60 juju@lists.ubuntu.com and join us on #juju on freenode. We would love to hear
61 your feedback and usage of juju.
62
63+
64 ## Anything else?
65
66 You can read more information about what's in this release by viewing the
67@@ -51,20 +55,22 @@
68 """
69
70 PROPOSED_TEMPLATE = """\
71-# Juju {version}
72-
73-A new proposed stable release of Juju, juju {version}, is now available.
74+A new proposed stable release of Juju, {version}, is here!
75 This release may replace version {previous} on {release_date}.
76
77
78-## Getting Juju
79-
80-juju {version} is available for Yakkety and backported to earlier
81-series in the following PPA:
82+## What's new?
83+
84+{notable}
85+
86+
87+## How do I get it?
88+
89+If you are running ubuntu, you can get it from the juju proposed ppa:
90
91 https://launchpad.net/~juju/+archive/proposed
92
93-Windows, Centos, and OS X users will find installers at:
94+Windows, Centos, and OS X users can get a corresponding installer at:
95
96 https://launchpad.net/juju-core/+milestone/{version}
97
98@@ -73,20 +79,17 @@
99 juju agents.
100
101
102-## Notable Changes
103+## Feedback Appreciated!
104
105-{notable}
106+We encourage everyone to subscribe the mailing list at
107+juju@lists.ubuntu.com and join us on #juju on freenode. We would love to hear
108+your feedback and usage of juju.
109
110
111 ## Resolved issues
112
113 {resolved_text}
114
115-
116-Finally
117-
118-We encourage everyone to subscribe the mailing list at
119-juju@lists.ubuntu.com, or join us on #juju on freenode.
120 """
121
122
123
124=== modified file 'tests/test_build_package.py'
125--- tests/test_build_package.py 2016-05-20 14:42:31 +0000
126+++ tests/test_build_package.py 2016-07-15 22:07:12 +0000
127@@ -63,7 +63,7 @@
128 def test_get_living_names(self):
129 juju_series = _JujuSeries()
130 self.assertEqual(
131- ['precise', 'trusty', 'wily', 'xenial', 'yakkety'],
132+ ['precise', 'trusty', 'xenial', 'yakkety'],
133 juju_series.get_living_names())
134
135 def test_get_name(self):
136
137=== modified file 'tests/test_make_release_notes.py'
138--- tests/test_make_release_notes.py 2016-06-03 23:12:06 +0000
139+++ tests/test_make_release_notes.py 2016-07-15 22:07:12 +0000
140@@ -71,29 +71,20 @@
141 # Proposed purpose points to the proposed PPA without a warning.
142 text = make_notes('1.20.0', PROPOSED, " * One\n Lp 1")
143 self.assertIn(
144- 'A new proposed stable release of Juju, juju 1.20.0, '
145- 'is now available.',
146- text)
147- self.assertIn(
148- 'https://launchpad.net/~juju/+archive/proposed',
149- text)
150- self.assertIn(
151- ' * One\n Lp 1',
152- text)
153+ 'A new proposed stable release of Juju, 1.20.0, is here!', text)
154+ self.assertIn(
155+ 'https://launchpad.net/~juju/+archive/proposed', text)
156+ self.assertIn(' * One\n Lp 1', text)
157
158 def test_make_notes_with_devel_purpose(self):
159 # Devel purpose points to the devel PPA and a warning is included.
160 text = make_notes('1.21-alpha1', DEVEL, " * One\n Lp 1")
161 self.assertIn(
162- 'A new development release of Juju, juju 1.21-alpha1,'
163- ' is now available.',
164- text)
165- self.assertIn(
166- 'https://launchpad.net/~juju/+archive/devel',
167- text)
168- self.assertIn(
169- ' * One\n Lp 1',
170- text)
171+ 'A new development release of Juju, 1.21-alpha1, is here!',
172+ text)
173+ self.assertIn('sudo apt-add-repository ppa/devel', text)
174+ self.assertIn(
175+ 'https://jujucharms.com/docs/devel/temp-release-notes', text)
176
177 def test_make_notes_with_notable(self):
178 # The default value of None implies a stable bug fix release.

Subscribers

People subscribed via source and target branches