You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by "warren830 (via GitHub)" <gi...@apache.org> on 2023/03/09 03:39:48 UTC

[GitHub] [incubator-devlake-website] warren830 opened a new pull request, #463: docs: update docker compose setup according to review

warren830 opened a new pull request, #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463

   # Summary
   
   Update how to configure grafana if user wants to use external db
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] hezyin commented on a diff in pull request #463: docs: update docker compose setup according to review

Posted by "hezyin (via GitHub)" <gi...@apache.org>.
hezyin commented on code in PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463#discussion_r1130547058


##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??

Review Comment:
   There's an extra "?"



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??
 
-    Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
+Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:

Review Comment:
   => "Yes, please follow the steps below:" (this is to simplify this paragraph, otherwise you also need to mention the changes in the 'grafana' section)



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -80,12 +89,11 @@ DB_URL="mysql://YOUR_USER:YOUR_PASSWORD@YOUR_IP:YOUR_PORT/lake?charset=utf8mb4&p
 #      collation-server=utf8mb4_bin
 ```
 
-
-5. Final step: `docker compose up -d`
+6. Final step: `docker compose up -d`
 
 ### Can I use an external Grafana instead of running Grafana in Docker?

Review Comment:
   I think it's missing two steps:
   
   1. How do users connect their own Grafana to our database?
   2. How do they import JSON dashboards to their Grafana instance?



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -57,21 +57,30 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
       --collation-server=utf8mb4_bin
 ```
 
-2. Comment out the 'mysql' volume:
+2. Comment out the 'mysql' volume in `docker-compose.yml`:
 
 ```yaml
 volumes:
    mysql-storage:
 ```
 
-3. Comment out the 'depends_on mysql' part:
+3. Comment out the 'depends_on mysql' part in `docker-compose.yml`:
 
 ```yaml
     depends_on:
       - mysql
 ```
 
-4. Set DB_URL to your own DB_URL in .env
+4. Edit `MYSQL_*` inside 'grafana.environment' part in `docker-compose.yml`:
+
+```yaml
+      MYSQL_URL: DB_URL:DB_PORT

Review Comment:
   @warren830 For each of these env vars, please let users know what they need to provide. Feel free to use https://devlake.apache.org/docs/GettingStarted/HelmSetup#faq as a reference.



##########
versioned_docs/version-v0.15/GettingStarted/DockerComposeSetup.md:
##########
@@ -36,11 +36,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?

Review Comment:
   Please replicate all changes to this file as well.



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??
 
-    Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
+Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
 
-1. Comment out mysql part:
+1. Comment out 'mysql' part in `docker-compose.yml`:

Review Comment:
   "Comment out 'mysql' part..." => "Comment out the 'mysql' section..."



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -57,21 +57,30 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
       --collation-server=utf8mb4_bin
 ```
 
-2. Comment out the 'mysql' volume:
+2. Comment out the 'mysql' volume in `docker-compose.yml`:
 
 ```yaml
 volumes:
    mysql-storage:
 ```
 
-3. Comment out the 'depends_on mysql' part:
+3. Comment out the 'depends_on mysql' part in `docker-compose.yml`:
 
 ```yaml
     depends_on:
       - mysql
 ```
 
-4. Set DB_URL to your own DB_URL in .env
+4. Edit `MYSQL_*` inside 'grafana.environment' part in `docker-compose.yml`:

Review Comment:
   => "4. Provide your managed cloud database connection info in the 'grafana' section in `docker-compose.yml`"



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] hezyin merged pull request #463: docs: update docker compose setup according to review

Posted by "hezyin (via GitHub)" <gi...@apache.org>.
hezyin merged PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] warren830 commented on a diff in pull request #463: docs: update docker compose setup according to review

Posted by "warren830 (via GitHub)" <gi...@apache.org>.
warren830 commented on code in PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463#discussion_r1130679225


##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??
 
-    Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
+Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
 
-1. Comment out mysql part:
+1. Comment out 'mysql' part in `docker-compose.yml`:

Review Comment:
   fixed



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] warren830 commented on a diff in pull request #463: docs: update docker compose setup according to review

Posted by "warren830 (via GitHub)" <gi...@apache.org>.
warren830 commented on code in PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463#discussion_r1130678335


##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??

Review Comment:
   fixed



##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -34,11 +34,11 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
 <br/>
 
 ## FAQ
-### Can I use an external Database service instead of running Database in Docker?
+### Can I use a managed Cloud database service instead of running database in Docker??
 
-    Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:
+Yes, you can simply comment out the 'mysql' part in 'docker-compose.yml' and update some configurations in '.env' before you run docker compose up -d, here are the steps:

Review Comment:
   fixed



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] warren830 commented on a diff in pull request #463: docs: update docker compose setup according to review

Posted by "warren830 (via GitHub)" <gi...@apache.org>.
warren830 commented on code in PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463#discussion_r1130679923


##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -57,21 +57,30 @@ Support for database schema migration was introduced to DevLake in v0.10.0. From
       --collation-server=utf8mb4_bin
 ```
 
-2. Comment out the 'mysql' volume:
+2. Comment out the 'mysql' volume in `docker-compose.yml`:
 
 ```yaml
 volumes:
    mysql-storage:
 ```
 
-3. Comment out the 'depends_on mysql' part:
+3. Comment out the 'depends_on mysql' part in `docker-compose.yml`:
 
 ```yaml
     depends_on:
       - mysql
 ```
 
-4. Set DB_URL to your own DB_URL in .env
+4. Edit `MYSQL_*` inside 'grafana.environment' part in `docker-compose.yml`:

Review Comment:
   fixed



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] hezyin commented on a diff in pull request #463: docs: update docker compose setup according to review

Posted by "hezyin (via GitHub)" <gi...@apache.org>.
hezyin commented on code in PR #463:
URL: https://github.com/apache/incubator-devlake-website/pull/463#discussion_r1131878199


##########
docs/GettingStarted/DockerComposeSetup.md:
##########
@@ -114,12 +128,18 @@ DB_URL="mysql://YOUR_USER:YOUR_PASSWORD@YOUR_IP:YOUR_PORT/lake?charset=utf8mb4&p
 
 ```yaml
 volumes:
-   grafana-storage:
+  grafana-storage:
 ```
 
 3. Set config-ui.environment.GRAFANA_ENDPOINT to your own grafana url in docker-compose.yml
 
-4. Final step: `docker compose up -d`
+4. Please set up a database for Grafana according to https://grafana.com/docs/grafana/latest/administration/data-source-management/

Review Comment:
   => "Please connect your Grafana to DevLake's database by following..."



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org