Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #93, Remove file age check bug #99

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Mar 22, 2023

Checklist (Please check before submitting)

Describe the contribution

Testing performed
CI and local check that files get produced as requested (1 per minute when configured to do so)

Expected behavior changes
Will produce the expected files, vs 17 per 10 minutes when configured to produce 1 per minute

System(s) tested on
Ubuntu 20.04

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper
Copy link
Contributor Author

skliper commented Mar 22, 2023

Note, could have also just broken the coupling between HK messages and age checks (either do it on a timer or w/ a separate processing message). Or just do a real age check instead of accumulating age. This change just makes it work like it used to when there wasn't a SB timeout.

Copy link
Contributor

@jphickey jphickey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK for a basic workaround. I think we should revisit this issue with a more robust design, though - this still requires DS_SECS_PER_HK_CYCLE to be manually set to the same thing it was set to in SCH to generate the HK request. This seems fragile.

I'd recommend to at least have DS_FileTestAge do its own clock sample and figure out the real elapsed time since it was last invoked using the monotonic clock provided by the PSP (e.g. via CFE_PSP_GetTime) rather than believing some constant value. This shouldn't need to be configured at all.

@skliper
Copy link
Contributor Author

skliper commented Mar 22, 2023

I agree completely, this is just the basic revert of the bug introduced in 2.6.0. I only put this merge in to get a quick fix if possible.

@dzbaker dzbaker merged commit 591233e into nasa:main Mar 23, 2023
@dmknutsen dmknutsen added this to the Equuleus milestone May 26, 2023
@chillfig chillfig added the bug label Sep 5, 2023
@skliper skliper deleted the fix93_agebug branch April 1, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File age check logic is wrong
5 participants