Ali Rizvi-Santiago
4783b6508e
Fix common/download_test.go to avoid formatting the volume name to a hidden windows share when not on windows.
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
d275bacb0f
go fmt builder/vmware/iso/step_create_vmx.go to calm down Travis CI.
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
e940dc7e42
Fixed a config_test that should've failed but didn't because ftp:// uris work now. HeH!
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
e42a23ecb5
Ugh..added dependency for gopkg.in/cheggaaa/pb.v1: github.com/matn/go-runewidth. Also added github.com/jlaffaye/ftp for ftp support.
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
c29e3915be
Added gopkg.in/cheggaaa/pb.v1 to vendor/vendor.json via govendor.
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
b1ff14714b
go fmt
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
11ff4439a6
Moved the setting of HTTPDownloader's current progress to after the object actually gets instantiated. ;)
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
0fa6c3782e
Added a progressbar using gopkg.in/cheggaaa/pb.v1 as per #3578 for all the DownloadClients in common/download.go.
2018-01-16 13:37:31 -06:00
Ali Rizvi-Santiago
2f1104625d
Fixed some of the unit-tests in common/ due to the changes made in {config,download}.go
...
config.go:
Fixed some issues related to the url scheme not being lowercased which broke some of the tests.
config_test.go:
Removed the UNC share test for \\host\share\file since SMB support has been moved to a different uri scheme.
download_test.go:
Explicitly set the CopyFile configuration option for all the unit-tests that test file copying capability.
Removed the UNC share testcase since it's under a different uri scheme now.
Modified the file:// UNC share testcase to explicitly test the smb:// uri.
Changed the incorrect t.Errorf calls to t.Logf so that the tests can pass.
2018-01-16 13:37:30 -06:00
Ali Rizvi-Santiago
6170e24ecb
Refactored the code a bit to move the CopyFile hack out of DownloadClient and instead into each protocol.
...
config.go:
Removed all of the windows-specific net/url hackery since it's now handled mostly by download.go
Removed the replacement of '\' with '/' since url.Parse does it now.
Added knowledge of the other protocols implemented in download.go (ftp, smb)
Removed some modules that were unused in this commit.
download.go:
Moved the file-path conversions for the different protocols into their own internally callable functions.
Shuffled some of the functions around in case someone wants to implement the ability to resume.
Modified DownloadClient.Get to remove the CopyFile special case and trust the protocol implementations if a user doesn't want to copy the file.
Since all the protocols except for HTTPDownloader implement Cancel, added a Resume method as a placeholder for another developer to implement.
Added a few missing names from their function definitions.
Fixed the syntax in a few lines due to my suckage at go.
Adjusted the types for progress and total so that they support 64-bit sizes.
Removed the usage of the bufio library since it wasn't really being used.
2018-01-16 13:37:30 -06:00
Ali Rizvi-Santiago
60831801a7
Added the file, ftp, and smb downloaders to common/download.go
2018-01-16 13:36:18 -06:00
Ali Rizvi-Santiago
da9c94b345
Added some testcases for the various file uri transforms to download_test.go
...
Moved some of the code for normalizing a Windows file uri to a regular path into it's own function NormalizeWindowsURL
2018-01-16 13:36:18 -06:00
Ali Rizvi-Santiago
281dd1258a
Added proper support for downloading via a Windows UNC path or a relative uri.
...
Added proper support for validating a downloadableURL containing a UNC or relative uri.
Removed the workaround for an earlier Go issue that had remained dormant in common/download.go (issue #5927 ).
When building a .vmx file via the vmware-iso builder, transform the path to the correct os-formatted one (using filepath.FromSlash).
2018-01-16 13:36:17 -06:00
Matthew Hooker
e222d60b5a
update changelog
2018-01-12 15:11:50 -08:00
Matthew Hooker
cea2ab8c6d
move kms/encrypted validation to block devices
2018-01-12 15:10:51 -08:00
Matthew Hooker
0023aa11cf
add kms_key_id to block device docs
2018-01-12 14:48:18 -08:00
Matthew Hooker
e968f9d6cc
spelling/style fixes
2018-01-12 14:26:56 -08:00
Matthew Hooker
cfd24eddbf
Merge pull request #5774 from jwieringa/f-kms-key-id
...
Added KmsKeyID to Amazon Builder DeviceBlock
2018-01-12 14:23:19 -08:00
Jason Wieringa
f6772f06be
builder/amazon: Raise error when ebsvolume kms_key_id is without encrypted = true
2018-01-11 20:27:38 -08:00
Jason Wieringa
2ac59b3c27
builder/amazon: Added KmsKeyID to BlockDevice
...
+ Adds kms_key_id to list of options
+ Tests that configuraiton is set along with encrypted
+ Updates documentation on ebsvolume builder
2018-01-11 20:27:38 -08:00
Jason Wieringa
78ff4d1eed
Updated github.com/aws/aws-sdk-go/service/ec2
...
Upgrades to v1.12.57 for the field KmsKeyID on EbsBlockDevice
introduced in v1.12.35 on November 2017.
2018-01-11 20:27:38 -08:00
Matthew Hooker
6f2669c044
Revert "add note about vix api libraries"
...
This reverts commit c5ec92c88b
.
2018-01-11 16:48:39 -08:00
Matthew Hooker
c5ec92c88b
add note about vix api libraries
2018-01-11 15:44:49 -08:00
Matthew Hooker
6d6216419c
update changelog
2018-01-11 12:14:34 -08:00
Matthew Hooker
2ce3ca1538
Merge pull request #5773 from hashicorp/fix5705
...
Use SDK's InstanceReady waiter
2018-01-11 12:12:03 -08:00
SwampDragons
fb730cf521
Merge pull request #5761 from hashicorp/fix_5713
...
fix nasty edge case where we can't find guest additions on windows if they are on a different drive
2018-01-11 11:48:26 -08:00
Megan Marsh
bdd186fa2b
add tests for fileexistslocally helper function
2018-01-10 16:44:27 -08:00
Megan Marsh
3ace5bb91b
simplify FileExistsLocally
2018-01-10 16:11:17 -08:00
Matthew Hooker
f5ea1e8312
Use WaitUntilInstanceReady waiter
2018-01-10 15:57:31 -08:00
Megan Marsh
55ddbf4765
sloppy copypasta
2018-01-10 10:08:23 -08:00
Megan Marsh
898dadd53c
re-add this block. I still don't think we need it but I don't want to risk breaking things with this bugfix.
2018-01-10 10:03:36 -08:00
SwampDragons
4b8476b3f4
Merge pull request #5781 from diegogoding/fix5779
...
fixes issue 5779
2018-01-10 09:59:37 -08:00
Diego Goding
38b1cdd8c4
fixed issue 5779
2018-01-10 08:56:59 -06:00
Megan Marsh
a04a921c2d
add UNC path to test cases, so I can try to enable it in future
2018-01-09 17:14:32 -08:00
Megan Marsh
154973241f
add a bunch of windows filepath tests
2018-01-09 16:57:52 -08:00
Megan Marsh
40f0cc6dfe
I don't think this is needed anymore
2018-01-09 15:53:54 -08:00
Megan Marsh
2838a2371d
disambiguate url variable from url library
2018-01-09 15:47:18 -08:00
Megan Marsh
216c44b153
fix FileExistsLocally
2018-01-09 15:47:07 -08:00
Matthew Hooker
60a136b20a
Merge pull request #5776 from hashicorp/hashicorp-middleman-0.3.29
...
Update hashicorp-middleman to 0.3.29
2018-01-09 13:16:01 -08:00
Matthew Hooker
2e8561a28d
update middleman-hashicorp
2018-01-09 11:39:55 -08:00
Justin Campbell
e4c1805f81
Update hashicorp-middleman to 0.3.29
...
Also includes Nokogiri 1.8.1, to fix https://github.com/hashicorp/packer/network/dependencies#30455077
2018-01-09 13:11:22 -05:00
SwampDragons
0d711bf2eb
Merge pull request #5768 from hashicorp/better_download_tests
...
reformat TestDownloadableURL into a table test to allow adding more URLS
2018-01-09 09:59:43 -08:00
Matthew Hooker
dfd5f07145
fix ebs acc test
2018-01-08 12:27:10 -08:00
SwampDragons
df790a889d
Merge pull request #5765 from lmayorga1980/master
...
improve documentation when vmware-iso is used on amazon-import post-p…
2018-01-08 09:12:01 -08:00
lmayorga
ccf404ea75
fix typo
2018-01-06 13:13:19 -05:00
Rickard von Essen
37ab70c4ae
Updated CHANGELOG.md
2018-01-06 11:23:41 +01:00
Rickard von Essen
15eccb1aa5
Merge pull request #5758 from GSA/better-user
...
make user retrieval for Ansible provisioner more robust
2018-01-06 11:22:03 +01:00
Megan Marsh
a3d5d40f78
reformat TestDownloadableURL into a table test to allow adding more URLS
2018-01-05 11:06:26 -08:00
Megan Marsh
e164621bfe
log line to help prevent confusion when guest_additions_url is set from env variable that is empty
2018-01-05 09:19:38 -08:00
lmayorga
7a4709400b
improve documentation when vmware-iso is used on amazon-import post-processr
2018-01-04 21:49:36 -05:00