Thanks to visit codestin.com
Credit goes to github.com

Skip to content

supress warnings#304

Merged
treydock merged 5 commits into
treydock:masterfrom
cirrax:dev_fix_warnings
Feb 10, 2026
Merged

supress warnings#304
treydock merged 5 commits into
treydock:masterfrom
cirrax:dev_fix_warnings

Conversation

@trefzer

@trefzer trefzer commented Oct 13, 2023

Copy link
Copy Markdown
Contributor

Add parameters to supress warnings, since my monitoring is not happy with warnings !

@treydock

Copy link
Copy Markdown
Owner

What about the ability to suppress all warnings with a single property?

https://github.com/treydock/puppet-module-keycloak/blob/master/lib/puppet/type/keycloak_api.rb

That allows setting things that can be accessed by all types/providers.

They are set here: https://github.com/treydock/puppet-module-keycloak/blob/master/lib/puppet/provider/keycloak_api.rb#L13-L22

You can then do like if self.class.suppress_warnings in instance methods.

@treydock

treydock commented Feb 6, 2026

Copy link
Copy Markdown
Owner

Please rebase against master branch as I've recently fixed tests.

@trefzer

trefzer commented Feb 6, 2026

Copy link
Copy Markdown
Contributor Author

fyi, I do currently not have access to a working keycloack, so I rebased but did not check if it's still a valid change !

Comment thread spec/unit/puppet/type/keycloak_identity_provider_spec.rb Outdated
set this to true, to not display the puppet warning that
the current value cannot be checked.
set this to true, to not display the puppet warning that
the current value cannot be checked.
set this to true, to not display the puppet warning that
the current value cannot be checked.
This test seems not to work with the parent: Puppet::Parameter::Boolean
setting.
Comment thread spec/unit/puppet/type/keycloak_realm_spec.rb Outdated
Comment thread spec/unit/puppet/type/keycloak_ldap_user_provider_spec.rb Outdated
@treydock

Copy link
Copy Markdown
Owner

If you are able to update so I can modify your fork's branch via this PR, I can make the updates.

@trefzer

trefzer commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

@treydock I think at least one statement of your last comment should be negated ;)
feel free to take over but hopefully it will pass now with all the removes !

@treydock treydock merged commit f805d31 into treydock:master Feb 10, 2026
10 checks passed
@trefzer trefzer deleted the dev_fix_warnings branch February 10, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants