public_git
/
kubernetes-operator
mirror of https://github.com/jenkinsci/kubernetes-operator.git
1
0
Fork 0
Code Issues Releases Wiki Activity

refactor service name and service fqdn fuctions

This commit is contained in:
Seshu Kumar Alluvada 2020-03-24 00:30:24 -04:00
parent 0082392dbf
commit d71d105986
5 changed files with 14 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/controller/jenkins/configuration/base/reconcile.go
View File

@ -183,11 +183,11 @@ func (r *ReconcileJenkinsBaseConfiguration) ensureResourcesRequiredForJenkinsPod
} }
r.logger.V(log.VDebug).Info("Extra role bindings are present") r.logger.V(log.VDebug).Info("Extra role bindings are present")
if err := r.createService(metaObject, resources.GetJenkinsHTTPServiceName(r.Configuration.Jenkins), r.Configuration.Jenkins.Spec.Service); err != nil { if err := r.createService(metaObject, resources.GetJenkinsServiceName(r.Configuration.Jenkins, "http"), r.Configuration.Jenkins.Spec.Service); err != nil {
return err return err
} }
r.logger.V(log.VDebug).Info("Jenkins HTTP Service is present") r.logger.V(log.VDebug).Info("Jenkins HTTP Service is present")
if err := r.createService(metaObject, resources.GetJenkinsSlavesServiceName(r.Configuration.Jenkins), r.Configuration.Jenkins.Spec.SlaveService); err != nil { if err := r.createService(metaObject, resources.GetJenkinsServiceName(r.Configuration.Jenkins, "slave"), r.Configuration.Jenkins.Spec.SlaveService); err != nil {
return err return err
} }
r.logger.V(log.VDebug).Info("Jenkins slave Service is present") r.logger.V(log.VDebug).Info("Jenkins slave Service is present")
@ -951,7 +951,7 @@ func (r *ReconcileJenkinsBaseConfiguration) getJenkinsAPIUrl() (string, error) {
err := r.Client.Get(context.TODO(), types.NamespacedName{ err := r.Client.Get(context.TODO(), types.NamespacedName{
Namespace: r.Configuration.Jenkins.ObjectMeta.Namespace, Namespace: r.Configuration.Jenkins.ObjectMeta.Namespace,
Name: resources.GetJenkinsHTTPServiceName(r.Configuration.Jenkins), Name: resources.GetJenkinsServiceName(r.Configuration.Jenkins, "http"),
}, &service) }, &service)
if err != nil { if err != nil {

4
pkg/controller/jenkins/configuration/base/resources/base_configuration_configmap.go
View File

@ -188,8 +188,8 @@ func NewBaseConfigurationConfigMap(meta metav1.ObjectMeta, jenkins *v1alpha2.Jen
disableInsecureFeaturesGroovyScriptName: disableInsecureFeatures, disableInsecureFeaturesGroovyScriptName: disableInsecureFeatures,
configureKubernetesPluginGroovyScriptName: fmt.Sprintf(configureKubernetesPluginFmt, configureKubernetesPluginGroovyScriptName: fmt.Sprintf(configureKubernetesPluginFmt,
jenkins.ObjectMeta.Namespace, jenkins.ObjectMeta.Namespace,
fmt.Sprintf("http://%s:%d", GetJenkinsHTTPServiceFQDN(jenkins), jenkins.Spec.Service.Port), fmt.Sprintf("http://%s:%d", GetJenkinsServiceFQDN(jenkins, "http"), jenkins.Spec.Service.Port),
fmt.Sprintf("%s:%d", GetJenkinsSlavesServiceFQDN(jenkins), jenkins.Spec.SlaveService.Port), fmt.Sprintf("%s:%d", GetJenkinsServiceFQDN(jenkins, "slave"), jenkins.Spec.SlaveService.Port),
), ),
configureViewsGroovyScriptName: configureViews, configureViewsGroovyScriptName: configureViews,
disableJobDslScriptApprovalGroovyScriptName: disableJobDSLScriptApproval, disableJobDslScriptApprovalGroovyScriptName: disableJobDSLScriptApproval,

24
pkg/controller/jenkins/configuration/base/resources/service.go
View File

@ -32,28 +32,16 @@ func UpdateService(actual corev1.Service, config v1alpha2.Service) corev1.Servic
return actual return actual
} }
// GetJenkinsHTTPServiceName returns Kubernetes service name used for expose Jenkins HTTP endpoint // GetJenkinsServiceName returns Kubernetes service name used for expose Jenkins HTTP and Slaves endpoint
func GetJenkinsHTTPServiceName(jenkins *v1alpha2.Jenkins) string { func GetJenkinsServiceName(jenkins *v1alpha2.Jenkins, serviceType string) string {
return fmt.Sprintf("%s-http-%s", constants.OperatorName, jenkins.ObjectMeta.Name) return fmt.Sprintf("%s-%s-%s", constants.OperatorName, serviceType, jenkins.ObjectMeta.Name)
} }
// GetJenkinsSlavesServiceName returns Kubernetes service name used for expose Jenkins slave endpoint // GetJenkinsServiceFQDN returns Kubernetes service FQDN used for expose Jenkins HTTP and Slaves endpoint
func GetJenkinsSlavesServiceName(jenkins *v1alpha2.Jenkins) string { func GetJenkinsServiceFQDN(jenkins *v1alpha2.Jenkins, serviceType string) string {
return fmt.Sprintf("%s-slave-%s", constants.OperatorName, jenkins.ObjectMeta.Name)
}
// GetJenkinsHTTPServiceFQDN returns Kubernetes service FQDN used for expose Jenkins HTTP endpoint
func GetJenkinsHTTPServiceFQDN(jenkins *v1alpha2.Jenkins) string {
clusterDomain := getClusterDomain() clusterDomain := getClusterDomain()
return fmt.Sprintf("%s-http-%s.%s", constants.OperatorName, jenkins.ObjectMeta.Name, clusterDomain) return fmt.Sprintf("%s-%s-%s.%s", constants.OperatorName, serviceType, jenkins.ObjectMeta.Name, clusterDomain)
}
// GetJenkinsSlavesServiceFQDN returns Kubernetes service FQDN used for expose Jenkins slave endpoint
func GetJenkinsSlavesServiceFQDN(jenkins *v1alpha2.Jenkins) string {
clusterDomain := getClusterDomain()
return fmt.Sprintf("%s-slave-%s.%s", constants.OperatorName, jenkins.ObjectMeta.Name, clusterDomain)
} }
// GetClusterDomain returns Kubernetes cluster domain, default to "cluster.local" // GetClusterDomain returns Kubernetes cluster domain, default to "cluster.local"

4
pkg/controller/jenkins/configuration/user/seedjobs/seedjobs.go
View File

@ -403,7 +403,7 @@ func agentDeployment(jenkins *v1alpha2.Jenkins, namespace string, agentName stri
{ {
Name: "JENKINS_TUNNEL", Name: "JENKINS_TUNNEL",
Value: fmt.Sprintf("%s:%d", Value: fmt.Sprintf("%s:%d",
resources.GetJenkinsSlavesServiceFQDN(jenkins), resources.GetJenkinsServiceFQDN(jenkins, "slave"),
jenkins.Spec.SlaveService.Port), jenkins.Spec.SlaveService.Port),
}, },
{ {
@ -417,7 +417,7 @@ func agentDeployment(jenkins *v1alpha2.Jenkins, namespace string, agentName stri
{ {
Name: "JENKINS_URL", Name: "JENKINS_URL",
Value: fmt.Sprintf("http://%s:%d", Value: fmt.Sprintf("http://%s:%d",
resources.GetJenkinsHTTPServiceFQDN(jenkins), resources.GetJenkinsServiceFQDN(jenkins, "http"),
jenkins.Spec.Service.Port, jenkins.Spec.Service.Port,
), ),
}, },

2
test/e2e/jenkins.go
View File

@ -57,7 +57,7 @@ func createJenkinsAPIClient(jenkins *v1alpha2.Jenkins, hostname string, port int
err := framework.Global.Client.Get(context.TODO(), types.NamespacedName{ err := framework.Global.Client.Get(context.TODO(), types.NamespacedName{
Namespace: jenkins.Namespace, Namespace: jenkins.Namespace,
Name: resources.GetJenkinsHTTPServiceName(jenkins), Name: resources.GetJenkinsServiceName(jenkins, "http"),
}, &service) }, &service)
if err != nil { if err != nil {