Skip to content

Commit bd3dde4

Browse files
authored
Merge pull request #1120 from nginx-proxy/letsencrypt-ghcr.io
test: upgrade pebble to 2.5.2
2 parents 32c6771 + 973a723 commit bd3dde4

File tree

6 files changed

+45
-56
lines changed

6 files changed

+45
-56
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ on:
1414
- "*.md"
1515

1616
env:
17+
ACME_CA: pebble
1718
DOCKER_GEN_CONTAINER_NAME: nginx-proxy-gen
1819
IMAGE: nginxproxy/acme-companion
1920
NGINX_CONTAINER_NAME: nginx-proxy
@@ -97,25 +98,17 @@ jobs:
9798
permissions_custom,
9899
symlinks,
99100
acme_hooks,
101+
ocsp_must_staple,
100102
]
101103
setup: [2containers, 3containers]
102-
acme-ca: [pebble]
103104
pebble-config: [pebble-config.json]
104105
include:
105106
- test-name: acme_eab
106107
setup: 2containers
107-
acme-ca: pebble
108108
pebble-config: pebble-config-eab.json
109109
- test-name: acme_eab
110110
setup: 3containers
111-
acme-ca: pebble
112111
pebble-config: pebble-config-eab.json
113-
- test-name: ocsp_must_staple
114-
setup: 2containers
115-
acme-ca: boulder
116-
- test-name: ocsp_must_staple
117-
setup: 3containers
118-
acme-ca: boulder
119112
runs-on: ubuntu-latest
120113

121114
steps:
@@ -133,19 +126,13 @@ jobs:
133126
done
134127
135128
- name: Setup Pebble
136-
if: ${{ matrix.acme-ca == 'pebble' }}
137129
env:
138130
PEBBLE_CONFIG: ${{ matrix.pebble-config }}
139131
run: test/setup/pebble/setup-pebble.sh
140132

141-
- name: Setup Boulder
142-
if: ${{ matrix.acme-ca == 'boulder' }}
143-
run: test/setup/setup-boulder.sh
144-
145133
- name: Setup nginx-proxy
146134
env:
147135
SETUP: ${{ matrix.setup }}
148-
ACME_CA: ${{ matrix.acme-ca }}
149136
run: test/setup/setup-nginx-proxy.sh
150137

151138
# ADD BUILT IMAGE
@@ -161,13 +148,11 @@ jobs:
161148
- name: Integration Testing
162149
env:
163150
SETUP: ${{ matrix.setup }}
164-
ACME_CA: ${{ matrix.acme-ca }}
165151
PEBBLE_CONFIG: ${{ matrix.pebble-config }}
166152
run: test/run.sh -t ${{ matrix.test-name }} "$IMAGE"
167153

168154
- name: Display containers logs
169155
if: ${{ failure() }}
170156
env:
171157
SETUP: ${{ matrix.setup }}
172-
ACME_CA: ${{ matrix.acme-ca }}
173158
run: test/github_actions/containers-logs.sh

test/config.sh

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ globalTests+=(
1818
permissions_custom
1919
symlinks
2020
acme_hooks
21+
ocsp_must_staple
2122
)
2223

2324
# The acme_eab test requires Pebble with a specific configuration
@@ -26,10 +27,3 @@ if [[ "$ACME_CA" == 'pebble' && "$PEBBLE_CONFIG" == 'pebble-config-eab.json' ]];
2627
acme_eab
2728
)
2829
fi
29-
30-
# The ocsp_must_staple test does not work with Pebble
31-
if [[ "$ACME_CA" == 'boulder' ]]; then
32-
globalTests+=(
33-
ocsp_must_staple
34-
)
35-
fi

test/setup/pebble/.env

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
PEBBLE_VERSION='v2.3.1'
1+
PEBBLE_VERSION='2.5.2'
22
PEBBLE_CONFIG='pebble-config.json'

test/setup/pebble/docker-compose.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
version: '3'
1+
version: "3"
22

33
services:
44
pebble:
5-
image: "letsencrypt/pebble:${PEBBLE_VERSION}"
5+
image: "ghcr.io/letsencrypt/pebble:${PEBBLE_VERSION}"
66
container_name: pebble
77
volumes:
88
- "./${PEBBLE_CONFIG}:/test/config/pebble-config.json"
99
environment:
1010
- PEBBLE_VA_NOSLEEP=1
11-
command: pebble -config /test/config/pebble-config.json -dnsserver 10.30.50.3:8053
11+
command: -config /test/config/pebble-config.json -dnsserver 10.30.50.3:8053
1212
ports:
1313
- 14000:14000 # HTTPS ACME API
1414
- 15000:15000 # HTTPS Management API
@@ -17,9 +17,9 @@ services:
1717
ipv4_address: 10.30.50.2
1818

1919
challtestsrv:
20-
image: "letsencrypt/pebble-challtestsrv:${PEBBLE_VERSION}"
20+
image: "ghcr.io/letsencrypt/pebble-challtestsrv:${PEBBLE_VERSION}"
2121
container_name: challtestserv
22-
command: pebble-challtestsrv -tlsalpn01 ""
22+
command: -defaultIPv6 "" -defaultIPv4 10.30.50.3
2323
ports:
2424
- 8055:8055 # HTTP Management API
2525
networks:
Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,21 @@
11
{
2-
"pebble": {
3-
"listenAddress": "0.0.0.0:14000",
4-
"managementListenAddress": "0.0.0.0:15000",
5-
"certificate": "test/certs/localhost/cert.pem",
6-
"privateKey": "test/certs/localhost/key.pem",
7-
"httpPort": 80,
8-
"tlsPort": 443,
9-
"ocspResponderURL": "",
10-
"externalAccountBindingRequired": true,
11-
"externalAccountMACKeys": {
12-
"kid-1": "zWNDZM6eQGHWpSRTPal5eIUYFTu7EajVIoguysqZ9wG44nMEtx3MUAsUDkMTQ12W",
13-
"kid-2": "b10lLJs8l1GPIzsLP0s6pMt8O0XVGnfTaCeROxQM0BIt2XrJMDHJZBM5NuQmQJQH"
14-
}
15-
}
16-
}
2+
"pebble": {
3+
"listenAddress": "0.0.0.0:14000",
4+
"managementListenAddress": "0.0.0.0:15000",
5+
"certificate": "test/certs/localhost/cert.pem",
6+
"privateKey": "test/certs/localhost/key.pem",
7+
"httpPort": 80,
8+
"tlsPort": 443,
9+
"ocspResponderURL": "",
10+
"externalAccountBindingRequired": true,
11+
"externalAccountMACKeys": {
12+
"kid-1": "zWNDZM6eQGHWpSRTPal5eIUYFTu7EajVIoguysqZ9wG44nMEtx3MUAsUDkMTQ12W",
13+
"kid-2": "b10lLJs8l1GPIzsLP0s6pMt8O0XVGnfTaCeROxQM0BIt2XrJMDHJZBM5NuQmQJQH"
14+
},
15+
"retryAfter": {
16+
"authz": 3,
17+
"order": 5
18+
},
19+
"certificateValidityPeriod": 157766400
20+
}
21+
}
Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,17 @@
11
{
2-
"pebble": {
3-
"listenAddress": "0.0.0.0:14000",
4-
"managementListenAddress": "0.0.0.0:15000",
5-
"certificate": "test/certs/localhost/cert.pem",
6-
"privateKey": "test/certs/localhost/key.pem",
7-
"httpPort": 80,
8-
"tlsPort": 443,
9-
"ocspResponderURL": "",
10-
"externalAccountBindingRequired": false
11-
}
12-
}
2+
"pebble": {
3+
"listenAddress": "0.0.0.0:14000",
4+
"managementListenAddress": "0.0.0.0:15000",
5+
"certificate": "test/certs/localhost/cert.pem",
6+
"privateKey": "test/certs/localhost/key.pem",
7+
"httpPort": 80,
8+
"tlsPort": 443,
9+
"ocspResponderURL": "",
10+
"externalAccountBindingRequired": false,
11+
"retryAfter": {
12+
"authz": 3,
13+
"order": 5
14+
},
15+
"certificateValidityPeriod": 157766400
16+
}
17+
}

0 commit comments

Comments
 (0)