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

Backend Architecture Rework - API Gateway #260

Merged
merged 22 commits into from
Jun 10, 2024
Merged

Backend Architecture Rework - API Gateway #260

merged 22 commits into from
Jun 10, 2024

Conversation

Corgam
Copy link
Collaborator

@Corgam Corgam commented Jun 9, 2024 •
edited
Loading

  • Renamed old API Gateway into API Composer
  • Implemented new API Gateway
    • A single endpoint for getting the Viewport dataset (it propagates requests to the API Composer)
  • Added new MongoDB Metadata DB container (currently not used)
  • Updated docker-compose file
  • Updated CI-CD Pipeline for easier extension of new docker images

Corgam added 11 commits June 9, 2024 15:43
@Corgam
Started working on new CI-CD for docker compose
37e8ab2 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed the .hcl file
74bb9f4 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed the .hcl file part 2
0362f0f 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed the .hcl file part 3
bd2fd97 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed the .hcl file part 4
1e4e2fe 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Added mongodb container to docker compose
fec7a9d 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Finished mongodb dockerfile
3e1e7b7 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Updated production deployment
848a1d5 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Added basic new api-gateway
75d1bf9 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Implemented the getViewportData endpoint
bc78670 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Included the new api-gateway inside the docker compose
0728855 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam Corgam added backend Issues related to the data, logic, or communication of the application devops Tracks issues related to system operations, deployment pipelines, and automation of processes labels Jun 9, 2024
@Corgam Corgam marked this pull request as draft June 9, 2024 18:40
@Corgam Corgam changed the title Backend Architecture Rework Backend Architecture Rework - API Gateway Jun 9, 2024
@Corgam Corgam self-assigned this Jun 9, 2024
Corgam added 2 commits June 9, 2024 20:45
@Corgam
Updated github workflows
9e84eff 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed .hcl file
05e4048 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam Corgam marked this pull request as ready for review June 9, 2024 18:50
@Corgam Corgam marked this pull request as draft June 9, 2024 19:55
Corgam added 4 commits June 9, 2024 22:10
@Corgam
Fixed github action
bd4e597 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed github action for production
9305075 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed github action build part 1
044634c 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed github action build part 2
ce6fb3d 
Signed-off-by: Emil Balitzki <[email protected]>
Corgam added 3 commits June 9, 2024 22:41
@Corgam
Quick text fix
bc34652 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Disabled temp. the https redirect
63ea85a 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Changed swagger for api composer
ec3d806 
Signed-off-by: Emil Balitzki <[email protected]>
@Lucas-Nan Lucas-Nan marked this pull request as ready for review June 10, 2024 07:42
Lucas-Nan
Lucas-Nan approved these changes Jun 10, 2024
View reviewed changes
Copy link
Contributor

@Lucas-Nan Lucas-Nan left a comment

Choose a reason for hiding this comment

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

working extra containers (requests are still transferred), small issue encountered with insuffifient waiting time in entrypoint.sh (20s fail vs 50s success) - up for discussion

@Lucas-Nan Lucas-Nan merged commit fffaed1 into sprint-release Jun 10, 2024
17 checks passed
@Corgam
Copy link
Collaborator Author

Corgam commented Jun 10, 2024

working extra containers (requests are still transferred), small issue encountered with insuffifient waiting time in entrypoint.sh (20s fail vs 50s success) - up for discussion

Discussed in Discord, will work on it in the next PRs.

@Corgam Corgam mentioned this pull request Jun 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lucas-Nan Lucas-Nan Lucas-Nan approved these changes

@Ahsankkhan Ahsankkhan Awaiting requested review from Ahsankkhan

@Superschnizel Superschnizel Awaiting requested review from Superschnizel

@nicolasbandel nicolasbandel Awaiting requested review from nicolasbandel

Assignees

@Corgam Corgam

Labels
backend Issues related to the data, logic, or communication of the application devops Tracks issues related to system operations, deployment pipelines, and automation of processes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Corgam @Lucas-Nan