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

feat(get deployed): Add subcommand "get deployed" #2

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Bhargav-InfraCloud
Copy link
Owner

What this PR does / why we need it:

The command to print the list of resources under a named release:

helm get deployed <RELEASE_NAME>

Optional flags:
-o / --output <format> - Prints the output in the specified format. Allowed values: table, json, yaml (default table)

Fixes 12722

Special notes for your reviewer:

If applicable:

  • this PR contains documentation
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

    This is a new subcommand, hence shouldn't break anything in older versions.

@Bhargav-InfraCloud Bhargav-InfraCloud self-assigned this Mar 9, 2024
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch from 33ce9fe to 4ab0430 Compare March 10, 2024 18:57
@Bhargav-InfraCloud Bhargav-InfraCloud marked this pull request as draft March 10, 2024 18:57
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch 7 times, most recently from 66b48d3 to 12cb2cb Compare March 13, 2024 18:00
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch 3 times, most recently from 4310c8c to 14edcaa Compare June 5, 2024 23:32
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch 2 times, most recently from 5f7278d to 3f64509 Compare October 27, 2024 22:19
Pass these extra arguments in Helm test cases:
1. REST client getter to be used with Helm action config
2. Kube client options for be used with fake printer kube client

Related to helm#12722

Signed-off-by: Bhargav Ravuri <[email protected]>
Add new command `helm get deployed` to list the
resources in the release.

Fixes helm#12722

Signed-off-by: Bhargav Ravuri <[email protected]>
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch from 3f64509 to 4384396 Compare October 27, 2024 22:52
Add cmd/helm level tests related to `helm get deployed` command.

Related to helm#12722

Signed-off-by: Bhargav Ravuri <[email protected]>
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch from 4384396 to dec4d5b Compare November 3, 2024 01:05
Add pkg/action level tests related to `helm get deployed` command.

Related to helm#12722

Signed-off-by: Bhargav Ravuri <[email protected]>
@Bhargav-InfraCloud Bhargav-InfraCloud force-pushed the br_add_helm_get_deployed_command branch from dec4d5b to b314998 Compare November 3, 2024 01:10
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.

1 participant