Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Polling tentacles need service restarted #55

Closed
spuder opened this issue Aug 17, 2016 · 3 comments · Fixed by #57
Closed

Polling tentacles need service restarted #55

spuder opened this issue Aug 17, 2016 · 3 comments · Fixed by #57
Labels

Comments

@spuder
Copy link
Contributor

spuder commented Aug 17, 2016

Trying to deploy octopus in mass. I find polling tentacles are installed properly, however they always fail their health check until I manually restart the service.

This is strange because the configure tentacle resource notifies the windows service to restart with a :delayed timer, yet it looks like the restart happens before the register action happens.

Recipe: nd-octopus::tentacle
  * octopus_deploy_tentacle[Tentacle] action install
    * remote_file[C:\chef\cache/octopus-tentacle.msi] action create (up to date)
    * windows_package[Octopus Deploy Tentacle] action install (up to date)
     (up to date)
  * octopus_deploy_tentacle[Tentacle] action configure
    * windows_firewall_rule[Octopus Deploy Tentacle] action create (skipped due to only_if)
    * octopus_deploy_tentacle[Tentacle] action install
      * remote_file[C:\chef\cache/octopus-tentacle.msi] action create (up to date)
      * windows_package[Octopus Deploy Tentacle] action install (up to date)
       (up to date)
    * directory[C:\Octopus] action create (up to date)
    * powershell_script[generate-tentacle-cert] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-2116-176koe7.ps1"
    * powershell_script[create-instance-Tentacle] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-2116-1h1q6d6.ps1"
    * powershell_script[configure-tentacle-Tentacle] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-2116-qnlanm.ps1"
    * windows_service[OctopusDeploy Tentacle] action enable (up to date)
    * windows_service[OctopusDeploy Tentacle] action start (up to date)
    * windows_service[OctopusDeploy Tentacle] action restart
      - restart service windows_service[OctopusDeploy Tentacle]

  * octopus_deploy_tentacle[Tentacle] action register
    * powershell_script[register-Tentacle-tentacle] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-2116-1913boz.ps1"

Starting audit phase�����������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������
@spuder
Copy link
Contributor Author

spuder commented Aug 17, 2016

I thought this might have been some interaction between new_resource and the fact that my wrapper cookbook has the configure and register resources as 2 separate resources

octopus_deploy_tentacle 'Tentacle' do
  action [:install, :configure]
  version node['nd-octopus']['version']
  checksum node['nd-octopus']['checksum']
  trusted_cert node['nd-octopus']['trusted_cert']
end

octopus_deploy_tentacle 'Tentacle' do
  action :register
  server node['nd-octopus']['server']
  api_key node['nd-octopus']['api_key']
  roles node['nd-octopus']['roles']
  environment node.chef_environment.to_s
  polling node['nd-octopus']['polling']
  configure_firewall node['nd-octopus']['configure_firewall']
end

I've combined my 2 resources down to 1, but I'm still finding that the service restart is happening too early in the chef run.

   * powershell_script[create-instance-Tentacle] action run (skipped due to not_if)
    * powershell_script[configure-tentacle-Tentacle] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-6000-13uugvk.ps1"
    * windows_service[OctopusDeploy Tentacle] action enable (up to date)
    * windows_service[OctopusDeploy Tentacle] action start (up to date)
    * windows_service[OctopusDeploy Tentacle] action restart
      - restart service windows_service[OctopusDeploy Tentacle]

  * octopus_deploy_tentacle[Tentacle] action register
    * powershell_script[register-Tentacle-tentacle] action run
      - execute "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/sowen/AppData/Local/Temp/2/chef-script20160817-6000-xribau.ps1"����������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������

@brentm5
Copy link
Member

brentm5 commented Aug 17, 2016

Should the register resource restart the tentacle after it runs?

@brentm5
Copy link
Member

brentm5 commented Aug 17, 2016

Just saw #56 and merged it. Can you confirm if this fixes the issue or not?

@brentm5 brentm5 added the bug label Aug 17, 2016
spuder added a commit to spuder/octopus-deploy-cookbook that referenced this issue Aug 18, 2016
spuder added a commit to spuder/octopus-deploy-cookbook that referenced this issue Aug 18, 2016
spuder added a commit to spuder/octopus-deploy-cookbook that referenced this issue Aug 18, 2016
spuder added a commit to spuder/octopus-deploy-cookbook that referenced this issue Aug 18, 2016
brentm5 added a commit that referenced this issue Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants