Closed
Bug 1379211
Opened 7 years ago
Closed 7 years ago
Add Travis tests for the Terraform configs
Categories
(Tree Management :: Treeherder: Infrastructure, enhancement, P2)
Tree Management
Treeherder: Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
Details
Attachments
(1 file)
Currently there are no tests run against the mozilla-platform-ops/devservices-aws repo where the Treeherder RDS configs live, making it harder for us to know we're not breaking things (exacerbated by our lack of familiarity with Terraform). I spotted there are a few nice ways to test Terraform configs plus related assets (eg json/shell files), for example what Travis use for their Terraform test run: https://github.com/travis-infrastructure/terraform-config/blob/7ed91676a42e17e2e6d03ecaa699c3f615acd4bd/runtests#L43
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8884332 -
Flags: review?(jwatkins)
Comment 2•7 years ago
|
||
Comment on attachment 8884332 [details] [review] devservices-aws PR#55: Add some tests and a Travis configuration Review of attachment 8884332 [details] [review]: ----------------------------------------------------------------- This is great! I'm all for CI checks although we might want to consider adding a notifications section. And since there are multiple stakeholders with regards to this repo I think Kendall would probably like to have some input here. Let's wait will he is back from PTO (next week) for his input.
Attachment #8884332 -
Attachment is patch: true
Attachment #8884332 -
Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8884332 -
Flags: review?(klibby)
Attachment #8884332 -
Flags: review?(jwatkins)
Attachment #8884332 -
Flags: review+
Updated•7 years ago
|
Attachment #8884332 -
Attachment is patch: false
Attachment #8884332 -
Attachment mime type: text/plain → text/x-github-pull-request
Assignee | ||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•7 years ago
|
||
(In reply to Jake Watkins [:dividehex] from comment #2) > This is great! I'm all for CI checks although we might want to consider > adding a notifications section. Yeah definitely worth considering, though even without it the PR status still shows up so warns people before merging (I see preventing a bad merge as the desired goal rather than notifying retrospectively that master is broken etc). For bonus points, someone with repo admin access can enable the "block merges unless CI has passed" option after this lands. > Let's wait will he is back from PTO (next week) for his input. Sounds good :-) Whilst working on this I also spotted some bugs in the upstream Travis test wrapper script that I used as inspiration, for which I've opened: https://github.com/travis-infrastructure/terraform-config/issues/185
Comment 4•7 years ago
|
||
Merged. Waiting on github support to unflag the new account I set up (devsvcs-travis) to connect the travis tests.
Assignee | ||
Comment 5•7 years ago
|
||
Great - thank you :-)
Comment 6•7 years ago
|
||
Done and working! Thanks!
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Attachment #8884332 -
Flags: review?(klibby)
Attachment #8884332 -
Flags: review+
Attachment #8884332 -
Flags: checkin+
Assignee | ||
Comment 7•7 years ago
|
||
Looking good :-) https://travis-ci.org/mozilla-platform-ops/devservices-aws/builds/256888397
You need to log in
before you can comment on or make changes to this bug.
Description
•