docs: update docs for k8s install with new helm chart (#14)
Merged
Commits were merged into target branch
Khaliq opened 11 months ago

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

New commits added 11 months ago
Khaliq commented 11 months ago
New commits added 11 months ago
New commits added 11 months ago
New commits added 11 months ago
Robin Shen commented 11 months 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 11 months 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.

New commits added 11 months ago
Khaliq commented 11 months ago

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

Robin Shen commented 11 months 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 11 months 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 11 months ago

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

Robin Shen merged 11 months ago
Referenced from commit 11 months ago
Merges pull request #14
docs: update docs for k8s install with new helm chart
Robin Shen committed 11 months ago
pull request 1 of 1
Assignees
Merge Strategy
Create Merge Commit
Watchers (4)
Reference
pull request onedev/docs#14
Please wait...
Page is in error, reload to recover