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

Feature/kubernetes 4 #10

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Feature/kubernetes 4 #10

wants to merge 40 commits into from

Conversation

DmitriySh
Copy link
Owner

For hw 31

Copy link
Collaborator

@chromko chromko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Советую поменять структуру репозитория:

  1. исходники приложения вытащить из директории swarm (например в sources),
  2. директорию minikube лучше назвать чем-то вроде manifests (ничто не мешает нам запускать в minikube те же helm-чарты)

Само дз принимаю, но прошу прочитать мои комментарии и поправить по ним, что следует.

externalPort: 9292

image:
repository: chromko/ui
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поправь, пожалйста, образ

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ок

@@ -0,0 +1,17 @@
-----BEGIN CERTIFICATE-----
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удалить из репы. Хоть в .gitignore запись и есть, но файл как-то попал в индекс

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ок

@@ -0,0 +1,28 @@
-----BEGIN PRIVATE KEY-----
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удалить из репы. Хоть в .gitignore запись и есть, но файл как-то попал в индекс

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ок

@DmitriySh
Copy link
Owner Author

Выполнение ДЗ имеет описание для повторения в будущем (это шпаргалка для себя в будущем), но оно невозможно из-за постоянных изменений в последующих ДЗ. Именно из-за этого я решил создать директории для docker compose/swarm. Имена директорий по номеру ДЗ возможно тоже было бы нормальным решением если делать изначально.

@chromko
Copy link
Collaborator

chromko commented Jan 16, 2018

Ну, недостатки текущей структуры я указал выше - сорцы из swarm вытащить.
Minikube в Manifests это просто мое имхо, поэтому как удобнее.
Мы не стали делать структуру по номерам ДЗ, чтобы репозиторий был более-менее похож на реальный результат работы. Результаты работы многих дз связаны с предыдущими и располагать их в такой структуре было бы просто лишним гемором.
В качестве памяток самим студентам о проделанной работе в конкретный момент должны служить PR, их описания и README-файлы.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants