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