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

(GH-3912) add include_catalog_edges config to terminus #3913

Merged
merged 3 commits into from
Nov 30, 2023

Conversation

austb
Copy link
Contributor

@austb austb commented Nov 27, 2023

Add configuration option to the terminus, include_catalog_edges that, when set to false, will disable sending any catalog edges to PuppetDB. Profiling of PuppetDB command processing has shown catalog edges can take as much as one third of the total time to process a catalog. Users with many nodes that don't use edges in their queries could benefit quite substantially from using this option. It is not enabled by default to maintain backwards compatibility during PuppetDB 8.

Resolves #3912.

Add configuration option to the terminus, `include_catalog_edges` that,
when set to `false`, will disable sending any catalog edges to PuppetDB.
Profiling of PuppetDB command processing has shown catalog edges can
take as much as one third of the total time to process a catalog. Users
with many nodes that don't use edges in their queries could benefit
quite substantially from using this option. It is not enabled by default
to maintain backwards compatibility during PuppetDB 8.

Resolves puppetlabs#3912.
@austb austb force-pushed the gh-3912/main/omit-catalog-edges branch from a71c40c to 8cf4aed Compare November 28, 2023 17:09
@austb austb force-pushed the gh-3912/main/omit-catalog-edges branch from 8cf4aed to f90549d Compare November 28, 2023 17:23
@austb austb marked this pull request as ready for review November 28, 2023 18:33
@austb austb requested review from a team as code owners November 28, 2023 18:33
Copy link
Contributor

@jpartlow jpartlow left a comment

Choose a reason for hiding this comment

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

👍 This looks pretty straight forward.

@rbrw rbrw merged commit 661bff6 into puppetlabs:main Nov 30, 2023
10 checks passed
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.

Add configuration option to disable sending of catalog edges from terminus
3 participants