Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Pebble for tests. #743

Merged
merged 9 commits into from Dec 28, 2020
Merged

Use Pebble for tests. #743

merged 9 commits into from Dec 28, 2020

Conversation

buchdag
Copy link
Member

@buchdag buchdag commented Dec 28, 2020

The PR replace Boulder with Pebble where possible (the only test that require Boulder for now is ocsp_must_staple).

Pebble is specifically made for tests, way easier and faster to setup (~5s on GitHub Actions vs ~100s with Boulder).

Quoting Pebble's doc:

Pebble has several top level goals:

  1. Provide a simplified ACME testing front end
  2. Provide a test-bed for new and compatibility breaking ACME features
  3. Encourage ACME client best-practices
  4. Aggressively build in guardrails against non-testing usage

Using Pebble also opens up testing on arm/v7 and arm64 archs, as it is relatively easy to build on those archs.

Local tests can now be run against either Pebble or Boulder, similar to the choice between 2 and 3 containers setups.

@buchdag buchdag added the type/ci PR that change the CI configuration files and scripts label Dec 28, 2020
@buchdag buchdag merged commit b91ec66 into nginx-proxy:master Dec 28, 2020
@buchdag buchdag deleted the gha-pebble branch December 28, 2020 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/ci PR that change the CI configuration files and scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant