mirror of
https://github.com/inspec/inspec
synced 2024-11-10 07:04:15 +00:00
2601 commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
Sonu Saha
|
16a74f39ae
|
CHEF-3162: Implement unit test for inspec-sign plugin (#6959)
* test: add unit test for inspec sign #profile_verify Signed-off-by: Sonu Saha <sonu.saha@progress.com> * enhance: add support for custom config dir to be used during testing Signed-off-by: Sonu Saha <sonu.saha@progress.com> * test: add unit test for inspec sign #keygen and #profile_sign Signed-off-by: Sonu Saha <sonu.saha@progress.com> * test: remove test for profile sign Signed-off-by: Sonu Saha <sonu.saha@progress.com> * test: add test for methods in inspec sign Signed-off-by: Sonu Saha <sonu.saha@progress.com> --------- Signed-off-by: Sonu Saha <sonu.saha@progress.com> |
||
|
Sonu Saha
|
d4208cb63c
|
CHEF-4571: Silence warning about transforming into git fetcher (#6946)
* chore: change log-level for url fetcher transformation Signed-off-by: Sonu Saha <sonu.saha@progress.com> * test: remove warning logs from url fetcher test Signed-off-by: Sonu Saha <sonu.saha@progress.com> --------- Signed-off-by: Sonu Saha <sonu.saha@progress.com> |
||
Vasundhara Jagdale
|
ebd3eeca55
|
CHEF-6412: Fix for virutalization resource undefined method file_read (#6947)
* Fix the undefined method file_read error which is causing virtualizaiton resource to set nil for virtualization platform check Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Adds unit test for virtualization resource detect podman container usind detect_container method. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Vasundhara Jagdale
|
dc7631b8ac
|
Skip audit log test on Windows (#6950)
* Skip audit log test on Windows Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix the text in the audit log test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
Nikita Mathur
|
7ac8ccf7c1
|
Unit tests added for enhanced outcomes (#6924)
Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Nikita Mathur
|
75fe6a0e27
|
Unit tests added for iaf_file (#6933)
Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Vasundhara Jagdale
|
e48e0c7786
|
Fix failing ssh_key test on Windows (#6948)
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Vasundhara Jagdale
|
b946f5454d
|
CHEF-3962: Adds ability to verify ssh keys (#6656)
* ssh_key resource file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Initial commit for unit test for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting and added resource in resources file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * extend the ssh key resource to use file resource properties Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates the ssh_key resource to get length and type of key. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates unit test for ssh_key resource. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * ADDS Docs for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix Review: Empty file handling Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fixed review comments and few code refactoring for ssh_key resource Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Doc edits Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> * Updates test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> Co-authored-by: Ian Maddaus <ian.maddaus@progress.com> |
||
|
Nikita Mathur
|
18847ac69d
|
Fix CI test for windows (#6913)
* Skip functional tests to match exit code for mandatory profile signing error in windows Signed-off-by: Nik08 <nikita.mathur@progress.com> * Addded comment to clarify skipping exit code matching test in windows Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Sonu Saha
|
b79ad46e5a
|
Fix examples for file resource provided in docs (#6899)
* Update docs and introduce test for file content Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Fix broken test due to changes in fixture Signed-off-by: Sonu Saha <sonu.saha@progress.com> --------- Signed-off-by: Sonu Saha <sonu.saha@progress.com> |
||
|
Vasundhara Jagdale
|
20d0a9dc0d
|
CHEF-6440: Integrates audit logging feature (#6809)
* Adds cli options to enable audit log and configure the audit log Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updated code to validate the audit log options. Audit log options are only valid for inspec exec and inspec shell command as those commands use the backend to execute commands and for file operations Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates cli options documentation for audit log options Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Disable audit log in test environment unless and until explicitly --enable-audit-log option is provided in test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Rename enable-audit-log cli option to disable-audit-log Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * CHEF-8210 Enables feature preview flag for audit logging Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Revert "Disable audit log in test environment unless and until explicitly --enable-audit-log option is provided in test" This reverts commit 50a42f0b44cf2fde6d927a00e444370cdd288a5d. * Adds functional test for audit logging feature Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates cli doc for audit loggin feature flag Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Debug why audit log functional test fails in ci environment Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Revert "Rename enable-audit-log cli option to disable-audit-log" This reverts commit |
||
|
Nikita Mathur
|
fd4e6d97a6
|
CHEF-6439 Mandatory Profile Signing (Preview) (#6705)
* Updated exec option to allow unsigned profiles run Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added method to verify signed profile and to check for signed profile Signed-off-by: Nik08 <nikita.mathur@progress.com> * Invoked logic on each run to verify profiles if signed else raise sig req error Signed-off-by: Nik08 <nikita.mathur@progress.com> * Tests cases added to validate behaviour of inspec exec with signed and unsigned profiles with --chef-allow-unsigned flag Signed-off-by: Nik08 <nikita.mathur@progress.com> * Refactored and moved delete_signing_keys to common helper library for tests Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated code comments for more information and clarity on security update of signed profiles inspec exec Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test cases to validate inspec run with combination of signed and unsigned profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Documented usage of flag --chef-allow-unsigned Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renamed the flag to run unsigned profiles to --allow-unsigned Signed-off-by: Nik08 <nikita.mathur@progress.com> * Refactored logic on profile level for profile signing verification Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renaming the argument variable - from runner_call to silent Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added profile mandate check for other inspec commands running profile evaluation Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated error message for profile sign requirement Signed-off-by: Nik08 <nikita.mathur@progress.com> * Updated test helper to fix inspec json test Signed-off-by: Nik08 <nikita.mathur@progress.com> * Fixed inspec json ability to use cli options successfully Signed-off-by: Nik08 <nikita.mathur@progress.com> * Documentation added for signed profiles mandatory usage with CLI commands Signed-off-by: Nik08 <nikita.mathur@progress.com> * Flow changes of raising exception when unsigned instead of direct exit Signed-off-by: Nik08 <nikita.mathur@progress.com> * Renamed unsigned profile flags Signed-off-by: Nik08 <nikita.mathur@progress.com> * Extracted out allow unsigned condition to config and modified comment info Signed-off-by: Nik08 <nikita.mathur@progress.com> * Doc update on consent of using signed and unsigned profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Fix in signing mandatin check and added additional check on runner for better error UI for exec command Signed-off-by: Nik08 <nikita.mathur@progress.com> * Removed repeated allow-unsigned-profile defination from exec_options Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test fixes Signed-off-by: Nik08 <nikita.mathur@progress.com> * Enabled feature preview flag for mandatory signing Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test fixes after feature flag usage for mandatory signing Signed-off-by: Nik08 <nikita.mathur@progress.com> * Doc changes using feature preview flag for mandatory signing feature Signed-off-by: Nik08 <nikita.mathur@progress.com> * Inspec exec tests fixes for ENV values and parallel test fix using default option --allow-unsigned-profile false Signed-off-by: Nik08 <nikita.mathur@progress.com> * Kitchen fix while using signed profiles with inspec Signed-off-by: Nik08 <nikita.mathur@progress.com> * Unit test fix for profile resource exception Signed-off-by: Nik08 <nikita.mathur@progress.com> * Virtual profile detection improved Signed-off-by: Nik08 <nikita.mathur@progress.com> * Move mandatory profile sigining info to sigining page Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Renamed flag from --allow-unsigned-profile to --allow-unsigned-profiles Signed-off-by: Nik08 <nikita.mathur@progress.com> * Typo fix in signing doc Signed-off-by: Nik08 <nikita.mathur@progress.com> * Trim note in cli.md about mandatory profile signing Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Docs changes Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> * Correct docs regarding exit code 5 Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Signed-off-by: Ian Maddaus <ian.maddaus@progress.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Ian Maddaus <ian.maddaus@progress.com> |
||
|
Vasundhara Jagdale
|
ed033b9ac2
|
CHEF-7694 (CHEF-8271) : Fix security_policy resource returns array for non comma separated values (#6838)
* Adds unit test for more security policies Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix for secuity_policy resource failing if policy includes single value in string format it returns array instead of string Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Update code comments Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Sonu Saha
|
b5fcc141d2
|
CHEF-6437: Implement different version of inspec export (#6816)
* Failing test for export - should not evaluate Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Sketch out a info_from_parse method Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Temporary commit to checkpoint experimental work Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic control ids extraction Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Modify to capture entire block Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Ability to parse desc, impact and title of a control (#6662) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Rework per-control metadata collectors to be class-based Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * REFACTOR: make a common base class for collectors Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * memoise `info_from_parse` Signed-off-by: Sathish <sbabu@progress.com> * Add --legacy-export option to inspec export (#6661) * support legacy export option Signed-off-by: Sathish <sbabu@progress.com> * ability to run legacy export option Signed-off-by: Sathish <sbabu@progress.com> --------- Signed-off-by: Sathish <sbabu@progress.com> * Improve ControlIDCollector and other fields of export data (#6686) * Parse tags & refs from the ast nodes Signed-off-by: Sonu Saha <sonu.saha@progress.com> * ENHANCE: Improve Desc collector to collect description Signed-off-by: Sonu Saha <sonu.saha@progress.com> * ENHANCE: Only loop through the child node of begin block Signed-off-by: Sonu Saha <sonu.saha@progress.com> * FIX: Fix bug/todo to handle duplicacy of control ids Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST - a profile which fails to properly be exported but is likely to be used by MITRE Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Revert "FIX: Fix bug/todo to handle duplicacy of control ids" This reverts commit |
||
|
Nikita Mathur
|
456fa5c8f5
|
CHEF-7670 Ability to set feature preview flag using ENV (#6833)
* Implemented ability to add feature flags in Inspec feature subsystem Signed-off-by: Nik08 <nikita.mathur@progress.com> * Added usage of env_preview for conditionalising features Signed-off-by: Nik08 <nikita.mathur@progress.com> * Test cases added for feature preview flag ability Signed-off-by: Nik08 <nikita.mathur@progress.com> * Minor chnages - Better commenting & smarter methods Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
|
Nikita Mathur
|
b360d64a1c
|
Changed params usage in inspec archive to skip evaluation (#6741)
* Using metadata to fetch basic info as name and version required in archive Signed-off-by: Nik08 <nikita.mathur@progress.com> * Typo fixes Signed-off-by: Nik08 <nikita.mathur@progress.com> --------- Signed-off-by: Nik08 <nikita.mathur@progress.com> |
||
Sathish Babu
|
e1d39e581a
|
CHEF-6422 Modify inspec archive to not check or export by default (#6659)
* make profile check optional Signed-off-by: Sathish <sbabu@progress.com> * Clarify error message when making inspec check optional Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add test fixture profile that emits evaluation markers on stderr Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Make export to inspec.json optional, not enabled by default Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add test confirming non-evaluative archive Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Fix failing plugin search test, failing since we published a new test gem Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Sathish <sbabu@progress.com> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Vasundhara Jagdale
|
95c17d4e7f
|
CHEF-3849: InSpec should exit quickly and clearly if waiver file is malformed/corrupt (#6644)
* Functional test for malformed waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Raise error for malformed yaml content and exit Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates functional test for malformed yaml waiver file and for empty waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Raises error in case of missing required parameters in waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Update functional test for missing parameters, extra parameters or column without headers in waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix warning and error messages Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates nil and false check for yaml data and adds additional empty check. Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com> Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Adds more generic message as this yaml reader is now getting used by other functionalities like waiver file Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fixed test description to reflect correct use case Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix validate headers was not validating the required parametes for all the data fields as it was not called inside the loop where we are iterating over the data and fetching the headers. Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates the test files for the use case to missing parameters and extra parameters Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updates code to better handle errors and warnings related to missing required parameters and extra parameters in waivers file in all format i.e (yaml, json and csv). Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Updated functional test to capture the updated error and warning messages for waiver file validation Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Fix linting Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Deleted fixture file which is not required Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Refactor: Renamed method validate_headers to reflect whats it's doing and instead of return data in array it will now return the data in hash Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
8ee7bc946d
|
CHEF-642 Fix for inspec exec fails with git fetcher if current directory does not have .git directory (#6626)
* Functional Test to verify git fetcher able to work in a directory where git is not initialized or does not contain the .git directory Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix: If git branch, tag or reference information is not given it should clone the repo and checkout the default HEAD Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Remove the logic to find default branch in case of branch, tag or ref is not provided as it's not required logically as we will be cloning the and checkout the default HEAD Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * This is a temporary change to make verify test working on this PR. Revert once the work is done for the git fetcher fix Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Unit Test: Fix unit test Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Verifies the git url before creating cache_key, if given relative path does not exist make sure it deletes the cache directory created to copy the cloned repo. It raises error if any git command fails Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Reverted verify git repo url call from cache_key as cache_key and shifted that to resolve_ref which is already raising the error in case of git ls remote command fails Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> * Unit test to cover the changes done for resolve_ref when branch, tag or ref is not given and unit test for invalid git repo Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com> |
||
|
Sathish Babu
|
5d0ae06a01
|
Missing changes from main (#6564)
* Sync up the default branch as main (#6239) Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> * Bump version to 5.20.2 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * resolves inspec/inspec#6042 (#6043) Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> * Enhance `lxc` resource to test properties (#6243) * ENHANCE: Extend lxc resource to test the properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * FIX: Correct exception class to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * FIX: Handle specific exception to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> * Bump version to 5.20.3 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-462: Fixes inspec sign breaks when there is period or dot in the profile name (#6261) Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Bump version to 5.20.4 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-374 DSL keyword `only_applicable_if` added (#6229) * Not applicable if logic addition Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Changes from not applicable if to only applicable if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fix to enable placing only_applicable_if at any position in control and for keeping impact zero intact Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc change added for only_applicable_if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Added generic examples that supports cross platform for testing Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Yet another build fix due to changes in test Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> * Bump version to 5.21.0 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * CFINSPEC-458 Oracle DB session resource compatibility with AIX-C shell (#6257) * Oracle DB session resource compatibility with AIX-Cshell Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fixes in oracledb session unit test using oracle_query_string Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Bump version to 5.21.1 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Replace faraday_middleware with faraday-follow_redirects (#6238) faraday_middleware was already DEPRECATED in 2022-01. Only FaradayMiddleware::FollowRedirects from faraday_middleware has been used, and so we can replace it with faraday-follow_redirects. faraday-follow_redirects 0.3.0 requires faraday ~> 1.0, which is as same as "faraday_middleware 1.0.0 or higher. Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> * Bump version to 5.21.2 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Update Faraday gem (#6267) Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Bump version to 5.21.3 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Bump test-kitchen from 3.3.1 to 3.3.2 in /omnibus (#6227) Bumps [test-kitchen](https://github.com/test-kitchen/test-kitchen) from 3.3.1 to 3.3.2. - [Release notes](https://github.com/test-kitchen/test-kitchen/releases) - [Changelog](https://github.com/test-kitchen/test-kitchen/blob/main/CHANGELOG.md) - [Commits](https://github.com/test-kitchen/test-kitchen/commits) --- updated-dependencies: - dependency-name: test-kitchen dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump version to 5.21.4 by Chef Expeditor Obvious fix; these changes are the result of automation not creative thinking. * Bump omnibus-software from `1d540dc` to `7d0e0fe` in /omnibus (#6274) Bumps [omnibus-software](https://github.com/chef/omnibus-software) from `1d540dc` to `7d0e0fe`. - [Release notes](https://github.com/chef/omnibus-software/releases) - [Commits]( |
||
Clinton Wolfe
|
eff13369f1
|
CHEF-4115 Added ability to merge reporter configurations from both CLI and config (#6568) (#6579)
* Added ability to merge cli and config reporter options * Test cases to validate working of reporter configuration using cli and config * Documentation change to add information on reporter configurations usage with both * Added Doc review changes and text fixture for config json * Verify fix * Verify pipeline test fixes for reporter options to be read successfully * Test changes in both cli and config reporter usage scenarios to fix verify pipeline * Review comments to improvise * Renamed testing fixture file for reporter cli config file --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
dfb0172992
|
Adds test for licensing_config (#57)
* Adds test for licensing_config Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Fix lint Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Read the value of license server url from env for test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Spell InSpec corrrectly in test Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Linting Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Clinton Wolfe
|
a3028b1fc8
|
For #6493 : Add postgres_session support for custom port with a socket connection (#6494) (#40)
* Add support for custom port with a socket connection * Add tests for the postgres_session with custom port --------- Signed-off-by: Pg <pg.developper.fr@gmail.com> Co-authored-by: Pg <pg.developper.fr@gmail.com> |
||
|
Sathish Babu
|
4ec735d09d
|
CHEF 83 Revert attestations changes (#47)
* revert attestation related files Signed-off-by: Sathish <sbabu@progress.com> * revert attestation changes to existing files Signed-off-by: Sathish <sbabu@progress.com> * update signature Signed-off-by: Sathish <sbabu@progress.com> --------- Signed-off-by: Sathish <sbabu@progress.com> |
||
|
Sonu Saha
|
b4648e0193
|
CHEF-1458 Multiple values changes in SimpleConfig library (#6489) (#28)
* Simple config changes to return array of values with multiple values opt * New option added to split multiple values with a delimiter regex --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Clinton Wolfe
|
4fce6845e5
|
Add nftables resources (#6499) (#44)
* Fix systemd path for Leap image
* Use vhef client version 17 as doocker cookbook do not support >= 18
* Add nftables resource
* Add nftables tests
* Add fixtures for nftables tests
* enable nftables only when attr is true - then disable iptables
* By default test iptables, not nftables
* Fix tests and lint errors
* Increase unit test coverage for nftables
* Do not use -nn nft option as behaviour changes based on nft version
* Base nft params identification on its version, not os version
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Make test more human friendly by reversing unless/if logic
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Update mocked cmds with nft params
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Fix quoting issue with rubocop
* Fix uninitiallized class vars
* Fix unit test by adding nft version mocking
* Clean nftables doc
---------
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
Co-authored-by: jjacque <jeremy.jacque@algolia.com>
|
||
|
Clinton Wolfe
|
dd9dccd84e
|
Update host resource to resolve all ipaddresses (#6481) (#39)
Signed-off-by: Alan Foster <alan@alanfoster.me> Co-authored-by: Alan Foster <alan@alanfoster.me> |
||
|
Clinton Wolfe
|
1ab146d028
|
fix: ensure Invoke-WebRequest headers can be configured (#6484) (#41)
* fix: ensure Invoke-WebRequest headers can be configured * fix: use array for custom headers * test: add unit test for Windows simple HTTP request with headers --------- Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net> Co-authored-by: amlodzianowski <adrianmlodzianowski@gmail.com> Co-authored-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net> |
||
|
Clinton Wolfe
|
cfde3f5bd8
|
CHEF-2438 Add train-kubernetes to inspec gemspec (#6512) (#43)
* Add train-kubernetes to inspec gemspec, fix spacing * Fix failing test to detect system plugins --------- Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Co-authored-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
699d73993c
|
Licensing - Integrates Software Entitlement (#13)
* CFINSPEC-24:Integrate License Software Entitlment. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-506 license execution check Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Software entitlement call changes and handling of error Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Error handling from chef licensing error to standard error Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Updated Gemfile to pick the chef licensing changes from git url Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Trying with https url Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Revert "Trying with https url" This reverts commit 33f1f4c0ecacf9ba2826e25e55b41219903ea736. * Adds the license not entitled exit code Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updates the require statement Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix typo Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Set the licensing configure for entitlement id failure Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Use ChefLicensing::Error instead of standard error Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * REFACTOR Move out licensing cong=fig to its own file and make unconditional Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * FIX TESTS - Add reference to licensing config for places where we call runner explicitly. Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Update inputs_test to use license configuration from utils Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Vasundhara Jagdale
|
530d2d1ef2
|
CFINSPEC-580 Dependent profiles fix for any scheme of version used in profiles (#6410) (#6468)
* Dependent profiles fix for any scheme of version used in profiles * Dependent profile fix when version is blank in dependent profiles --------- Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Vasundhara Jagdale
|
ee35f4ed08
|
CFINSPEC-576 : Fix profile gem dependency loading issue when dependent gem is required inside profile libraries. (#6408) (#6467)
* Test profile consist of custom resource which has a gem dependency * CFINSPEC-576 Fix for profile gem depedency loading issue if depedent gem is required in libraries files of the profile * Fix windows test failure for profile gem dependency --------- Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
812001abcc
|
CFINSPEC-542 Bug fix for profiles with dependent profiles (#6377) (#6435)
* Matching semver using regex to determine profile name without version in dsl logic * Testing for selecting control using version in require control dsl * Documentation for using version with profile name when including or selecting controls in a profile * Commenting in dsl library about logic for fetching version and matching it * Comment for regex used to fetch version * Minor improvement for regex variable placement Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> |
||
|
Nikita Mathur
|
440934ca13
|
Foreport 6369 to inspec-6 Extended file format support for waivers: JSON & CSV only (#6371)
* Revert "Revert "CFINSPEC-240 Extended file format support for waivers (#6193)" (#6366)"
This reverts commit
|
||
|
Vasundhara Jagdale
|
35f8c4829f
|
CFINSPEC-533: Fix undefined method 'summary' for Gem::SourceFetchProblem (NoMethodError) when airgapped (#6337) (#6434)
* This fixes the undefined method 'summary' for #<Gem::SourceFetchProblem:0x000000000139dc70> (NoMethodError) while invoking any inspec command if the custom plugin is installed and if inspec commands are run in an air gap environment. This fix removes an earlier call for getting gem specification from rubygems.org which is the cause for above error and replaces it with fetching the summary from a locally saved gemspec file. This will work in both airgap and non-airgap environments. * Adds test for reading description from local gemspec file. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Vasundhara Jagdale
|
4376365304
|
Fix env smoke test by updating ERB.new in inspec env; add additional test (#6342) (#6440)
* Failing test for Ruby 3.1 - ERB.new call options changed * Fix for Ruby 3.1 by using named params to ERB.new * Restrict test to non-windows systems * Restrict test to non-windows systems Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
|
Nikita Mathur
|
3d54b17049
|
Upgraded mock ubuntu test from 20.04 to 22.04 (#6456)
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |
||
|
Vasundhara Jagdale
|
1e1e804b87
|
Prevent negative status from crashing launchd service resource (#6262) (#6433)
Signed-off-by: dtheyer <davidheyer2016@gmail.com> Co-authored-by: dtheyer <davidheyer2016@gmail.com> |
||
|
Nikita Mathur
|
b593c2ff34
|
Foreport 6257 to inspec 6 (#6279)
* Oracle DB session resource compatibility with AIX-Cshell Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fixes in oracledb session unit test using oracle_query_string Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |
||
|
Nikita Mathur
|
15433e8661
|
Foreport 6229 to inspec 6 (#6277)
* Not applicable if logic addition Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Changes from not applicable if to only applicable if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fix to enable placing only_applicable_if at any position in control and for keeping impact zero intact Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc change added for only_applicable_if Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Added generic examples that supports cross platform for testing Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Yet another build fix due to changes in test Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> |
||
|
Clinton Wolfe
|
2d886e3128
|
resolves inspec/inspec#6042 (#6043) (#6278)
Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> Signed-off-by: Marijus Ravickas <marijus.ravickas@gmail.com> Co-authored-by: Marijus Ravickas <marijus.ravickas@gmail.com> |
||
|
Clinton Wolfe
|
9e4c4994c6
|
Enhance lxc resource to test properties (#6243) (#6275)
* ENHANCE: Extend lxc resource to test the properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Extend test for introduced properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * FIX: Correct exception class to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * FIX: Handle specific exception to handle bad yaml Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> Signed-off-by: Sonu Saha <sonu.saha@progress.com> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Sonu Saha <98935583+ahasunos@users.noreply.github.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> |
||
|
Vasundhara Jagdale
|
3e3b2c9b5b
|
Replace faraday_middleware with faraday-follow_redirects (#6238) (#6280)
faraday_middleware was already DEPRECATED in 2022-01. Only FaradayMiddleware::FollowRedirects from faraday_middleware has been used, and so we can replace it with faraday-follow_redirects. faraday-follow_redirects 0.3.0 requires faraday ~> 1.0, which is as same as "faraday_middleware 1.0.0 or higher. Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> Co-authored-by: Takuya N <takninnovationresearch@gmail.com> |
||
|
Clinton Wolfe
|
393b8072e8
|
Feature Config File and Logger Support (#6260)
* Convenience method skeleton Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic support for a config file Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add features() array method Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Accept config as an option to with_feature Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Accept logger as an option to with_feature Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Refactor to push code into its own files Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic logger integration Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Crude validation of feature names, simply issues a warning log message Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Linting Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add basic tamperproofing to feature config Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Convenience method skeleton Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic support for a config file Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add features() array method Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Accept config as an option to with_feature Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Accept logger as an option to with_feature Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Refactor to push code into its own files Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Basic logger integration Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Crude validation of feature names, simply issues a warning log message Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Linting Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Add basic tamperproofing to feature config Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * CFINSPEC-464 CLI commands declaration using with_feature functionality (#6263) * Declared inspec cli commands within feature_with function Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added enhanced outcomes, waivers, reporters and streaming reporters within with_feature block Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added with_feature declaration for attestations Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Update features.yaml signature after adding in new feature flags Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> * Fix lint Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Resolved undefined method with_feature in reporters.rb Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix for features tampered file test failing Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fixed the failing functional test for junit2: Missing the entry in the features.yaml Added progress-bar reporter entry in features.yaml Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> Co-authored-by: Nikita Mathur <Nik08@users.noreply.github.com> Co-authored-by: Vasu1105 <vasundhara.jagdale@chef.io> |
||
|
Nikita Mathur
|
efc6f2c63a
|
CFINSPEC-246/CFINSPEC-247 Attestation changes for N/R outcomes (#6222)
* Added attestation file option to read attestation in various formats Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added method to add attestation data on control level Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Enhanced outcomes flag to be true when attestation file is passed Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added logic for attestation for reporters Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Attestation integration with streaming reporters and lots of refactoring Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Support for mitre - with frequency, updated and explanation fields' Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * To only revise enhanced outcomes when attestation data is present for the control - fix in streaming reporter Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added test cases for attestation and also added validation warnings Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Attestation test for different formats of attestation file Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Validating presence of status column to be mandtory for attestation files Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Build fix Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Attestation build fix for windows Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * No justification and no status graceful handling Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * New class attestations added for logic and added missing test attestations file Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Code comments and cli doc changes for attestation option Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Moved logic of attestations and enhanced outcomes to the base of streaming reporter Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Attestation documentation added Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added information on what happens if justification is missing in attestation file Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Attestation doc changes as per the review Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * File fields doc changes in attestation doc Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Content Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Edits Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Attestation test changes matching regex and separated logic for expiration using frequency and updated date Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Proof-read Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Name changes for expiry calculation method Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Generic tests in attestations for cross platform Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> |
||
|
Vasundhara Jagdale
|
183d09c534
|
Adds podman resources. (#6183)
* CFINSPEC-368 Adds podman resource. Adds PodmanContainerFilter to handle podman.containers plural resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fix lint errors Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Adds style simple to the filter table fields to flatten the array Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to work with podman images plural resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to work with podman networks plural resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to work with podman pods plural resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to work with info and version parameter Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to work with podman volumes plural resource. Also updated the docs to add pods, networks and other resource parameters details. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-368 Updated podman resource to aspect object id to verify values of specified Podman object. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updated docs Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updated column names to keep them same as the field names Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Adds style simple to commands column for podman containers plural resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-360 Adds podman_container resource. Adds podman_object module Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fixed the require path Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fixed deprecation warning in unit test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fixed indentation issue in the docs. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Renamed the methods names Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Adds skip resource test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updated podman.images to fetch only high level information as using query with low level information does not have required fields and represents the data in different way. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * added method to parse command which uses format option to fetch specific placeholders Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Update the podman,containers to fetch only high level information Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Update podman.networks to fetch only hight level information Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Removed style for where it is not required Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Lint fix and some code improvisation Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * It turn out to be the docker object module can be utilized but still there is scope to rename that module to generic and not specific if going to use with other container management tools Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Missing file in earlier commit Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Content Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * CFINSPEC-361: Test for podman_image matchers and properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Add podman_image matchers and properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Documentation for podman_image matchers and properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Fix Rubocop lint issues Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Test for more properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Add more podman_image properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Documentation for podman_image properties Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Test for low-level information of image Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Add inspec_info property to test low-level info Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Documentation for inspec_info property Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Modify resource id to avoid dependency on other methods Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Decouple podman_image from podman plural resource Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Modify unit test and reqd fixtures Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Handle nil scenario of inspect_info Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Add missing property: id Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Update documentation Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Rename method and attribute name as suggested Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Add stderr message in exception Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Remove comment Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-361: Move json_key mapping to a method Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Content Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * CFINSPEC-361: Remove exception and add nil Signed-off-by: Sonu Saha <sonu.saha@progress.com> * CFINSPEC-351: failing tests for podman_network resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * CFINSPEC-351 Adds podman_network resource properties Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Fixed typo in the test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Refactored the code for podman_image to handle non exisiting image and for creating dynamic methods for properties Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Uncommented the image test Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Update the example for podman_network Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Missing test fixture file Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Adds doc for podman_network resource Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Doc Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> * Adds module for podman to group all common methods Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * Updated code to match the stderr string for non existing image and network. Also added the test for the same. Added code comments. Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io> * TEST: Add failing test for podman_volume (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * RESOURCE: Add podman_volume properties & matcher (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Add mock cmds and fixtures for test to pass (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Update documentation for podman_volume (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Add failing test for podman_pod (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * RESOURCE: Add podman_pod properties & matcher (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * TEST: Add mock cmds and fixtures for test to pass (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Update documentation for podman_pod (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * DOCS: Update properties of podman_volume (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * LINT: Remove trailing whitespaces (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * REFACTOR: Extend existing fixture, remove new fixtures (CFINSPEC-351) Signed-off-by: Sonu Saha <sonu.saha@progress.com> * Content Review Signed-off-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Deepa Kumaraswamy <dkumaras@progress.com> Co-authored-by: Sonu Saha <sonu.saha@progress.com> |
||
|
Nikita Mathur
|
b7ddac9dcc
|
CFINSPEC-240 Extended file format support for waivers (#6193)
* Added separate waiver file reader and support for csv Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added support for json format waivers Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Added support for xls and xlsx Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Build issues and updated description of gems Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc changes for waivers about supports Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Fix added to check final options presense in config Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Renamed variables from inputs to waivers Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Validation changes with other small changes Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Headers validation added for json and yaml * Linter issues resolved Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Some refactoring and message change Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * exit code check removed from test cases since not req Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> * Doc change for waiver support for excel by showing example Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com> |
||
Nikita Mathur
|
249f0aec9a |
Enhanced outcomes build fix for windows
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |
||
|
Nikita Mathur
|
c8351c9323 |
Added doc for only if impact change and modified test cases
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |
||
|
Nikita Mathur
|
1274a39690 |
Minitest deprecation warning fix for nil check
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io> |