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

Disable directory polling in default configuration table #319

Closed
3 tasks done
semaldona opened this issue Aug 24, 2022 · 0 comments · Fixed by #320
Closed
3 tasks done

Disable directory polling in default configuration table #319

semaldona opened this issue Aug 24, 2022 · 0 comments · Fixed by #320
Assignees
Milestone

Comments

@semaldona
Copy link

semaldona commented Aug 24, 2022

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the CF README.md file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe.
The default configuration table cf_def_config.c enables channel 0 directory polling on /cf/poll_dir at app startup. Unless that directory already exists on the target filesystem, CF will periodically emit the error event 65 message "CF: failed to open playback directory /cf/poll_dir, error=-1". When testing and validating the CF app running defaults, it is not ideal to have unnecessary error events being periodically reported.

Describe the solution you'd like
Disable all directory polling in the default configuration table. Because this feature requires foreknowledge of specific directories resident on the target filesystem, it should only be configured and enabled by end users. All channels should have polling disabled and no polling settings in the CF_PollDir_t structure.

Describe alternatives you've considered
N/A

Additional context
N/A

Requester Info
Sergio Maldonado, NASA GSFC, Arctic Slope Technical Services

@skliper skliper self-assigned this Aug 24, 2022
@skliper skliper added this to the Draco milestone Aug 24, 2022
skliper added a commit to skliper/CF that referenced this issue Aug 24, 2022
skliper added a commit to skliper/CF that referenced this issue Aug 24, 2022
dzbaker added a commit that referenced this issue Aug 25, 2022
Fix #319, Disable polling directory in default table
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants