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 pkidestroy to support ACME #4852

Merged
merged 1 commit into from
Sep 20, 2024
Merged

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Sep 20, 2024

pkidestroy has been updated to support removing ACME from PKI server. If it is the last subsystem on the server, the server will be removed as well.

The ACMESubsystem.create() has been modified to create a base dir (i.e. /var/lib/pki/<instance>/<subsystem>) which is used by PKIServer.load_subsystems() to determine if the subsystem exists. The code that creates the conf and logs folders has been moved into create_conf() and create_logs(), respectively.

The pki-server acme-remove has been updated to provide options to remove the conf and logs folders.

The tests that use pkispawn to install ACME have been updated to use pkidestroy to remove ACME.

pkidestroy has been updated to support removing ACME from
PKI server. If it is the last subsystem on the server, the
server will be removed as well.

The ACMESubsystem.create() has been modified to create a base
dir (i.e. /var/lib/pki/<instance>/<subsystem>) which is used
by PKIServer.load_subsystems() to determine if the subsystem
exists. The code that creates the conf and logs folders has
been moved into create_conf() and create_logs(), respectively.

The pki-server acme-remove has been updated to provide options
to remove the conf and logs folders.

The tests that use pkispawn to install ACME have been updated
to use pkidestroy to remove ACME.
Copy link

sonarcloud bot commented Sep 20, 2024

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 Sep 20, 2024

@fmarco76 Thanks!

@edewata edewata merged commit 2217c89 into dogtagpki:master Sep 20, 2024
151 of 159 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