Skip to content

wrapper for extension type: allow extension config from server.pp#1674

Open
ikonia wants to merge 1 commit into
puppetlabs:mainfrom
ikonia:extensions_loop
Open

wrapper for extension type: allow extension config from server.pp#1674
ikonia wants to merge 1 commit into
puppetlabs:mainfrom
ikonia:extensions_loop

Conversation

@ikonia
Copy link
Copy Markdown
Contributor

@ikonia ikonia commented May 28, 2026

Summary

using the same merge pattern as server::roles to the role type, server::grants to the grant type this follows the same pattern creating the parameter extensions to access the extension type allowing extensions to be configured against database directly from server.pp

There is additional logic as there is a condition where you can define an extension against a database that has not yet been created (if the database has not been defined or server::database has not executed in this scenario there is logic to create the database first with the concept that there is no point trying to configure against an extension against a database you don't want to exist, so it is likely this database is either managed outside of the module, managed in the module but has not yet been properly configured (server::databases has not run) in which case the initial creation is done here and then the actual full database configuration will happen post the extension creation. This seems an acceptable way to handle this problem.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • [ x] Thought process behind the implementation.

following a proven pattern with roles -> server::roles , grants -> server::grants (and other variations that are still pending merge)

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • [x ] 🟢 Spec tests.
  • [x ] 🟢 Acceptance tests.
  • [x ] Manually verified. (For example puppet apply)

remove white space linting

linting alignment correction
@ikonia ikonia force-pushed the extensions_loop branch from 13d9e1a to 57f0d1d Compare May 28, 2026 09:17
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.

1 participant