WIP: Start impl auto switch feature #811

Closed
muellerr wants to merge 11 commits from auto-switch-image-feature into main
Owner
TMTC PR: https://egit.irs.uni-stuttgart.de/eive/eive-tmtc/pulls/246/files
muellerr added 1 commit 2023-10-19 14:30:22 +02:00
EIVE/eive-obsw/pipeline/head This commit looks good Details
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
2e210a0572
start impl auto switch feature
muellerr added this to the v7.2.0 milestone 2023-10-19 14:30:30 +02:00
muellerr added 1 commit 2023-10-19 14:43:52 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
01ce1f154e
that should get the job done
muellerr added 1 commit 2023-10-19 15:06:27 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
befe7ec441
bump tmtc
muellerr added 2 commits 2023-10-19 15:18:41 +02:00
EIVE/eive-obsw/pipeline/pr-main There was a failure building this commit Details
674202c6fb
delay
muellerr added 1 commit 2023-10-19 15:23:24 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
f0ade7274a
works
muellerr changed title from WIP: Start impl auto switch feature to Start impl auto switch feature 2023-10-19 15:23:39 +02:00
muellerr requested review from meggert 2023-10-19 15:24:02 +02:00
muellerr added 1 commit 2023-10-19 15:24:53 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
18bcb434e9
tiny tweak
meggert requested changes 2023-10-20 13:08:29 +02:00
meggert left a comment
Owner

Changelog missing.
Also not sure what this is. To me it looks like the OBSW will always boot to the Target Image, no matter what. Am I missing something? Or is the mechanism not finished yet?

Changelog missing. Also not sure what this is. To me it looks like the OBSW will always boot to the Target Image, no matter what. Am I missing something? Or is the mechanism not finished yet?
muellerr added 1 commit 2023-10-23 13:31:03 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
4fd18e94fd
changelog
muellerr added 1 commit 2023-10-23 13:39:48 +02:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
4e0842c607
add link
Author
Owner

Added changelog entry. For more information, also see new redmine entry: https://egit.irs.uni-stuttgart.de/redmine/projects/eive-flight-manual/wiki/OBC_Auto_Switch_Image

This mechanism will only trigger once it has been activated explicitely. It is also self-clearing / one-shot: The key value pair triggering the mechanism will be cleared after the reboot.

Added changelog entry. For more information, also see new redmine entry: https://egit.irs.uni-stuttgart.de/redmine/projects/eive-flight-manual/wiki/OBC_Auto_Switch_Image This mechanism will only trigger once it has been activated explicitely. It is also self-clearing / one-shot: The key value pair triggering the mechanism will be cleared after the reboot.
Owner

Added changelog entry. For more information, also see new redmine entry: https://egit.irs.uni-stuttgart.de/redmine/projects/eive-flight-manual/wiki/OBC_Auto_Switch_Image

This mechanism will only trigger once it has been activated explicitely. It is also self-clearing / one-shot: The key value pair triggering the mechanism will be cleared after the reboot.

I do understand how it works, I am just not sure it is usefull like this. If we update 0 0, what information do we gain about the updated image with this feature, if it reboots more or less instantly.

Also this way a manual reboot SELF will lead to 2 reboots to the same image.

> Added changelog entry. For more information, also see new redmine entry: https://egit.irs.uni-stuttgart.de/redmine/projects/eive-flight-manual/wiki/OBC_Auto_Switch_Image > > This mechanism will only trigger once it has been activated explicitely. It is also self-clearing / one-shot: The key value pair triggering the mechanism will be cleared after the reboot. I do understand how it works, I am just not sure it is usefull like this. If we update 0 0, what information do we gain about the updated image with this feature, if it reboots more or less instantly. Also this way a manual reboot SELF will lead to 2 reboots to the same image.
Author
Owner

It's not about information, it is about safety. If the 0 0 image OBSW binary is updated, but not the firmware/dtb (HPC, connection loss, cosmic snipe whatever..) , the Linux/OBSW might crash horribly. We still have the other reboot mechanism, but I do not 100 % trust only this mechanism. The auto switch image would be a one-shot fallback to just prevent the OBC from booting a (temporarily) broken image. After successfully updating everythng (nominal case unless we're terribly unlucky), we clear the flag and test the image normally. It's just a temporary safety layer for the 0 0 update I guess.

It's not about information, it is about safety. If the 0 0 image OBSW binary is updated, but not the firmware/dtb (HPC, connection loss, cosmic snipe whatever..) , the Linux/OBSW might crash horribly. We still have the other reboot mechanism, but I do not 100 % trust only this mechanism. The auto switch image would be a one-shot fallback to just prevent the OBC from booting a (temporarily) broken image. After successfully updating everythng (nominal case unless we're terribly unlucky), we clear the flag and test the image normally. It's just a temporary safety layer for the 0 0 update I guess.
muellerr modified the milestone from v7.2.0 to v7.3.0 2023-10-27 12:49:24 +02:00
muellerr added 1 commit 2023-10-30 14:45:10 +01:00
muellerr added 1 commit 2023-10-30 15:02:27 +01:00
EIVE/eive-obsw/pipeline/pr-main This commit looks good Details
EIVE/eive-obsw/pipeline/head This commit looks good Details
c12706ef05
Merge branch 'main' into auto-switch-image-feature
muellerr changed title from Start impl auto switch feature to WIP: Start impl auto switch feature 2023-10-30 15:50:10 +01:00
muellerr modified the milestone from v7.3.0 to v7.3.0 2023-10-30 15:54:57 +01:00
muellerr modified the milestone from v7.3.0 to v7.4.0 2023-11-06 14:39:18 +01:00
meggert modified the milestone from v7.4.0 to v7.4.1 2023-11-24 09:04:10 +01:00
meggert modified the milestone from v7.4.1 to v7.5.1 2023-12-06 11:53:06 +01:00
meggert modified the milestone from v7.5.1 to v7.5.2 2023-12-13 16:57:41 +01:00
meggert modified the milestone from v7.5.2 to v7.5.3 2023-12-13 18:03:13 +01:00
muellerr modified the milestone from v7.5.3 to v7.7.1 2023-12-19 13:02:06 +01:00
meggert removed this from the v7.7.1 milestone 2024-02-29 16:07:49 +01:00
Owner

no plan on implementing this anytime soon

no plan on implementing this anytime soon
meggert closed this pull request 2024-03-27 11:44:34 +01:00
All checks were successful
EIVE/eive-obsw/pipeline/pr-main This commit looks good
EIVE/eive-obsw/pipeline/head This commit looks good

Pull request closed

Sign in to join this conversation.
No description provided.