Closed
Bug 1474682
Opened 8 years ago
Closed 8 years ago
Submit for review and merge unpublished terraform changes
Categories
(Release Engineering :: General, enhancement)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dividehex, Assigned: garbas)
Details
/cc fubar, jlund and sylvestre
Hi Rok,
I think you might have unpublished changes to the build cloud tools terraform configs that are blocking this PR: https://github.com/mozilla-releng/build-cloud-tools/pull/344
Therefore, I can't merge and apply the PR without destroying the uncommited/unpublished aws resources you may have setup outside of the master branch.
Can you please publish this code for review so it can be merged and we can unblock this PR.
Thanks,
Jake
<snippet>
An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
destroy
Terraform will perform the following actions:
aws_route53_record.heroku-mapper-cname-prod
aws_route53_record.heroku-tokens-cname-prod
aws_route53_record.heroku-tokens-cname-stage
aws_route53_record.heroku-tokens-cname-test
Plan: 0 to add, 0 to change, 4 to destroy.
| Reporter | ||
Updated•8 years ago
|
Flags: needinfo?(rgarbas)
Updated•8 years ago
|
Flags: needinfo?(jlund)
| Reporter | ||
Comment 1•8 years ago
|
||
If this becomes urgent and we need to move forward without the missing config resources, it is possible to remove the 4 resources from the terraform state file while leaving them intact in aws. This would cause terraform to "forget" about the records and not attempt to destroy them. In order to pull them back into the state file so that terraform can manage them again after the fact, they would either need to be destroyed (via the aws console) or imported back into the state file with terraform import option. Either way the missing resources would need to be defined in the config and merged to master.
| Assignee | ||
Comment 2•8 years ago
|
||
:dividehex
looks i forgot to push last week since i was in a hurry to finish relengapi migration. i will make sure everything is pushed so we can merge https://github.com/mozilla-releng/build-cloud-tools/pull/344
Flags: needinfo?(rgarbas)
| Assignee | ||
Comment 3•8 years ago
|
||
This was fixed in https://github.com/mozilla-releng/build-cloud-tools/pull/349
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(jlund)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•