#294 Docker support for local development

닫힘
senooken senooken/docker-support 에서 diogo/master 로 3 commits 를 머지하려 합니다
senooken 코멘트됨, 2 년 전

I created Docker support PR on v2 for bug reproducing, standardization/normalization of development environment.

I checked #293/ #254 on this docker image.

In #293 comment, @someonewithpc prepare docker support for v3.

But I think v2 not yet, and it looks like complex. So I created this PR for our development.

To consider someonewithpc's directory structure, I put files to docker/localhost.

I created Docker support PR on v2 for bug reproducing, standardization/normalization of development environment. I checked #293/ #254 on this docker image. In [#293 comment](https://notabug.org/diogo/gnu-social/pulls/293#issuecomment-33609), @someonewithpc prepare docker support for v3. But I think v2 not yet, and it looks like complex. So I created this PR for our development. To consider [someonewithpc's directory structure](https://notabug.org/someonewithpc/gnu-social/src/nightly/docker), I put files to `docker/localhost`.
someonewithpc 코멘트됨, 2 년 전
협업자

I said docker support wasn't merged, but it was ready in https://notabug.org/someonewithpc/gnu-social/src/docker (needs a rebase with the main repo's nightly)

This PR looks good, apart from the small issues below. However, we were mostly using nginx and postgres, while this PR uses Apache and MariaDB, which might allow us to spot any differences.

Issues:

  • There's a typo here

  • The database doesn't persist if the container is stopped. I think it would be helpful to add an explicit volume

  • MariaDB ports don't need to be mapped to outside the container, but it's useful for using external tools to visualize or manage the DB. It should use the standard port for this, though, not 13306

  • The directory docker/localhost should be named docker/development, in my opinion

  • Here ROOT is set but not used, so there's no point in including it in the instructions

  • docker-compose is replaced with docker compose on new systems

  • docker-compose build is redundant, docker-compose up builds if necessary

I said docker support wasn't merged, but it was ready in https://notabug.org/someonewithpc/gnu-social/src/docker (needs a rebase with the main repo's nightly) This PR looks good, apart from the small issues below. However, we were mostly using nginx and postgres, while this PR uses Apache and MariaDB, which might allow us to spot any differences. Issues: - There's a typo [here](https://notabug.org/diogo/gnu-social/pulls/294/files#diff-b1d63a1fcfb303b193447eafcae02391121bb97R10) - The database doesn't persist if the container is stopped. I think it would be helpful to add an explicit volume - MariaDB ports don't need to be mapped to outside the container, but it's useful for using external tools to visualize or manage the DB. It should use the standard port for this, though, not `13306` - The directory `docker/localhost` should be named `docker/development`, in my opinion - [Here](https://notabug.org/diogo/gnu-social/pulls/294/files#diff-b1d63a1fcfb303b193447eafcae02391121bb97R8) `ROOT` is set but not used, so there's no point in including it in the instructions - `docker-compose` is replaced with `docker compose` on new systems - `docker-compose build` is redundant, `docker-compose up` builds if necessary
senooken 코멘트됨, 2 년 전
포스터

@someonewithpc Thanks for review. I fixed review. When you merge, please use squashed merge because intermediate commit is no meaning for master.

However, we were mostly using nginx and postgres, while this PR uses Apache and MariaDB,

I think either is OK. You may fix to nginx/postgres later.

I am familiar with LAMP (Apache/MariaDB) than nginx/postgres. And docker-hub have official php-apache image, so I used apache for simple. There is no php-nginx image. And GNU social supports MariaDB mainly originally (https://notabug.org/diogo/gnu-social/src/master/INSTALL.md).

And I think Apache/MariaDB is more than nginx/postgres on web shared hosting. In Japan, there is no available nginx for all web shared hosting as long as I know.

These are my reason.

@someonewithpc Thanks for review. I fixed review. When you merge, please use squashed merge because intermediate commit is no meaning for master. > However, we were mostly using nginx and postgres, while this PR uses Apache and MariaDB, I think either is OK. You may fix to nginx/postgres later. I am familiar with LAMP (Apache/MariaDB) than nginx/postgres. And docker-hub have official php-apache image, so I used apache for simple. There is no php-nginx image. And GNU social supports MariaDB mainly originally (<https://notabug.org/diogo/gnu-social/src/master/INSTALL.md>). And I think Apache/MariaDB is more than nginx/postgres on web shared hosting. In Japan, there is no available nginx for all web shared hosting as long as I know. These are my reason.
someonewithpc 코멘트됨, 2 년 전
협업자

I don't think the notabug UI allows me to squash commits, so I'll close the PR and merge to nightly (development branch)

BTW, we normally rebase, don't be afraid to rewrite history, especially in PRs

I don't think the notabug UI allows me to squash commits, so I'll close the PR and merge to nightly (development branch) BTW, we normally rebase, don't be afraid to rewrite history, especially in PRs
senooken 코멘트됨, 2 년 전
포스터

OK. In this time, I did not rebase to make the difference clearer for your check. I rebased on #293 after review.

OK. In this time, I did not rebase to make the difference clearer for your check. I rebased on #293 after review.
someonewithpc 코멘트됨, 2 년 전
협업자

Sorry for the delay

Commited in c378019deb

Sorry for the delay Commited in https://notabug.org/diogo/gnu-social/commit/c378019deb06a03f75c55d5edd7042b51e7f24b1
머지 작업을 수행하려면 이 풀리퀘스트를 다시 열어주세요.
로그인하여 이 대화에 참여
마일스톤 없음
담당자 없음
참여자 2명
로딩중...
취소
저장
아직 콘텐츠가 없습니다.