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

Fixed boolean check for boolean parameter #121

Merged
merged 1 commit into from Apr 14, 2023

Conversation

abujeda
Copy link
Contributor

@abujeda abujeda commented Mar 31, 2023

Fixed the check for @job_strict_host_checking in the openondemand::cluster template. As it is a boolean and when set to false, the check fails and will not render the value.

@abujeda
Copy link
Contributor Author

abujeda commented Apr 11, 2023

@treydock Is this the best way to submit changes/requests for the Puppet Module?

CC: @johrstrom

@treydock treydock added the bugfix Something isn't working label Apr 13, 2023
@treydock
Copy link
Collaborator

@abujeda You will need to rebase against master branch to resolve the tests not running. There were some changes recently to fix the testing and looks like your branch doesn't have them.

@abujeda abujeda force-pushed the cluster_strict_host_checking_fix branch from c75c695 to a624953 Compare April 14, 2023 07:59
@abujeda abujeda force-pushed the cluster_strict_host_checking_fix branch from a624953 to f0216c9 Compare April 14, 2023 08:06
@abujeda
Copy link
Contributor Author

abujeda commented Apr 14, 2023

Thanks @treydock.

Rebased completed.

@treydock treydock merged commit d5778cf into OSC:master Apr 14, 2023
12 checks passed
@treydock
Copy link
Collaborator

This is released as v3.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants