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 2 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
OPS TODO
api change
breaking api change
bug
documentation
duplicate
feature
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
v1.28.0
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

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?