(QA-2603) update MSI path for "install_pe_client_tools_on"

[zreichert]

No description provided.

[puppetlabs-jenkins]

Can one of the admins verify this patch?

[johnduarte]

I believe the spec tests need to be updated as well.

[johnduarte]

spec test diff

diff --git a/spec/beaker-pe/pe-client-tools/installer_helper_spec.rb b/spec/beaker-pe/pe-client-tools/installer_helper_spec.rb
index b3fb0c3..13c6d73 100644
--- a/spec/beaker-pe/pe-client-tools/installer_helper_spec.rb
+++ b/spec/beaker-pe/pe-client-tools/installer_helper_spec.rb
@@ -47,10 +47,10 @@ describe ClassPEClientToolsMixedWithPatterns do
       let(:platform) { Beaker::Platform.new('windows-2012r2-x86_64') }
       it 'installs' do
         hosts.each do |host|
-          allow(subject). to receive(:fetch_http_file).with("http://builds.delivery.puppetlabs.net/pe-client-tools/#{opts[:pe_client_tools_sha]}/artifacts/deb/xenial/PC1", "pe-client-tools_#{opts[:pe_client_tools_version]}-1xenial_x86_64.deb", "tmp/repo_configs")
+          allow(subject). to receive(:fetch_http_file).with("http://builds.delivery.puppetlabs.net/pe-client-tools/#{opts[:pe_client_tools_sha]}/artifacts/windows", "pe-client-tools_#{opts[:pe_client_tools_version]}-x86_64.msi", "tmp/repo_configs")
           allow(host). to receive(:external_copy_base)
           expect(subject).to receive(:generic_install_msi_on).with( host,
-                                                                   "http://builds.delivery.puppetlabs.net/pe-client-tools/#{opts[:pe_client_tools_sha]}/artifacts/windows/PC1/xx86_64/pe-client-tools-#{opts[:pe_client_tools_version]}.1-xx86_64_VANAGON.msi",
+                                                                   "http://builds.delivery.puppetlabs.net/pe-client-tools/#{opts[:pe_client_tools_sha]}/artifacts/windows/pe-client-tools-#{opts[:pe_client_tools_version]}-xx86_64.msi",
                                                                    {},
                                                                    { :debug => true }
                                                                   )

[zreichert]

@johnduarte Thanks! PR is updated.

[puppetlabs-jenkins]

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker-pe_btc-intn/51/

[puppetlabs-jenkins]

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker-pe_btc-intn/52/

[johnduarte]

👍

[johnduarte]

ping @kevpl or @tvpartytonight when you get a chance to look at this.

[tvpartytonight]

LGTM: I'll release this next week so we don't have to worry about broken builds over the weekend.