Skip to content

cisco-nxos-provider: handle existing trustpoint#46

Merged
felix-kaestner merged 1 commit intomainfrom
trustpoint
Sep 2, 2025
Merged

cisco-nxos-provider: handle existing trustpoint#46
felix-kaestner merged 1 commit intomainfrom
trustpoint

Conversation

@felix-kaestner
Copy link
Contributor

Trying to replace the configuration of an existing trustpoint will fail with "disassociating rsa key-pair not allowed when identity certificate exists".

In order to avoid this, we check if the trustpoint already exists and return an error if it does. This can be checked in the caller and handled appropriately.

Additionally, when deleting a trustpoint, it now also delete the associated keyring that is not automatically garbage collected by NX-OS.

Trying to replace the configuration of an existing trustpoint will fail
with "disassociating rsa key-pair not allowed when identity certificate exists".

In order to avoid this, we check if the trustpoint already exists and return
an error if it does. This can be checked in the caller and handled appropriately.

Additionally, when deleting a trustpoint, it now also delete the associated
keyring that is not automatically garbage collected by NX-OS.
@felix-kaestner felix-kaestner requested a review from a team as a code owner September 2, 2025 14:04
@felix-kaestner felix-kaestner merged commit 46337a5 into main Sep 2, 2025
8 checks passed
@felix-kaestner felix-kaestner deleted the trustpoint branch September 2, 2025 14:31
@hardikdr hardikdr added the area/metal-automation Automation processes within the Metal project. label Sep 3, 2025
@hardikdr hardikdr added this to Roadmap Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/metal-automation Automation processes within the Metal project.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants