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

Replace deprecated terminology #11

Closed
timja opened this issue Aug 26, 2020 · 5 comments
Closed

Replace deprecated terminology #11

timja opened this issue Aug 26, 2020 · 5 comments
Labels
lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.

Comments

@timja
Copy link
Member

timja commented Aug 26, 2020

Slave has been deprecated for 4 years
'slave' -> 'agent'

Master recently became a deprecated term and was replaced with controller
'master' -> 'controller'

This can definitely be delivered incrementally.

@torstenwalter
Copy link
Member

Already replaced "Jenkins master" with "Jenkins server" in the readme. Would controller be the preferred term? It conflict a bit with Kubernetes terminology

Updating all values will take a while and be a breaking change. I suggest to do it either after or together with #13

@timja
Copy link
Member Author

timja commented Aug 26, 2020

Yes controller is the term,

cc @slide and @markyjackson-taulia @oleg-nenashev who are the ones mostly involved in this I believe and can provide more information, and I believe a blog is in progress.

@markjacksonfishing
Copy link

You are correct @timja

@slide
Copy link
Member

slide commented Aug 26, 2020

It would be hard not to conflict with another piece of software in terms. We recently had a public poll and controller was the top term and the Governance Meeting approved it on August 12 as the replacement.

@torstenwalter
Copy link
Member

torstenwalter commented Sep 2, 2020

I did a quick research in the chart to look for things which should be addressed:

  • master key in values file => should be renamed to controller

  • master.componentName: "jenkins-master" in values file => should be changed to controller.componentName: "controller"

  • many comments

  • slave in values.yaml

    slaveListenerPort: 50000
    slaveHostPort:
    # slaveListenerServiceType is the Kubernetes Service type for the JNLP agent service,
    slaveListenerServiceType: "ClusterIP"
    # Optionally assign an IP to the LoadBalancer slaveListenerService LoadBalancer
    # slaveListenerLoadBalancerIP: 1.2.3.4
    slaveListenerServiceAnnotations: {}
    slaveDefaultsProviderTemplate: ""
    slaveConnectTimeout: 5
    slaveReadTimeout: 15
    slaveKubernetesNamespace:
    slaveJenkinsUrl:
    slaveJenkinsTunnel:
    # slaveListenerServiceType: LoadBalancer
    # slaveListenerServiceAnnotations:
    componentName: "jenkins-slave"
    slaveConnectTimeout: 100
    
  • template file names:

    jenkins-agent-svc.yaml
    jenkins-master-alerting-rules.yaml
    jenkins-master-backendconfig.yaml
    jenkins-master-deployment.yaml
    jenkins-master-ingress.yaml
    jenkins-master-networkpolicy.yaml
    jenkins-master-route.yaml
    jenkins-master-servicemonitor.yaml
    jenkins-master-svc.yaml
    

    We could replace master with controller everywhere.

@torstenwalter torstenwalter added this to the v3.0.0 release milestone Sep 28, 2020
@torstenwalter torstenwalter added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. hacktoberfest labels Oct 5, 2020
torstenwalter added a commit that referenced this issue Oct 16, 2020
Part-of: #11

Signed-off-by: Torsten Walter <mail@torstenwalter.de>
@torstenwalter torstenwalter mentioned this issue Oct 18, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness.
Projects
Development

No branches or pull requests

4 participants