Skip to content

Allow interval to be configurable#86

Open
faisal-memon wants to merge 3 commits intolinuxserver:masterfrom
faisal-memon:interval
Open

Allow interval to be configurable#86
faisal-memon wants to merge 3 commits intolinuxserver:masterfrom
faisal-memon:interval

Conversation

@faisal-memon
Copy link
Copy Markdown

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

  • Allows the update interval to be configurable using a 60m, 3h, etc. format.
  • Can configure hours and minutes. If unit is not specified defaults to minutes
  • Default is 5 minutes if not set. Maintaining existing functionality.

Benefits of this PR and context:

Can configure to update less often, reducing load on duck dns servers. Most people probably don't need to update every 5 minutes.

How Has This Been Tested?

  • Running locally on my home server with a few different values using this image
  • Negative testing of bad values

Source / References:

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/duckdns/33acdda4-pkg-33acdda4-dev-d6e2fa3787949664710922dc1b5c84d99c31a308-pr-86/index.html
https://ci-tests.linuxserver.io/lspipepr/duckdns/33acdda4-pkg-33acdda4-dev-d6e2fa3787949664710922dc1b5c84d99c31a308-pr-86/shellcheck-result.xml

Tag Passed
amd64-33acdda4-pkg-33acdda4-dev-d6e2fa3787949664710922dc1b5c84d99c31a308-pr-86
arm64v8-33acdda4-pkg-33acdda4-dev-d6e2fa3787949664710922dc1b5c84d99c31a308-pr-86

@aptalca
Copy link
Copy Markdown
Member

aptalca commented Feb 19, 2026

#8

@faisal-memon
Copy link
Copy Markdown
Author

#8

Thanks @aptalca. Makes sense if you want to close. Doesn't hurt to have it though.

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/duckdns/02448f72-pkg-02448f72-dev-019a547869225f9f61fb90d0e767abc0f8344a3e-pr-86/index.html
https://ci-tests.linuxserver.io/lspipepr/duckdns/02448f72-pkg-02448f72-dev-019a547869225f9f61fb90d0e767abc0f8344a3e-pr-86/shellcheck-result.xml

Tag Passed
amd64-02448f72-pkg-02448f72-dev-019a547869225f9f61fb90d0e767abc0f8344a3e-pr-86
arm64v8-02448f72-pkg-02448f72-dev-019a547869225f9f61fb90d0e767abc0f8344a3e-pr-86

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/duckdns/1e65555f-pkg-1e65555f-dev-4e731ab91122235226bb1d2effaaca6d00bf2d6e-pr-86/index.html
https://ci-tests.linuxserver.io/lspipepr/duckdns/1e65555f-pkg-1e65555f-dev-4e731ab91122235226bb1d2effaaca6d00bf2d6e-pr-86/shellcheck-result.xml

Tag Passed
amd64-1e65555f-pkg-1e65555f-dev-4e731ab91122235226bb1d2effaaca6d00bf2d6e-pr-86
arm64v8-1e65555f-pkg-1e65555f-dev-4e731ab91122235226bb1d2effaaca6d00bf2d6e-pr-86

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/duckdns/36a2e3a3-pkg-36a2e3a3-dev-a9487f766a2ef737f485cdb8099e889512c01c7f-pr-86/index.html
https://ci-tests.linuxserver.io/lspipepr/duckdns/36a2e3a3-pkg-36a2e3a3-dev-a9487f766a2ef737f485cdb8099e889512c01c7f-pr-86/shellcheck-result.xml

Tag Passed
amd64-36a2e3a3-pkg-36a2e3a3-dev-a9487f766a2ef737f485cdb8099e889512c01c7f-pr-86
arm64v8-36a2e3a3-pkg-36a2e3a3-dev-a9487f766a2ef737f485cdb8099e889512c01c7f-pr-86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants