Allow to configure Terraform State#2438
Conversation
|
@sachilles @sameersbn is there any chance this PR will be accepted in the near future? |
|
@peter-jozsa I think so. But I'll need some time to understand the details in your PR (and get warm with Terraform). |
|
@peter-jozsa Sorry for my late reply. I will include your PR before the #2465 . However, I will not be able to check it until the weekend or next week. |
|
alright, thank you! anyway in the end we ended up not using terraform state management but we are planning to use it in the near future |
|
Hi, I think that for now I will rather update GitLab to stay up to date with the upstream. |
|
Hi, this feature would be wonderful for people using terraform. As of now, the state is stored using the tmp local folder which does not seem very safe. Any change to merge it soon? Or at least to change default terraform state storage location to something like |
|
Hi everyone, @sachilles are there any chances to have this one merged ? |
a9f0df2 to
06cbb56
Compare
|
hi 👋 |
bd655de to
82e88ff
Compare
|
thanks a lot @peter-jozsa |
|
Hi, I'll do my very best to understand the terraform configuration. |
This feature would be great! Hope to see this merged soon! Thank you for your work! |
|
@sachilles are we still in time to have this one merged before 14.10 ? |
|
@sachilles sorry for chasing, are you planning to merge this one for the next release ? |
|
Hi, I'm planning to merge this PR. Please be patient. I think I will get to the final review tonight. |
sachilles
left a comment
There was a problem hiding this comment.
Thanks for this very valuable post and sorry for the long wait (and subsequent rework). Nevertheless, could you include the small changes in the documentation?
|
|
||
| ##### `GITLAB_TERRAFORM_STATE_OBJECT_STORE_CONNECTION_AWS_PATH_STYLE` | ||
|
|
||
| Changes AWS Path Style to 'host/bucket_name/object' instead of 'bucket_name.host/object'. Defaults to `AWS_PATH_STYLE` |
There was a problem hiding this comment.
Please change AWS_PATH_STYLE to $AWS_PATH_STYLE
README.md
Outdated
|
|
||
| ##### `GITLAB_TERRAFORM_STATE_OBJECT_STORE_CONNECTION_PROVIDER` | ||
|
|
||
| Connection Provider for the Object Store. (`AWS` or `Google`) Defaults to `$GITLAB_OBJECT_STORE_CONNECTION_PROVIDER` (`AWS`) |
There was a problem hiding this comment.
Please change this sentence to:
Connection Provider for the Object Store (AWS or Google). Defaults to $GITLAB_OBJECT_STORE_CONNECTION_PROVIDER (i.e. AWS).
Add proposed changes (see sameersbn#2438).
The goal of this PR is to introduce new configuration variables for Terraform State management feature of GitLab. The new params let users:
AWSbucket for Object StoreGCSbucket for Object Store