Commit Graph

253 Commits

Author SHA1 Message Date
6e43296466 Merge pull request 'feature/icecheng/role_management1' (#17) from feature/icecheng/role_management into dev
Reviewed-on: freeleaps/freeleaps-service-hub#17
Reviewed-by: jingyao1991 <jingyao1991@noreply.gitea.freeleaps.mathmast.com>
2025-08-12 04:18:04 +00:00
77a1005871 Merge pull request 'modify PR template' (#28) from modify_template into dev
Reviewed-on: freeleaps/freeleaps-service-hub#28
2025-08-12 04:09:56 +00:00
dongli
fcee95d609 modify PR template 2025-08-11 21:09:25 -07:00
6132f98283 Merge branch 'dev' into feature/icecheng/role_management 2025-08-12 03:24:03 +00:00
icecheng
e2f51d09da feat(role): update docs 2025-08-12 11:17:52 +08:00
e16c1dc240 Merge pull request 'Add dummy change to trigger ci/cd' (#25) from dax/minor into dev
Reviewed-on: freeleaps/freeleaps-service-hub#25
2025-08-11 01:22:38 +00:00
dongli
8caa8e6b31 Add dummy change to trigger ci/cd 2025-08-10 18:21:39 -07:00
6e1cb5962f Merge pull request 'fix: make startup and shutdown event handlers async in common provider' (#24) from Nicolas_devops_reconciler into dev
Reviewed-on: freeleaps/freeleaps-service-hub#24
2025-08-08 05:02:21 +00:00
Nicolas
04acd78d78 fix: connect to existing named queue instead of creating anonymous queue
- Change AsyncMQClient to connect to existing persistent queue by name
- Fix issue where DevOps Service created temporary anonymous queues instead of consuming from the correct named queue
- This allows consuming the 42 backlogged messages in freeleaps.devops.reconciler.output queue
- Change queue properties: exclusive=False, auto_delete=False, durable=True
- Resolves the core issue where messages were split between persistent and temporary queues
2025-08-08 12:45:21 +08:00
Nicolas
4340949f57 fix: make startup and shutdown event handlers async in common provider
- Fix TypeError: object NoneType can't be used in 'await' expression
- FastAPI requires event handlers to be async functions
- This was blocking the entire application startup sequence
- Fixes the issue where message queue consumers were not starting properly
2025-08-08 12:33:14 +08:00
fc4c8e7b14 Merge pull request 'fix: use environment variable for RabbitMQ output queue name' (#23) from Nicolas_devops_reconciler into dev
Reviewed-on: freeleaps/freeleaps-service-hub#23
2025-08-08 04:06:43 +00:00
Nicolas
e4fe9394b1 fix: use environment variable for RabbitMQ output queue name
- Add RABBITMQ_OUTPUT_QUEUE_NAME environment variable support
- Fix hardcoded queue name 'reconciler.output' to use configurable queue name
- Default to 'freeleaps.devops.reconciler.output' if env var not set
- Add debug logging to show which queue name is being used
- This fixes the issue where 42 messages were stuck in the output queue due to queue name mismatch
2025-08-08 12:01:47 +08:00
52ec8f8f03 Merge pull request 'fix: update devops service to subscribe to correct reconciler heartbeat queue' (#21) from Nicolas_devops_reconciler into dev
Reviewed-on: freeleaps/freeleaps-service-hub#21
2025-08-08 03:32:51 +00:00
Nicolas
b5ca0cf0f5 fix: update devops service to subscribe to correct reconciler heartbeat queue 2025-08-07 18:29:59 +08:00
aa89bceac0 Merge pull request 'feat: Fix check_deployment_status' (#19) from dax/fix_check_deployment_status into dev
Reviewed-on: freeleaps/freeleaps-service-hub#19
2025-08-05 05:10:18 +00:00
dongli
d680b9e015 Fix check_deployment_status 2025-08-03 21:39:39 -07:00
zhenyus
879e45eeb5 fix: correct deployment TTL calculation in DeploymentService
Updated the TTL calculation in the DeploymentService to multiply by 3600 instead of 60, ensuring accurate time-to-live settings for deployments.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 12:22:24 +08:00
zhenyus
6c60a0b6a5 fix: update deployment service to use correct product ID
Changed the parameter name from deployment_project_id to deployment_product_id in the DevOpsReconcileRequest to ensure accurate referencing of the product associated with the deployment.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 04:04:19 +08:00
zhenyus
5f3368cbdc fix: correct typo in BASE_RECONCILE_URL parameter in DeploymentService
Updated the parameter name in the _start_deployment method to match the corrected constant in site_settings.py, ensuring consistency across the codebase.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 03:51:15 +08:00
zhenyus
e41aa3d983 fix: correct typo in BASE_RECONCILE_URL in site_settings.py
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 03:41:26 +08:00
zhenyus
6ebac252b0 refactor: change DevOpsReconcileOperationType to inherit from str and Enum
Updated the DevOpsReconcileOperationType class to inherit from both str and Enum, enhancing its usability as a string enumeration in the application.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 03:28:06 +08:00
zhenyus
2dd73c0734 refactor: enhance lazy initialization and error handling in message consumer startup
Updated the message consumer initialization to occur lazily during the startup event, improving application startup performance. Added detailed logging for service initialization, registration, and error handling to enhance traceability and robustness during startup and shutdown processes.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 03:01:11 +08:00
zhenyus
95b6560ffd refactor: improve clarity of subscriber startup process in message_queue.py
Updated comment to specify that the subscriber starts in the background, enhancing code readability and understanding of the asynchronous behavior.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 02:48:55 +08:00
zhenyus
9283e52e48 feat: enhance message consumer startup and shutdown logging in message_queue.py
Added print statements to log the status of message consumers during startup and shutdown processes, improving traceability and debugging capabilities.

Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-03 02:38:33 +08:00
zhenyus
113bc2bc75 fix: ensure consistent formatting in requirements.txt by removing redundant line
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-01 09:12:19 +08:00
zhenyus
ebc37bbd0d refactor: streamline DevOpsReconcileRequest by removing redundant code
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-01 09:06:42 +08:00
zhenyus
14eb5abda6 refactor: remove unnecessary @dataclass decorator from DevOpsReconcileRequest
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-01 09:03:39 +08:00
zhenyus
d74967db55 feat: add RabbitMQ configuration to app settings
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-08-01 00:15:06 +08:00
6ea40142ac Merge pull request 'Add template fastapi app, this can be used for future extension' (#11) from feature/dax/devops-svc into dev
Reviewed-on: freeleaps/freeleaps-service-hub#11
2025-07-31 16:11:36 +00:00
zhenyus
d0aa2ba14b Add message queue registration in application bootstrap and update dependencies
Signed-off-by: zhenyus <zhenyus@mathmast.com>
2025-07-30 10:50:22 +08:00
icecheng
69d1007ddf feat(test): add unittest for role management, add coverage report for role management 2025-07-25 16:20:50 +08:00
icecheng
30e6ca72a7 feat(refactor): remove duplicate get_current_user 2025-07-25 10:26:09 +08:00
icecheng
0f7d63f4a2 feat(role_management): add test report for role management 2025-07-22 17:14:30 +08:00
223457162f Merge pull request 'feat(storing payment method): adding payment method storing feature consistent with the updated payment logic in freeleaps backend' (#16) from haolou_local into dev
Reviewed-on: freeleaps/freeleaps-service-hub#16
Reviewed-by: jingyao1991 <jingyao1991@noreply.gitea.freeleaps.mathmast.com>
2025-07-22 06:32:36 +00:00
sunhaolou
d01468f89a fix: refractor the length functions and add comments, and address the issues 2025-07-22 13:16:14 +08:00
icecheng
55e0bebbef feat(role_management): update apitest for role management 2025-07-22 12:27:26 +08:00
icecheng
5be8403c75 feat(role_management): update apitest for role management 2025-07-22 12:14:58 +08:00
icecheng
6b6b52a599 feat(role_management): Add apitest for test_assign_roles 2025-07-22 11:52:41 +08:00
icecheng
9dcd27bc8a feat(role_management): Add logic for creating temporary users to apitest. 2025-07-22 11:30:16 +08:00
icecheng
beba0f5fe5 feat(role_management): add api test case for permission api 2025-07-22 10:28:05 +08:00
icecheng
641281066c feat(role_management): Add pytest for auth services, and add api test case for role management 2025-07-21 17:51:08 +08:00
fca4216043 Merge branch 'dev' into haolou_local 2025-07-21 07:37:02 +00:00
sunhaolou
a8f0a714ae refractor: clean up the codes for online services 2025-07-21 15:23:19 +08:00
icecheng
b8be65615b feat(role_management): Add a universal Depends for permission verification. 2025-07-21 15:10:27 +08:00
icecheng
4109018692 feat(role_management): Update the JWT token generation logic to include new fields user_roles and user_permissions in the payload. 2025-07-21 13:09:59 +08:00
sunhaolou
ccc995f599 refractor: a current working version before cleaning up. 2025-07-21 12:50:51 +08:00
icecheng
419e58da0c feat(role_management): add delete role api 2025-07-21 10:53:37 +08:00
icecheng
4f39f888c4 feat(role_management): add delete permission api 2025-07-21 10:33:21 +08:00
icecheng
ca0bfb155f feat(role_management): add assign roles to user api 2025-07-21 10:20:46 +08:00
icecheng
795c1262c6 feat(role_management): add assign permissions to role api 2025-07-21 09:31:37 +08:00