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: split and run grpc_server, ota_core and otaclient main as separated processes #431

Draft
wants to merge 88 commits into
base: main
Choose a base branch
from
Draft

feat: split and run grpc_server, ota_core and otaclient main as separated processes #431

wants to merge 88 commits into from

Conversation

Bodong-Yang
Copy link
Member

@Bodong-Yang Bodong-Yang commented Nov 27, 2024 •
edited
Loading

Introduction

Note

This PR is the core PR and the final piece of the whole otaclient re-architecture project.

https://tier4.atlassian.net/browse/RT4-12276

@Bodong-Yang
otaclient._utils
7c5e02d
@Bodong-Yang
add otaclient_common.shm_status module
141dba9
@Bodong-Yang
utils: add SharedOTAClientStatusWriter/Reader types
8617f5f
@Bodong-Yang
ecu_tracker: use new shm_status
9564b52
@Bodong-Yang
_status_monitor: integrate shm
1ffbf7d
@Bodong-Yang
ecu_tracker: cleanup at exit
dfb231f
@Bodong-Yang
ota_core: control_flag now becomes mp_sync.Event, cleanup unused code
82f945c
@Bodong-Yang
ota_core: use session_id from request
3a518b2
@Bodong-Yang
move gen_session_id to otaclient._utils
c83d17c
@Bodong-Yang
_types: add types for IPC
19e4e34
@Bodong-Yang
api_v2.servicer: integrate IPC
4d9b43d
@Bodong-Yang
minor fix
747d247
@Bodong-Yang
types: cleanup unused
89e3ca0
@Bodong-Yang
ota_core: implement IPC interface
3ae0f4b
@Bodong-Yang
minor update
8448628
@Bodong-Yang
api_v2.servicer: nolonger in charge of launching/shutting down otaproxy
e473dcc
@Bodong-Yang
api_v2.ecu_status: expose any_requires_network and all_ecus_succeeded…
dec832a 
… flags
@Bodong-Yang
WIP: main
72cd46c
@Bodong-Yang
re-implement _otaproxy_ctx, move it to otaclient package
2009c09
@Bodong-Yang
_otaproxy_ctx: minor cleanup
544e356
@Bodong-Yang
WIP: main
a26a4e8
@Bodong-Yang
implement grpc.api_v2.main
467de3f
@Bodong-Yang
move otaproxy control thread logic into otaproxy_ctx
d6df546
@Bodong-Yang
move ota_core_process into ota_core module
256d822
@Bodong-Yang
otaproxy_ctx: simplify the implementation of otaproxy control
bd7a183
@Bodong-Yang
finish up main
4149c39
@Bodong-Yang
fix main
772c5ba
@Bodong-Yang
status_monitor: not unlink the shm
57d5947
@Bodong-Yang
api_v2.ecu_tracker: actively polling until we get the first valid res…
3575406 
…p when startup
@Bodong-Yang
ecu_tracker: minor cleanup
712b441
Bodong-Yang and others added 11 commits November 27, 2024 01:44
@Bodong-Yang
revert src/ota_proxy to main branch's status for merge
e31859e
@Bodong-Yang
revert src/otaclient_common/cmdhelper.py to main branch's status for …
a9bf9e5 
…merge
@Bodong-Yang
Merge remote-tracking branch 'origin/main' into feat/otaclient_split_…
95f43f9 
…processes
@Bodong-Yang
do not install SIGINT handler; SIGNTERM handler now only raises a Sys…
4648dc7 
…temExit exception
@Bodong-Yang
ota_core: do not replace the original SIGINT handler
8275cf9
@Bodong-Yang
signame -> signal_value
fbb1272
@Bodong-Yang
ota_core: not install SIGTERM handler for now
6934021
@Bodong-Yang
main: call on_shutdown on sigint and sigterm
66ae925
@Bodong-Yang
main: still use sys.exit
732b5e1
@Bodong-Yang
Revert "ota_core: not install SIGTERM handler for now"
7f1c7e5 
This reverts commit 6934021.
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
187d8dc 
for more information, see https://pre-commit.ci
@Bodong-Yang Bodong-Yang added the feature New feature is introduced label Nov 27, 2024
Bodong-Yang and others added 18 commits November 28, 2024 01:28
@Bodong-Yang
ready for merging main branch's shm_status
f1c187a
@Bodong-Yang
Merge remote-tracking branch 'origin/main' into feat/otaclient_split_…
108d6e1 
…processes
@Bodong-Yang
_types: add MultipleECUStatusFlags
5645585
@Bodong-Yang
ecu_status: use MultipleECUStatusFlags
17a58c8
@Bodong-Yang
otaproxy_ctx: use MultipleECUStatusFlags
95e300b
@Bodong-Yang
api_v2.servicer: not manage otaclient control flags here
d9d791e
@Bodong-Yang
api_v2.main: cleanup accordingly
a0cee32
@Bodong-Yang
main: finish up integration
d45dd27
@Bodong-Yang
utils.wait_and_log: take a func that returns bool
674ea3e
@Bodong-Yang
ota_core: use MultipleECUStatusFlags instead
f1be14b
@Bodong-Yang
fix up test_utils
aaa1984
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
c2d8f5e 
for more information, see https://pre-commit.ci
@Bodong-Yang
Merge branch 'main' into feat/otaclient_split_processes
6339bd3
@Bodong-Yang
status_monitor: reduce min collect interval to 0.5
c075d90
@Bodong-Yang
grpc.api_v2.ECUTracker: reduce local active poll interval on startup …
77c4e94 
…to 0.1
@Bodong-Yang
Merge branch 'main' into feat/otaclient_split_processes
1bd6c4d
@Bodong-Yang
squashed merge from fix/ota_image_invalid_handling
2b72d57
@Bodong-Yang
Merge remote-tracking branch 'origin/main' into feat/otaclient_split_…
ec8bd67 
…processes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature New feature is introduced
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Bodong-Yang