Skip to content

Add CLI commands for cloudnativepg installation#1528

Merged
dannyzaken merged 1 commit intonoobaa:masterfrom
dannyzaken:danny-cnpg
Feb 17, 2025
Merged

Add CLI commands for cloudnativepg installation#1528
dannyzaken merged 1 commit intonoobaa:masterfrom
dannyzaken:danny-cnpg

Conversation

@dannyzaken
Copy link
Member

@dannyzaken dannyzaken commented Feb 13, 2025

This is an initial commit of the Cloudnative-pg integration.

Explain the changes

  • Add CmdCNPG with install, uninstall, status, and yaml subcommands
  • Implement YAML manifest loading and parsing with API group customization
  • Add resource validation
  • Add resource modifications according to options.
  • Support installation and uninstallation of cnpg operator resources
  • Add simple status check for cnpg operator deployment

Changes to install command:

  • For now, by default the install command does not install cnpg operator.
  • When using the flag --use-cnpg the cnpg operator and all its resources will be installed. It is still not used to create a postgres cluster.

Testing Instructions:

  1. cnpg installation alone:
noobaa cnpg install
  1. cnpg installation as part of noobaa install:
noobaa install --use-cnpg

@dannyzaken dannyzaken requested review from a team, aayushchouhan09, alphaprinz and guymguym and removed request for a team February 13, 2025 14:43
- Add CmdCNPG with install, uninstall, status, and yaml subcommands
- Implement YAML manifest loading and parsing with API group customization
- Add resource validation
- Add resource modifications according to options.
- Support installation and uninstallation of cnpg operator resources
- Add simple status check for cnpg operator deployment

Added CNPG yamls under deploy/cnpg

undo first commit

Signed-off-by: Danny Zaken <dannyzaken@gmail.com>
@dannyzaken dannyzaken merged commit 702ba6e into noobaa:master Feb 17, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants