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
=== modified file 'build_package.py'
--- build_package.py 2016-05-20 14:42:31 +0000
+++ build_package.py 2016-07-15 22:07:12 +0000
@@ -30,7 +30,7 @@
3014.04 trusty LTS3014.04 trusty LTS
3114.10 utopic HISTORIC3114.10 utopic HISTORIC
3215.04 vivid HISTORIC3215.04 vivid HISTORIC
3315.10 wily SUPPORTED3315.10 wily HITORIC
3416.04 xenial LTS3416.04 xenial LTS
3516.10 yakkety DEVEL3516.10 yakkety DEVEL
36"""36"""
3737
=== modified file 'generate_agents.py'
--- generate_agents.py 2016-06-17 21:06:59 +0000
+++ generate_agents.py 2016-07-15 22:07:12 +0000
@@ -119,7 +119,7 @@
119 'control'])119 'control'])
120 control = deb822.Deb822(control_str)120 control = deb822.Deb822(control_str)
121 control_version = control['Version']121 control_version = control['Version']
122 base_version = re.sub('-0ubuntu.*$', '', control_version)122 base_version = re.sub('(~|-0ubuntu).*$', '', control_version)
123 major_minor = '.'.join(base_version.split('-')[0].split('.')[0:2])123 major_minor = '.'.join(base_version.split('-')[0].split('.')[0:2])
124 series = juju_series.get_name_from_package_version(control_version)124 series = juju_series.get_name_from_package_version(control_version)
125 architecture = control['Architecture']125 architecture = control['Architecture']
126126
=== modified file 'make_release_notes.py'
--- make_release_notes.py 2016-07-15 13:50:50 +0000
+++ make_release_notes.py 2016-07-15 22:07:12 +0000
@@ -18,29 +18,33 @@
18PROPOSED = 'proposed'18PROPOSED = 'proposed'
1919
20DEVEL_TEMPLATE = """\20DEVEL_TEMPLATE = """\
21A new development release of Juju, {version} is here!21A new development release of Juju, {version}, is here!
22
2223
23## What's new?24## What's new?
2425
25{notable}26{notable}
2627
28
27## How do I get it?29## How do I get it?
2830
29If you are running ubuntu, you can get it from the juju devel ppa:31If you are running ubuntu, you can get it from the juju devel ppa:
3032
31sudo apt-add-repository ppa/devel33 sudo apt-add-repository ppa/devel
32sudo apt update; sudo apt install juju-2.034 sudo apt update; sudo apt install juju-2.0
3335
34Windows, Centos, and OS X users can get a corresponding installer at:36Windows, Centos, and OS X users can get a corresponding installer at:
3537
36 https://launchpad.net/juju-core/+milestone/{version}38 https://launchpad.net/juju-core/+milestone/{version}
3739
40
38## Feedback Appreciated!41## Feedback Appreciated!
3942
40We encourage everyone to subscribe the mailing list at43We encourage everyone to subscribe the mailing list at
41juju@lists.ubuntu.com and join us on #juju on freenode. We would love to hear44juju@lists.ubuntu.com and join us on #juju on freenode. We would love to hear
42your feedback and usage of juju.45your feedback and usage of juju.
4346
47
44## Anything else?48## Anything else?
4549
46You can read more information about what's in this release by viewing the50You can read more information about what's in this release by viewing the
@@ -51,20 +55,22 @@
51"""55"""
5256
53PROPOSED_TEMPLATE = """\57PROPOSED_TEMPLATE = """\
54# Juju {version}58A new proposed stable release of Juju, {version}, is here!
55
56A new proposed stable release of Juju, juju {version}, is now available.
57This release may replace version {previous} on {release_date}.59This release may replace version {previous} on {release_date}.
5860
5961
60## Getting Juju62## What's new?
6163
62juju {version} is available for Yakkety and backported to earlier64{notable}
63series in the following PPA:65
66
67## How do I get it?
68
69If you are running ubuntu, you can get it from the juju proposed ppa:
6470
65 https://launchpad.net/~juju/+archive/proposed71 https://launchpad.net/~juju/+archive/proposed
6672
67Windows, Centos, and OS X users will find installers at:73Windows, Centos, and OS X users can get a corresponding installer at:
6874
69 https://launchpad.net/juju-core/+milestone/{version}75 https://launchpad.net/juju-core/+milestone/{version}
7076
@@ -73,20 +79,17 @@
73juju agents.79juju agents.
7480
7581
76## Notable Changes82## Feedback Appreciated!
7783
78{notable}84We encourage everyone to subscribe the mailing list at
85juju@lists.ubuntu.com and join us on #juju on freenode. We would love to hear
86your feedback and usage of juju.
7987
8088
81## Resolved issues89## Resolved issues
8290
83{resolved_text}91{resolved_text}
8492
85
86Finally
87
88We encourage everyone to subscribe the mailing list at
89juju@lists.ubuntu.com, or join us on #juju on freenode.
90"""93"""
9194
9295
9396
=== modified file 'tests/test_build_package.py'
--- tests/test_build_package.py 2016-05-20 14:42:31 +0000
+++ tests/test_build_package.py 2016-07-15 22:07:12 +0000
@@ -63,7 +63,7 @@
63 def test_get_living_names(self):63 def test_get_living_names(self):
64 juju_series = _JujuSeries()64 juju_series = _JujuSeries()
65 self.assertEqual(65 self.assertEqual(
66 ['precise', 'trusty', 'wily', 'xenial', 'yakkety'],66 ['precise', 'trusty', 'xenial', 'yakkety'],
67 juju_series.get_living_names())67 juju_series.get_living_names())
6868
69 def test_get_name(self):69 def test_get_name(self):
7070
=== modified file 'tests/test_make_release_notes.py'
--- tests/test_make_release_notes.py 2016-06-03 23:12:06 +0000
+++ tests/test_make_release_notes.py 2016-07-15 22:07:12 +0000
@@ -71,29 +71,20 @@
71 # Proposed purpose points to the proposed PPA without a warning.71 # Proposed purpose points to the proposed PPA without a warning.
72 text = make_notes('1.20.0', PROPOSED, " * One\n Lp 1")72 text = make_notes('1.20.0', PROPOSED, " * One\n Lp 1")
73 self.assertIn(73 self.assertIn(
74 'A new proposed stable release of Juju, juju 1.20.0, '74 'A new proposed stable release of Juju, 1.20.0, is here!', text)
75 'is now available.',75 self.assertIn(
76 text)76 'https://launchpad.net/~juju/+archive/proposed', text)
77 self.assertIn(77 self.assertIn(' * One\n Lp 1', text)
78 'https://launchpad.net/~juju/+archive/proposed',
79 text)
80 self.assertIn(
81 ' * One\n Lp 1',
82 text)
8378
84 def test_make_notes_with_devel_purpose(self):79 def test_make_notes_with_devel_purpose(self):
85 # Devel purpose points to the devel PPA and a warning is included.80 # Devel purpose points to the devel PPA and a warning is included.
86 text = make_notes('1.21-alpha1', DEVEL, " * One\n Lp 1")81 text = make_notes('1.21-alpha1', DEVEL, " * One\n Lp 1")
87 self.assertIn(82 self.assertIn(
88 'A new development release of Juju, juju 1.21-alpha1,'83 'A new development release of Juju, 1.21-alpha1, is here!',
89 ' is now available.',84 text)
90 text)85 self.assertIn('sudo apt-add-repository ppa/devel', text)
91 self.assertIn(86 self.assertIn(
92 'https://launchpad.net/~juju/+archive/devel',87 'https://jujucharms.com/docs/devel/temp-release-notes', text)
93 text)
94 self.assertIn(
95 ' * One\n Lp 1',
96 text)
9788
98 def test_make_notes_with_notable(self):89 def test_make_notes_with_notable(self):
99 # The default value of None implies a stable bug fix release.90 # The default value of None implies a stable bug fix release.

Subscribers

People subscribed via source and target branches