CFDP Update #682

Merged
gaisser merged 158 commits from mueller/cfdp-update-without-handlers into development 2022-11-14 15:04:45 +01:00
Owner

A lot of refactoring and improvements which happended during implementation of the CFDP source and destination handlers.

The handlers itself are not included yet.

Changelog contains more details.

This PR was the baseline for the following successful end-to-end tests with a corresponding Python client:

  • Transfer of small file, no closure, class 1 unacknowledged transfer
  • Transfer of 400kB file, no closure, class 1 unacknowledged transfer
A lot of refactoring and improvements which happended during implementation of the CFDP source and destination handlers. The handlers itself are not included yet. Changelog contains more details. This PR was the baseline for the following successful end-to-end tests with a corresponding Python client: - Transfer of small file, no closure, class 1 unacknowledged transfer - Transfer of 400kB file, no closure, class 1 unacknowledged transfer
muellerr added 138 commits 2022-09-14 19:16:39 +02:00
fsfw/fsfw/pipeline/head This commit looks good Details
6c5bbfa080
some renaming
fsfw/fsfw/pipeline/head This commit looks good Details
f4beef8c9f
run auto-formatter
fsfw/fsfw/pipeline/head There was a failure building this commit Details
458fe460eb
TMTC Routing Refactoring
1. Make TcDistributor less powerful / more flexible and rename it to TcDistributorBase
2. Require AcceptsTelecommandsIF to have an abstract function returning the name
3. Allows the CCSDS distributor to remove the CCSDS header before rerouting. This is useful
for protocols like CFDP which are CCSDS agnostic.

WIP: Still need to implement the removal of the CCSDS header
fsfw/fsfw/pipeline/head There was a failure building this commit Details
4334106ad1
applied auto-formatter
fsfw/fsfw/pipeline/head This commit looks good Details
fsfw/fsfw/pipeline/pr-development This commit looks good Details
902a4bfa9c
unittests for TC and CCSDS distributor
fsfw/fsfw/pipeline/pr-development This commit looks good Details
a426aef7c7
add some more tests for CCSDS distributor
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b0c479cab9
remove some shadowing warnings
fsfw/fsfw/pipeline/pr-development This commit looks good Details
df690b9628
some re-ordering
fsfw/fsfw/pipeline/pr-development This commit looks good Details
60ea9a9a7c
clean up a few tests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
cc98512caf
use tags properly
fsfw/fsfw/pipeline/pr-development This commit looks good Details
29bcaee196
some tests for new var len field
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b20e8a9679
continue new cfdp router
fsfw/fsfw/pipeline/pr-development This commit looks good Details
7fb9e14555
better names for CFDP components
fsfw/fsfw/pipeline/pr-development This commit looks good Details
acf5c2a56d
better names
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c6a7a0fec8
use distributor instead of router
fsfw/fsfw/pipeline/pr-development This commit looks good Details
269a3052ca
added stubs for source and dest handler
fsfw/fsfw/pipeline/pr-development This commit looks good Details
fdb0cc0e44
tweaks for CFDP distributor
fsfw/fsfw/pipeline/pr-development This commit looks good Details
65a47c7c57
add distributor test stub
fsfw/fsfw/pipeline/pr-development This commit looks good Details
43fb6ef5cb
some tag replacements
fsfw/fsfw/pipeline/pr-development This commit looks good Details
37c60d1dd0
add source and dest test stubs
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
202d9341d8
add CFDP unit tests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
904abfba28
clean up CFDP stack a bit
fsfw/fsfw/pipeline/pr-development This commit looks good Details
4f1fe39182
fix unittests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
06bea2f621
test inNull and bool operator
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2f8020baac
move parsing steps into parse function
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c832bffdb0
clean up header test
fsfw/fsfw/pipeline/pr-development This commit looks good Details
1c0b778848
continue CFDP distrib unittests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
bddf5bded1
move some storage manager impl to IF
- Add StorageManagerMock
fsfw/fsfw/pipeline/pr-development This commit looks good Details
22e1555f50
basic storage manager mock
fsfw/fsfw/pipeline/pr-development This commit looks good Details
192956c2c7
use new storage manager mock
fsfw/fsfw/pipeline/pr-development This commit looks good Details
0d26a0f54b
fnish CFDP distributor unittests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
7fb906a0ac
add first fault handler base class
fsfw/fsfw/pipeline/pr-development This commit looks good Details
eccb629ba8
fnished fault handler base
fsfw/fsfw/pipeline/pr-development This commit looks good Details
dba3f9960e
add CFDP fault handler mock
fsfw/fsfw/pipeline/pr-development This commit looks good Details
7f9401cf63
some bugfixes
fsfw/fsfw/pipeline/pr-development This commit looks good Details
4ee01e395b
finished fault handler unit tests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
dce1ebcf65
add CFDP user base
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
7a20412305
add user base documentation
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
bba2d883b6
added some documentation
fsfw/fsfw/pipeline/pr-development This commit looks good Details
5bea92d2a4
some doc updates
fsfw/fsfw/pipeline/pr-development This commit looks good Details
8c059f8f32
new filesystem module
fsfw/fsfw/pipeline/pr-development This commit looks good Details
d45108e3c2
add remote config table provider class
fsfw/fsfw/pipeline/pr-development This commit looks good Details
0cb15e901e
new cfdp mock folder, added additional cfdp mocks
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
507c6ddff1
added some param structs
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
96dfe46e25
added string LV tests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
e45a99143c
added const char* test
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
3e1fd15613
completed all indication parameters
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
28c8248f26
add FS mock and improve HasFilesystemIF
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
a2e0646ed4
fix duplicate naming error
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
c86adf8e10
Merge branch 'development' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development This commit looks good Details
3df1161560
this should do the job
fsfw/fsfw/pipeline/pr-development This commit looks good Details
aca8b53a59
start host FS impl
fsfw/fsfw/pipeline/pr-development This commit looks good Details
20eee2c469
finished host FS impl
fsfw/fsfw/pipeline/pr-development This commit looks good Details
e796e025b6
continue host fs unittests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
8aaabc5d73
basic host FS unittests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
c12492df03
continued host FS test
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
2339c48756
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development This commit looks good Details
23f514039a
host filesystem continued
fsfw/fsfw/pipeline/pr-development This commit looks good Details
12c452e7ce
finished host FS unittests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
0bb82e0da2
add fs mock feed function
fsfw/fsfw/pipeline/pr-development This commit looks good Details
6930656d4e
added some more basic tests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
11a699c3ce
a list should do the job
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
3b33b429e6
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
52f8c5038b
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
9e5dddf79c
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development This commit looks good Details
d12c59c8ac
all retval replacements
fsfw/fsfw/pipeline/pr-development This commit looks good Details
9441b4a70e
continue dest handler
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b1bd631322
implemented metadata handler
fsfw/fsfw/pipeline/pr-development This commit looks good Details
26ea6606bf
metadata recvd indication
fsfw/fsfw/pipeline/pr-development This commit looks good Details
eb29b79467
some more docs
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2c730c8632
use const struct ref instead
fsfw/fsfw/pipeline/pr-development This commit looks good Details
267466be9d
pass const references to user handler now
fsfw/fsfw/pipeline/pr-development This commit looks good Details
81a7c21cd1
continued router component
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
0de7b66218
new retval for permission errors
fsfw/fsfw/pipeline/pr-development This commit looks good Details
db84dcd3ce
afmt
fsfw/fsfw/pipeline/pr-development This commit looks good Details
dc79b7ba00
delete metadata after processing
fsfw/fsfw/pipeline/pr-development This commit looks good Details
eae75b29e7
start file data pdu processing
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c90d1c8071
continue dest handler impl
fsfw/fsfw/pipeline/pr-development This commit looks good Details
108e7737e2
continued dest handler
fsfw/fsfw/pipeline/pr-development This commit looks good Details
3147f67fbd
basic dest handler framework almost complete
fsfw/fsfw/pipeline/pr-development This commit looks good Details
134d5a1411
basic impl which allows naked no closure file transfer
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b984128de5
stub for creating finished PDU
fsfw/fsfw/pipeline/pr-development This commit looks good Details
3dfc882226
impl finish PDU sending
fsfw/fsfw/pipeline/pr-development This commit looks good Details
0dd2b5ddd4
need an event reporter proxy
fsfw/fsfw/pipeline/pr-development This commit looks good Details
d2c4d546c3
first dest handler unittests
- Add new mock class for event reporting proxies
- Add basic setup for unittesting the CFDP destination handler
fsfw/fsfw/pipeline/pr-development This commit looks good Details
52802f127b
basic test stubs
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
5ce1e76723
init dest handler unittests
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
fb96250e36
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
a5d1c38b22
Merge branch 'mueller/refactor-tmtc-stack' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development This commit looks good Details
afc48726b9
comment out check
fsfw/fsfw/pipeline/pr-development This commit looks good Details
5a3f05fa79
return struct ref instead of code
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2e4cdb7366
additional filesystem abstractions
fsfw/fsfw/pipeline/pr-development This commit looks good Details
7b97c8a182
continue tests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
806ae9b41a
check that indication was called
fsfw/fsfw/pipeline/pr-development This commit looks good Details
428da017ba
empty file transfer unittest almost done
fsfw/fsfw/pipeline/pr-development This commit looks good Details
ba046cebd9
empty file unittest complete
fsfw/fsfw/pipeline/pr-development This commit looks good Details
94c9800fae
refactored out generic metadata part
fsfw/fsfw/pipeline/pr-development This commit looks good Details
af851165b4
continue basic small file transfer unittest
fsfw/fsfw/pipeline/pr-development This commit looks good Details
67439b4285
refactor away some duplication
fsfw/fsfw/pipeline/pr-development This commit looks good Details
aebd401d5b
start segmentes file unittest
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c54caf134a
segmented file transfer working
fsfw/fsfw/pipeline/pr-development This commit looks good Details
8971eb386e
retval replacement
fsfw/fsfw/pipeline/pr-development This commit looks good Details
770463e618
remove old code
fsfw/fsfw/pipeline/pr-development This commit looks good Details
11a4b27642
allow deferred init
fsfw/fsfw/pipeline/pr-development This commit looks good Details
683cf8a047
some additional tests
fsfw/fsfw/pipeline/pr-development This commit looks good Details
c87667c03f
some API improvements
fsfw/fsfw/pipeline/pr-development This commit looks good Details
ec1e07b466
some more returncodes
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b73754dfd6
implemented basic PDU forwarding
fsfw/fsfw/pipeline/pr-development This commit looks good Details
34dd478848
helper class OneRemoteConfigProvider
fsfw/fsfw/pipeline/pr-development This commit looks good Details
3e9b47d3a2
update docs release in conf.py
fsfw/fsfw/pipeline/pr-development This commit looks good Details
26b3e5a013
fix misc def config
fsfw/fsfw/pipeline/pr-development This commit looks good Details
a1ea671e2f
header include update
fsfw/fsfw/pipeline/pr-development This commit looks good Details
382543fc59
important bugfix
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
44615c150b
add printout capabilities
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
389d804735
Merge remote-tracking branch 'origin/development' into mueller/cfdp-routers
fsfw/fsfw/pipeline/pr-development This commit looks good Details
fsfw/fsfw/pipeline/head This commit looks good Details
ac7dc55fc1
fix unittest
fsfw/fsfw/pipeline/head Build started... Details
fsfw/fsfw/pipeline/pr-development This commit looks good Details
3a52454949
removed handlers
muellerr requested review from gaisser 2022-09-14 19:18:22 +02:00
muellerr added this to the v6.0.0 milestone 2022-09-14 19:18:28 +02:00
muellerr added 1 commit 2022-09-14 19:25:49 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
73f1917c81
update changelog
muellerr added 1 commit 2022-09-14 19:47:55 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2e27a85c95
finished renaming operation
muellerr added 1 commit 2022-09-14 19:52:17 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
723f8749d4
bump changelog
muellerr added the
Breaking API Change
label 2022-09-15 10:33:16 +02:00
muellerr added a new dependency 2022-09-15 10:35:51 +02:00
muellerr added a new dependency 2022-09-15 10:35:58 +02:00
muellerr added 4 commits 2022-09-15 10:40:59 +02:00
fsfw/fsfw/pipeline/head Build started... Details
fsfw/fsfw/pipeline/pr-development This commit looks good Details
237e29cc59
all necessary modifications
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
1efc0d2855
update changelog
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
75ddfdc65d
Merge branch 'mueller/refactor-accepts-tc-tm-if' into mueller/cfdp-update-without-handlers
muellerr added 5 commits 2022-09-15 10:50:16 +02:00
fsfw/fsfw/pipeline/head Build started... Details
fsfw/fsfw/pipeline/pr-development This commit looks good Details
b3c0e24611
bump changelog
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
e063b44899
add link to PR
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
1db77753e3
Merge remote-tracking branch 'origin/development' into mueller/local-pool-update
muellerr added 2 commits 2022-09-15 10:51:53 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
8501477a78
Merge remote-tracking branch 'origin/development' into mueller/refactor-accepts-tc-tm-if
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
36cf59cc8e
Merge branch 'mueller/refactor-accepts-tc-tm-if' into mueller/cfdp-update-without-handlers
muellerr added 1 commit 2022-09-15 11:01:43 +02:00
muellerr added 1 commit 2022-09-15 16:02:31 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
954c749de0
additional docs
muellerr added 1 commit 2022-09-15 16:20:38 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
2fee2fdff5
unittest which would have caught this
muellerr added 1 commit 2022-09-15 16:45:53 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
54762232a4
refactoring and condition code to strin converter
muellerr added 1 commit 2022-09-15 18:35:27 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
67f1cd0b5f
Merge branch 'development' into mueller/cfdp-update-without-handlers
muellerr added 1 commit 2022-09-15 18:40:56 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
6f8ccf83e7
use singular enum names
muellerr added 1 commit 2022-09-26 15:38:45 +02:00
muellerr added 1 commit 2022-10-17 14:30:29 +02:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
dd636b186b
size check bugfix
gaisser added 1 commit 2022-10-17 14:44:35 +02:00
gaisser approved these changes 2022-10-17 14:55:48 +02:00
gaisser left a comment
Owner

Mainly new feature CFDP. Small linter fixes in other files. LGTM

Mainly new feature CFDP. Small linter fixes in other files. LGTM
muellerr added 1 commit 2022-10-17 17:25:21 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
652c31a683
missing validity check
gaisser reviewed 2022-10-24 15:06:56 +02:00
@ -0,0 +21,4 @@
ReturnValue_t rename(const char *oldPath, const char *newPath, FileSystemArgsIF *args) override;
std::error_code errorCode;
using HasFileSystemIF::createDirectory;
Owner

Check for same issue as #685

Check for same issue as #685
muellerr added 1 commit 2022-10-28 10:04:16 +02:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
c11af63015
Merge remote-tracking branch 'origin/development' into mueller/cfdp-update-without-handlers
muellerr added 1 commit 2022-11-14 14:25:19 +01:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
b48e0fdc0d
Merge remote-tracking branch 'origin/development' into mueller/cfdp-update-without-handlers
gaisser added a new dependency 2022-11-14 14:48:27 +01:00
muellerr added 1 commit 2022-11-14 14:51:16 +01:00
fsfw/fsfw/pipeline/pr-development There was a failure building this commit Details
ab9b6c8c89
Merge remote-tracking branch 'origin/development' into mueller/cfdp-update-without-handlers
muellerr added 1 commit 2022-11-14 14:57:31 +01:00
fsfw/fsfw/pipeline/pr-development This commit looks good Details
72d7c43445
updates for storage manager mock
gaisser merged commit 1b8fc2af19 into development 2022-11-14 15:04:45 +01:00
gaisser deleted branch mueller/cfdp-update-without-handlers 2022-11-14 15:04:49 +01:00
Sign in to join this conversation.
No description provided.