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

[ci-skip] utilize the common vault-k8s-mgmt Helm chart #2311

Merged
merged 1 commit into from
Jul 17, 2023
Merged

[ci-skip] utilize the common vault-k8s-mgmt Helm chart #2311

merged 1 commit into from
Jul 17, 2023

Conversation

saurabhkumarkardam
Copy link
Contributor

Commit to be reviewed


feat(shared): utilize shared vault-k8s-mgmt Helm chart for Fabric and R3-Corda-Ent platforms.

This commit implements the following changes:

- Updated the shared role setup/vault-kubernetes to support both the Fabric and R3-Corda-Ent platforms.
- Implemented vault policy templates, fabric-policy.tpl and r3-corda-ent-policy.tpl, for Fabric and R3-Corda-Ent respectively.
- Updated the variables in the shared role setup/vault_kubernetes to ensure compatibility with both the Fabric and R3-Corda-Ent platforms.
- Removed the setup/vault-kubernetes from the Corda platform, enabling the use of the shared role setup/vault-kubernetes for the same purpose.
- Removed the setup/vault-kubernetes from the Fabric platform, enabling the use of the shared role setup/vault-kubernetes for the same purpose.
- Removed the Helm Chart vault_kubernetes from the Fabric platform, enabling the use of the shared Helm Chart vault-k8s-mgmt.

These changes enable code reuse and ensure consistency across the Fabric and R3-Corda-Ent platforms, as both now utilize the shared vault-k8s-mgmt Helm chart.

fixes #2291

@saurabhkumarkardam saurabhkumarkardam changed the title [ci-skip] utilize shared vault-k8s-mgmt Helm chart [ci-skip] utilize the common vault-k8s-mgmt Helm chart Jul 11, 2023
… R3-Corda-Ent platforms.

This commit implements the following changes:

 • Updated the shared role setup/vault-kubernetes to support both the Fabric and R3-Corda-Ent platforms.
 • Implemented vault policy templates, fabric-policy.tpl and r3-corda-ent-policy.tpl, for Fabric and R3-Corda-Ent respectively.
 • Updated the variables in the shared role setup/vault_kubernetes to ensure compatibility with both the Fabric and R3-Corda-Ent platforms.
 • Removed the setup/vault-kubernetes from the Corda platform, enabling the use of the shared role setup/vault-kubernetes for the same purpose.
 • Removed the setup/vault-kubernetes from the Fabric platform, enabling the use of the shared role setup/vault-kubernetes for the same purpose.
 • Removed the Helm Chart vault_kubernetes from the Fabric platform, enabling the use of the shared Helm Chart vault-k8s-mgmt.

These changes enable code reuse and ensure consistency across the Fabric and R3-Corda-Ent platforms, as both now utilize the shared vault-k8s-mgmt Helm chart.

fixes #2291

Signed-off-by: saurabhkumarkardam <[email protected]>
Copy link
Contributor

@suvajit-sarkar suvajit-sarkar left a comment

Choose a reason for hiding this comment

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

LGTM

@suvajit-sarkar suvajit-sarkar merged commit bb23f6b into hyperledger-bevel:develop Jul 17, 2023
4 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.

3 participants