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

Fix 1399: added pod matching label in service selector #1404

Merged
merged 5 commits into from
Sep 14, 2021

Conversation

deepak-muley
Copy link
Contributor

@deepak-muley deepak-muley commented Sep 14, 2021

ref: #1399

Testing Details:

k -n kubeflow get pod -o wide
...
training-operator-749dfc7446-ktmjk                           1/1     Running   0          65s   172.20.16.63    karbon-dm-demo-3-9b4ea8-k8s-worker-1              
...
 k -n kubeflow describe service training-operator
...
Name:              training-operator
Namespace:         kubeflow
Labels:            app=training-operator
Annotations:       prometheus.io/path: /metrics
                   prometheus.io/port: 8443
                   prometheus.io/scrape: true
Selector:          control-plane=kubeflow-training-operator
Type:              ClusterIP
IP Families:       
IP:                172.19.137.178
IPs:               
Port:              monitoring-port  8443/TCP
TargetPort:        8443/TCP
Endpoints:         172.20.16.63:8443
...

@deepak-muley
Copy link
Contributor Author

/hold

@deepak-muley
Copy link
Contributor Author

/remove-hold

@deepak-muley
Copy link
Contributor Author

@Jeffwan Could you please review?

@Jeffwan
Copy link
Member

Jeffwan commented Sep 14, 2021

/retest
/lgtm
/approve

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Jeffwan

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

@deepak-muley
Copy link
Contributor Author

/test kubeflow-tf-operator-presubmit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants