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.

Sign up for GitHub

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

Jump to bottom

feat: poetry for django pipeline step #79

Merged
merged 8 commits into from
Aug 11, 2023
Merged

feat: poetry for django pipeline step #79

merged 8 commits into from
Aug 11, 2023

Conversation

MikhailKorol-saritasa
Copy link
Contributor

@MikhailKorol-saritasa MikhailKorol-saritasa commented Jul 6, 2023 •
edited
Loading

Add poetry option for django requirements step

Task: https://saritasa.atlassian.net/browse/SD-543

Tested on saritasa-ci-experiments-django-backend. Build failed due argocd credentials, everything else is fine.

@MikhailKorol-saritasa MikhailKorol-saritasa self-assigned this Jul 6, 2023
@MikhailKorol-saritasa MikhailKorol-saritasa added enhancement New feature or request Needs Review and removed wip labels Jul 6, 2023
@MikhailKorol-saritasa MikhailKorol-saritasa marked this pull request as ready for review July 6, 2023 13:32
@dmitry-mightydevops
Copy link
Contributor

@MikhailKorol-saritasa I have updated the secret - pls try entire cycle again

✗  for secret in `kgsec -n ci-experiments --no-headers --output=custom-columns=NAME:.metadata.name | grep argocd-secre`; do kubectl -n ci-experiments patch secret $secret -p '{"stringData": {"ARGOCD_PASSWORD": "'$NEW_PASS'"}}'; done
secret/saritasa-ci-experiments-django-backend-dev-argocd-secret patched
secret/saritasa-ci-experiments-dotnet-backend-dev-argocd-secret patched
secret/saritasa-ci-experiments-frontend-dev-argocd-secret patched
secret/saritasa-ci-experiments-kaniko-backend-dev-argocd-secret patched
secret/saritasa-ci-experiments-kaniko-frontend-dev-argocd-secret patched
secret/saritasa-ci-experiments-php-backend-dev-argocd-secret patched

luciano-buono
luciano-buono previously approved these changes Jul 7, 2023
View reviewed changes
@kseniyashaydurova kseniyashaydurova changed the base branch from main to feature/add-post-deploy-task July 24, 2023 08:25
@kseniyashaydurova kseniyashaydurova marked this pull request as ready for review July 24, 2023 08:25
populov
populov previously approved these changes Jul 24, 2023
View reviewed changes
luciano-buono
luciano-buono previously approved these changes Jul 26, 2023
View reviewed changes
udaltsovra
udaltsovra previously approved these changes Jul 27, 2023
View reviewed changes
@kseniyashaydurova kseniyashaydurova changed the base branch from feature/add-post-deploy-task to main August 10, 2023 09:08
@kseniyashaydurova kseniyashaydurova dismissed stale reviews from udaltsovra, luciano-buono, roman-mightydevops, SergeyDeminSaritasa, populov, and themself August 10, 2023 09:08

The base branch was changed.

@kseniyashaydurova kseniyashaydurova merged commit 9f99e46 into main Aug 11, 2023
2 checks passed
@populov populov deleted the feature/tekton-pipelines-poetry branch April 23, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@udaltsovra udaltsovra udaltsovra approved these changes

@awameg awameg awameg approved these changes

@roman-mightydevops roman-mightydevops roman-mightydevops approved these changes

@SergeyDeminSaritasa SergeyDeminSaritasa SergeyDeminSaritasa approved these changes

@populov populov populov approved these changes

@luciano-buono luciano-buono luciano-buono left review comments

@kseniyashaydurova kseniyashaydurova kseniyashaydurova left review comments

@DanilaKazakevich DanilaKazakevich Awaiting requested review from DanilaKazakevich DanilaKazakevich is a code owner

@dmitry-mightydevops dmitry-mightydevops Awaiting requested review from dmitry-mightydevops dmitry-mightydevops is a code owner

Assignees

@MikhailKorol-saritasa MikhailKorol-saritasa

Labels
chart/tekton-pipelines enhancement New feature or request Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@MikhailKorol-saritasa @dmitry-mightydevops @populov @kseniyashaydurova @luciano-buono @SergeyDeminSaritasa @roman-mightydevops @awameg @udaltsovra