Skip to content

Commit

Permalink
add dedicated website profile for web + downloads
Browse files Browse the repository at this point in the history
  • Loading branch information
evgeni committed Nov 15, 2024
1 parent d54cd57 commit bfe2b00
Show file tree
Hide file tree
Showing 3 changed files with 47 additions and 0 deletions.
5 changes: 5 additions & 0 deletions puppet/manifests/site.pp
Original file line number Diff line number Diff line change
Expand Up @@ -51,3 +51,8 @@
include profiles::base
include profiles::repo::rpm
}

node /^website\d+\.[a-z]+\.theforeman\.org$/ {
include profiles::base
include profiles::website
}
25 changes: 25 additions & 0 deletions puppet/modules/profiles/manifests/website.pp
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
# @summary A profile for the machines hosting the website and downloads
#
# @param stable
# Latest release that users expect
#
# @param https
# Whether to enable HTTPS. This is typically wanted but can only be enabled
# in a 2 pass setup. First Apache needs to run for Letsencrypt to function.
# Then Letsencrypt can be enabled. Also useful to turn off in test setups.
class profiles::website (
String[1] $stable,
Boolean $https = true,
) {
class { 'web':
https => $https,
}
contain web

contain web::vhost::downloads

class { 'web::vhost::web':
stable => $stable,
}
contain web::vhost::web
}
17 changes: 17 additions & 0 deletions puppet/spec/classes/profiles_website_spec.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
require 'spec_helper'

describe 'profiles::website' do
on_supported_os.each do |os, os_facts|
context "on #{os}" do
let(:facts) { os_facts }

it { is_expected.to compile.with_all_deps }

context 'without https' do
let(:params) { { https: false } }

it { is_expected.to compile.with_all_deps }
end
end
end
end

0 comments on commit bfe2b00

Please sign in to comment.