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

Allow enrollmentJwt from secret (ziti-router-chart) #289

Conversation

jan94
Copy link
Contributor

@jan94 jan94 commented Jan 22, 2025

Hi @qrkourier,

Currently, the only way to specify the enrollmentJwt is by setting the enrollmentJwt property in the values for the ziti-router chart. This approach can be cumbersome when automating the deployment process of Ziti controllers and routers, as the enrollmentJwt must be actively acquired from the Ziti controller by calling its API.

Proposal:
I propose allowing the specification of a secret name (for a secret which could be created before the actual Ziti router deployment) that holds the enrollmentJwt. This way, an automated process can interact with the Ziti controller API to create the secret in the respective Kubernetes namespace before triggering the router deployment.

Benefits:

  • Streamlines the deployment process by reducing manual steps.
  • Enhances automation capabilities for Ziti controllers and routers.

This enhancement would significantly improve my deployment procedures and likely benefit others facing similar challenges.
BR
Jan

@qrkourier qrkourier changed the base branch from main to jan94_feature-allowEnrollmentJwtFromSecret January 22, 2025 21:30
Copy link
Member

@qrkourier qrkourier left a comment

Choose a reason for hiding this comment

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

Wonderful idea. Thank you.

@qrkourier qrkourier merged commit 6cd6472 into openziti:jan94_feature-allowEnrollmentJwtFromSecret Jan 22, 2025
1 check passed
@jan94
Copy link
Contributor Author

jan94 commented Jan 23, 2025

You are welcome :)

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