eive/eive-obsw
eive/eive-obsw
13
4
Fork 0
Code Issues 4 Pull Requests 1 Releases 120 Wiki Activity

RW Status Check for ACS Ctrl #382

Merged
muellerr merged 7 commits from eggert/rw-status-check into develop 2023-02-17 10:21:34 +01:00
Conversation 3 Commits 7 Files Changed 11 +79 -119
meggert commented 2023-02-16 16:29:59 +01:00
Owner
Copy Link

ACS Ctrl needs to know which RWs are available

ACS Ctrl needs to know which RWs are available
meggert added this to the v1.28.0 milestone 2023-02-16 16:29:59 +01:00
meggert added 3 commits 2023-02-16 16:30:01 +01:00
matched naming to actual RWs
Some checks failed
EIVE/eive-obsw/pipeline/head There was a failure building this commit
Details
4b8f5992b5
RW pseudo inverses now set from RW state. multiple invalids trigger event and disable ptgCtrl
All checks were successful
EIVE/eive-obsw/pipeline/head This commit looks good
Details
97f7f7c973
added event handling for failure of more than one RW
Some checks are pending
EIVE/eive-obsw/pipeline/head Build started...
Details
EIVE/eive-obsw/pipeline/pr-develop This commit looks good
Details
8b1d6cf07d
meggert added 3 commits 2023-02-17 09:22:13 +01:00
rwAntistiction now gets actual RW states
All checks were successful
EIVE/eive-obsw/pipeline/pr-develop This commit looks good
Details
7ee9bf6edf
meggert added 1 commit 2023-02-17 09:46:44 +01:00
changelog
All checks were successful
EIVE/eive-obsw/pipeline/pr-develop This commit looks good
Details
f54945de12
meggert changed title from WIP: RW Status Check for ACS Ctrl to RW Status Check for ACS Ctrl 2023-02-17 09:47:01 +01:00
meggert commented 2023-02-17 09:59:29 +01:00
Author
Owner
Copy Link

i dont think more than 1 invalid RW should ever happen because the RwAss should handle these situations anyways

i dont think more than 1 invalid RW should ever happen because the RwAss should handle these situations anyways
meggert requested review from muellerr 2023-02-17 09:59:39 +01:00
muellerr approved these changes 2023-02-17 10:18:20 +01:00
muellerr commented 2023-02-17 10:19:07 +01:00
Owner
Copy Link

Yes, that's what I thought as well. Still merge it?

Yes, that's what I thought as well. Still merge it?
muellerr merged commit f41cf14df5 into develop 2023-02-17 10:21:34 +01:00
muellerr deleted branch eggert/rw-status-check 2023-02-17 10:21:35 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
muellerr
Labels
Clear labels
OPS TODO
Issues identified by OPS
api change
API change
breaking api change
API change which can break existing code
bug
Something is not working
documentation
Documentation
duplicate
This issue or pull request already exists
feature
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone v1.28.0
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: eive/eive-obsw#382
No description provided.
Delete Branch "eggert/rw-status-check"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?