-
Notifications
You must be signed in to change notification settings - Fork 611
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
(MODULES-5062) release prep 5.0.0 #883
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Prevents warnings like: Scope(Concat[/var/lib/pgsql/data/pg_ident.conf]): The $force parameter to concat is deprecated and has no effect
The $force parameter to concat is deprecated and has no effect.
…eters Fix deprecated apt::source parameters
- remove the old openSUSE 13.2 - add default value '94' to cover all the recent and supported openSUSE distros - use the old initscript for SLE 11 SPx, systemd commands for anything else
fix default params for SUSE family OSes
Caused by float passed to versioncmp function which expects a string, which is only executed when OS == SLES
(maint) Fix CI issue where acceptance tests fail on SLES
As the values for the postgis version is hard-coded, this module currently installs by default older, unsupported versions of postgis. This commit updates the module to align with current default postgis versions for each related postgres version.
Align postgis default version with postgres version
Change - Remove deprecated force parameter on concat resources
[MODULES-4528] Replace Puppet.version.to_f version comparison from spec_helper.rb
(FM-6116) - Adding POT file for metadata.json
This reverts commit b6fbbb3.
[MODULES-4598] Revert "fix default params for SUSE family OSes"
…son from locales config
[msync] 786266 Implement puppet-module-gems, a45803 Remove metadata.json from locales config
[MODULES-4598] Revert "Revert "fix default params for SUSE family OSes""
This includes updates to the README to prepare for being sent away for Translation.
…ates MODULES-4324 - README Updates for Translation.
Remove deprecated force parameter from concat
Allowo to disable managing passwords for users
we recently switched apt back to the params pattern and it now uses structured facts. since postgres includes apt, some postgres tests needed to have structured facts mocked out because most of the tests still use legacy facts. the mocking gets redundant and we should make an effor to go in at some point and switch postgres to structured facts completely but this works for now.
we recently switched apt back to the params pattern and it now uses structured facts. since postgres includes apt, some postgres tests needed to have structured facts mocked out because most of the tests still use legacy facts. the mocking gets redundant and we should make an effor to go in at some point and switch postgres to structured facts completely but this works for now.
The goal of this is to get rid of most of the validate_* calls, not to introduce datatypes to _all_ params.
Cleanup README
This replaces the defined type `validate_db_connection` with a new custom type called `postgresql_conn_validator`. It's functionality is nearly a copy of its predecessor with the exception of the `create_db_first` parameter which is taken care of with a collector in service.pp. The old type is still intact but all docs have been removed and a warning has been attached to it announcing its deprecation.
Addresses a few of @hasegeli\'s comments. Adds a `command` parameter for more customizability.
What appeared to be a breakage because of puppet 4 types ended up being an issue where auth_option needs to be passed to pg_hba_rule on postgres 8.1. This fixes the problem and should kick the ci clog
What appeared to be a breakage because of puppet 4 types ended up being an issue where auth_option needs to be passed to pg_hba_rule on postgres 8.1. This fixes the problem and should kick the ci clog
This would let us use the list on multiple places. It was also continuous source of source code conflicts.
postgresql::server::reload class requires postgresql::server::service. We don't need to repeat it in here.
They do no longer require restart on PostgreSQL 10.
"silent_mode" has been removed by the upstream with commit f7ea6beaf4ca02b8e6dc576255e35a5b86035cb9. "unix_socket_directory" has been turned into "unix_socket_directories" with commit c9b0cbe98bd783e24a8c4d8d8ac472a494b81292. It wouldn't matter if we trigger restart for those configuration variables, as they are not valid so cannot be declared anymore, but why not do it as we have the infrastructure. PostgreSQL wouldn't start when they are on the configuration file. Preventing the restart can save people from having their database shut-down, because of not-anymore-valid entry. The change is also good for maintenance of the configuration variables. We wouldn't forget to remove them after the releases are not supported anymore.
(MODULES-5050) Fix for grant_schema_spec
Maintain config entries for PostgreSQL 10
(MODULES-1394) replace validate_db_connection type with custom type
last fix for postgres 8 ever. postgres 8 does not support the CONNECT privilege or equals signs in the command. --no-password doesn't seem to be required. Also cleaned up conditional in the util script.
last fix for postgres 8 ever. postgres 8 does not support the CONNECT privilege or equals signs in the command. --no-password doesn't seem to be required. Also cleaned up conditional in the util script.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.