-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add --tf-download-url
configuration option
#787
Add --tf-download-url
configuration option
#787
Conversation
c864731
to
fc2c66a
Compare
Codecov Report
@@ Coverage Diff @@
## master #787 +/- ##
==========================================
+ Coverage 71.83% 71.86% +0.02%
==========================================
Files 65 65
Lines 5213 5217 +4
==========================================
+ Hits 3745 3749 +4
Misses 1183 1183
Partials 285 285
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Almost there. I don't think it makes sense to be in Downloader
actually, I like where you put it. Downloader
is just used as a shim during testing.
Ok(t, err) | ||
|
||
Ok(t, err) | ||
Equals(t, "0.11.10", c.DefaultVersion().String()) | ||
baseURL := "https://releases.hashicorp.com/terraform/0.11.10" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe set this to https://my-mirror.releases.mycompany.com
in this test to make it clear it's being tested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I adjusted this to match what you are asking, can you double check?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes this looks good
fc2c66a
to
ac008b1
Compare
Co-Authored-By: Luke Kysow <1034429+lkysow@users.noreply.github.com>
I just tried to test this with
|
Atlantis runs |
This fixes #614
I'm new to Go so I'm sure I'll need plenty of changes before this can be merged. The main thing I'm unsure about is how I implemented the downloadUrl. I feel as though the modifications I made would fit better as part of
Downloader
however I'm not entirely sure the best way to go about modifying that.I don't currently have access to an internal mirror of releases.hashicorp.com so I haven't had a chance to fully test this functionality myself. I'm hoping to have the time to do that later this week.