Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 6, 2026

Containerized deployments default to ftslave=false (using dafilesrv) and parallelRequestLimit=20, while bare-metal defaults were ftslave=true and had inconsistent parallelRequestLimit values across configuration files. This PR aligns both deployment types to use consistent defaults.

Changes

ftslave default

  • dali/dfu/dfurun.cpp: Changed bare-metal default from isContainerized() ? false : true to false
  • Both deployment types now use dafilesrv by default

parallelRequestLimit default (20 → 80)

Updated to match code default DEFAULT_STDCMD_PARALLELREQUESTLIMIT:

  • initfiles/etc/DIR_NAME/environment.xml.in: DafilesrvProcess and Instance elements
  • testing/regress/environment.xml.in: DafilesrvProcess and Instance elements
  • initfiles/componentfiles/configxml/dafilesrv.xsd: Global and Instance attribute defaults
  • initfiles/componentfiles/configschema/xsd/dafilesrv.xsd: presetValue attribute
  • helm/hpcc/values.schema.json: Default value (also fixed type from string to integer)

Incidental fixes

  • Corrected spelling: "settting" → "setting" in tooltip text
  • Fixed grammar: "This a global setting" → "This is a global setting"
Original prompt

Currently, containerized defaults to ftslave=false and uses dafilesrv, but bare-metal (non-containerized) uses ftslave=true by default.
Change the default in bare-metal so it is also ftslave=false and hence uses dafilesrv.

Also, change the defaults for DEFAULT_STDCMD_PARALLELREQUESTLIMIT and DEFAULT_SLOWCMD_PARALLELREQUESTLIMIT, so that k8s containerized is the same as bare-metal. Note there are defaults in environment.xml.in that also need to be udpated.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update default settings for bare-metal to match containerized Align bare-metal and containerized defaults: ftslave=false, parallelRequestLimit=80 Jan 6, 2026
Copilot AI requested a review from jakesmith January 6, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants