Page MenuHomeSolus
Differential D3290

Initial inclusion of terraform
ClosedPublic
Actions

Authored by der_eismann on Jul 13 2018, 8:15 PM.
Tags
None
Referenced Files
F11054825: D3290.id8129.diff
Fri, Aug 11, 1:13 AM
F11054824: D3290.id.diff
Fri, Aug 11, 1:13 AM
F11054823: D3290.id8293.diff
Fri, Aug 11, 1:13 AM
F11054821: D3290.id8294.diff
Fri, Aug 11, 1:13 AM
F11038250: D3290.diff
Wed, Aug 9, 10:24 PM
F10927919: D3290.id8129.diff
Jul 10 2023, 9:17 PM
F10908366: D3290.id.diff
Jul 6 2023, 11:59 PM
F10845392: D3290.diff
Jun 11 2023, 12:05 AM
View All 14 Files
Subscribers
JoshStrobl

Details

Reviewers
JoshStrobl
Group Reviewers
Triage Team
Maniphest Tasks
T5362: Terraform
Commits
R4354:45aa5e5d8226: Initial inclusion of terraform
Summary

Initial commit of terraform, fixes T5362.

Test Plan
  • Cloned terraform config to local folder
  • run terraform init to create a key pair
  • change config files
  • run terraform plan to see changes
  • run terraform apply to apply changes

Diff Detail

Repository
R4354 terraform
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

der_eismann created this revision.Jul 13 2018, 8:15 PM
Herald added a reviewer: Triage Team. · View Herald TranscriptJul 13 2018, 8:15 PM
der_eismann requested review of this revision.Jul 13 2018, 8:15 PM
der_eismann mentioned this in T5362: Terraform.Jul 13 2018, 8:17 PM
JoshStrobl requested changes to this revision.Jul 20 2018, 1:59 PM
JoshStrobl added a subscriber: JoshStrobl.

Additionally completions from https://github.com/hashicorp/terraform/tree/master/contrib, namely fish and zsh should be installed.

package.yml
23

No quotes are needed. Should also be -Dm00755 so we're ensuring there is no uid and guid bits set.

This revision now requires changes to proceed.Jul 20 2018, 1:59 PM
der_eismann added a comment.Jul 20 2018, 2:10 PM

fish completions:

Please note that these completions have been merged upstream and should be bundled with fish 2.6 or later.

We already have 2.7.0 and it is indeed included.

zsh completions:
Was removed 4 months ago.

JoshStrobl added a comment.Jul 20 2018, 2:12 PM

Well okay then, not sure why they'd bother to have either of those folders in the first place :D

der_eismann updated this revision to Diff 8293.Jul 20 2018, 2:14 PM

Change install command

der_eismann added a comment.Jul 20 2018, 2:15 PM

I have no idea why they still have those folders :D But good idea to include it.

der_eismann marked an inline comment as done.Jul 20 2018, 2:15 PM

Done done done.

JoshStrobl accepted this revision.Jul 20 2018, 2:48 PM
This revision is now accepted and ready to land.Jul 20 2018, 2:48 PM
Closed by commit R4354:45aa5e5d8226: Initial inclusion of terraform (authored by der_eismann, committed by JoshStrobl). · Explain WhyJul 20 2018, 2:49 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
1 line
2 lines
22 lines
36 lines

Diff 8294

View Options

Makefile

Loading...
View Options

abi_used_libs

Loading...
View Options

package.yml

Loading...
View Options

pspec_x86_64.xml

Loading...
Copyright © 2015-2021 Solus Project. The Solus logo is Copyright © 2016-2021 Solus Project. All Rights Reserved.