summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Stub test cert body to fix tests (#13740)Thomas Powell2023-05-041-0/+25
| | | Signed-off-by: Thomas Powell <thomas.powell@progress.com>
* feat(apt_repository): Allow specifying arbitrary options (#13728)Thayne McCombs2023-05-032-7/+22
| | | | | | | | | This allows specifying additional options to apt repositories, in addition to `trusted` and `arch`. By using an array of strings we also allow using multivalue operators like -= and += Fixes: #13727 Signed-off-by: Thayne McCombs <thayne@lucid.co>
* zypper_package: Fix idempotency issue (#13691)Lance Albertson2023-04-181-0/+10
| | | | | | | | | | | | Installing the apache2-mod_wsgi-python3 package is not idempotent due to the fact that zypper will also return the apache2-mod_wsgi package. The resolve_available_version method, incorrectly assumes this will only return one item, but in this case it returns four items and will pick the first package listed. To work around this, we should verify that the package name listed in the output matches what we are expecting to install. Signed-off-by: Lance Albertson <lance@osuosl.org>
* Correcting a typo in the resource and now the tests as wellJohn2023-03-311-4/+4
| | | | Signed-off-by: John <john.mccrae@progress.com>
* new certs from digicert website (#13625)Vikram Karve2023-03-102-81/+71
| | | Signed-off-by: Vikram Karve <vikram.karve@progress.com>
* set default values for user and host on userdefaults (#12791)Rishi Kumar Chawda2023-03-072-8/+8
| | | | | | * set default values for user and host on userdefaults Signed-off-by: rishichawda <rishichawda@users.noreply.github.com>
* Add selinux_user and selinux_login resources (#13511)wheatevo2023-03-072-0/+165
| | | | | | | | | * Add `selinux_user` resource to manage SELinux users. * Add `selinux_login` resource to manage OS login to SELinux user mapping. Related to PR https://github.com/sous-chefs/selinux/pull/92 Signed-off-by: Matthew Newell <matthew.newell@oracle.com>
* Merge pull request #13552 from chef/jfm/chef18_cert_retrieval2John McCrae2023-03-023-29/+112
|\ | | | | Correcting cert retrieval issues for multiple user scenarios
| * Correcting cert retrieval issues for multiple user scenariosJohn McCrae2023-02-283-29/+112
| | | | | | | | Signed-off-by: John McCrae <mccrae@progress.com>
* | fix(apt_repository): Detect changes to expiration in key (#13535)Thayne McCombs2023-02-071-5/+26
|/ | | | | | | | | This makes it so that the apt keys are updated if the expiration of a key is updated without changing the fingerprint. See: https://gitlab.com/gitlab-cookbooks/cookbook-gitlab-runner/-/merge_requests/37/diffs#35cd6a49d9e22469db42af7a30c5f3a76487e3c7 Fixes: #13308 Signed-off-by: Thayne McCombs <thayne@lucid.co>
* add missing compliance phase test (#13471)Corey Hemminger2023-01-311-0/+8
| | | | | | | * add missing test Customer ran into this scenario where audit cookbook is present due to base dependencies, profiles are set to pull from automate. CB is using policyfile and setting in kitchen.yml attributes: audit: compliance_phase: false to avoid run failures in test-kitchen. Compliance phase still runs though. Signed-off-by: Corey Hemminger <hemminger@hotmail.com>
* revert chef-foundation changes in chef omnibus config (#13528)Evan Ahlberg2023-01-251-2/+2
| | | | | | | | * revert chef-foundation changes in chef omnibus config * change fips test for windows * update pipeline names to validate and increase timeout for windows omnibus builds * remove omnibus build and test for now Signed-off-by: Evan Ahlberg <evanahlberg@gmail.com>
* update verify pipeline and omnibus build/test to use containers (#13489)Evan Ahlberg2023-01-231-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * update verify pipeline and omnibus build/test to use containers * add execution permission on scripts * add fips back in for opensuse but skip for windows * jesseprieur/BS-159 - Add RPM Signing Key for RHEL/SLES/CentOS * evanahlberg/BS-152 - Add MSI Signing to Windows Builds * fix label on windows platforms * fix role and add check for organization for aws credentials * chmod omnibus test script and fix windows 2019 queue for omnibus test * fix windows queue * Remove dependencies that are part of chef-foundation. * Disable s3 caching. * Rename chef-gem as it conflicts with another omnibus-software. * Speeding up debugging; Removing more-ruby-cleanup gem from omnibus chef package * Adding more-ruby-cleanup back in * Removing more-ruby-cleanup; Removing debugging steps * Remove ruby dependency from more-ruby-cleanup. * add retry and timeouts to all steps * move omnibus test and build to own file and create ad hoc pipeline * Adding in changes to use heredoc in verify pipeline * Adding in retries/timeouts; Fixing missing agent * allow for filtering of the omnibus build and test platforms * only block on chef-oss org * add canary adhoc pipeline and add back trigger for omnibus release * Make omnibus s3 caching configurable via env. Signed-off-by: Gregory Schofield <grschofi@progress.com> Signed-off-by: Evan Ahlberg <evanahlberg@gmail.com> Signed-off-by: Jesse Prieur <jesse.prieur@gmail.com>
* Adds 'makecache_fast' property to `yum_repository` minimise cached repodata ↵Stuart Sears2023-01-031-0/+4
| | | | | | | | | | (#12915) * Adds 'makecache_fast' property to minimise cached repodata * add makecache_fast property to unit test * no makecache fast on dnf systems * correct property in unit test Signed-off-by: Stuart Sears <stuart@sjsears.com>
* updating file_system to send back the copy status (#13107)Vinay Satish2023-01-031-0/+2
| | | | | | * updating file_system to send back the copy status * Updating Gemfile.lock for cheffish bump Signed-off-by: Vinay Satish <vinay.satish@progress.com>
* Fix indentation of condition for disabling repos (#13327)Davi Arnaut2022-12-138-0/+71
| | | | | | | | | | * Fix indentation of condition for disabling repos The helper was only disabling repos if the disable action appeared last in the repos array. This was also hiding a problem where the helper failed to re-enable repos when all repos are disabled using a wildcard (e.g. "*"). Signed-off-by: Davi Arnaut <davi@verdesmares.com>
* Update omnibus-software for FIPS issues (#13375)tpowell-progress2022-11-283-0/+28
| | | | | | | | | | | | | | * Update omnibus-software for FIPS issues New version of omnibus-software should have disable-dynamicbase option on for the all DLLs involve with openssl for FIPS build as well as keying off of mingw64 "platform" argument to ./Configure script. Ensure FIPS mode turned off after test Disable all opensuse fips_mode tags, due to broken FIPS build in rubydistros for OpenSUSE. Signed-off-by: Thomas Powell <powell@progress.com>
* Squashed commit of the following: (#13069)Steve Abatangle2022-11-221-0/+30
| | | | | | | | | Res validation should be aware of action prop reqs. The get and set methods in the Property class (for resource validation) check that a property is required if its value is nil. However, they don't check to see whether the property was required for the specific action the resource is using, so any use of get and set will throw an exception if used with a resource that doesn't use a required property, even when the property is not required for that action. What I'm describing is easier to understand with an example: the current version of the chef-client cookbook has a recipe (cron.rb) with a resource that uses the :delete action. When you try to get that resource, it throws an exception, explaining that :command is a required property—but it's only required for the :create action, not for :delete. Co-authored-by: Steve Abatangle <steve_abatangle@gap.com>
* Fix false updates on cron resource when using integers (#13147)Jeremy Kimber2022-11-081-0/+36
| | | | | | | | | | | | | | | | * Fix false updates on cron resource when using integers Signed-off-by: Jeremy Kimber <kimbernator@state.local> * move new cron idempotency test to provider spec and coerce data to string in provider Signed-off-by: Jeremy Kimber <kimbernator@state.local> * Compare cron variables by class Signed-off-by: Jeremy Kimber <kimbernator@state.local> Signed-off-by: Jeremy Kimber <kimbernator@state.local> Co-authored-by: Jeremy Kimber <kimbernator@state.local>
* Bugfix: checksum validation (#13210)Joseph Larionov2022-11-011-0/+28
| | | | | | | | | | * Add a case-insensitive helper to determine if two checksums match * Use case-insensitive checksum match helper for checksum validation This fixes a bug where checksum validation fails because the two checksums are in different letter cases. The existing checksum validation logic was using case sensitive equality checks. Signed-off-by: Joseph Larionov <jlarionov@webmd.net>
* Add linux_only: true to new linux user specs (#13252)tpowell-progress2022-10-182-2/+9
| | | | | | * Add linux_only: true to new linux user specs * Address Trailing Comma and nonexisting group in SLES 15 Signed-off-by: Thomas Powell <powell@progress.com>
* Merge pull request #13223 from fretb/bug/linux-user-comparetpowell-progress2022-10-142-0/+165
|\ | | | | Fixes bug in compare_user on Linux systems
| * Fixes bug in compare_user on Linux systemsFrederik Thuysbaert2022-10-072-0/+165
| | | | | | | | | | | | | | | | d4c3b8ad111ec7ff6b32ebccf9edd1b6a2f9e8e2 introduced a bug making `compare_user` on Linux systems return `%i{expire_date inactive}` instead of wether `@change_desc` is empty. Signed-off-by: Frederik Thuysbaert <frederik.thuysbaert@combell.group>
* | Merge pull request #13229 from chef/mp/pipeline-workaround-server2012-2Marc A. Paradise2022-10-101-4/+0
|\ \ | | | | | | Do not raise if we can't close win resource handle
| * | Allow dsc_refresh_mode_disabled to be invoked in testMarc A. Paradise2022-10-071-4/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | Currently we're failing this test on a pre-emptive direct execution of Get-LocalDscConfigurationManager that we're doing in the test. It is not returning the expected result, but it's hiding the errors. Instead, we'll let the actual thing we want to test occur (dsc_refresh_mode_disabled) - because it runs the same command, and raises an error if it fails. Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
* | Fix term and privilege related broken tests in Solaris build (#13230)tpowell-progress2022-10-072-6/+7
| | | | | | | | | | | | INFC-274 Fix Solaris tests * Do not match on OS specific message, only on generic description. * Update omnibus version to fix read_shared_libs * Force `ENV["TERM"]` to `"vt100"` for tests
* | On AIX, add delay to ensure members added to group are visible to testsNeha Pansare2022-10-072-12/+16
|/ | | | | | | | | | | | | | | | | | | | | We have found on AIX 7.x, it can be a varying amount of time before the system reflects that member(s) have been added to a group. This change increases the delay on AIX from 2 seconds to 3 after adding a user to a group. This ensures that the new group members are visible before we verify that they are added correctly. This change also removes the limitation that only applies this delay on AIX 7.2 since we have found it affects 7.3 as well. For consistency, it is now applied on all AIX platform versions. With these tests passing, it is now possible to re-enable the AIX pipeline (which has also been done in this change). Signed-off-by: Neha Pansare <neha.pansare@progress.com> Signed-off-by: John McCrae <john.mccrae@progress.com> Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
* Allow user providers to declare no support for ruby-shadowMarc A. Paradise2022-09-281-6/+24
| | | | | | | | | | | | | The AIX platform does not support ruby-shadow, but we always attempt to load it. The load fails on the AIX platform, which later causes our requirement assertion that `@shadow_lib_ok` be true to fail. Instead we allow user resource providers to override `supports_ruby_shadow?` if they do not support it, so that we don't assert that it is required in those cases. We default the return value to `true` because most platforms are supported by this gem. Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
* Integrate ruby 3.1 into buildsJohn McCrae2022-08-236-12/+17
| | | | | | | | | This is a combination of 64 commits, the originals were mostly repeating one-line messages so that history has not been kept here. Signed-off-by: Thomas Powell <thomas.powell@progress.com> Signed-off-by: Marc Paradise <marc@chef.io> Signed-off-by: John McCrae <john.mccrae@progress.com>
* Merge pull request #13118 from tmccombs/sysctl-newlineMarc A. Paradise2022-08-161-2/+2
|\ | | | | | | | | Add newline to end of sysctl files Current pipeline failures are environmental and not related to this change. Failing tests are kitchen, which does not have coverage for this resource.
| * Add newline to end of sysctl filesThayne McCombs2022-08-051-2/+2
| | | | | | | | | | | | | | | | | | | | If the sysctl file doesn't end with a newline, then Ubuntu's CIS audit won't correctly detect the sysctl configuration. Also, it is UNIX best practice to include a newline at the end of the last line of text files. Obvious fix. Signed-off-by: Thayne McCombs <thayne@lucid.co>
* | add splay 0 test to chef_client_scheduled_taskCorey Hemminger2022-08-021-0/+5
| | | | | | | | Signed-off-by: Corey Hemminger <hemminger@hotmail.com>
* | Update chef_client_systemd_timer_spec.rbCorey Hemminger2022-08-021-5/+0
| |
* | testCorey Hemminger2022-08-021-10/+1
| | | | | | | | Signed-off-by: Corey Hemminger <hemminger@hotmail.com>
* | add tests for splay 0Corey Hemminger2022-07-263-1/+25
|/ | | | Signed-off-by: Corey Hemminger <hemminger@hotmail.com>
* Revert "Revert "Use new msys2 based devkit for windows" (#13063)" (#13097)neha-p62022-07-221-2/+2
| | | This reverts commit 4797e01535af8e33ace9ef3a7404dd8256913bce.
* Correct offensesPoornima2022-07-131-1/+1
| | | | Signed-off-by: Poornima <94897054+poorndm@users.noreply.github.com>
* Skip tests on FreeBSD-13Poornima2022-07-133-2/+7
| | | | Signed-off-by: Poornima <94897054+poorndm@users.noreply.github.com>
* Revert "Use new msys2 based devkit for windows" (#13063)neha-p62022-07-111-2/+2
|
* Merge pull request #13014 from chef/fix_desired_state_issueblabade2022-07-071-1/+21
|\ | | | | Fixed sensitive properties unsuppressed content issue
| * Merge branch 'main' into fix_desired_state_issueBapu L2022-06-272-11/+38
| |\
| * | Fixed test case property name code styleBapu L2022-06-221-6/+6
| | | | | | | | | | | | Signed-off-by: Bapu L <bapu.labade@progress.com>
| * | Fixed code style issueBapu L2022-06-221-3/+6
| | | | | | | | | | | | Signed-off-by: Bapu L <bapu.labade@progress.com>
| * | Fixed build issuesBapu L2022-06-211-3/+3
| | |
| * | test case description changesBapu L2022-06-211-2/+2
| | |
| * | Doc string message changes, test case cleanupBapu L2022-06-211-1/+8
| | |
| * | Property values should be suppressed irrespective of desired stateBapu L2022-06-211-6/+16
| | |
* | | use new msys2 devkit for windowsMatt Wrock2022-06-291-2/+2
| |/ |/| | | | | Signed-off-by: Matt Wrock <matt@mattwrock.com>
* | Added a new test, discovered a new bug in the tests and corrected that. All ↵John McCrae2022-06-241-1/+1
| | | | | | | | | | | | tests passing locally Signed-off-by: John McCrae <john.mccrae@progress.com>
* | Added a new test, discovered a new bug in the tests and corrected that. All ↵John McCrae2022-06-241-2/+19
| | | | | | | | | | | | tests passing locally Signed-off-by: John McCrae <john.mccrae@progress.com>