Skip to content

Commit bbc048b

Browse files
committed
Update ODL to Beryllium official release
Uses the 40-release repo to pin to ODL Be official, vs 4-testing, which will always serve the latest Be build. Signed-off-by: Daniel Farrell <[email protected]>
1 parent d8cd994 commit bbc048b

File tree

7 files changed

+21
-19
lines changed

7 files changed

+21
-19
lines changed

CHANGELOG

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,3 +16,5 @@
1616
- Update ODL to Beryllium ERP RC3
1717
2016-02-16 Release 3.2.8
1818
- Update ODL to Beryllium ERP RC3.1
19+
2016-02-16 Release 3.2.9
20+
- Update ODL to Beryllium

README.markdown

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ module's [params](#parameters). If you need a new knob, [please raise an
3636
Issue][8].
3737

3838
Both supported [install methods](#install-method) default to the latest
39-
stable OpenDaylight release, which is currently [a Beryllium ERP][18].
39+
OpenDaylight release, which is currently [Beryllium][18].
4040

4141
## Setup
4242

@@ -364,4 +364,4 @@ See our [git commit history][17] for contributor information.
364364
[15]: https://github.com/dfarrell07/puppet-opendaylight/blob/master/CHANGELOG
365365
[16]: https://github.com/dfarrell07/puppet-opendaylight/releases
366366
[17]: https://github.com/dfarrell07/puppet-opendaylight/commits/master
367-
[18]: http://cbs.centos.org/repos/nfv7-opendaylight-4-testing/x86_64/os/Packages/
367+
[18]: http://cbs.centos.org/repos/nfv7-opendaylight-40-release/x86_64/os/Packages/

manifests/install.pp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,12 @@
1111
# Choose Yum URL based on OS (CentOS vs Fedora)
1212
# NB: Currently using the CentOS CBS for both Fedora and CentOS
1313
$base_url = $::operatingsystem ? {
14-
'CentOS' => 'http://cbs.centos.org/repos/nfv7-opendaylight-4-testing/$basearch/os/',
15-
'Fedora' => 'http://cbs.centos.org/repos/nfv7-opendaylight-4-testing/$basearch/os/',
14+
'CentOS' => 'http://cbs.centos.org/repos/nfv7-opendaylight-40-release/$basearch/os/',
15+
'Fedora' => 'http://cbs.centos.org/repos/nfv7-opendaylight-40-release/$basearch/os/',
1616
}
1717

1818
# Add OpenDaylight's Yum repository
19-
yumrepo { 'opendaylight-4-testing':
19+
yumrepo { 'opendaylight-40-release':
2020
# 'ensure' isn't supported with Puppet <3.5
2121
# Seems to default to present, but docs don't say
2222
# https://docs.puppetlabs.com/references/3.4.0/type.html#yumrepo
@@ -33,7 +33,7 @@
3333
# Install the OpenDaylight RPM
3434
package { 'opendaylight':
3535
ensure => present,
36-
require => Yumrepo['opendaylight-4-testing'],
36+
require => Yumrepo['opendaylight-40-release'],
3737
}
3838
}
3939
elsif $opendaylight::install_method == 'tarball' {

manifests/params.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
$extra_features = []
1313
$odl_rest_port = '8080'
1414
$install_method = 'rpm'
15-
$tarball_url = 'https://nexus.opendaylight.org/content/repositories/autorelease-1079/org/opendaylight/integration/distribution-karaf/0.4.0-Beryllium/distribution-karaf-0.4.0-Beryllium.tar.gz'
15+
$tarball_url = 'https://nexus.opendaylight.org/content/groups/public/org/opendaylight/integration/distribution-karaf/0.4.0-Beryllium/distribution-karaf-0.4.0-Beryllium.tar.gz'
1616
$unitfile_url = 'https://github.com/dfarrell07/opendaylight-systemd/archive/master/opendaylight-unitfile.tar.gz'
1717
$enable_l3 = 'no'
1818
$log_levels = {}

metadata.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "dfarrell07-opendaylight",
3-
"version": "3.2.8",
3+
"version": "3.2.9",
44
"author": "Daniel Farrell",
55
"summary": "Puppet module that installs and configures the OpenDaylight SDN controller",
66
"license": "BSD-2-Clause",

spec/spec_helper.rb

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ def tarball_install_tests(options = {})
231231
# Extract params
232232
# NB: These default values should be the same as ones in opendaylight::params
233233
# TODO: Remove this possible source of bugs^^
234-
tarball_url = options.fetch(:tarball_url, 'https://nexus.opendaylight.org/content/repositories/autorelease-1079/org/opendaylight/integration/distribution-karaf/0.4.0-Beryllium/distribution-karaf-0.4.0-Beryllium.tar.gz')
234+
tarball_url = options.fetch(:tarball_url, 'https://nexus.opendaylight.org/content/groups/public/org/opendaylight/integration/distribution-karaf/0.4.0-Beryllium/distribution-karaf-0.4.0-Beryllium.tar.gz')
235235
unitfile_url = options.fetch(:unitfile_url, 'https://github.com/dfarrell07/opendaylight-systemd/archive/master/opendaylight-unitfile.tar.gz')
236236
osfamily = options.fetch(:osfamily, 'RedHat')
237237

@@ -355,7 +355,7 @@ def tarball_install_tests(options = {})
355355
end
356356

357357
# Verify that there are no unexpected resources from RPM-type installs
358-
it { should_not contain_yumrepo('opendaylight-4-testing') }
358+
it { should_not contain_yumrepo('opendaylight-40-release') }
359359
it { should_not contain_package('opendaylight') }
360360
end
361361

@@ -366,28 +366,28 @@ def rpm_install_tests(options = {})
366366
operatingsystem = options.fetch(:operatingsystem, 'CentOS')
367367
case operatingsystem
368368
when 'CentOS'
369-
yum_repo = 'http://cbs.centos.org/repos/nfv7-opendaylight-4-testing/$basearch/os/'
369+
yum_repo = 'http://cbs.centos.org/repos/nfv7-opendaylight-40-release/$basearch/os/'
370370
when 'Fedora'
371-
yum_repo = 'http://cbs.centos.org/repos/nfv7-opendaylight-4-testing/$basearch/os/'
371+
yum_repo = 'http://cbs.centos.org/repos/nfv7-opendaylight-40-release/$basearch/os/'
372372
else
373373
fail("Unknown operatingsystem: #{operatingsystem}")
374374
end
375375

376376
# Default to CentOS 7 Yum repo URL
377377

378378
# Confirm presence of RPM-related resources
379-
it { should contain_yumrepo('opendaylight-4-testing') }
379+
it { should contain_yumrepo('opendaylight-40-release') }
380380
it { should contain_package('opendaylight') }
381381

382382
# Confirm relationships between RPM-related resources
383-
it { should contain_package('opendaylight').that_requires('Yumrepo[opendaylight-4-testing]') }
384-
it { should contain_yumrepo('opendaylight-4-testing').that_comes_before('Package[opendaylight]') }
383+
it { should contain_package('opendaylight').that_requires('Yumrepo[opendaylight-40-release]') }
384+
it { should contain_yumrepo('opendaylight-40-release').that_comes_before('Package[opendaylight]') }
385385

386386
# Confirm properties of RPM-related resources
387387
# NB: These hashes don't work with Ruby 1.8.7, but we
388388
# don't support 1.8.7 so that's okay. See issue #36.
389389
it {
390-
should contain_yumrepo('opendaylight-4-testing').with(
390+
should contain_yumrepo('opendaylight-40-release').with(
391391
'enabled' => '1',
392392
'gpgcheck' => '0',
393393
'descr' => 'CentOS CBS OpenDaylight Berillium testing repository',
@@ -413,7 +413,7 @@ def unsupported_os_tests(options = {})
413413
it { expect { should contain_class('opendaylight::service') }.to raise_error(Puppet::Error, /#{expected_msg}/) }
414414

415415
# Confirm that other resources fail on unsupported OSs
416-
it { expect { should contain_yumrepo('opendaylight-4-testing') }.to raise_error(Puppet::Error, /#{expected_msg}/) }
416+
it { expect { should contain_yumrepo('opendaylight-40-release') }.to raise_error(Puppet::Error, /#{expected_msg}/) }
417417
it { expect { should contain_package('opendaylight') }.to raise_error(Puppet::Error, /#{expected_msg}/) }
418418
it { expect { should contain_service('opendaylight') }.to raise_error(Puppet::Error, /#{expected_msg}/) }
419419
it { expect { should contain_file('org.apache.karaf.features.cfg') }.to raise_error(Puppet::Error, /#{expected_msg}/) }

spec/spec_helper_acceptance.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ def enable_l3_validations(options = {})
303303

304304
# Shared function that handles validations specific to RPM-type installs
305305
def rpm_validations()
306-
describe yumrepo('opendaylight-4-testing') do
306+
describe yumrepo('opendaylight-40-release') do
307307
it { should exist }
308308
it { should be_enabled }
309309
end
@@ -321,7 +321,7 @@ def tarball_validations()
321321

322322
# Repo checks break (not fail) when yum doesn't make sense (Ubuntu)
323323
if ['centos-7', 'fedora-22', 'fedora-23-docker'].include? ENV['RS_SET']
324-
describe yumrepo('opendaylight-4-testing') do
324+
describe yumrepo('opendaylight-40-release') do
325325
it { should_not exist }
326326
it { should_not be_enabled }
327327
end

0 commit comments

Comments
 (0)