Simulation-2d docker Image #7

Pvcunha · 2023-01-12T00:30:36Z

Adds the simulation-2d dockerfile

fnalmeidap · 2023-01-12T01:50:01Z

containers/simulation-2d.Dockerfile

+FROM robocin/cpp-ubuntu-20.04:latest
+
+# packages setup
+RUN apt update


Já é resolvido em robocin/cpp-ubuntu-20.04:latest. Não seria redundante?

E além disso, apenas o upstream das fontes é atualizado com apt update, não há mudança no sistema.

Sinceramente, eu só copiei o Dockerfile.exodia do simulation-2d-cpp e queria ver a action em ação. Tem que mudar algumas coisas até pra começar a usar o robocin-install

Pode remover sim [link]

fnalmeidap · 2023-01-12T01:55:34Z

containers/simulation-2d.Dockerfile

+  cd rcssmonitor && ./bootstrap && ./configure && \
+  make && make install && \
+  cd /root && rm -rf rcssmonitor
+


Podemos ter aqui também o Fedit e o Soccerwindow2.

fnalmeidap · 2023-01-12T01:56:52Z

containers/simulation-2d.Dockerfile

+
+# Set environment variables.
+ENV HOME /root
+ENV DISPLAY :0


Variável mágica? Podes explicar o uso dela?

Kkkkkk mágica demais, é pra definir o display padrão pro localhost

Gui apps em docker https://link.medium.com/MDgtowncwwb

O que é a variavel de ambiente display https://askubuntu.com/questions/432255/what-is-the-display-environment-variable

Basicamente quando nós rodamos um container docker temos que definir a flag -e DISPLAY=$DISPLAY pra usar a env do PC host, aí eu tô definindo para a padrao ser :0 que é a normalmente utilizada só pra evitar a flag em momentos futuros

Acho show, mas essa imagem ainda é meio que uma base, poderíamos puxar pro primeiro pai, ou colocar nas imagens filhas / no devcontainer.json

fnalmeidap · 2023-01-12T01:58:40Z

containers/simulation-2d.Dockerfile

+# rcssserver install
+WORKDIR /root
+RUN git clone https://github.com/rcsoccersim/rcssserver.git
+RUN \


Podemos incluir a primeira chamada do rcssserver para ele criar as configurações do server no arquivo/pasta oculta autogerada por ele.

fnalmeidap · 2023-01-12T02:05:13Z

containers/simulation-2d.Dockerfile

@@ -0,0 +1,49 @@
+# Disclaimer:


O dockerfile não utiliza módulos já baixados, não entendi o disclaimer. Podes clarificar?

O disclaimer é pq eu literalmente copiei, te amo não me bate

fnalmeidap · 2023-01-12T02:08:55Z

containers/simulation-2d.Dockerfile

+
+RUN apt install -yq x11-apps
+
+# Define default command.


Os comandos do docker são autoexplicativos. Usar comentários apenas em situações que a informação contida no código não seja suficiente para o seu entendimento.

joseviccruz

Sinceramente, eu só copiei o Dockerfile.exodia do simulation-2d-cpp e queria ver a action em ação. Tem que mudar algumas coisas até pra começar a usar o robocin-install

Opa, eu vi isso, mas comentei coisas pra deixar normalizado a imagem, no final ela poderia ficar assim:

FROM robocin/cpp-ubuntu-20.04:latest

ARG QT=5.15.2

RUN robocin-install \
    boost \
    'qt ${QT}' \
    rcssserver \
    rcssmonitor # \
    # fedit \
    # soccerwindow2

ENV PATH /opt/qt/${QT}/gcc_64:$PATH

Sobre colocar as deps em /opt no lugar de /root, é pq é exatamente o diretório que cumpre com esse próposito [link]

joseviccruz · 2023-01-12T05:02:54Z

containers/simulation-2d.Dockerfile

+# Define default command.
+CMD ["bash"]


CMD ["bash"] já é setado previamente, pode remover

joseviccruz · 2023-01-12T05:05:10Z

containers/simulation-2d.Dockerfile

+
+# Setup tzdata
+RUN \ 
+  apt-get install -yq tzdata && \


tzdata parece ser instalado previamente, pode remover

joseviccruz · 2023-01-12T05:06:50Z

containers/simulation-2d.Dockerfile

+FROM robocin/cpp-ubuntu-20.04:latest
+
+# packages setup
+RUN apt update


Pode remover sim [link]

joseviccruz · 2023-01-12T05:08:50Z

containers/simulation-2d.Dockerfile

+  ln -fs /usr/share/zoneinfo/America/New_York /etc/localtime && \
+  dpkg-reconfigure -f noninteractive tzdata


Essas configs são realmente necessárias?

Então, quando eu tava fazendo os updates pro Ubuntu 20.04 ele travava numa parte que ficava pedindo pra definir a timezone, e a solução que eu achei foi esse troço aí kkkkkk

https://dev.to/0xbf/set-timezone-in-your-docker-image-d22

Qual o sw que exigiu? Eu já me deparei com esse problema mesmo, lembrei agr, mas era quando a gente puxava da imagem do ubuntu padrão
Como nossa imagem puxa de uma imagem de devcontainer, acho que isso é resolvido por baixo dos panos... Tenta seguir sem isso 🤔

Hmmm não lembro ao certo, era algum desses pacotes que vem logo em sequencia. Creio que tenha sido o build-essential.

Pronto massa, vou tentar sem

joseviccruz · 2023-01-12T05:22:39Z

containers/simulation-2d.Dockerfile

+RUN apt install -yq \
+  git build-essential automake autoconf libtool flex bison libboost-all-dev \
+  qt5-default libfontconfig1-dev libaudio-dev libxt-dev libglib2.0-dev libxi-dev libxrender-dev


Acho que tudo aqui já foi instalado, exceto o qt5-default. Sugestão de substituição:

ARG QT=5.15.2 RUN robocin-install 'qt ${QT}' ENV PATH /opt/qt/${QT}/gcc_64:$PATH

Baseado no link.

joseviccruz · 2023-01-12T06:02:15Z

containers/simulation-2d.Dockerfile

+RUN git clone https://github.com/rcsoccersim/rcssserver.git
+RUN \
+  cd rcssserver && ./bootstrap && ./configure && \
+  make && make install && \
+  cd /root && rm -rf rcssserver


Adicionar rcssserver.sh no repo de scripts

exemplo: googletest.sh

joseviccruz · 2023-01-12T06:02:44Z

containers/simulation-2d.Dockerfile

+WORKDIR /root
+RUN git clone https://github.com/rcsoccersim/rcssmonitor.git
+RUN \
+  cd rcssmonitor && ./bootstrap && ./configure && \
+  make && make install && \
+  cd /root && rm -rf rcssmonitor
+


Adicionar rcssmonitor.sh no repo de scripts

exemplo: googletest.sh

joseviccruz · 2023-01-12T14:56:52Z

containers/simulation-2d.Dockerfile

+  cd /root && rm -rf rcssserver
+
+# rcssmonitor install
+WORKDIR /root


Poderia ser na pasta /opt ao invés de /root?

joseviccruz · 2023-01-12T14:56:59Z

containers/simulation-2d.Dockerfile

+ENV DISPLAY :0
+
+# Define working directory.
+WORKDIR /root


Poderia ser na pasta /opt ao invés de /root?

joseviccruz · 2023-01-12T14:58:28Z

containers/simulation-2d.Dockerfile

+
+# Set environment variables.
+ENV HOME /root
+ENV DISPLAY :0


Acho show, mas essa imagem ainda é meio que uma base, poderíamos puxar pro primeiro pai, ou colocar nas imagens filhas / no devcontainer.json

joseviccruz

Imagens anteriores a essa, pra ver o que já foi instalado:

ersaraujo · 2023-12-07T19:26:30Z

@Pvcunha qual o status?

Pvcunha · 2023-12-07T19:38:18Z

@Pvcunha qual o status?

@ersaraujo
Esse PR foi descontinuado. Ele pode ser encontrado e o devcontainer ja foi resolvido por essa imagem https://github.com/robocin/simulation-2d-cpp/tree/main/.devcontainer

Alem disso, continua sendo adicionadas outras necessidades que surgiram por binho como pode ser visto nesse PR atual https://github.com/robocin/simulation-2d-cpp/pull/283

feat(simulation-2d.Dockerfile): base image for 2d

90120b2

Pvcunha requested review from aalmds, fnalmeidap, joseviccruz and cstiano January 12, 2023 00:31

fnalmeidap requested changes Jan 12, 2023

View reviewed changes

joseviccruz reviewed Jan 12, 2023

View reviewed changes

Merge branch 'main' into feat/docker_2d

9b3dfe3

Pvcunha closed this Dec 7, 2023

Pvcunha deleted the feat/docker_2d branch December 7, 2023 19:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Simulation-2d docker Image #7

Simulation-2d docker Image #7

Pvcunha commented Jan 12, 2023

fnalmeidap Jan 12, 2023 •

edited

Loading

Pvcunha Jan 12, 2023

joseviccruz Jan 12, 2023

fnalmeidap Jan 12, 2023

Pvcunha Jan 12, 2023

fnalmeidap Jan 12, 2023

Pvcunha Jan 12, 2023

joseviccruz Jan 12, 2023

fnalmeidap Jan 12, 2023

fnalmeidap Jan 12, 2023

Pvcunha Jan 12, 2023

fnalmeidap Jan 12, 2023

joseviccruz left a comment •

edited

Loading

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

Pvcunha Jan 12, 2023

joseviccruz Jan 12, 2023

Pvcunha Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz Jan 12, 2023

joseviccruz left a comment

ersaraujo commented Dec 7, 2023

Pvcunha commented Dec 7, 2023

		ln -fs /usr/share/zoneinfo/America/New_York /etc/localtime && \
		dpkg-reconfigure -f noninteractive tzdata

Simulation-2d docker Image #7

Simulation-2d docker Image #7

Conversation

Pvcunha commented Jan 12, 2023

fnalmeidap Jan 12, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

joseviccruz left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

joseviccruz left a comment

Choose a reason for hiding this comment

ersaraujo commented Dec 7, 2023

Pvcunha commented Dec 7, 2023

fnalmeidap Jan 12, 2023 •

edited

Loading

joseviccruz left a comment •

edited

Loading