docs: update docs for k8s install with new helm chart (#14)
Merged
Khaliq opened 3 years ago

This PR update documentation for configuring and installing OneDev in kubernetes cluster with the new helm chart.

Commits were merged into target branch
  • Robin Shen commented 3 years ago

    Thanks for also taking docs into account. It is very well organized and informative.

    Can we also include the command for typical use cases? For instance in original docs for ingress setup, there is a step by step guide telling which command to execute to get things done. I am not a k8s/helm expert, and guess so for some OneDev users.

  • Khaliq commented 3 years ago

    You are welcome, hopefully it will be useful for the community.

    I merged the previous information to setup ingress controller , cert-manger and lets encrypt. I will change the title to reflect this.

  • Khaliq commented 3 years ago

    Information about upgrading from an older version of chart has also been added.

  • Robin Shen commented 3 years ago

    Tried to go through the procedure of upgrading an old chart. After created mysql service and deployed new chart of OneDev, it fails with:

    java.lang.RuntimeException: java.sql.SQLException: Access denied for user 'root'@'10.108.0.18' (using password: YES)
    	at io.onedev.commons.bootstrap.Bootstrap.unchecked(Bootstrap.java:337)
    	at io.onedev.commons.utils.ExceptionUtils.unchecked(ExceptionUtils.java:35)
    	at io.onedev.server.persistence.PersistenceUtils.openConnection(PersistenceUtils.java:38)
    	at io.onedev.server.persistence.DefaultPersistenceManager.openConnection(DefaultPersistenceManager.java:232)
    	at io.onedev.server.cluster.DefaultClusterManager.start(DefaultClusterManager.java:85)
    	at io.onedev.server.OneDev.start(OneDev.java:133)
    	at io.onedev.commons.loader.DefaultPluginManager.start(DefaultPluginManager.java:44)
    	at io.onedev.commons.loader.AppLoader.start(AppLoader.java:60)
    	at io.onedev.commons.bootstrap.Bootstrap.main(Bootstrap.java:196)
    	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
    	at org.tanukisoftware.wrapper.WrapperSimpleApp.run(WrapperSimpleApp.java:349)
    	at java.base/java.lang.Thread.run(Thread.java:829)
    
    Caused by: java.sql.SQLException: Access denied for user 'root'@'10.108.0.18' (using password: YES)
    	at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:129)
    	at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:97)
    	at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:122)
    	at com.mysql.cj.jdbc.ConnectionImpl.createNewIO(ConnectionImpl.java:832)
    	at com.mysql.cj.jdbc.ConnectionImpl.<init>(ConnectionImpl.java:456)
    	at com.mysql.cj.jdbc.ConnectionImpl.getInstance(ConnectionImpl.java:240)
    	at com.mysql.cj.jdbc.NonRegisteringDriver.connect(NonRegisteringDriver.java:207)
    	at io.onedev.server.persistence.PersistenceUtils.openConnection(PersistenceUtils.java:36)
    

    Apparently we need some tunings for mysql.yaml to allow remote access

  • Khaliq commented 3 years ago

    This error is related to username and password, i am unable to reproduce this. Can you check username/password and database name for mysql and then provide those values.

    Based on the error message looks like root user does not have access to database on 10.108.0.108

  • Robin Shen commented 3 years ago

    Sorry my mistake. I am taking changeme for changeit. Works like a charm now, 👍

  • Robin Shen merged 3 years ago
pull request 1/1
Assignees
Merge Strategy
Create Merge Commit
Watchers (4)
Reference
pull request onedev/docs#14
Please wait...
Connection lost or session expired, reload to recover
Page is in error, reload to recover