| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
| |
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
|
| |
|
|
|
|
| |
Autocorrected from RuboCop Performance which is now smart enough to detect when you use the match and when you don't. Using match? does not create any objects so it's slightly faster and uses less memory.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
|
|
| |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
|
|
| |
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
|
| |
|
|
|
|
| |
Thanks for pointing this out Ian
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
|
|
| |
Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
|
| |
|
|
| |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
|
|
| |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
|
|
|
|
|
|
|
|
|
|
| |
While Chef already implements a nice amount of information
in `TemplateError#to_s`, when using ChefSpec with RSpec, the `#to_s`
is not called, leaving us with no source information for template
errors.
By adding the filename to `Erubis::Eruby` initialization, we can get
a better backtrace that will show up in RSpec output.
Signed-off-by: Gabriel Mazetto <brodock@gmail.com>
|
| |\
| |
| |
| |
| | |
MsysTechnologiesllc/Kapil/MSYS-986_Move_response_file_and_response_file_variables_out_of_base_package_resource
package: move response_file and response_file_variables out of base package resource
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
Fix for Move response_file and response_file_variables out of base package resource
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
update require changes
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
Updated require changes
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
Fixed some unit test cases
Signed-off-by: Kapil Chouhan <kapil.chouhan@msystechnologies.com>
|
| |/
|
|
|
|
|
| |
Fixes #6281 which is a frequently reported issue where md files and swap
files and other junk causes the cookbook compiler to barf.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|
|
|
|
|
|
| |
When no certs are available, test with an actual empty certs dir.
This works around a problem with the mock where on the Windows
platform, a buried call to Dir.glob is returning the actual certificates
directory content instead of the empty list required for the test.
Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
|
| |
|
|
|
|
|
|
|
|
| |
certificates
- P7B might contain multiple certificates and we should not miss the internal one while import.
- Added test cases and ensured chef-style
- Fixes: MSYS-977
Signed-off-by: Nimesh-Msys <nimesh.patni@msystechnologies.com>
|
| |
|
|
|
|
|
|
|
|
| |
- Till now, cer certificates were only considered to be in binary format (DER)
- They can also be base-64 encoded(PEM)
- We should only append "inform DER" only if it is a binary certificate, otherwise, default ("inform PEM") would support base64 encoded certificates.
- Added test caes
- Ensured Chef style
Signed-off-by: Nimesh-Msys <nimesh.patni@msystechnologies.com>
|
| |
|
|
|
|
| |
- Mostly focus on all major actions and their idempotency.
Signed-off-by: Nimesh-Msys <nimesh.patni@msystechnologies.com>
|
| |
|
|
| |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
|
|
| |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
|
|
|
|
| |
We can greatly simplify our dscl user provider by removing support for macOS 10.7 and 10.7 which has been upgraded to 10.8. No one is using this anyone as it's beyond EOL with Apple and has been for years.
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
|
|
|
| |
move snap api json response into fixture data - chefstyle was very unhappy with inline data defs
Signed-off-by: S.Cavallo <smcavallo@hotmail.com>
|
| |
|
|
|
|
| |
This new field tracks the file from which the object was loaded.
Signed-off-by: Zeal Jagannatha <zealjagannatha@gmail.com>
|
| |
|
|
|
|
| |
Fixes #6680
Signed-off-by: Allan Espinosa <aespinosa33@bloomberg.net>
|
| |\
| |
| | |
Improve coverage of root alias tests
|
| | |
| |
| |
| | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |\ \
| |/
| |
| | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| | |
| |
| |
| | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| | |
| |
| |
| | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |/
|
|
|
|
| |
feel like updating them this second.
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |
|
|
| |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| |
|
|
| |
Signed-off-by: Salim Afiune <afiune@chef.io>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
most of this deletes useless old code.
the change to lookup_provider_constant changes to more strictly
stop using class name based lookups and to go through the resource
resolver and provider resolver.
as a result in order to find the provider class for a given dsl name
you have to go through the resource resolver to find the resource in
order to be able to pass a resource instance through the provider
resolver. since the provider resolver api passes resources into blocks
passed into provides api you must construct a resource instance.
that means that providers need to be associated with resources in order
to be looked up (which makes sense in Real Life(tm) use of Chef, but
breaks quite a few lazy tests we had where we constructed providers
without doing the work of wrapping them in a resource.
note that as the deploy resource shows this filters into a changed
behavior of the `provider` syntax where before `provider :revision`
would look up Chef::Provider::Deploy::Revision via class-name based
magic. this breaks that API so that `provider :deploy_revision` is
used instead -- the symbol (or string) there is turned into a resource
first via the Chef::ResourceResolver and then looked up via the
Chef::ProviderResolver into Chef::Provider::Deploy::Revision. this
is a breaking change but is also a bug fix so that the symbol here
goes through the same lookup that you get when you type it in the DSL.
i had considered implementing a lookup from a resource_name symbol to
a provider, but in looking at how to implement that in the
ProviderResolver the issue is that we really need to have a resource
instance order to pass to the ProviderResolver.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|
|
|
|
| |
make sure that if we have a metadata.rb that is not valid to read that
we never parse that and use compiled metadata.json instead.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|
|
|
| |
Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
|
| |
|
|
|
|
| |
contain a common name. Stores the certificate under the URI host instead of the common name in that case
Signed-off-by: tyler-ball <tyleraball@gmail.com>
|
| |
|
|
| |
Signed-off-by: Vasu1105 <vasundhara.jagdale@clogeny.com>
|
| |
|
|
|
|
|
|
|
|
|
| |
- fixes *_unless behavior and set_unless_value_present hack from Chef 12
- simplifies rm_* code
- introduces functional read/write/unlink/exist? API
- deprecates method_missing access to attributes for Chef 13
- deprecates set/set_unless aliases for Chef 14
- removes MultiMash mess that I wrote for Chef 13
https://github.com/chef/chef/pull/5029 for more details
|
| | |
|
| | |
|
| |
|
|
|
| |
client.d/*.rb will be read in sorted order. All directories will be
ignored.
|
| |
|
|
|
| |
When knife finds a client.d/ directory, it will upload
all files nested under that directory.
|
| | |
|
| |
|
| |
Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
|
| |
|
| |
Generated via git ls-files | xargs perl -pi -e "s/[Cc]opyright (?:\([Cc]\) )?((?\!$(date +%Y))\\d{4})(-\\d{4})?([, ][ \d]+)*(,|(?= ))/Copyright \\1-$(date +%Y),/g"
|
| |\
| |
| | |
more rubocop fixes
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
fixes:
Lint/UnreachableCode
Lint/NonLocalExitFromIterator
Lint/LiteralInCondition
Lint/EmptyEnsure
Lint/DefEndAlignment
Lint/CircularArgumentReference
|
| |/
|
|
|
| |
This cookbook metadata uses the supports method, which currently parses versions
using the cookbook version parser, when it should be using the platform parser.
|
| | |
|
| |
|
|
|
|
|
|
| |
- multipackages dpkg_package
- fixes edge conditions in load-current-resource around purging/removing
packages that threw errors before
- fixes the ability to purge packages that have been removed
- adds a lot of functional tests for dpkg_package
|
| |\
| |
| | |
Fixing Issue #2513 - the broken render of nested partial templates
|