iis_config fails with certain commands #89

kb18951452 · 2014-11-14T05:15:46Z

I'd love someone else to verify this:

iis_config "-section:system.webServer/handlers /[name=\'ExtensionlessUrlHandler-Integrated-4.0\'].verb:\"GET,HEAD,POST,DEBUG,PUT,DELETE\"  /commit:apphost" do
    action :config
end

getting this error

==> default: Ran 
C:\Windows\System32\inetsrv\appcmd.exe set config -section:system.webServer/handlers /[name='ExtensionlessUrlHandler-Integrated-4.0'].verb:"GET,HEAD,POST,DEBUG,PUT,DELETE"  /commit:apphost returned 4312

Running the command it says it ran works......

C:\Windows\System32\inetsrv\appcmd.exe set config -section:system.webServer/handlers /[name='ExtensionlessUrlHandler-Integrated-4.0'].verb:"GET,HEAD,POST,DEBUG,PUT,DELETE"  /commit:apphost

The text was updated successfully, but these errors were encountered:

EasyAsABC123 · 2014-12-22T17:39:19Z

Just ran:

iis_config "-section:system.webServer/handlers /[name=\'ExtensionlessUrlHandler-Integrated-4.0\'].verb:\"GET,HEAD,POST,DEBUG,PUT,DELETE\" /commit:apphost" do
    action :config
end

this is the chef result:

[2014-12-22T12:36:18-05:00] INFO: Processing iis_config[-section:system.webServer/handlers /[name='ExtensionlessUrlHandler-Integrated-4.0'].verb:"GET,HEAD,POST,DEBUG,PUT,DELETE" /commit:apphost] action config (hps-atlas-web::default line 23)

@kbarry-x Can you give the following information:
chef version?
iis version?

Thanks

kb18951452 · 2014-12-22T21:09:44Z

chef-server 11.0.10

iis version is 8.5

EasyAsABC123 · 2014-12-23T05:14:56Z

Cool, i just tested that with the latest code on a vagrant Windows Server 2012 x64 without error.
Are you on the latest iis cookbook?

webframp · 2015-01-28T21:40:56Z

Saw something similar with this snippet in a recipe:

  iis_config "/section:staticContent /+\"[fileExtension='.gem',mimeType='application/octet-stream ']\"" do
    action :config
  end

---- Begin output of C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" ----
STDOUT: ERROR ( message:New mimeMap object missing required attributes. Cannot add duplicate collection entry of type 'mimeMap' with unique key attribute 'fileExtension' set to '.gem'. )
STDERR:
---- End output of C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" ----
Ran C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" returned 183

This worked the first time it ran. It could just be my ignorance with how appcmd work, but it seems like maybe the resource isn't really idempotent in some cases?

chef-client 11.16.2, iis version 7.5 and iis cookbook version 2.1.6

kb18951452 · 2015-01-28T22:27:53Z

In '/+"[fileExtension .... ' the "+" means "Add", exit 183 means "This
already exists, you can't add it again"

You can change the expected exit codes to include 183, as its not really a
bad exit, it simply means "I am already there".

On Wed, Jan 28, 2015 at 1:40 PM, Sean Escriva notifications@github.com
wrote:

Saw something similar with this snippet in a recipe:

iis_config "/section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']"" do
action :config
end

---- Begin output of C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" ----
STDOUT: ERROR ( message:New mimeMap object missing required attributes. Cannot add duplicate collection entry of type 'mimeMap' with unique key attribute 'fileExtension' set to '.gem'. )
STDERR:
---- End output of C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" ----
Ran C:\Windows\System32\inetsrv\appcmd.exe set config /section:staticContent /+"[fileExtension='.gem',mimeType='application/octet-stream ']" returned 183

This worked the first time it ran. It could just be my ignorance with how
appcmd work, but it seems like maybe the resource isn't really idempotent
in some cases?

chef-client 11.16.2, iis version 7.5 and iis cookbook version 2.1.6

—
Reply to this email directly or view it on GitHub
#89 (comment)
.

Kenneth Barry
TuneIn | Build and Release Engineer
M: 409-673-0544
www.tunein.com

EasyAsABC123 · 2015-01-29T00:27:23Z

@webframp since you can run anything and everything via iis_config it can't really be idempotent...it is going to run everytime chef-client runs. With this specific example is there a reason you don't require the developers to code this into the web.config

require 'chef/mixin/shell_out'

include Chef::Mixin::ShellOut
include Opscode::IIS::Helper

action :config do
  cmd = "#{appcmd(node)} set config #{@new_resource.cfg_cmd}"
  Chef::Log.debug(cmd)
  shell_out!(cmd, :returns => @new_resource.returns)
  Chef::Log.info("IIS Config command run")
  @new_resource.updated_by_last_action(true)
end

this is all there is to the iis_config provider

@kbarry-x adding 183 doesn't look like a bad idea at all, let me think on it and see if I can find an issue.
one other solution would be make it accept any error (since it just runs anything given to it)...trying to make it idempotent is an impossible task I believe...

good find though

kb18951452 · 2015-02-05T21:44:23Z

I was not using the latest version of the cookbook. I am getting the latest version now. When I attempted to earlier, I ran into #95

I am updating, and testing now. Wish me luck.

ksubrama · 2015-02-12T22:17:22Z

The /+ config commands are currently not idempotent and I think there is some lingering issue in detecting them correctly. Let me know if the newest version works for you. If not, I'll keep an careful eye for this when doing a bug-fix pass.

kb18951452 · 2015-03-31T19:08:21Z

I'm still getting the erro, using the latest of both the windows and iis cookbooks.

I'm going to change

resources/config.rb

attribute :returns, :kind_of => [Integer, Array], :default => 0
to
attribute :returns, :kind_of => [Integer, Array], :default => [0,183]

EasyAsABC123 · 2015-03-31T19:11:54Z

@kbarry-x please test that and submit a pull request.

kb18951452 · 2015-03-31T19:16:56Z

183 is the "Its already there" code. And i believe it was a hack/fix in a
previous version employed by others. Lemme see if I can figure out a
pull-request ;)

On Tue, Mar 31, 2015 at 12:11 PM, Justin Schuhmann <notifications@github.com

wrote:

@kbarry-x https://github.com/kbarry-x please test that and submit a
pull request.

—
Reply to this email directly or view it on GitHub
#89 (comment)
.

Kenneth Barry
TuneIn | Build and Release Engineer
M: 409-673-0544
www.tunein.com

EasyAsABC123 · 2015-03-31T20:29:49Z

Actually this isn't a pull request, here is the solution...

add an attribute to your provider call

iis_config "-section:system.webServer/handlers /[name=\'ExtensionlessUrlHandler-Integrated-4.0\'].verb:\"GET,HEAD,POST,DEBUG,PUT,DELETE\"  /commit:apphost" do
    returns [0, 183]
end

A global change isn't a great idea here but this will definitely solve your use case.

@ksubrama mind validating that you agree changing this globally isn't the best idea?

kb18951452 · 2015-03-31T21:00:12Z

Awesome. Thanks very much. I wasnt aware of "returns" but I suspect its a
"common attribute"

On Tue, Mar 31, 2015 at 1:29 PM, Justin Schuhmann notifications@github.com
wrote:

Actually this isn't a pull request, here is the solution...

add an attribute to your provider call

iis_config "-section:system.webServer/handlers /[name='ExtensionlessUrlHandler-Integrated-4.0'].verb:"GET,HEAD,POST,DEBUG,PUT,DELETE" /commit:apphost" do
action :config
returns [0, 183]end

A global change isn't a great idea here but this will definitely solve
your use case.

—
Reply to this email directly or view it on GitHub
#89 (comment)
.

Kenneth Barry
TuneIn | Build and Release Engineer
M: 409-673-0544
www.tunein.com

EasyAsABC123 · 2015-03-31T22:10:33Z

@kbarry-x never a problem, this is definitely a permanent solution for you. We just need to decide if that's the permanent solution for everyone via default.

kb18951452 · 2015-03-31T22:31:00Z

How do we handle configs that are in the form of an "addition"?
On Tue, Mar 31, 2015 at 3:10 PM, Justin Schuhmann notifications@github.com
wrote:

@kbarry-x https://github.com/kbarry-x never a problem, this is
definitely a permanent solution for you. We just need to decide if that's
the permanent solution for everyone via default.

—
Reply to this email directly or view it on GitHub
#89 (comment)
.

Kenneth Barry
TuneIn | Build and Release Engineer
M: 409-673-0544
www.tunein.com

EasyAsABC123 · 2015-03-31T23:06:27Z

@kbarry-x can you provide an example?

EasyAsABC123 · 2015-07-27T22:42:50Z

@kbarry-x Closing issue, appears to be resolved. If you have an example of the issue persisting please re-open. Thanks

EasyAsABC123 closed this as completed Jul 27, 2015

spuder mentioned this issue Aug 24, 2015

Set logging custom fields #205

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

iis_config fails with certain commands #89

iis_config fails with certain commands #89

kb18951452 commented Nov 14, 2014

EasyAsABC123 commented Dec 22, 2014

kb18951452 commented Dec 22, 2014

EasyAsABC123 commented Dec 23, 2014

webframp commented Jan 28, 2015

kb18951452 commented Jan 28, 2015

EasyAsABC123 commented Jan 29, 2015

kb18951452 commented Feb 5, 2015

ksubrama commented Feb 12, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

EasyAsABC123 commented Jul 27, 2015

iis_config fails with certain commands #89

iis_config fails with certain commands #89

Comments

kb18951452 commented Nov 14, 2014

EasyAsABC123 commented Dec 22, 2014

kb18951452 commented Dec 22, 2014

EasyAsABC123 commented Dec 23, 2014

webframp commented Jan 28, 2015

kb18951452 commented Jan 28, 2015

EasyAsABC123 commented Jan 29, 2015

kb18951452 commented Feb 5, 2015

ksubrama commented Feb 12, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

kb18951452 commented Mar 31, 2015

EasyAsABC123 commented Mar 31, 2015

EasyAsABC123 commented Jul 27, 2015