Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Add kapp-controller 0.30.0 package #2558

Conversation

danniel1205
Copy link
Contributor

Signed-off-by: Daniel Guo danniel1205@gmail.com

What this PR does / why we need it

This PR adds kapp-controller 0.30.0 package under addons. The 0.30.0 version of kapp-controller supports a shorthand for easily adding the KUBERNETES_SERVICE_HOST environment variable to kapp-controller’s noProxy controller config property. This can help when a Kubernetes cluster is configured with a proxy and the kapp-controller-config is created with the http and https proxy URL. More details could be found at: https://carvel.dev/kapp-controller/docs/latest/controller-config/#config-shorthands

Details for the Release Notes (PLEASE PROVIDE)

Add kapp-controller 0.30.0 package

Which issue(s) this PR fixes

Fixes: #2556

Describe testing done for PR

  • Manually run ytt command and verified the manifests are rendered properly.

Special notes for your reviewer

Signed-off-by: Daniel Guo <danniel1205@gmail.com>
@danniel1205 danniel1205 requested a review from a team as a code owner November 16, 2021 21:53
@github-actions github-actions bot added owner/docs Work executed by VMware documentation team owner/packages Work executed by a package's maintainer labels Nov 16, 2021
@github-actions
Copy link

Hi @danniel1205! And thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Tanzu Community Edition better.

Copy link
Contributor

@blc1996 blc1996 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@shivaani0505 shivaani0505 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@seemiller seemiller left a comment

Choose a reason for hiding this comment

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

LGTM

@seemiller seemiller merged commit 87e39a6 into vmware-tanzu:main Nov 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/docs Work executed by VMware documentation team owner/packages Work executed by a package's maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add kapp-controller 0.30.0 package
5 participants