puppet 6.24.0-x64-mingw32 → 7.0.0-x64-mingw32
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of puppet might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/CODEOWNERS +16 -2
- data/CONTRIBUTING.md +5 -5
- data/Gemfile +1 -3
- data/Gemfile.lock +35 -47
- data/README.md +5 -5
- data/conf/fileserver.conf +5 -10
- data/ext/build_defaults.yaml +1 -1
- data/ext/osx/file_mapping.yaml +0 -5
- data/ext/osx/puppet.plist +0 -2
- data/ext/project_data.yaml +1 -15
- data/ext/redhat/puppet.spec.erb +0 -1
- data/ext/windows/service/daemon.rb +6 -5
- data/install.rb +21 -17
- data/lib/puppet.rb +14 -23
- data/lib/puppet/application.rb +178 -108
- data/lib/puppet/application/agent.rb +4 -12
- data/lib/puppet/application/apply.rb +2 -4
- data/lib/puppet/application/device.rb +100 -106
- data/lib/puppet/application/filebucket.rb +13 -10
- data/lib/puppet/application/resource.rb +3 -17
- data/lib/puppet/application/script.rb +0 -2
- data/lib/puppet/application/ssl.rb +1 -13
- data/lib/puppet/application_support.rb +0 -7
- data/lib/puppet/configurer.rb +30 -45
- data/lib/puppet/configurer/downloader.rb +1 -2
- data/lib/puppet/configurer/plugin_handler.rb +21 -19
- data/lib/puppet/defaults.rb +100 -192
- data/lib/puppet/environments.rb +60 -84
- data/lib/puppet/face/facts.rb +5 -103
- data/lib/puppet/face/help.rb +1 -1
- data/lib/puppet/face/help/action.erb +0 -1
- data/lib/puppet/face/help/face.erb +0 -1
- data/lib/puppet/face/node/clean.rb +0 -11
- data/lib/puppet/face/plugin.rb +5 -8
- data/lib/puppet/ffi/windows.rb +12 -0
- data/lib/puppet/ffi/windows/api_types.rb +311 -0
- data/lib/puppet/ffi/windows/constants.rb +404 -0
- data/lib/puppet/ffi/windows/functions.rb +628 -0
- data/lib/puppet/ffi/windows/structs.rb +338 -0
- data/lib/puppet/file_serving/configuration.rb +0 -5
- data/lib/puppet/file_serving/configuration/parser.rb +3 -32
- data/lib/puppet/file_serving/fileset.rb +2 -14
- data/lib/puppet/file_serving/http_metadata.rb +1 -1
- data/lib/puppet/file_serving/mount.rb +1 -2
- data/lib/puppet/file_system/file_impl.rb +1 -1
- data/lib/puppet/file_system/memory_file.rb +1 -8
- data/lib/puppet/file_system/windows.rb +2 -4
- data/lib/puppet/forge.rb +3 -3
- data/lib/puppet/forge/repository.rb +0 -1
- data/lib/puppet/functions/all.rb +1 -1
- data/lib/puppet/functions/camelcase.rb +1 -1
- data/lib/puppet/functions/capitalize.rb +2 -2
- data/lib/puppet/functions/downcase.rb +2 -2
- data/lib/puppet/functions/empty.rb +0 -8
- data/lib/puppet/functions/get.rb +5 -5
- data/lib/puppet/functions/group_by.rb +5 -13
- data/lib/puppet/functions/lest.rb +1 -1
- data/lib/puppet/functions/new.rb +100 -100
- data/lib/puppet/functions/partition.rb +4 -12
- data/lib/puppet/functions/require.rb +5 -5
- data/lib/puppet/functions/sort.rb +3 -3
- data/lib/puppet/functions/strftime.rb +0 -1
- data/lib/puppet/functions/tree_each.rb +9 -7
- data/lib/puppet/functions/type.rb +4 -4
- data/lib/puppet/functions/unwrap.rb +2 -17
- data/lib/puppet/functions/upcase.rb +2 -2
- data/lib/puppet/generate/models/type/type.rb +4 -1
- data/lib/puppet/http.rb +22 -13
- data/lib/puppet/http/client.rb +164 -114
- data/lib/puppet/{network/resolver.rb → http/dns.rb} +2 -2
- data/lib/puppet/http/errors.rb +16 -0
- data/lib/puppet/http/external_client.rb +5 -7
- data/lib/puppet/{network/http → http}/factory.rb +8 -15
- data/lib/puppet/{network/http → http}/pool.rb +61 -26
- data/lib/puppet/{network/http/session.rb → http/pool_entry.rb} +2 -3
- data/lib/puppet/http/proxy.rb +137 -0
- data/lib/puppet/http/redirector.rb +4 -12
- data/lib/puppet/http/resolver.rb +5 -15
- data/lib/puppet/http/resolver/server_list.rb +10 -25
- data/lib/puppet/http/resolver/settings.rb +4 -7
- data/lib/puppet/http/resolver/srv.rb +7 -11
- data/lib/puppet/http/response.rb +36 -54
- data/lib/puppet/http/response_converter.rb +24 -0
- data/lib/puppet/http/response_net_http.rb +42 -0
- data/lib/puppet/http/retry_after_handler.rb +4 -13
- data/lib/puppet/http/service.rb +12 -26
- data/lib/puppet/http/service/ca.rb +11 -22
- data/lib/puppet/http/service/compiler.rb +22 -138
- data/lib/puppet/http/service/file_server.rb +19 -29
- data/lib/puppet/http/service/puppetserver.rb +26 -12
- data/lib/puppet/http/service/report.rb +8 -10
- data/lib/puppet/http/session.rb +11 -20
- data/lib/puppet/{network/http → http}/site.rb +1 -2
- data/lib/puppet/indirector/catalog/compiler.rb +0 -1
- data/lib/puppet/indirector/catalog/rest.rb +2 -4
- data/lib/puppet/indirector/facts/rest.rb +3 -22
- data/lib/puppet/indirector/file_bucket_file/rest.rb +3 -9
- data/lib/puppet/indirector/file_content/rest.rb +2 -6
- data/lib/puppet/indirector/file_metadata/rest.rb +3 -10
- data/lib/puppet/indirector/file_server.rb +1 -8
- data/lib/puppet/indirector/generic_http.rb +0 -11
- data/lib/puppet/indirector/node/rest.rb +2 -4
- data/lib/puppet/indirector/report/rest.rb +3 -8
- data/lib/puppet/indirector/request.rb +0 -101
- data/lib/puppet/indirector/resource/ral.rb +1 -6
- data/lib/puppet/indirector/rest.rb +12 -263
- data/lib/puppet/interface/documentation.rb +0 -1
- data/lib/puppet/module_tool/applications.rb +0 -1
- data/lib/puppet/module_tool/applications/installer.rb +2 -52
- data/lib/puppet/module_tool/errors/shared.rb +2 -34
- data/lib/puppet/network/authconfig.rb +2 -96
- data/lib/puppet/network/authorization.rb +13 -35
- data/lib/puppet/network/formats.rb +0 -67
- data/lib/puppet/network/http.rb +3 -3
- data/lib/puppet/network/http/api/indirected_routes.rb +2 -20
- data/lib/puppet/network/http/api/master/v3.rb +11 -13
- data/lib/puppet/network/http/connection.rb +247 -316
- data/lib/puppet/network/http/handler.rb +0 -1
- data/lib/puppet/network/http_pool.rb +16 -34
- data/lib/puppet/node.rb +1 -30
- data/lib/puppet/pal/json_catalog_encoder.rb +4 -0
- data/lib/puppet/pal/pal_impl.rb +3 -1
- data/lib/puppet/parser/ast/leaf.rb +2 -3
- data/lib/puppet/parser/ast/pops_bridge.rb +0 -38
- data/lib/puppet/parser/compiler.rb +0 -198
- data/lib/puppet/parser/compiler/catalog_validator/relationship_validator.rb +14 -39
- data/lib/puppet/parser/functions/fqdn_rand.rb +6 -14
- data/lib/puppet/parser/resource.rb +0 -69
- data/lib/puppet/parser/templatewrapper.rb +1 -1
- data/lib/puppet/pops/evaluator/deferred_resolver.rb +3 -5
- data/lib/puppet/pops/evaluator/evaluator_impl.rb +0 -5
- data/lib/puppet/pops/evaluator/runtime3_resource_support.rb +3 -3
- data/lib/puppet/pops/evaluator/runtime3_support.rb +1 -1
- data/lib/puppet/pops/issues.rb +0 -5
- data/lib/puppet/pops/loader/ruby_legacy_function_instantiator.rb +6 -8
- data/lib/puppet/pops/model/ast.pp +0 -42
- data/lib/puppet/pops/model/ast.rb +0 -290
- data/lib/puppet/pops/model/ast_transformer.rb +1 -1
- data/lib/puppet/pops/model/factory.rb +0 -45
- data/lib/puppet/pops/model/model_label_provider.rb +0 -5
- data/lib/puppet/pops/model/model_tree_dumper.rb +0 -22
- data/lib/puppet/pops/model/pn_transformer.rb +0 -16
- data/lib/puppet/pops/parser/egrammar.ra +0 -56
- data/lib/puppet/pops/parser/eparser.rb +1520 -1712
- data/lib/puppet/pops/parser/lexer2.rb +4 -4
- data/lib/puppet/pops/parser/parser_support.rb +0 -5
- data/lib/puppet/pops/resource/resource_type_impl.rb +2 -24
- data/lib/puppet/pops/types/p_sem_ver_type.rb +2 -8
- data/lib/puppet/pops/types/p_sensitive_type.rb +0 -10
- data/lib/puppet/pops/types/type_calculator.rb +0 -7
- data/lib/puppet/pops/types/type_mismatch_describer.rb +1 -1
- data/lib/puppet/pops/types/type_parser.rb +0 -4
- data/lib/puppet/pops/types/types.rb +0 -1
- data/lib/puppet/pops/validation/checker4_0.rb +9 -37
- data/lib/puppet/pops/validation/tasks_checker.rb +0 -12
- data/lib/puppet/pops/validation/validator_factory_4_0.rb +1 -2
- data/lib/puppet/property/list.rb +1 -1
- data/lib/puppet/provider.rb +0 -13
- data/lib/puppet/provider/exec/posix.rb +4 -16
- data/lib/puppet/provider/group/groupadd.rb +8 -13
- data/lib/puppet/provider/nameservice.rb +0 -18
- data/lib/puppet/provider/package/apt.rb +2 -34
- data/lib/puppet/provider/package/aptitude.rb +0 -6
- data/lib/puppet/provider/package/dnfmodule.rb +1 -1
- data/lib/puppet/provider/package/dpkg.rb +0 -10
- data/lib/puppet/provider/package/gem.rb +23 -3
- data/lib/puppet/provider/package/nim.rb +6 -11
- data/lib/puppet/provider/package/pip.rb +3 -16
- data/lib/puppet/provider/package/pkg.rb +0 -4
- data/lib/puppet/provider/package/portage.rb +1 -1
- data/lib/puppet/provider/package/puppet_gem.rb +1 -4
- data/lib/puppet/provider/parsedfile.rb +0 -3
- data/lib/puppet/provider/service/debian.rb +0 -2
- data/lib/puppet/provider/service/smf.rb +191 -73
- data/lib/puppet/provider/service/systemd.rb +4 -14
- data/lib/puppet/provider/service/windows.rb +0 -38
- data/lib/puppet/provider/user/aix.rb +2 -2
- data/lib/puppet/provider/user/directoryservice.rb +10 -33
- data/lib/puppet/provider/user/useradd.rb +8 -62
- data/lib/puppet/reference/configuration.rb +8 -7
- data/lib/puppet/reference/indirection.rb +1 -1
- data/lib/puppet/resource.rb +1 -89
- data/lib/puppet/resource/catalog.rb +1 -14
- data/lib/puppet/resource/type.rb +3 -119
- data/lib/puppet/resource/type_collection.rb +3 -48
- data/lib/puppet/runtime.rb +1 -2
- data/lib/puppet/settings.rb +80 -96
- data/lib/puppet/settings/environment_conf.rb +0 -1
- data/lib/puppet/settings/integer_setting.rb +17 -0
- data/lib/puppet/settings/port_setting.rb +15 -0
- data/lib/puppet/settings/priority_setting.rb +5 -4
- data/lib/puppet/ssl.rb +10 -6
- data/lib/puppet/ssl/base.rb +3 -5
- data/lib/puppet/ssl/certificate.rb +0 -6
- data/lib/puppet/ssl/certificate_request.rb +1 -12
- data/lib/puppet/ssl/certificate_signer.rb +6 -0
- data/lib/puppet/ssl/oids.rb +3 -1
- data/lib/puppet/ssl/ssl_provider.rb +17 -0
- data/lib/puppet/ssl/state_machine.rb +3 -1
- data/lib/puppet/ssl/verifier.rb +2 -0
- data/lib/puppet/test/test_helper.rb +1 -3
- data/lib/puppet/transaction.rb +1 -7
- data/lib/puppet/transaction/additional_resource_generator.rb +1 -1
- data/lib/puppet/transaction/report.rb +2 -4
- data/lib/puppet/type.rb +0 -76
- data/lib/puppet/type/exec.rb +3 -16
- data/lib/puppet/type/file.rb +6 -26
- data/lib/puppet/type/file/checksum.rb +1 -1
- data/lib/puppet/type/file/mode.rb +0 -6
- data/lib/puppet/type/file/selcontext.rb +1 -1
- data/lib/puppet/type/file/source.rb +1 -1
- data/lib/puppet/type/filebucket.rb +3 -3
- data/lib/puppet/type/package.rb +8 -16
- data/lib/puppet/type/service.rb +38 -18
- data/lib/puppet/type/tidy.rb +3 -22
- data/lib/puppet/type/user.rb +20 -38
- data/lib/puppet/util/autoload.rb +8 -1
- data/lib/puppet/util/execution.rb +0 -11
- data/lib/puppet/util/http_proxy.rb +2 -215
- data/lib/puppet/util/monkey_patches.rb +0 -53
- data/lib/puppet/util/posix.rb +5 -54
- data/lib/puppet/util/rdoc.rb +0 -7
- data/lib/puppet/util/retry_action.rb +1 -1
- data/lib/puppet/util/run_mode.rb +9 -1
- data/lib/puppet/util/selinux.rb +4 -30
- data/lib/puppet/util/symbolic_file_mode.rb +17 -29
- data/lib/puppet/util/windows.rb +3 -8
- data/lib/puppet/util/windows/adsi.rb +0 -46
- data/lib/puppet/util/windows/daemon.rb +360 -0
- data/lib/puppet/util/windows/error.rb +1 -0
- data/lib/puppet/util/windows/eventlog.rb +4 -9
- data/lib/puppet/util/windows/file.rb +8 -242
- data/lib/puppet/util/windows/monkey_patches/process.rb +414 -0
- data/lib/puppet/util/windows/principal.rb +2 -9
- data/lib/puppet/util/windows/process.rb +4 -226
- data/lib/puppet/util/windows/service.rb +9 -460
- data/lib/puppet/util/windows/sid.rb +2 -6
- data/lib/puppet/util/windows/string.rb +12 -13
- data/lib/puppet/util/yaml.rb +0 -22
- data/lib/puppet/vendor/require_vendored.rb +0 -1
- data/lib/puppet/version.rb +1 -1
- data/lib/puppet/x509.rb +5 -1
- data/lib/puppet/x509/cert_provider.rb +29 -1
- data/locales/puppet.pot +722 -1527
- data/man/man5/puppet.conf.5 +266 -354
- data/man/man8/puppet-agent.8 +2 -2
- data/man/man8/puppet-apply.8 +2 -2
- data/man/man8/puppet-catalog.8 +9 -9
- data/man/man8/puppet-config.8 +1 -1
- data/man/man8/puppet-describe.8 +1 -1
- data/man/man8/puppet-device.8 +2 -2
- data/man/man8/puppet-doc.8 +1 -1
- data/man/man8/puppet-epp.8 +1 -1
- data/man/man8/puppet-facts.8 +8 -51
- data/man/man8/puppet-filebucket.8 +4 -4
- data/man/man8/puppet-generate.8 +1 -1
- data/man/man8/puppet-help.8 +1 -1
- data/man/man8/puppet-lookup.8 +1 -1
- data/man/man8/puppet-module.8 +1 -58
- data/man/man8/puppet-node.8 +5 -5
- data/man/man8/puppet-parser.8 +1 -1
- data/man/man8/puppet-plugin.8 +1 -1
- data/man/man8/puppet-report.8 +5 -5
- data/man/man8/puppet-resource.8 +1 -1
- data/man/man8/puppet-script.8 +2 -2
- data/man/man8/puppet-ssl.8 +1 -5
- data/man/man8/puppet.8 +2 -2
- data/spec/fixtures/ssl/127.0.0.1-key.pem +57 -107
- data/spec/fixtures/ssl/127.0.0.1.pem +31 -52
- data/spec/fixtures/ssl/bad-basic-constraints.pem +35 -57
- data/spec/fixtures/ssl/bad-int-basic-constraints.pem +35 -57
- data/spec/fixtures/ssl/ca.pem +35 -57
- data/spec/fixtures/ssl/crl.pem +18 -28
- data/spec/fixtures/ssl/ec-key.pem +11 -11
- data/spec/fixtures/ssl/ec.pem +24 -33
- data/spec/fixtures/ssl/encrypted-ec-key.pem +12 -12
- data/spec/fixtures/ssl/encrypted-key.pem +58 -108
- data/spec/fixtures/ssl/intermediate-agent-crl.pem +19 -28
- data/spec/fixtures/ssl/intermediate-agent.pem +36 -57
- data/spec/fixtures/ssl/intermediate-crl.pem +21 -31
- data/spec/fixtures/ssl/intermediate.pem +36 -57
- data/spec/fixtures/ssl/pluto-key.pem +57 -107
- data/spec/fixtures/ssl/pluto.pem +30 -52
- data/spec/fixtures/ssl/request-key.pem +57 -107
- data/spec/fixtures/ssl/request.pem +26 -47
- data/spec/fixtures/ssl/revoked-key.pem +57 -107
- data/spec/fixtures/ssl/revoked.pem +30 -52
- data/spec/fixtures/ssl/signed-key.pem +57 -107
- data/spec/fixtures/ssl/signed.pem +30 -52
- data/spec/fixtures/ssl/tampered-cert.pem +30 -52
- data/spec/fixtures/ssl/tampered-csr.pem +26 -47
- data/spec/fixtures/ssl/unknown-127.0.0.1-key.pem +57 -107
- data/spec/fixtures/ssl/unknown-127.0.0.1.pem +29 -50
- data/spec/fixtures/ssl/unknown-ca-key.pem +57 -107
- data/spec/fixtures/ssl/unknown-ca.pem +33 -55
- data/spec/fixtures/unit/provider/service/smf/{svcs.out → svcs_instances.out} +0 -0
- data/spec/fixtures/unit/provider/user/aix/aix_passwd_file.out +0 -4
- data/spec/integration/application/agent_spec.rb +27 -171
- data/spec/integration/application/apply_spec.rb +1 -20
- data/spec/integration/application/filebucket_spec.rb +16 -27
- data/spec/integration/application/help_spec.rb +2 -0
- data/spec/integration/application/module_spec.rb +0 -21
- data/spec/integration/application/plugin_spec.rb +24 -2
- data/spec/integration/defaults_spec.rb +14 -3
- data/spec/integration/environments/settings_interpolation_spec.rb +4 -0
- data/spec/integration/http/client_spec.rb +0 -12
- data/spec/integration/indirector/direct_file_server_spec.rb +3 -1
- data/spec/integration/indirector/facts/facter_spec.rb +36 -90
- data/spec/integration/network/http_pool_spec.rb +3 -21
- data/spec/integration/parser/catalog_spec.rb +0 -38
- data/spec/integration/parser/node_spec.rb +0 -9
- data/spec/integration/parser/pcore_resource_spec.rb +0 -37
- data/spec/integration/resource/type_collection_spec.rb +6 -2
- data/spec/integration/transaction_spec.rb +9 -4
- data/spec/integration/type/exec_spec.rb +45 -70
- data/spec/integration/type/file_spec.rb +5 -4
- data/spec/integration/util/windows/adsi_spec.rb +1 -21
- data/spec/integration/util/windows/monkey_patches/process_spec.rb +231 -0
- data/spec/integration/util/windows/principal_spec.rb +0 -21
- data/spec/integration/util/windows/registry_spec.rb +10 -6
- data/spec/integration/util/windows/security_spec.rb +1 -1
- data/spec/lib/matchers/include.rb +27 -0
- data/spec/lib/matchers/include_spec.rb +32 -0
- data/spec/lib/puppet/test_ca.rb +2 -7
- data/spec/lib/puppet_spec/puppetserver.rb +1 -1
- data/spec/lib/puppet_spec/settings.rb +1 -1
- data/spec/spec_helper.rb +7 -12
- data/spec/unit/agent_spec.rb +6 -10
- data/spec/unit/application/agent_spec.rb +3 -7
- data/spec/unit/application/facts_spec.rb +12 -456
- data/spec/unit/application/filebucket_spec.rb +43 -39
- data/spec/unit/application/ssl_spec.rb +2 -25
- data/spec/unit/application_spec.rb +9 -51
- data/spec/unit/certificate_factory_spec.rb +1 -1
- data/spec/unit/configurer/downloader_spec.rb +6 -8
- data/spec/unit/configurer/plugin_handler_spec.rb +56 -18
- data/spec/unit/configurer_spec.rb +12 -32
- data/spec/unit/confine/feature_spec.rb +1 -1
- data/spec/unit/confine_spec.rb +2 -8
- data/spec/unit/context/trusted_information_spec.rb +2 -6
- data/spec/unit/defaults_spec.rb +68 -54
- data/spec/unit/environments_spec.rb +68 -259
- data/spec/unit/face/node_spec.rb +11 -0
- data/spec/unit/face/plugin_spec.rb +73 -33
- data/spec/unit/file_bucket/file_spec.rb +1 -1
- data/spec/unit/file_serving/configuration/parser_spec.rb +15 -18
- data/spec/unit/file_serving/configuration_spec.rb +6 -12
- data/spec/unit/file_serving/fileset_spec.rb +0 -60
- data/spec/unit/file_serving/metadata_spec.rb +3 -3
- data/spec/unit/file_serving/terminus_helper_spec.rb +4 -11
- data/spec/unit/file_system_spec.rb +0 -15
- data/spec/unit/forge/module_release_spec.rb +7 -2
- data/spec/unit/functions/assert_type_spec.rb +1 -1
- data/spec/unit/functions/camelcase_spec.rb +1 -1
- data/spec/unit/functions/capitalize_spec.rb +1 -1
- data/spec/unit/functions/downcase_spec.rb +1 -1
- data/spec/unit/functions/empty_spec.rb +0 -10
- data/spec/unit/functions/unwrap_spec.rb +0 -8
- data/spec/unit/functions/upcase_spec.rb +1 -1
- data/spec/unit/functions4_spec.rb +2 -2
- data/spec/unit/gettext/config_spec.rb +0 -12
- data/spec/unit/http/client_spec.rb +7 -8
- data/spec/unit/{network/resolver_spec.rb → http/dns_spec.rb} +3 -3
- data/spec/unit/http/external_client_spec.rb +4 -4
- data/spec/unit/{network/http → http}/factory_spec.rb +5 -30
- data/spec/unit/{network/http/session_spec.rb → http/pool_entry_spec.rb} +3 -3
- data/spec/unit/{network/http → http}/pool_spec.rb +12 -17
- data/spec/unit/{util/http_proxy_spec.rb → http/proxy_spec.rb} +2 -69
- data/spec/unit/http/resolver_spec.rb +13 -13
- data/spec/unit/http/service/compiler_spec.rb +0 -185
- data/spec/unit/http/service/file_server_spec.rb +3 -3
- data/spec/unit/http/service/puppetserver_spec.rb +34 -4
- data/spec/unit/http/service_spec.rb +0 -1
- data/spec/unit/http/session_spec.rb +16 -14
- data/spec/unit/{network/http → http}/site_spec.rb +3 -3
- data/spec/unit/indirector/catalog/compiler_spec.rb +10 -14
- data/spec/unit/indirector/face_spec.rb +1 -0
- data/spec/unit/indirector/facts/facter_spec.rb +3 -0
- data/spec/unit/indirector/file_bucket_file/file_spec.rb +5 -3
- data/spec/unit/indirector/file_bucket_file/selector_spec.rb +8 -26
- data/spec/unit/indirector/file_content/rest_spec.rb +0 -4
- data/spec/unit/indirector/file_metadata/rest_spec.rb +0 -4
- data/spec/unit/indirector/file_server_spec.rb +1 -15
- data/spec/unit/indirector/indirection_spec.rb +12 -8
- data/spec/unit/indirector/report/rest_spec.rb +2 -17
- data/spec/unit/indirector/request_spec.rb +0 -264
- data/spec/unit/indirector/resource/ral_spec.rb +75 -40
- data/spec/unit/indirector/rest_spec.rb +98 -752
- data/spec/unit/indirector_spec.rb +2 -2
- data/spec/unit/module_tool/applications/installer_spec.rb +0 -78
- data/spec/unit/network/authconfig_spec.rb +2 -129
- data/spec/unit/network/authorization_spec.rb +2 -55
- data/spec/unit/network/formats_spec.rb +4 -45
- data/spec/unit/network/http/api/indirected_routes_spec.rb +5 -92
- data/spec/unit/network/http/api/master/v3_spec.rb +28 -7
- data/spec/unit/network/http/api_spec.rb +10 -0
- data/spec/unit/network/http/connection_spec.rb +19 -41
- data/spec/unit/network/http/handler_spec.rb +0 -1
- data/spec/unit/network/http_pool_spec.rb +0 -4
- data/spec/unit/node/environment_spec.rb +33 -21
- data/spec/unit/node_spec.rb +2 -54
- data/spec/unit/parser/compiler_spec.rb +19 -3
- data/spec/unit/parser/functions/create_resources_spec.rb +2 -20
- data/spec/unit/parser/functions/fqdn_rand_spec.rb +1 -15
- data/spec/unit/parser/resource_spec.rb +8 -14
- data/spec/unit/parser/templatewrapper_spec.rb +5 -16
- data/spec/unit/pops/evaluator/evaluating_parser_spec.rb +4 -7
- data/spec/unit/pops/loaders/loaders_spec.rb +6 -21
- data/spec/unit/pops/parser/parse_application_spec.rb +4 -22
- data/spec/unit/pops/parser/parse_basic_expressions_spec.rb +0 -1
- data/spec/unit/pops/parser/parse_capabilities_spec.rb +8 -21
- data/spec/unit/pops/parser/parse_site_spec.rb +20 -24
- data/spec/unit/pops/resource/resource_type_impl_spec.rb +0 -71
- data/spec/unit/pops/serialization/to_from_hr_spec.rb +1 -1
- data/spec/unit/pops/types/p_sem_ver_type_spec.rb +0 -18
- data/spec/unit/pops/types/p_sensitive_type_spec.rb +0 -18
- data/spec/unit/pops/types/type_calculator_spec.rb +6 -6
- data/spec/unit/pops/types/type_factory_spec.rb +1 -1
- data/spec/unit/pops/validator/validator_spec.rb +61 -46
- data/spec/unit/pops/visitor_spec.rb +1 -1
- data/spec/unit/property_spec.rb +0 -1
- data/spec/unit/provider/group/groupadd_spec.rb +2 -5
- data/spec/unit/provider/nameservice_spec.rb +64 -122
- data/spec/unit/provider/package/apt_spec.rb +23 -28
- data/spec/unit/provider/package/aptitude_spec.rb +1 -1
- data/spec/unit/provider/package/base_spec.rb +5 -6
- data/spec/unit/provider/package/dnfmodule_spec.rb +1 -10
- data/spec/unit/provider/package/dpkg_spec.rb +0 -48
- data/spec/unit/provider/package/gem_spec.rb +32 -0
- data/spec/unit/provider/package/nim_spec.rb +0 -42
- data/spec/unit/provider/package/pacman_spec.rb +12 -18
- data/spec/unit/provider/package/pip_spec.rb +11 -43
- data/spec/unit/provider/package/pkgdmg_spec.rb +4 -0
- data/spec/unit/provider/package/puppet_gem_spec.rb +3 -2
- data/spec/unit/provider/parsedfile_spec.rb +0 -10
- data/spec/unit/provider/service/init_spec.rb +0 -1
- data/spec/unit/provider/service/openwrt_spec.rb +1 -3
- data/spec/unit/provider/service/smf_spec.rb +401 -165
- data/spec/unit/provider/service/systemd_spec.rb +8 -53
- data/spec/unit/provider/service/windows_spec.rb +0 -203
- data/spec/unit/provider/user/aix_spec.rb +0 -5
- data/spec/unit/provider/user/directoryservice_spec.rb +35 -67
- data/spec/unit/provider/user/hpux_spec.rb +1 -1
- data/spec/unit/provider/user/pw_spec.rb +0 -2
- data/spec/unit/provider/user/useradd_spec.rb +3 -71
- data/spec/unit/provider_spec.rb +8 -18
- data/spec/unit/resource/catalog_spec.rb +1 -1
- data/spec/unit/resource/type_collection_spec.rb +2 -22
- data/spec/unit/resource/type_spec.rb +1 -1
- data/spec/unit/resource_spec.rb +10 -67
- data/spec/unit/settings/http_extra_headers_spec.rb +2 -4
- data/spec/unit/settings/integer_setting_spec.rb +42 -0
- data/spec/unit/settings/port_setting_spec.rb +31 -0
- data/spec/unit/settings/priority_setting_spec.rb +4 -4
- data/spec/unit/settings_spec.rb +79 -110
- data/spec/unit/ssl/base_spec.rb +37 -3
- data/spec/unit/ssl/certificate_request_spec.rb +15 -45
- data/spec/unit/ssl/certificate_spec.rb +2 -11
- data/spec/unit/ssl/ssl_provider_spec.rb +2 -5
- data/spec/unit/ssl/state_machine_spec.rb +5 -20
- data/spec/unit/ssl/verifier_spec.rb +0 -21
- data/spec/unit/transaction/additional_resource_generator_spec.rb +9 -3
- data/spec/unit/transaction/event_manager_spec.rb +11 -14
- data/spec/unit/transaction/report_spec.rb +0 -2
- data/spec/unit/transaction/resource_harness_spec.rb +2 -2
- data/spec/unit/transaction_spec.rb +55 -96
- data/spec/unit/type/exec_spec.rb +29 -76
- data/spec/unit/type/file/checksum_spec.rb +6 -6
- data/spec/unit/type/file/content_spec.rb +2 -1
- data/spec/unit/type/file/ensure_spec.rb +1 -1
- data/spec/unit/type/file/mode_spec.rb +1 -1
- data/spec/unit/type/file/selinux_spec.rb +5 -3
- data/spec/unit/type/file/source_spec.rb +4 -5
- data/spec/unit/type/file_spec.rb +18 -6
- data/spec/unit/type/group_spec.rb +6 -13
- data/spec/unit/type/package_spec.rb +1 -1
- data/spec/unit/type/resources_spec.rb +7 -7
- data/spec/unit/type/service_spec.rb +189 -60
- data/spec/unit/type/tidy_spec.rb +8 -24
- data/spec/unit/type/user_spec.rb +0 -45
- data/spec/unit/type_spec.rb +22 -2
- data/spec/unit/util/at_fork_spec.rb +2 -2
- data/spec/unit/util/autoload_spec.rb +1 -5
- data/spec/unit/util/backups_spec.rb +2 -3
- data/spec/unit/util/execution_spec.rb +11 -44
- data/spec/unit/util/inifile_spec.rb +14 -6
- data/spec/unit/util/log_spec.rb +7 -8
- data/spec/unit/util/logging_spec.rb +3 -3
- data/spec/unit/util/monkey_patches_spec.rb +0 -6
- data/spec/unit/util/posix_spec.rb +15 -363
- data/spec/unit/util/run_mode_spec.rb +21 -121
- data/spec/unit/util/selinux_spec.rb +68 -163
- data/spec/unit/util/storage_spec.rb +1 -3
- data/spec/unit/util/suidmanager_spec.rb +41 -44
- data/spec/unit/util/windows/sid_spec.rb +0 -41
- data/spec/unit/util/windows/string_spec.rb +1 -3
- data/spec/unit/util/yaml_spec.rb +0 -54
- data/spec/unit/util_spec.rb +6 -31
- data/tasks/generate_cert_fixtures.rake +3 -12
- metadata +45 -253
- data/conf/auth.conf +0 -150
- data/lib/puppet/application/cert.rb +0 -76
- data/lib/puppet/application/key.rb +0 -4
- data/lib/puppet/application/man.rb +0 -4
- data/lib/puppet/application/status.rb +0 -4
- data/lib/puppet/face/key.rb +0 -16
- data/lib/puppet/face/man.rb +0 -145
- data/lib/puppet/face/module/build.rb +0 -14
- data/lib/puppet/face/module/generate.rb +0 -14
- data/lib/puppet/face/module/search.rb +0 -103
- data/lib/puppet/face/status.rb +0 -51
- data/lib/puppet/ffi/posix.rb +0 -10
- data/lib/puppet/ffi/posix/constants.rb +0 -14
- data/lib/puppet/ffi/posix/functions.rb +0 -24
- data/lib/puppet/indirector/certificate/file.rb +0 -9
- data/lib/puppet/indirector/certificate/rest.rb +0 -18
- data/lib/puppet/indirector/certificate_request/file.rb +0 -9
- data/lib/puppet/indirector/certificate_request/memory.rb +0 -7
- data/lib/puppet/indirector/certificate_request/rest.rb +0 -11
- data/lib/puppet/indirector/file_content/http.rb +0 -22
- data/lib/puppet/indirector/key/file.rb +0 -46
- data/lib/puppet/indirector/key/memory.rb +0 -7
- data/lib/puppet/indirector/ssl_file.rb +0 -162
- data/lib/puppet/indirector/status.rb +0 -3
- data/lib/puppet/indirector/status/local.rb +0 -12
- data/lib/puppet/indirector/status/rest.rb +0 -27
- data/lib/puppet/module_tool/applications/searcher.rb +0 -29
- data/lib/puppet/network/auth_config_parser.rb +0 -90
- data/lib/puppet/network/authstore.rb +0 -283
- data/lib/puppet/network/http/api/master/v3/authorization.rb +0 -18
- data/lib/puppet/network/http/api/master/v3/environment.rb +0 -88
- data/lib/puppet/network/http/base_pool.rb +0 -36
- data/lib/puppet/network/http/compression.rb +0 -127
- data/lib/puppet/network/http/connection_adapter.rb +0 -184
- data/lib/puppet/network/http/nocache_pool.rb +0 -28
- data/lib/puppet/network/rest_controller.rb +0 -2
- data/lib/puppet/network/rights.rb +0 -210
- data/lib/puppet/parser/compiler/catalog_validator/env_relationship_validator.rb +0 -66
- data/lib/puppet/parser/compiler/catalog_validator/site_validator.rb +0 -22
- data/lib/puppet/parser/environment_compiler.rb +0 -202
- data/lib/puppet/pops/types/enumeration.rb +0 -16
- data/lib/puppet/resource/capability_finder.rb +0 -154
- data/lib/puppet/rest/errors.rb +0 -15
- data/lib/puppet/rest/response.rb +0 -35
- data/lib/puppet/rest/route.rb +0 -85
- data/lib/puppet/rest/routes.rb +0 -135
- data/lib/puppet/settings/alias_setting.rb +0 -37
- data/lib/puppet/ssl/host.rb +0 -505
- data/lib/puppet/ssl/key.rb +0 -61
- data/lib/puppet/ssl/validator.rb +0 -61
- data/lib/puppet/ssl/validator/default_validator.rb +0 -209
- data/lib/puppet/ssl/validator/no_validator.rb +0 -22
- data/lib/puppet/ssl/verifier_adapter.rb +0 -58
- data/lib/puppet/status.rb +0 -40
- data/lib/puppet/util/connection.rb +0 -88
- data/lib/puppet/util/fact_dif.rb +0 -81
- data/lib/puppet/util/ssl.rb +0 -83
- data/lib/puppet/util/windows/api_types.rb +0 -309
- data/lib/puppet/util/windows/monkey_patches/dir.rb +0 -40
- data/lib/puppet/vendor/load_pathspec.rb +0 -1
- data/lib/puppet/vendor/pathspec/CHANGELOG.md +0 -2
- data/lib/puppet/vendor/pathspec/LICENSE +0 -201
- data/lib/puppet/vendor/pathspec/PUPPET_README.md +0 -6
- data/lib/puppet/vendor/pathspec/README.md +0 -53
- data/lib/puppet/vendor/pathspec/lib/pathspec.rb +0 -122
- data/lib/puppet/vendor/pathspec/lib/pathspec/gitignorespec.rb +0 -275
- data/lib/puppet/vendor/pathspec/lib/pathspec/regexspec.rb +0 -17
- data/lib/puppet/vendor/pathspec/lib/pathspec/spec.rb +0 -14
- data/man/man8/puppet-key.8 +0 -126
- data/man/man8/puppet-man.8 +0 -76
- data/man/man8/puppet-status.8 +0 -108
- data/spec/fixtures/integration/application/agent/cached_deferred_catalog.json +0 -91
- data/spec/fixtures/ssl/oid-key.pem +0 -117
- data/spec/fixtures/ssl/oid.pem +0 -69
- data/spec/fixtures/ssl/trusted_oid_mapping.yaml +0 -5
- data/spec/fixtures/unit/provider/service/systemd/list_unit_files_services_vendor_preset +0 -9
- data/spec/integration/application/resource_spec.rb +0 -64
- data/spec/integration/application/ssl_spec.rb +0 -20
- data/spec/integration/network/authconfig_spec.rb +0 -256
- data/spec/integration/util/windows/monkey_patches/dir_spec.rb +0 -11
- data/spec/unit/application/man_spec.rb +0 -52
- data/spec/unit/capability_spec.rb +0 -414
- data/spec/unit/face/key_spec.rb +0 -9
- data/spec/unit/face/module/search_spec.rb +0 -231
- data/spec/unit/face/status_spec.rb +0 -9
- data/spec/unit/indirector/certificate/file_spec.rb +0 -14
- data/spec/unit/indirector/certificate/rest_spec.rb +0 -61
- data/spec/unit/indirector/certificate_request/file_spec.rb +0 -14
- data/spec/unit/indirector/certificate_request/rest_spec.rb +0 -25
- data/spec/unit/indirector/key/file_spec.rb +0 -78
- data/spec/unit/indirector/ssl_file_spec.rb +0 -305
- data/spec/unit/indirector/status/local_spec.rb +0 -10
- data/spec/unit/indirector/status/rest_spec.rb +0 -50
- data/spec/unit/module_tool/applications/searcher_spec.rb +0 -38
- data/spec/unit/network/auth_config_parser_spec.rb +0 -115
- data/spec/unit/network/authstore_spec.rb +0 -422
- data/spec/unit/network/http/api/master/v3/authorization_spec.rb +0 -57
- data/spec/unit/network/http/api/master/v3/environment_spec.rb +0 -185
- data/spec/unit/network/http/compression_spec.rb +0 -240
- data/spec/unit/network/http/nocache_pool_spec.rb +0 -64
- data/spec/unit/network/http_spec.rb +0 -9
- data/spec/unit/network/rights_spec.rb +0 -439
- data/spec/unit/parser/environment_compiler_spec.rb +0 -730
- data/spec/unit/pops/evaluator/deferred_resolver_spec.rb +0 -20
- data/spec/unit/pops/types/enumeration_spec.rb +0 -51
- data/spec/unit/resource/capability_finder_spec.rb +0 -148
- data/spec/unit/rest/route_spec.rb +0 -132
- data/spec/unit/ssl/host_spec.rb +0 -645
- data/spec/unit/ssl/key_spec.rb +0 -173
- data/spec/unit/ssl/validator_spec.rb +0 -278
- data/spec/unit/status_spec.rb +0 -45
- data/spec/unit/util/ssl_spec.rb +0 -91
data/spec/unit/confine_spec.rb
CHANGED
@@ -2,12 +2,6 @@ require 'spec_helper'
|
|
2
2
|
|
3
3
|
require 'puppet/confine'
|
4
4
|
|
5
|
-
class Puppet::TestConfine < Puppet::Confine
|
6
|
-
def pass?(value)
|
7
|
-
false
|
8
|
-
end
|
9
|
-
end
|
10
|
-
|
11
5
|
describe Puppet::Confine do
|
12
6
|
it "should require a value" do
|
13
7
|
expect { Puppet::Confine.new }.to raise_error(ArgumentError)
|
@@ -39,7 +33,7 @@ describe Puppet::Confine do
|
|
39
33
|
|
40
34
|
describe "when testing all values" do
|
41
35
|
before do
|
42
|
-
@confine = Puppet::
|
36
|
+
@confine = Puppet::Confine.new(%w{a b c})
|
43
37
|
@confine.label = "foo"
|
44
38
|
end
|
45
39
|
|
@@ -70,7 +64,7 @@ describe Puppet::Confine do
|
|
70
64
|
end
|
71
65
|
|
72
66
|
describe "when testing the result of the values" do
|
73
|
-
before { @confine = Puppet::
|
67
|
+
before { @confine = Puppet::Confine.new(%w{a b c d}) }
|
74
68
|
|
75
69
|
it "should return an array with the result of the test for each value" do
|
76
70
|
allow(@confine).to receive(:pass?).and_return(true)
|
@@ -4,11 +4,7 @@ require 'puppet/certificate_factory'
|
|
4
4
|
require 'puppet/context/trusted_information'
|
5
5
|
|
6
6
|
describe Puppet::Context::TrustedInformation, :unless => RUBY_PLATFORM == 'java' do
|
7
|
-
let(:key)
|
8
|
-
key = Puppet::SSL::Key.new("myname")
|
9
|
-
key.generate
|
10
|
-
key
|
11
|
-
end
|
7
|
+
let(:key) { OpenSSL::PKey::RSA.new(Puppet[:keylength]) }
|
12
8
|
|
13
9
|
let(:csr) do
|
14
10
|
csr = Puppet::SSL::CertificateRequest.new("csr")
|
@@ -26,7 +22,7 @@ describe Puppet::Context::TrustedInformation, :unless => RUBY_PLATFORM == 'java'
|
|
26
22
|
|
27
23
|
# The cert must be signed so that it can be successfully be DER-decoded later
|
28
24
|
signer = Puppet::SSL::CertificateSigner.new
|
29
|
-
signer.sign(cert.content, key
|
25
|
+
signer.sign(cert.content, key)
|
30
26
|
cert
|
31
27
|
end
|
32
28
|
|
data/spec/unit/defaults_spec.rb
CHANGED
@@ -65,9 +65,9 @@ describe "Defaults" do
|
|
65
65
|
end
|
66
66
|
|
67
67
|
describe '.default_digest_algorithm' do
|
68
|
-
it 'defaults to
|
68
|
+
it 'defaults to sha256 when FIPS is not enabled' do
|
69
69
|
allow(Puppet::Util::Platform).to receive(:fips_enabled?).and_return(false)
|
70
|
-
expect(Puppet.default_digest_algorithm).to eq('
|
70
|
+
expect(Puppet.default_digest_algorithm).to eq('sha256')
|
71
71
|
end
|
72
72
|
|
73
73
|
it 'defaults to sha256 when FIPS is enabled' do
|
@@ -77,9 +77,9 @@ describe "Defaults" do
|
|
77
77
|
end
|
78
78
|
|
79
79
|
describe '.supported_checksum_types' do
|
80
|
-
it 'defaults to
|
80
|
+
it 'defaults to sha256, sha384, sha512, sha224, md5 when FIPS is not enabled' do
|
81
81
|
allow(Puppet::Util::Platform).to receive(:fips_enabled?).and_return(false)
|
82
|
-
expect(Puppet.default_file_checksum_types).to eq(%w[
|
82
|
+
expect(Puppet.default_file_checksum_types).to eq(%w[sha256 sha384 sha512 sha224 md5])
|
83
83
|
end
|
84
84
|
|
85
85
|
it 'defaults to sha256, sha384, sha512, sha224 when FIPS is enabled' do
|
@@ -89,8 +89,8 @@ describe "Defaults" do
|
|
89
89
|
end
|
90
90
|
|
91
91
|
describe 'Puppet[:supported_checksum_types]' do
|
92
|
-
it 'defaults to
|
93
|
-
expect(Puppet.settings[:supported_checksum_types]).to eq(%w[
|
92
|
+
it 'defaults to sha256, sha512, sha384, sha224, md5' do
|
93
|
+
expect(Puppet.settings[:supported_checksum_types]).to eq(%w[sha256 sha384 sha512 sha224 md5])
|
94
94
|
end
|
95
95
|
|
96
96
|
it 'should raise an error on an unsupported checksum type' do
|
@@ -182,72 +182,86 @@ describe "Defaults" do
|
|
182
182
|
end
|
183
183
|
end
|
184
184
|
|
185
|
-
describe "
|
186
|
-
it
|
187
|
-
expect(Puppet
|
188
|
-
end
|
189
|
-
|
190
|
-
it "raises an exception if facter-ng could not be loaded" do
|
191
|
-
allow_any_instance_of(Puppet::Settings::BooleanSetting).to receive(:require).with('facter-ng').and_raise(LoadError)
|
192
|
-
allow(Facter).to receive(:value).with('facterversion').and_return('3.11.4')
|
185
|
+
describe "deprecated settings" do
|
186
|
+
it 'does not issue a deprecation warning by default' do
|
187
|
+
expect(Puppet).to receive(:deprecation_warning).never
|
193
188
|
|
194
|
-
|
189
|
+
Puppet.initialize_settings
|
195
190
|
end
|
191
|
+
end
|
196
192
|
|
197
|
-
|
198
|
-
|
199
|
-
|
200
|
-
|
201
|
-
|
202
|
-
|
203
|
-
|
204
|
-
|
205
|
-
|
206
|
-
|
207
|
-
|
208
|
-
|
209
|
-
|
210
|
-
|
211
|
-
Object.const_set(:Facter, @original_facter)
|
212
|
-
end
|
193
|
+
describe "the call hook for the cadir setting", unless: Puppet::Util::Platform.windows? do
|
194
|
+
it 'does not warn when the cadir is outside the puppet ssldir' do
|
195
|
+
FileUtils.mkdir_p(Puppet[:confdir])
|
196
|
+
# This is fun; in order to get the confdir setting from the spec_helper to
|
197
|
+
# not be deleted by the #unsafe_clear in Puppet::Settings#parse_config,
|
198
|
+
# we can code it into the confdir itself, and thus we can avoid getting
|
199
|
+
# that setting wiped out during Puppet.initialize_settings.
|
200
|
+
File.write(File.join(Puppet[:confdir], 'puppet.conf'),
|
201
|
+
"cadir = /my/cool/path/for/my/cadir\n
|
202
|
+
confdir = #{Puppet[:confdir]}")
|
203
|
+
expect(Puppet).to_not receive(:log_ca_migration_warning)
|
204
|
+
Puppet.initialize_settings
|
205
|
+
expect(Puppet[:cadir]).to eq("/my/cool/path/for/my/cadir")
|
206
|
+
end
|
213
207
|
|
214
|
-
|
215
|
-
|
216
|
-
|
217
|
-
|
208
|
+
it 'does warn when the cadir is inside the puppet ssldir' do
|
209
|
+
FileUtils.mkdir_p(Puppet[:confdir])
|
210
|
+
cadir_location = File.join(Puppet[:ssldir], 'still_inside_ssldir')
|
211
|
+
# This is fun; in order to get the confdir setting from the spec_helper to
|
212
|
+
# not be deleted by the #unsafe_clear in Puppet::Settings#parse_config,
|
213
|
+
# we can code it into the confdir itself, and thus we can avoid getting
|
214
|
+
# that setting wiped out during Puppet.initialize_settings.
|
215
|
+
File.write(File.join(Puppet[:confdir], 'puppet.conf'),
|
216
|
+
"cadir = #{cadir_location}\n
|
217
|
+
confdir = #{Puppet[:confdir]}")
|
218
|
+
expect(Puppet).to receive(:log_ca_migration_warning).twice
|
219
|
+
Puppet.initialize_settings
|
220
|
+
expect(Puppet[:cadir]).to eq(cadir_location)
|
218
221
|
end
|
219
|
-
end
|
220
222
|
end
|
221
223
|
|
222
|
-
describe "
|
223
|
-
it '
|
224
|
-
expect(Puppet).to receive(:deprecation_warning).never
|
225
|
-
|
224
|
+
describe "the default cadir", :unless => Puppet::Util::Platform.windows? do
|
225
|
+
it 'defaults to the puppetserver confdir' do
|
226
226
|
Puppet.initialize_settings
|
227
|
+
expect(Puppet[:cadir]).to eq('/etc/puppetlabs/puppetserver/ca')
|
227
228
|
end
|
228
229
|
|
229
|
-
it '
|
230
|
-
Puppet[:func3x_check] = false
|
231
|
-
|
232
|
-
expect(Puppet).to receive(:deprecation_warning).with("The 'func3x_check' setting is deprecated and will be removed in a future release.")
|
233
|
-
|
230
|
+
it 'returns an empty string for Windows platforms', :if => Puppet::Util::Platform.windows? do
|
234
231
|
Puppet.initialize_settings
|
232
|
+
expect(Puppet[:cadir]).to eq("")
|
235
233
|
end
|
236
234
|
end
|
237
235
|
|
238
|
-
describe
|
239
|
-
it '
|
240
|
-
expect(
|
241
|
-
expect(Puppet.default_cadir).to eq(
|
236
|
+
describe '#default_cadir', :unless => Puppet::Util::Platform.windows? do
|
237
|
+
it 'returns the new puppetserver directory when no ca dir is present' do
|
238
|
+
expect(Puppet).to_not receive(:log_ca_migration_warning)
|
239
|
+
expect(Puppet.default_cadir).to eq('/etc/puppetlabs/puppetserver/ca')
|
242
240
|
end
|
243
241
|
|
244
|
-
it '
|
245
|
-
|
246
|
-
|
242
|
+
it 'warns when a CA dir exists in the current ssldir' do
|
243
|
+
cadir = File.join(Puppet[:ssldir], 'ca')
|
244
|
+
FileUtils.mkdir_p(cadir)
|
245
|
+
expect(Puppet).to receive(:log_ca_migration_warning)
|
246
|
+
expect(Puppet.default_cadir).to eq(cadir)
|
247
247
|
end
|
248
248
|
|
249
|
-
it '
|
250
|
-
|
249
|
+
it 'warns when the cadir is a symlink still inside the ssldir' do
|
250
|
+
another_dir = File.join(Puppet[:ssldir], 'another_dir')
|
251
|
+
cadir = File.join(Puppet[:ssldir], 'ca')
|
252
|
+
FileUtils.mkdir_p(another_dir)
|
253
|
+
File.symlink(another_dir, cadir)
|
254
|
+
expect(Puppet).to receive(:log_ca_migration_warning)
|
255
|
+
expect(Puppet.default_cadir).to eq(another_dir)
|
256
|
+
end
|
257
|
+
|
258
|
+
it 'does not warn when the cadir is a symlink targeted outside the ssldir' do
|
259
|
+
another_dir = Dir.mktmpdir
|
260
|
+
cadir = File.join(Puppet[:ssldir], 'ca')
|
261
|
+
FileUtils.mkdir_p(Puppet[:ssldir])
|
262
|
+
File.symlink(another_dir, cadir)
|
263
|
+
expect(Puppet).to_not receive(:log_ca_migration_warning)
|
264
|
+
expect(Puppet.default_cadir).to eq(another_dir)
|
251
265
|
end
|
252
266
|
end
|
253
267
|
end
|
@@ -1,17 +1,14 @@
|
|
1
1
|
require 'spec_helper'
|
2
2
|
require 'puppet/environments'
|
3
3
|
require 'puppet/file_system'
|
4
|
+
require 'matchers/include'
|
5
|
+
require 'matchers/include_in_order'
|
4
6
|
|
7
|
+
module PuppetEnvironments
|
5
8
|
describe Puppet::Environments do
|
6
|
-
|
9
|
+
include Matchers::Include
|
7
10
|
|
8
|
-
|
9
|
-
def remove
|
10
|
-
@properties[:directory?] = false
|
11
|
-
@properties[:exist?] = false
|
12
|
-
@properties[:executable?] = false
|
13
|
-
end
|
14
|
-
end
|
11
|
+
FS = Puppet::FileSystem
|
15
12
|
|
16
13
|
before(:each) do
|
17
14
|
Puppet.settings.initialize_global_settings
|
@@ -52,7 +49,7 @@ describe Puppet::Environments do
|
|
52
49
|
loader_from(:filesystem => [directory_tree, global_path_1, global_path_2],
|
53
50
|
:directory => directory_tree.children.first,
|
54
51
|
:modulepath => [global_path_1_location, global_path_2_location]) do |loader|
|
55
|
-
expect(loader.list).to
|
52
|
+
expect(loader.list).to include_in_any_order(
|
56
53
|
environment(:an_environment).
|
57
54
|
with_manifest("#{FS.path_string(directory_tree)}/envdir/an_environment/manifests").
|
58
55
|
with_modulepath(["#{FS.path_string(directory_tree)}/envdir/an_environment/modules",
|
@@ -90,7 +87,7 @@ describe Puppet::Environments do
|
|
90
87
|
|
91
88
|
loader_from(:filesystem => [envdir],
|
92
89
|
:directory => envdir) do |loader|
|
93
|
-
expect(loader.list).to
|
90
|
+
expect(loader.list).to include_in_any_order(environment(:env1), environment(:env2))
|
94
91
|
end
|
95
92
|
end
|
96
93
|
|
@@ -409,29 +406,33 @@ config_version=$vardir/random/scripts
|
|
409
406
|
]),
|
410
407
|
])
|
411
408
|
|
412
|
-
|
413
|
-
|
414
|
-
|
415
|
-
|
416
|
-
|
417
|
-
|
418
|
-
|
419
|
-
|
420
|
-
|
421
|
-
|
422
|
-
|
423
|
-
|
409
|
+
FS.overlay(original_envdir) do
|
410
|
+
dir_loader = Puppet::Environments::Directories.new(original_envdir, [])
|
411
|
+
loader = Puppet::Environments::Cached.new(dir_loader)
|
412
|
+
Puppet.override(:environments => loader) do
|
413
|
+
original_env = loader.get("env3") # force the environment.conf to be read
|
414
|
+
|
415
|
+
changed_envdir = FS::MemoryFile.a_directory(base_dir, [
|
416
|
+
FS::MemoryFile.a_directory("env3", [
|
417
|
+
FS::MemoryFile.a_regular_file_containing("environment.conf", <<-EOF)
|
418
|
+
manifest=/manifest_changed
|
419
|
+
modulepath=/modules_changed
|
420
|
+
environment_timeout=0
|
421
|
+
EOF
|
422
|
+
]),
|
423
|
+
])
|
424
424
|
|
425
|
-
|
426
|
-
|
425
|
+
FS.overlay(changed_envdir) do
|
426
|
+
changed_env = loader.get("env3")
|
427
427
|
|
428
|
-
|
429
|
-
|
430
|
-
|
428
|
+
expect(original_env).to environment(:env3).
|
429
|
+
with_manifest(File.expand_path("/manifest_orig")).
|
430
|
+
with_full_modulepath([File.expand_path("/modules_orig")])
|
431
431
|
|
432
|
-
|
433
|
-
|
434
|
-
|
432
|
+
expect(changed_env).to environment(:env3).
|
433
|
+
with_manifest(File.expand_path("/manifest_changed")).
|
434
|
+
with_full_modulepath([File.expand_path("/modules_changed")])
|
435
|
+
end
|
435
436
|
end
|
436
437
|
end
|
437
438
|
end
|
@@ -557,49 +558,24 @@ config_version=$vardir/random/scripts
|
|
557
558
|
|
558
559
|
describe "cached loaders" do
|
559
560
|
it "lists environments" do
|
560
|
-
|
561
|
-
expect(loader.list).to
|
561
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
562
|
+
expect(Puppet::Environments::Cached.new(loader).list).to include_in_any_order(
|
562
563
|
environment(:an_environment),
|
563
564
|
environment(:another_environment),
|
564
565
|
environment(:symlinked_environment))
|
565
566
|
end
|
566
567
|
end
|
567
568
|
|
568
|
-
it "returns the same cached environment object for list and get methods" do
|
569
|
-
cached_loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
570
|
-
env = loader.list.find { |e| e.name == :an_environment }
|
571
|
-
|
572
|
-
expect(env).to equal(loader.get(:an_environment)) # same object
|
573
|
-
end
|
574
|
-
end
|
575
|
-
|
576
|
-
it "returns the same cached environment object for multiple list calls" do
|
577
|
-
cached_loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
578
|
-
expect(loader.list.first).to equal(loader.list.first) # same object
|
579
|
-
end
|
580
|
-
end
|
581
|
-
|
582
|
-
it "expires environments and returns a new environment object with the same value" do
|
583
|
-
Puppet[:environment_timeout] = "0"
|
584
|
-
|
585
|
-
cached_loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
586
|
-
a = loader.list.first
|
587
|
-
b = loader.list.first
|
588
|
-
expect(a).to eq(b) # same value
|
589
|
-
expect(a).to_not equal(b) # not same object
|
590
|
-
end
|
591
|
-
end
|
592
|
-
|
593
569
|
it "has search_paths" do
|
594
|
-
|
595
|
-
expect(loader.search_paths).to eq(["file://#{directory_tree.children.first}"])
|
570
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
571
|
+
expect(Puppet::Environments::Cached.new(loader).search_paths).to eq(["file://#{directory_tree.children.first}"])
|
596
572
|
end
|
597
573
|
end
|
598
574
|
|
599
575
|
context "#get" do
|
600
576
|
it "gets an environment" do
|
601
|
-
|
602
|
-
expect(loader.get(:an_environment)).to environment(:an_environment)
|
577
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
578
|
+
expect(Puppet::Environments::Cached.new(loader).get(:an_environment)).to environment(:an_environment)
|
603
579
|
end
|
604
580
|
end
|
605
581
|
|
@@ -615,44 +591,17 @@ config_version=$vardir/random/scripts
|
|
615
591
|
cached.get(:cached)
|
616
592
|
end
|
617
593
|
|
618
|
-
it "does not list deleted environments" do
|
619
|
-
env3 = FS::MemoryFile.a_directory("env3", [
|
620
|
-
FS::MemoryFile.a_regular_file_containing("environment.conf", '')
|
621
|
-
])
|
622
|
-
|
623
|
-
envdir = FS::MemoryFile.a_directory(File.expand_path("envdir"), [
|
624
|
-
FS::MemoryFile.a_directory("env1", [
|
625
|
-
FS::MemoryFile.a_regular_file_containing("environment.conf", '')
|
626
|
-
]),
|
627
|
-
FS::MemoryFile.a_directory("env2", [
|
628
|
-
FS::MemoryFile.a_regular_file_containing("environment.conf", '')
|
629
|
-
]),
|
630
|
-
env3
|
631
|
-
])
|
632
|
-
|
633
|
-
loader_from(:filesystem => [envdir], :directory => envdir) do |loader|
|
634
|
-
cached = Puppet::Environments::Cached.new(loader)
|
635
|
-
cached.get(:env1)
|
636
|
-
cached.get(:env2)
|
637
|
-
cached.get(:env3)
|
638
|
-
env3.extend(FsRemove).remove
|
639
|
-
|
640
|
-
expect(cached.list).to contain_exactly(environment(:env1),environment(:env2))
|
641
|
-
expect(cached.get(:env3)).to be_nil
|
642
|
-
end
|
643
|
-
end
|
644
|
-
|
645
594
|
it "returns nil if env not found" do
|
646
|
-
|
647
|
-
expect(loader.get(:doesnotexist)).to be_nil
|
595
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
596
|
+
expect(Puppet::Environments::Cached.new(loader).get(:doesnotexist)).to be_nil
|
648
597
|
end
|
649
598
|
end
|
650
599
|
end
|
651
600
|
|
652
601
|
context "#get!" do
|
653
602
|
it "gets an environment" do
|
654
|
-
|
655
|
-
expect(loader.get!(:an_environment)).to environment(:an_environment)
|
603
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
604
|
+
expect(Puppet::Environments::Cached.new(loader).get!(:an_environment)).to environment(:an_environment)
|
656
605
|
end
|
657
606
|
end
|
658
607
|
|
@@ -669,43 +618,29 @@ config_version=$vardir/random/scripts
|
|
669
618
|
end
|
670
619
|
|
671
620
|
it "raises error if environment is not found" do
|
672
|
-
|
621
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
673
622
|
expect do
|
674
|
-
loader.get!(:doesnotexist)
|
623
|
+
Puppet::Environments::Cached.new(loader).get!(:doesnotexist)
|
675
624
|
end.to raise_error(Puppet::Environments::EnvironmentNotFound)
|
676
625
|
end
|
677
626
|
end
|
678
627
|
end
|
679
628
|
|
680
|
-
context "
|
681
|
-
|
682
|
-
cached_loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
683
|
-
expect(loader.get_conf(:an_environment)).to match_environment_conf(:an_environment).
|
684
|
-
with_env_path(directory_tree.children.first).
|
685
|
-
with_global_module_path([])
|
686
|
-
end
|
687
|
-
end
|
688
|
-
|
689
|
-
it "always reloads environment.conf" do
|
690
|
-
env = Puppet::Node::Environment.create(:cached, [])
|
691
|
-
mocked_loader = double('loader')
|
692
|
-
expect(mocked_loader).to receive(:get_conf).with(:cached).and_return(Puppet::Settings::EnvironmentConf.static_for(env, 20)).twice
|
693
|
-
|
694
|
-
cached = Puppet::Environments::Cached.new(mocked_loader)
|
629
|
+
context "expiration policies" do
|
630
|
+
let(:service) { ReplayExpirationService.new }
|
695
631
|
|
696
|
-
|
697
|
-
|
698
|
-
|
632
|
+
# The environment named `:an_environment` will already be loaded when the
|
633
|
+
# block is yielded to
|
634
|
+
def with_environment_loaded(service, &block)
|
635
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
636
|
+
using_expiration_service(service) do
|
637
|
+
cached = Puppet::Environments::Cached.new(loader)
|
638
|
+
cached.get!(:an_environment)
|
699
639
|
|
700
|
-
|
701
|
-
|
702
|
-
expect(loader.get_conf(:doesnotexist)).to be_nil
|
640
|
+
yield cached if block_given?
|
641
|
+
end
|
703
642
|
end
|
704
643
|
end
|
705
|
-
end
|
706
|
-
|
707
|
-
context "expiration policies" do
|
708
|
-
let(:service) { ReplayExpirationService.new }
|
709
644
|
|
710
645
|
it "notifies when the environment is first created" do
|
711
646
|
with_environment_loaded(service)
|
@@ -725,7 +660,12 @@ config_version=$vardir/random/scripts
|
|
725
660
|
end
|
726
661
|
|
727
662
|
it "evicts an expired environment" do
|
728
|
-
|
663
|
+
service = ReplayExpirationService.new
|
664
|
+
|
665
|
+
# The `Cached#clear_all_expired` method tries to optimize the case where
|
666
|
+
# no entries are expired. But if `Time.now < @next_expiration` and there is
|
667
|
+
# an expired entry, then the `service#expired?` method is called twice.
|
668
|
+
expect(service).to receive(:expired?).twice.and_return(true)
|
729
669
|
|
730
670
|
with_environment_loaded(service) do |cached|
|
731
671
|
cached.get!(:an_environment)
|
@@ -737,7 +677,6 @@ config_version=$vardir/random/scripts
|
|
737
677
|
|
738
678
|
it "evicts an environment that hasn't been recently touched" do
|
739
679
|
Puppet[:environment_timeout] = 1
|
740
|
-
Puppet[:environment_timeout_mode] = :from_last_used
|
741
680
|
|
742
681
|
with_environment_loaded(service) do |cached|
|
743
682
|
future = Time.now + 60
|
@@ -754,7 +693,6 @@ config_version=$vardir/random/scripts
|
|
754
693
|
|
755
694
|
it "reuses an environment that was recently touched" do
|
756
695
|
Puppet[:environment_timeout] = 60
|
757
|
-
Puppet[:environment_timeout_mode] = :from_last_used
|
758
696
|
|
759
697
|
with_environment_loaded(service) do |cached|
|
760
698
|
# reuse the already cached environment
|
@@ -767,9 +705,9 @@ config_version=$vardir/random/scripts
|
|
767
705
|
|
768
706
|
it "evicts a recently touched environment" do
|
769
707
|
Puppet[:environment_timeout] = 60
|
770
|
-
Puppet[:environment_timeout_mode] = :from_last_used
|
771
708
|
|
772
|
-
|
709
|
+
# see note above about "twice"
|
710
|
+
expect(service).to receive(:expired?).twice.and_return(true)
|
773
711
|
|
774
712
|
with_environment_loaded(service) do |cached|
|
775
713
|
# even though the environment was recently touched, it's been expired
|
@@ -779,116 +717,13 @@ config_version=$vardir/random/scripts
|
|
779
717
|
expect(service.created_envs).to eq([:an_environment, :an_environment])
|
780
718
|
expect(service.evicted_envs).to eq([:an_environment])
|
781
719
|
end
|
782
|
-
|
783
|
-
it "evicts expired environments when listing" do
|
784
|
-
expect(service).to receive(:expired?).with(:an_environment).and_return(true)
|
785
|
-
|
786
|
-
with_environment_loaded(service) do |cached|
|
787
|
-
cached.list
|
788
|
-
end
|
789
|
-
|
790
|
-
expect(service.evicted_envs).to eq([:an_environment])
|
791
|
-
end
|
792
720
|
end
|
793
721
|
|
794
|
-
|
795
|
-
|
796
|
-
|
797
|
-
|
798
|
-
|
799
|
-
cached.clear(:an_environment)
|
800
|
-
end
|
801
|
-
|
802
|
-
expect(service.evicted_envs).to eq([:an_environment])
|
803
|
-
end
|
804
|
-
end
|
805
|
-
|
806
|
-
context '#clear_all' do
|
807
|
-
let(:service) { ReplayExpirationService.new }
|
808
|
-
let(:envdir) { File.expand_path("envdir") }
|
809
|
-
let(:default_dir) { File.join(envdir, "cached_env", "modules") }
|
810
|
-
let(:expected_dir) { File.join(envdir, "cached_env", "site") }
|
811
|
-
|
812
|
-
let(:base_dir) do
|
813
|
-
FS::MemoryFile.a_directory(envdir, [
|
814
|
-
FS::MemoryFile.a_directory("cached_env", [
|
815
|
-
FS::MemoryFile.a_missing_file("environment.conf")
|
816
|
-
])
|
817
|
-
])
|
818
|
-
end
|
819
|
-
|
820
|
-
let(:updated_dir) do
|
821
|
-
FS::MemoryFile.a_directory(envdir, [
|
822
|
-
FS::MemoryFile.a_directory("cached_env", [
|
823
|
-
FS::MemoryFile.a_directory("site"),
|
824
|
-
FS::MemoryFile.a_missing_directory("modules"),
|
825
|
-
FS::MemoryFile.a_regular_file_containing("environment.conf", <<-EOF)
|
826
|
-
modulepath=site
|
827
|
-
environment_timeout=unlimited
|
828
|
-
EOF
|
829
|
-
])
|
830
|
-
])
|
831
|
-
end
|
832
|
-
|
833
|
-
it 'evicts all environments' do
|
834
|
-
with_environment_loaded(service) do |cached|
|
835
|
-
cached.get(:an_environment)
|
836
|
-
cached.get(:another_environment)
|
837
|
-
cached.clear_all
|
838
|
-
|
839
|
-
expect(service.evicted_envs).to match([:an_environment, :another_environment])
|
840
|
-
end
|
841
|
-
end
|
842
|
-
|
843
|
-
it "recomputes modulepath if 'get' is called before 'clear_all'" do
|
844
|
-
cached_loader_from(:filesystem => [base_dir], :directory => base_dir) do |loader|
|
845
|
-
loader.get(:cached_env)
|
846
|
-
|
847
|
-
expect(Puppet.settings.value(:modulepath, :cached_env)).to eq(default_dir)
|
848
|
-
|
849
|
-
FS.overlay(updated_dir) do
|
850
|
-
loader.clear_all
|
851
|
-
|
852
|
-
expect(loader.get(:cached_env).modulepath).to contain_exactly(expected_dir)
|
853
|
-
end
|
854
|
-
end
|
855
|
-
end
|
856
|
-
|
857
|
-
it "recomputes modulepath if 'list' is called before 'clear_all'" do
|
858
|
-
cached_loader_from(:filesystem => [base_dir], :directory => base_dir) do |loader|
|
859
|
-
loader.list
|
860
|
-
|
861
|
-
expect(Puppet.settings.value(:modulepath, :cached_env)).to eq(default_dir)
|
862
|
-
|
863
|
-
FS.overlay(updated_dir) do
|
864
|
-
loader.clear_all
|
865
|
-
|
866
|
-
expect(loader.get(:cached_env).modulepath).to contain_exactly(expected_dir)
|
867
|
-
end
|
868
|
-
end
|
869
|
-
end
|
870
|
-
|
871
|
-
it "recomputes modulepath if 'get_conf' is called before 'clear_all'" do
|
872
|
-
cached_loader_from(:filesystem => [base_dir], :directory => base_dir) do |loader|
|
873
|
-
loader.get_conf(:cached_env)
|
874
|
-
|
875
|
-
expect(Puppet.settings.value(:modulepath, :cached_env)).to eq(default_dir)
|
876
|
-
|
877
|
-
FS.overlay(updated_dir) do
|
878
|
-
loader.clear_all
|
879
|
-
|
880
|
-
expect(loader.get(:cached_env).modulepath).to contain_exactly(expected_dir)
|
881
|
-
end
|
882
|
-
end
|
883
|
-
end
|
884
|
-
|
885
|
-
it 'deletes environment text domains' do
|
886
|
-
with_environment_loaded(service) do |cached|
|
887
|
-
cached.get(:an_environment)
|
888
|
-
cached.clear_all
|
889
|
-
|
890
|
-
expect(FastGettext.text_domain).to eq(Puppet::GettextConfig::DEFAULT_TEXT_DOMAIN)
|
891
|
-
end
|
722
|
+
it "gets an environment.conf" do
|
723
|
+
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
724
|
+
expect(Puppet::Environments::Cached.new(loader).get_conf(:an_environment)).to match_environment_conf(:an_environment).
|
725
|
+
with_env_path(directory_tree.children.first).
|
726
|
+
with_global_module_path([])
|
892
727
|
end
|
893
728
|
end
|
894
729
|
end
|
@@ -963,20 +798,6 @@ config_version=$vardir/random/scripts
|
|
963
798
|
end
|
964
799
|
end
|
965
800
|
|
966
|
-
def cached_loader_from(options, &block)
|
967
|
-
FS.overlay(*options[:filesystem]) do
|
968
|
-
environments = Puppet::Environments::Cached.new(
|
969
|
-
Puppet::Environments::Directories.new(
|
970
|
-
options[:directory],
|
971
|
-
options[:modulepath] || []
|
972
|
-
)
|
973
|
-
)
|
974
|
-
Puppet.override(:environments => environments) do
|
975
|
-
yield environments
|
976
|
-
end
|
977
|
-
end
|
978
|
-
end
|
979
|
-
|
980
801
|
def loader_from(options, &block)
|
981
802
|
FS.overlay(*options[:filesystem]) do
|
982
803
|
environments = Puppet::Environments::Directories.new(
|
@@ -999,19 +820,6 @@ config_version=$vardir/random/scripts
|
|
999
820
|
end
|
1000
821
|
end
|
1001
822
|
|
1002
|
-
# The environment named `:an_environment` will already be loaded when the
|
1003
|
-
# block is yielded to
|
1004
|
-
def with_environment_loaded(service, &block)
|
1005
|
-
loader_from(:filesystem => [directory_tree], :directory => directory_tree.children.first) do |loader|
|
1006
|
-
using_expiration_service(service) do
|
1007
|
-
cached = Puppet::Environments::Cached.new(loader)
|
1008
|
-
cached.get!(:an_environment)
|
1009
|
-
|
1010
|
-
yield cached if block_given?
|
1011
|
-
end
|
1012
|
-
end
|
1013
|
-
end
|
1014
|
-
|
1015
823
|
class ReplayExpirationService < Puppet::Environments::Cached::DefaultCacheExpirationService
|
1016
824
|
attr_reader :created_envs, :evicted_envs
|
1017
825
|
|
@@ -1029,3 +837,4 @@ config_version=$vardir/random/scripts
|
|
1029
837
|
end
|
1030
838
|
end
|
1031
839
|
end
|
840
|
+
end
|