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

Docker Develop Environment #65

Open
indiePeeters opened this issue Feb 14, 2019 · 11 comments
Open

Docker Develop Environment #65

indiePeeters opened this issue Feb 14, 2019 · 11 comments
Assignees
Labels
enhancement New feature or request low priority Low priority

Comments

@indiePeeters
Copy link
Contributor

voor het toegangelijker maken van de IC, moet de huidige website opgezet worden in docker. zodat na een commande kan begonnen worden met programeeren

@indiePeeters indiePeeters self-assigned this Feb 14, 2019
@wouterbles wouterbles added low priority Low priority enhancement New feature or request labels Feb 14, 2019
@indiePeeters indiePeeters changed the title Docker Develop Enviroment Docker Develop Environment Feb 25, 2019
@otherwise777
Copy link
Contributor

Is het idee om een docker image te maken voor de test omgeving? Of om ook de productie omgeving via een Docker te runnen?
Wat is je voortgang hiermee? Ik werk elke dag met Docker en container orchestraties voor mijn werk en kan wellicht makkelijk hiermee helpen

@indiePeeters
Copy link
Contributor Author

indiePeeters commented Mar 8, 2019

Voor nu weet ik niet echt of een testomgeving die bereikbaar vanaf je browser is een toegevoegde waarde heeft. Wat vindt de rest hiervan?

Momenteel staat er in de docker branch een docker folder waarbij met docker-compose de ontwikkel omgeving opgezet wordt.

Vrijwel alles werkt op de agenda na. Iets met een vue crossorigin error. Aangezien ik niet echt veel ervaring met Vue heb en het een instellingsfout lijkt te zijn ( had het zelfde probleem ook voordat dit met docker kon) kom ik hier niet erg snel uit.

Dus als een van jullie hier naar zou kunnen kijken zou dat top zijn!

@otherwise777
Copy link
Contributor

ik kijk hier wel naar, wellicht dat ik een nieuwe branch met een ontwikkel omgeving

@otherwise777
Copy link
Contributor

Zo dan, veel werk gedaan, grootste deel met details zit in de commit messages, kleine samenvatting:
-Containers opgeruimd om alles wat sneller/overzichtelijker te maken
-CORS ingeschakeld anders laad de agenda/zekeringen niet (dit is een aanpassing in de index.php file)
-aparte environment file toegevoegd voor docker
-alles in de root folder gegooid want laraval kan niet met environments overweg die niet in de root zitten
-readme update gegeven

Nog 2 TODO's
-zorgen dat de Database aangemaakt word on start, op dit moment gaat dit mis met het aanmaken van een nieuwe DB container
-npm install toevoegen in de app, deze werkt nu niet gebouwd

@otherwise777
Copy link
Contributor

kleine extra todo: composer erop gooien,
Ik kwam helaas weer wat bugs tegen met Docker caching, weer wat geleerd :)
Mijn idee is nu, de app container krijgt installatie van Composer, NodeJS en PHP, en de entrypoint word een shell script wat composer update uitvoerd, daarna npm install, de database leegt/vult en vervolgens de web server served.

Ik ben niet helemaal up to date met hoe de omgeving in productie draait, anders is het wellicht een idee om dat na te bootsen

@wouterbles
Copy link
Contributor

Ik ben niet helemaal up to date met hoe de omgeving in productie draait, anders is het wellicht een idee om dat na te bootsen

Weet niet wat ik daar over kan zeggen, het is gewoon een Ubuntu installatie waarop nginx draait die de website served.

@indiePeeters indiePeeters removed their assignment Jun 26, 2019
@indiePeeters
Copy link
Contributor Author

Npm dependencies moeten nog geladen worden, dan kan er een pull request gemaakt worden

@otherwise777
Copy link
Contributor

Ik denk dat hij zo goed als klaar is, een paar kleingiheidjes:

  • server start op op ip 0.0.0.0:8000, dit heeft niets te maken met de npm instellingen maar met docker compose die de container naar je pc exposed, dit gebeurd op 0.0.0.0, dit moet wel ergens te veranderen zijn, ik hoop eigenlijk op een domein zoals esac.test:80 oid maar ik weet niet of het dan nog overal werkt
  • bestanden staan nu in /docker en in de root, het liefst heb ik alles in /docker zodat we de root kunnen gebruiken voor Docker in productie

@otherwise777
Copy link
Contributor

ok, 1 ding gefxied, hij served hem nu op 127.0.0.1:80
nu nog het domein: https://docs.docker.com/v17.12/datacenter/ucp/2.1/guides/user/services/use-domain-names-to-access-services/#from-the-cli

@otherwise777
Copy link
Contributor

ik ga dit helemaal opnieuw doen, met als uitgangspunt hoe we docker in productie gebruiken. ik zet wel een aparte docker-compose in de repo die dan gebruikt kan worden voor local development, in eerste instantie wou ik de docker prod versie gebruiken maar dit betekend dat iedereen die wil ontwikkelen ook de andere repo moet pullen en instellen en dat lijkt me teveel werk

@otherwise777
Copy link
Contributor

Om dit voor elkaar te krijgen moeten de server_config en esac repo samengevoegd worden, dit zal ik uitvoeren en daarna dit afmaken

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low priority Low priority
Projects
None yet
Development

No branches or pull requests

3 participants