Skip to content

Commit

Permalink
(PE-37192) Removing default install version
Browse files Browse the repository at this point in the history
To prevent installation happening without user specifying a version, we are removing the default version.

Updating tests to match latest LTS version
  • Loading branch information
ragingra committed Nov 10, 2023
1 parent 71d97e7 commit b363daa
Show file tree
Hide file tree
Showing 16 changed files with 16 additions and 18 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test-add-compiler.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
version:
description: 'PE version to install'
required: true
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-add-replica.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
version:
description: 'PE version to install'
required: true
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-backup-restore.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
version:
description: 'PE version to install'
required: true
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-failover.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
version_to_upgrade:
description: 'PE version to upgrade to'
required: false
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-install.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
version:
description: 'PE version to install'
required: true
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-upgrade-latest-dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ on:
type: string
required: true
description: "The initial version of PE to install before upgrade"
default: "2021.7.4"
default: "2021.7.6"
ssh-debugging:
description: "Boolean; whether or not to pause for ssh debugging"
required: true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
architecture:
- "extra-large-with-dr"
version:
- "2021.7.4"
- "2021.7.6"
image:
- "almalinux-cloud/almalinux-8"

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-upgrade-matrix.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
version:
- '2019.8.12'
version_to_upgrade:
- '2021.7.4'
- '2021.7.6'
image:
- 'almalinux-cloud/almalinux-8'
download_mode:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-upgrade.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ on:
upgrade_version:
description: 'PE version to upgrade to'
required: true
default: '2021.7.4'
default: '2021.7.6'
ssh-debugging:
description: 'Boolean; whether or not to pause for ssh debugging'
required: true
Expand Down
1 change: 0 additions & 1 deletion REFERENCE.md
Original file line number Diff line number Diff line change
Expand Up @@ -1826,7 +1826,6 @@ Data type: `Peadm::Pe_version`



Default value: `'2021.7.4'`

##### <a name="-peadm--install--dns_alt_names"></a>`dns_alt_names`

Expand Down
3 changes: 1 addition & 2 deletions documentation/install.md
Original file line number Diff line number Diff line change
Expand Up @@ -112,10 +112,9 @@ Example params.json Bolt parameters file (shown: Standard):
```json
{
"primary_host": "pe-xl-core-0.lab1.puppet.vm",

"console_password": "puppetlabs",
"dns_alt_names": [ "puppet", "puppet.lab1.puppet.vm" ],
"version": "2021.5.0",
"version": "2021.5.0"
}
```

Expand Down
2 changes: 1 addition & 1 deletion plans/install.pp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@

# Common Configuration
String $console_password,
Peadm::Pe_version $version = '2021.7.4',
Peadm::Pe_version $version,
Optional[String] $pe_installer_source = undef,
Optional[Array[String]] $dns_alt_names = undef,
Optional[String] $compiler_pool_address = undef,
Expand Down
2 changes: 1 addition & 1 deletion spec/functions/assert_supported_pe_version_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
end

it 'accepts the newest supported version' do
is_expected.to run.with_params('2021.7.4').and_return({ 'supported' => true })
is_expected.to run.with_params('2021.7.6').and_return({ 'supported' => true })
end

it 'accepts a version in the middle' do
Expand Down
2 changes: 1 addition & 1 deletion spec/plans/convert_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
allow_apply

expect_task('peadm::cert_data').return_for_targets('primary' => trustedjson)
expect_task('peadm::read_file').always_return({ 'content' => '2021.7.4' })
expect_task('peadm::read_file').always_return({ 'content' => '2021.7.6' })

# For some reason, expect_plan() was not working??
allow_plan('peadm::modify_certificate').always_return({})
Expand Down
2 changes: 1 addition & 1 deletion spec/plans/install_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
it 'runs successfully with the minimum required parameters' do
expect_plan('peadm::subplans::install')
expect_plan('peadm::subplans::configure')
expect(run_plan('peadm::install', 'primary_host' => 'primary', 'console_password' => 'puppetlabs')).to be_ok
expect(run_plan('peadm::install', 'primary_host' => 'primary', 'console_password' => 'puppetlabs', 'version' => '2021.7.6')).to be_ok
end
end
end
4 changes: 2 additions & 2 deletions spec/plans/upgrade_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ def allow_standard_non_returning_calls

expect(run_plan('peadm::upgrade',
'primary_host' => 'primary',
'version' => '2021.7.4')).to be_ok
'version' => '2021.7.6')).to be_ok
end

it 'runs with a primary, compilers, but no replica' do
Expand All @@ -47,7 +47,7 @@ def allow_standard_non_returning_calls
expect(run_plan('peadm::upgrade',
'primary_host' => 'primary',
'compiler_hosts' => 'compiler',
'version' => '2021.7.4')).to be_ok
'version' => '2021.7.6')).to be_ok
end

it 'fails if the primary uses the pcp transport' do
Expand Down

0 comments on commit b363daa

Please sign in to comment.