-
Notifications
You must be signed in to change notification settings - Fork 1
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
base: master
Are you sure you want to change the base?
Conversation
app: default backend - 404
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Советую поменять структуру репозитория:
- исходники приложения вытащить из директории swarm (например в sources),
- директорию minikube лучше назвать чем-то вроде manifests (ничто не мешает нам запускать в minikube те же helm-чарты)
Само дз принимаю, но прошу прочитать мои комментарии и поправить по ним, что следует.
externalPort: 9292 | ||
|
||
image: | ||
repository: chromko/ui |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поправь, пожалйста, образ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ок
kubernetes/gke/tls.crt
Outdated
@@ -0,0 +1,17 @@ | |||
-----BEGIN CERTIFICATE----- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Удалить из репы. Хоть в .gitignore запись и есть, но файл как-то попал в индекс
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ок
kubernetes/gke/tls.key
Outdated
@@ -0,0 +1,28 @@ | |||
-----BEGIN PRIVATE KEY----- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Удалить из репы. Хоть в .gitignore запись и есть, но файл как-то попал в индекс
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ок
Выполнение ДЗ имеет описание для повторения в будущем (это шпаргалка для себя в будущем), но оно невозможно из-за постоянных изменений в последующих ДЗ. Именно из-за этого я решил создать директории для docker compose/swarm. Имена директорий по номеру ДЗ возможно тоже было бы нормальным решением если делать изначально. |
Ну, недостатки текущей структуры я указал выше - сорцы из swarm вытащить. |
For hw 31