I found this while trying to fix a sentinel CI test.
@hwware FYI

btw, would it be more appropriate to change to SRI_MASTER_REBOOTING?

I found this while trying to fix a sentinel CI test. @hwware FYI

btw, would it be more appropriate to change to SRI_MASTER_REBOOTING?

I do not think change to SRI_MASTER_REBOOTING is a good idea, because reboot do not take long time, the loading after reboot takes long time

I do not think change to SRI_MASTER_REBOOTING is a good idea, because reboot do not take long time, the loading after reboot takes long time

great, got it

i don't know anything about sentinel, so maybe i should leave this for Yossi.
is this flag meant to be used by users or just tests?
should it be mentioned in the release notes?
you said you noticed it's missing when working on a failed test, did you need it in order to fix the test? or just for debugging?

is this flag meant to be used by users or just tests?

I guess it was meant to be used by users

should it be mentioned in the release notes?

look like we don't.
#8682

you said you noticed it's missing when working on a failed test, did you need it in order to fix the test? or just for debugging?

currently i don't need it. Just look the code and find it

i don't know anything about sentinel, so maybe i should leave this for Yossi. is this flag meant to be used by users or just tests? should it be mentioned in the release notes? you said you noticed it's missing when working on a failed test, did you need it in order to fix the test? or just for debugging?

SRI_MASTER_REBOOT flag is only for the user (aka our codes), we do not need to add it in the release note.