Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: puppetlabs/puppetlabs-postgresql
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v7.2.0
Choose a base ref
...
head repository: puppetlabs/puppetlabs-postgresql
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v7.3.0
Choose a head ref
  • 18 commits
  • 23 files changed
  • 10 contributors

Commits on May 27, 2021

  1. Autorequire the service in postgresql_psql

    In d69ff1e I changed the require to be
    an autorequire. However, even in the PR I noted[1] it may not work. Now
    I realize this may not work because you can't autorequire on a class. In
    the same way, you can't collect classes.
    
    I started to see this on my Puppet 7 testing (edited for brevity):
    
        Notice: /Stage[main]/Postgresql::Client/Package[postgresql-client]/ensure: created
        Notice: /Stage[main]/Postgresql::Client/File[/usr/local/bin/validate_postgresql_connection.sh]/ensure: defined content as '{sha256}f34c53aa433754c6343e6382cd994e33f2887f9eb54503421eb0db55a55e14a6'
        Notice: /Stage[main]/Postgresql::Server::Install/Package[postgresql-server]/ensure: created
        Notice: /Stage[main]/Postgresql::Server::Initdb/File[/var/lib/pgsql/data]/ensure: created
        Notice: /Stage[main]/Postgresql::Server::Initdb/Exec[postgresql_initdb]/returns: The files belonging to this database system will be owned by user "postgres".
        Notice: /Stage[main]/Postgresql::Server::Initdb/Exec[postgresql_initdb]/returns: This user must also own the server process.
        Notice: /Stage[main]/Postgresql::Server::Initdb/Exec[postgresql_initdb]/returns:
        Notice: /Stage[main]/Postgresql::Server::Initdb/Exec[postgresql_initdb]/returns: initdb: error: invalid locale settings; check LANG and LC_* environment variables
        Error: '/usr/bin/initdb --pgdata '/var/lib/pgsql/data'' returned 1 instead of one of [0]
        Error: /Stage[main]/Postgresql::Server::Initdb/Exec[postgresql_initdb]/returns: change from 'notrun' to ['0'] failed: '/usr/bin/initdb --pgdata '/var/lib/pgsql/data'' returned 1 instead of one of [0]
        Info: Class[Postgresql::Server::Initdb]: Unscheduling all events on Class[Postgresql::Server::Initdb]
        Notice: /Stage[main]/Postgresql::Server::Config/File[/var/lib/pgsql/data/postgresql.conf]: Dependency Exec[postgresql_initdb] has failures: true
        Warning: /Stage[main]/Postgresql::Server::Config/File[/var/lib/pgsql/data/postgresql.conf]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/File[systemd-conf-dir]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/File[systemd-override]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/File[old-systemd-override]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Exec[restart-systemd]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_hba.conf]/Concat_file[/var/lib/pgsql/data/pg_hba.conf]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_hba.conf]/File[/var/lib/pgsql/data/pg_hba.conf]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_hba.conf]/Concat_fragment[/var/lib/pgsql/data/pg_hba.conf_header]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Config_entry[port]/Postgresql_conf[port]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Config_entry[data_directory]/Postgresql_conf[data_directory]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_ident.conf]/Concat_file[/var/lib/pgsql/data/pg_ident.conf]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_ident.conf]/File[/var/lib/pgsql/data/pg_ident.conf]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Concat[/var/lib/pgsql/data/pg_ident.conf]/Concat_fragment[/var/lib/pgsql/data/pg_ident.conf_header]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[local access as postgres user]/Concat::Fragment[pg_hba_rule_local access as postgres user]/Concat_fragment[pg_hba_rule_local access as postgres user]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[local access to database with same name]/Concat::Fragment[pg_hba_rule_local access to database with same name]/Concat_fragment[pg_hba_rule_local access to database with same name]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[allow localhost TCP access to postgresql user]/Concat::Fragment[pg_hba_rule_allow localhost TCP access to postgresql user]/Concat_fragment[pg_hba_rule_allow localhost TCP access to postgresql user]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[deny access to postgresql user]/Concat::Fragment[pg_hba_rule_deny access to postgresql user]/Concat_fragment[pg_hba_rule_deny access to postgresql user]: Skipping because of failed dependencies
          Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[allow access to all users]/Concat::Fragment[pg_hba_rule_allow access to all users]/Concat_fragment[pg_hba_rule_allow access to all users]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Config/Postgresql::Server::Pg_hba_rule[allow access to ipv6 localhost]/Concat::Fragment[pg_hba_rule_allow access to ipv6 localhost]/Concat_fragment[pg_hba_rule_allow access to ipv6 localhost]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Service/Anchor[postgresql::server::service::begin]: Skipping because of failed dependencies
        Warning: /Stage[main]/Postgresql::Server::Service/Service[postgresqld]: Skipping because of failed dependencies
        Info: Class[Postgresql::Server]: Unscheduling all events on Class[Postgresql::Server]
        Warning: /Stage[main]/Postgresql::Server::Service/Postgresql_conn_validator[validate_service_is_running]: Skipping because of failed dependencies
          Warning: /Stage[main]/Postgresql::Server::Service/Anchor[postgresql::server::service::end]: Skipping because of failed dependencies
          Warning: /Stage[main]/Postgresql::Server::Reload/Exec[postgresql_reload]: Skipping because of failed dependencies
        Info: Class[Foreman::Database::Postgresql]: Scheduling refresh of Postgresql::Server::Db[foreman]
        Info: Postgresql::Server::Db[foreman]: Scheduling refresh of Postgresql::Server::Database[foreman]
        Info: Postgresql::Server::Db[foreman]: Scheduling refresh of Postgresql::Server::Role[foreman]
        Info: Postgresql::Server::Db[foreman]: Scheduling refresh of Postgresql::Server::Database_grant[GRANT foreman - ALL - foreman]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[CREATE ROLE foreman ENCRYPTED PASSWORD ****]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" NOSUPERUSER]
          Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" NOCREATEDB]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" NOCREATEROLE]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" LOGIN]
          Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" INHERIT]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" NOREPLICATION]
        Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE "foreman" CONNECTION LIMIT -1]
          Info: Postgresql::Server::Role[foreman]: Scheduling refresh of Postgresql_psql[ALTER ROLE foreman ENCRYPTED PASSWORD ****]
        Info: Postgresql::Server::Database_grant[GRANT foreman - ALL - foreman]: Scheduling refresh of Postgresql::Server::Grant[database:GRANT foreman - ALL - foreman]
        Error: /Stage[main]/Foreman::Database::Postgresql/Postgresql::Server::Db[foreman]/Postgresql::Server::Role[foreman]/Postgresql_psql[CREATE ROLE foreman ENCRYPTED PASSWORD ****]: Could not evaluate: Error evaluating 'unless' clause, returned pid 1676 exit 2: 'psql: error: could not connect to server: No such file or directory
                Is the server running locally and accepting
                connections on Unix domain socket "/var/run/postgresql/.s.PGSQL.5432"?
          '
    
    As can be seen, various resources are correctly skipped due to failed
    dependencies but Postgresql_psql isn't.
    
    This changes it to an autorequire on the service. This should work since
    the name of the service resource is static.
    
    There is an edge case where the service isn't managed. For that the
    anchor is also auto-required. This gives less guarantee, but it's a
    best-effort for those users.
    
    [1]: #950 (comment)
    
    Fixes: d69ff1e
    ekohl committed May 27, 2021
    Configuration menu
    Copy the full SHA
    79fa355 View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2021

  1. (maint) pdksync - PDK Update

    Daniel Carabas committed Jun 7, 2021
    Configuration menu
    Copy the full SHA
    d7bd677 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2021

  1. Merge pull request #1278 from puppetlabs/pdksync_pdkupdate

    pdksync - (maint) PDKSync - PDK Update
    adrianiurca authored Jun 8, 2021
    Configuration menu
    Copy the full SHA
    37ced4c View commit details
    Browse the repository at this point in the history

Commits on Jun 11, 2021

  1. Configuration menu
    Copy the full SHA
    c96b073 View commit details
    Browse the repository at this point in the history

Commits on Jun 14, 2021

  1. Merge pull request #1267 from opus-codium/default-privileges

    MODULES-11049 - Implement default privileges changes
    sheenaajay authored Jun 14, 2021
    Configuration menu
    Copy the full SHA
    34d933a View commit details
    Browse the repository at this point in the history
  2. Merge pull request #1276 from ekohl/autorequire-service

    (MODULES-8700) Autorequire the service in postgresql_psql
    sheenaajay authored Jun 14, 2021
    Configuration menu
    Copy the full SHA
    a46036b View commit details
    Browse the repository at this point in the history

Commits on Jun 28, 2021

  1. Configuration menu
    Copy the full SHA
    b21cb9e View commit details
    Browse the repository at this point in the history

Commits on Jun 29, 2021

  1. Merge pull request #1281 from puppetlabs/updatetests

    (maint) update test with the changes from Autorequire the service
    adrianiurca authored Jun 29, 2021
    Configuration menu
    Copy the full SHA
    9f3c50e View commit details
    Browse the repository at this point in the history

Commits on Jun 30, 2021

  1. Configuration menu
    Copy the full SHA
    bcc1181 View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2021

  1. Configuration menu
    Copy the full SHA
    0e3e17a View commit details
    Browse the repository at this point in the history

Commits on Jul 5, 2021

  1. Merge pull request #1280 from pmcmaw/maint/main/update_changelog_config

    (maint) Updating changelog generator config
    sheenaajay authored Jul 5, 2021
    Configuration menu
    Copy the full SHA
    fca2052 View commit details
    Browse the repository at this point in the history

Commits on Jul 20, 2021

  1. Configuration menu
    Copy the full SHA
    e08a303 View commit details
    Browse the repository at this point in the history

Commits on Jul 21, 2021

  1. Merge pull request #1283 from puppetlabs/pdksync_iac-1149

    pdksync - (maint) - Updating Contribution Guidelines
    daianamezdrea authored Jul 21, 2021
    Configuration menu
    Copy the full SHA
    ffe418c View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2021

  1. Merge pull request #1282 from weastur/repo

    Do not add version component to repo definition
    david22swan authored Jul 26, 2021
    Configuration menu
    Copy the full SHA
    99a3fae View commit details
    Browse the repository at this point in the history

Commits on Aug 3, 2021

  1. Configuration menu
    Copy the full SHA
    15a4318 View commit details
    Browse the repository at this point in the history
  2. Release prep v7.3.0

    GitHub Action committed Aug 3, 2021
    Configuration menu
    Copy the full SHA
    b429781 View commit details
    Browse the repository at this point in the history
  3. Merge pull request #1285 from david22swan/maint/main/max_issues

    (maint) - Set max_issues for changelog generator to 500
    sheenaajay authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    a9779c1 View commit details
    Browse the repository at this point in the history
  4. Merge pull request #1287 from puppetlabs/release-prep

    Release prep v7.3.0
    sheenaajay authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    7f3ac0c View commit details
    Browse the repository at this point in the history
Loading