* fix: ensure Invoke-WebRequest headers can be configured
Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net>
* fix: use array for custom headers
Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net>
* test: add unit test for Windows simple HTTP request with headers
Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net>
---------
Signed-off-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net>
Co-authored-by: Adrian Mlodzianowski <amlodzianowski@logicworks.net>
* Fix systemd path for Leap image
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Use vhef client version 17 as doocker cookbook do not support >= 18
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Add nftables resource
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Add nftables tests
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Add fixtures for nftables tests
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* enable nftables only when attr is true - then disable iptables
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* By default test iptables, not nftables
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Fix tests and lint errors
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Increase unit test coverage for nftables
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Do not use -nn nft option as behaviour changes based on nft version
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* 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
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Fix uninitiallized class vars
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Fix unit test by adding nft version mocking
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Clean nftables doc
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
---------
Signed-off-by: Jeremy JACQUE <jeremy.jacque@algolia.com>
* Dependent profiles fix for any scheme of version used in profiles
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Dependent profile fix when version is blank in dependent profiles
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
---------
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Test profile consist of custom resource which has a gem dependency
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* CFINSPEC-576 Fix for profile gem depedency loading issue if depedent gem is required in libraries files of the profile
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Fix windows test failure for profile gem dependency
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
---------
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Matching semver using regex to determine profile name without version in dsl logic
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Testing for selecting control using version in require control dsl
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Documentation for using version with profile name when including or selecting controls in a profile
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Commenting in dsl library about logic for fetching version and matching it
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Comment for regex used to fetch version
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Minor improvement for regex variable placement
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Revert "Revert "CFINSPEC-240 Extended file format support for waivers (#6193)" (#6366)"
This reverts commit 30146e07fe.
* Waiver support for csv and json
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Removed the word excel from image tag
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Update waivers.md
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Co-authored-by: Ian Maddaus <ian.maddaus@progress.com>
* 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.
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Adds test for reading description from local gemspec file.
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* 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>
* 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>
* 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>
* 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>
* add ruby test 3.1 in verify pipeline
Signed-off-by: jayashri garud <jgarud@msystechnologies.com>
* updating rubygem to 3.3.3
Signed-off-by: jayashri garud <jgarud@msystechnologies.com>
* Test fixture plugin files for ABI 3.1.0
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Use kwargs to match new calling style for ERB in ruby 2.7 and 3.0
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Allow Date and Time data types, needed for waivers, since safe load is now the default for YAML
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Use Time.parse, not Time.new since the rules tightened for what you can pass the constructor for waiver dates
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Add ruby platform debug message to test parallel output
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Conditionalize permitted_classes in inputs yaml_load for <= 3.1.x only
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* In ruby 3.1.0 net/pop3 is no more a default gem and that will require us to add it in Gemfile so trying with the byebug gem which we are bundling already
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* CFINSPEC-166 Updated url fetcher to handle Net::OpenTimeout Exception on Windows
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Co-authored-by: Vasu1105 <vasundhara.jagdale@chef.io>
