Fix namespace/cluster name on templates. #244
Conversation
1. ConfigMap had Provider config pointing to kvcluster namespace instead of cluster namespace. 2. Deployment was looking for cluster named kvcluster instead of actual cluster name.
1. ConfigMap had Provider config pointing to kvcluster namespace instead of actual cluster namespace. 2. Deployment was looking for cluster kvcluster instead of actual cluster name.
1. ConfigMap config for Provider was looking for kvcluster namespace instead of actual cluster namespace. 2. Deployment was looking for kvcluster instead of actual cluster name.
1. Configmap data for Provider was looking for kvcluster namespace instead of actual cluster namespace. 2. Deployment is looking for kvcluster cluster instead of actual cluster name.
k8s-ci-robot
added
do-not-merge/work-in-progress
|
Welcome @feitnomore! |
k8s-ci-robot
removed
the
do-not-merge/work-in-progress
k8s-ci-robot
added
the
size/S
|
/lgtm |
k8s-ci-robot
added
the
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: feitnomore, qinqon 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 |
k8s-ci-robot
added
the
approved
What this PR does / why we need it: The templates provided have some caveats regarding cluster name and namespace.
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged): fixes # N/ASpecial notes for your reviewer: I was trying those templates and took a time to figure out the substitutions I had to perform here.