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

Deploy to production 2024-08-22 / 1 #54

Merged
merged 8 commits into from
Aug 23, 2024
Merged

Deploy to production 2024-08-22 / 1 #54

merged 8 commits into from
Aug 23, 2024

Conversation

bhvishal9
Copy link
Collaborator

@bhvishal9 bhvishal9 commented Aug 22, 2024 •
edited
Loading

Description:

The PR is code promotion from staging to production.

This PR includes:

  • Add main changes here

How to run:

  • Add instructions on how to run here

Demo:

  • Add here screenshots, or a loom, showcasing the changes

Checklist (based on PR guidelines):

  • I have self-reviewed this PR, according to these points
  • This PR falls into maximum three of these categories
  • I feel comfortable taking responsibility for merging this code to production
  • The code is high quality, well-tested, follows styles guides (Frontend), clean, and well documented

pietrobolcato and others added 8 commits September 21, 2023 09:50
@pietrobolcato
Add CI to vectorizing CU-86bvyavpu (#46)
93e4675 
* Add CI pipeline to run tests with gha
* Add system dependencies installer script shared between dev container and CI
* Update README documentation
* Remove 'python-dev0 as system dependency
@pietrobolcato
Upgrade vectorizing to python 3.11 CU-866aw6ung (#42)
cc237b5 
* Update dockerfile
* Update requirements
* Update environments
@pietrobolcato
Add linting and formatting CU-86bw0yqjw (#47)
cfbfc22 
* Linting and formatting setup using pre-commit
* Updated devcontainer with pre-commit
@pietrobolcato
Fix k8s deployment CU-86bwkb51r
80d4bd4
@pietrobolcato
Add PR template CU-86bx2ntyx (#50)
d04e388 
* Add PR template to the repository, with information aligned with guidelines in notion
@pietrobolcato
Remove PR template to use org one (#51)
5abd7f6 
* Remove PR template to use org one
@pietrobolcato
Fix UA (#52)
0b2b95c 
* Update UA
@bhvishal9
CU-86bze09vt Move vectorizing to new clusters (#53)
e74b781 
* Update Dockerfile and remove AWS AK/SAK

* Update deployment workflow

* Minor modification to the deploy workflow

* Add region_name to the client

* Upgrade boto3

* Update port

* Undo temp changes
pietrobolcato
pietrobolcato approved these changes Aug 22, 2024
View reviewed changes
Copy link
Collaborator

@pietrobolcato pietrobolcato left a comment

Choose a reason for hiding this comment

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

lgtm 🚀

super nit: usually the naming for production deployment pr follow the format: Deploy to production yyyy-mm-dd / k, eg: Deploy to production 2024-08-22 / 1 :)

@bhvishal9 bhvishal9 changed the title Staging to Production Deploy to production 2024-08-22 / 1 Aug 22, 2024
@bhvishal9 bhvishal9 merged commit 419e920 into production Aug 23, 2024
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pietrobolcato pietrobolcato pietrobolcato approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bhvishal9 @pietrobolcato