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

Add helm chart proto #3102

Merged
merged 20 commits into from Apr 27, 2021
Merged

Add helm chart proto #3102

merged 20 commits into from Apr 27, 2021

Conversation

Keramblock
Copy link
Contributor

@Keramblock Keramblock commented Apr 19, 2021

Motivation and context

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Closes #1087

helm-chart/templates/cvat_backend/deployment.yml Outdated Show resolved Hide resolved
helm-chart/templates/cvat_backend/service.yml Outdated Show resolved Hide resolved
helm-chart/templates/cvat_backend/storage.yml Outdated Show resolved Hide resolved
helm-chart/templates/cvat_frontend/service.yml Outdated Show resolved Hide resolved
helm-chart/templates/cvat_frontend/service.yml Outdated Show resolved Hide resolved
helm-chart/values.yaml Outdated Show resolved Hide resolved
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 74.071% when pulling 9d3c95a on Keramblock:add-helm-chart into d2e4ee4 on openvinotoolkit:develop.

@Keramblock Keramblock marked this pull request as ready for review April 20, 2021 12:01
@Keramblock
Copy link
Contributor Author

Keramblock commented Apr 20, 2021

Found some issues with ingress, will fix tomorrow
UPD: fixed

@Keramblock
Copy link
Contributor Author

@eremeevfd thanks a lot! =)

@azhavoro
Copy link
Contributor

@Keramblock Thanks for the contribution!
@ActiveChooN Could you please take a look?

Copy link
Contributor

@ActiveChooN ActiveChooN left a comment

Choose a reason for hiding this comment

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

Great work! Can you also add a small README file with quick guide how to install cvat with helm (helm dependency update, helm install cvat ., etc.), with advise to change postgres password and how use it with ingress (nginx or traefik)?

helm-chart/Chart.yaml Outdated Show resolved Hide resolved
helm-chart/values.yaml Show resolved Hide resolved
helm-chart/values.yaml Show resolved Hide resolved
helm-chart/README.md Outdated Show resolved Hide resolved
helm-chart/README.md Outdated Show resolved Hide resolved
Keramblock and others added 3 commits April 27, 2021 16:39
Co-authored-by: eremeevfd <dartfedor@mail.ru>
Co-authored-by: eremeevfd <dartfedor@mail.ru>
helm-chart/README.md Outdated Show resolved Hide resolved
helm-chart/README.md Outdated Show resolved Hide resolved
helm-chart/README.md Outdated Show resolved Hide resolved
helm-chart/README.md Outdated Show resolved Hide resolved
Kirpichev Michael and others added 4 commits April 27, 2021 16:43
Co-authored-by: eremeevfd <dartfedor@mail.ru>
Co-authored-by: eremeevfd <dartfedor@mail.ru>
Co-authored-by: eremeevfd <dartfedor@mail.ru>
Co-authored-by: eremeevfd <dartfedor@mail.ru>
@Keramblock
Copy link
Contributor Author

@ActiveChooN @azhavoro could please take a look one more time?

@nmanovic nmanovic mentioned this pull request Apr 27, 2021
8 tasks
Copy link
Contributor

@nmanovic nmanovic left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.

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.

CVAT deployment in Kubernetes
6 participants