* CHEF-3832: Fix for InSpec Parallel fails to fetch remote profiles due to cache contention.
This fix creates the write exclusive lock on the cache directory of profile till write operation is finished for git fetcher
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Refactored to have locking and unlocking logic inside cache. Cache directory is not required for archived directories and this is currently only required by git fetcher
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* REFACTOR: adds method in fetcher to return true if requires cache locking mechanism to avoid cache contention
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Adds exception handling
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Handles interruption in cache creation process to avoid empty cache directory creation or currupted data
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Refactored to handle interruption through SystemExit
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Fixed the failing test. While creating lock cache_key need to be extracted from the respective fetcher.
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
---------
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* FIX: Handle invalid license exception as handled in base_cli:57
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Use 174 error code when exiting with license subsystem error from license add
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* 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>
* Error logging changes to fix renaming of files in windows
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Parallel rename, delete and logging fix for windows
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Log path changes reverted after testing with static logs directory
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Fix lint
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Logic to rename error log files on interrupt
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* An attempt to reduce the delay on control c interrupt by killing child process instantly
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Only rename existing file of the child process to avoid conficts in renaming
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Fix in renaming with control-c interrupt by invoking from command library itself
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Added logic to wait for processes to die after process kill, to fix error on interrupt
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Removed idle inititation on status super reporter for parallel - needs future improvements
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* Fileutils Mkdir usage refactoring
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
---------
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
Co-authored-by: Nikita Mathur <nikita.mathur@chef.io>
Co-authored-by: Vasu1105 <vasundhara.jagdale@chef.io>
Sending EOF Marker at the end of of the whole suit run here instead of depending on the controls count
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Co-authored-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Update license commands description to be updated for local mode
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* To hide licensing information in help for CINC & other distros
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
---------
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
* 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>
* 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>
* 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>
