Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Added support for clusterIP and headless services. #8312

Merged
merged 1 commit into from
Oct 9, 2018

Conversation

guptaarvindk
Copy link
Contributor

Signed-off-by: Arvind Gupta guptaarvindk@gmail.com

What this PR does / why we need it:

support headless service or specific clusterIp for service.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

support headless service or specific clusterIp for service.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md

Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
@ey-bot ey-bot added the Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). label Oct 9, 2018
@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Oct 9, 2018
@jdolitsky
Copy link
Contributor

/ok-to-test

@k8s-ci-robot k8s-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Oct 9, 2018
@jdolitsky
Copy link
Contributor

@guptaarvindk thanks!
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 9, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: guptaarvindk, jdolitsky

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 9, 2018
@k8s-ci-robot k8s-ci-robot merged commit d2983fb into helm:master Oct 9, 2018
faizanahmad055 pushed a commit to stakater/helm-charts that referenced this pull request Oct 10, 2018
Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
Signed-off-by: faizanahmad055 <faizan.ahmad55@outlook.com>
darioblanco pushed a commit to minddocdev/charts that referenced this pull request Oct 22, 2018
Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
emas80 pushed a commit to faceit/charts that referenced this pull request Oct 24, 2018
Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
Jnig pushed a commit to Jnig/charts that referenced this pull request Nov 13, 2018
Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
Signed-off-by: Jakob Niggel <info@jakobniggel.de>
wgiddens pushed a commit to wgiddens/charts that referenced this pull request Jan 18, 2019
Signed-off-by: Arvind Gupta <guptaarvindk@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants