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

Update ACME tests to create DS indexes #4955

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Feb 6, 2025

The ACME tests have been updated to create DS indexes as described in the docs.

The tests have also been updated to no longer upload test artifacts to save space and reduce execution time.

The runner-init.sh has been updated to support multiple network aliases.

https://github.com/dogtagpki/pki/wiki/Installing-ACME-Responder-using-pkispawn
https://github.com/dogtagpki/pki/wiki/Installing-ACME-Responder-using-PKI-Server-ACME-CLI

The ACME tests have been updated to create DS indexes as
described in the docs.

The tests have also been updated to no longer upload test
artifacts to save space and reduce execution time.

The runner-init.sh has been updated to support multiple
network aliases.

https://github.com/dogtagpki/pki/wiki/Installing-ACME-Responder-using-pkispawn
https://github.com/dogtagpki/pki/wiki/Installing-ACME-Responder-using-PKI-Server-ACME-CLI
@edewata edewata requested a review from fmarco76 February 6, 2025 22:29
Copy link

sonarqubecloud bot commented Feb 6, 2025

Copy link
Member

@fmarco76 fmarco76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@edewata
Copy link
Contributor Author

edewata commented Feb 7, 2025

@fmarco76 Thanks!

@edewata edewata merged commit 4248ada into dogtagpki:master Feb 7, 2025
167 of 174 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants