Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Adds disaster recovery tooling for the terraform promotion script #542

Merged
merged 3 commits into from
May 6, 2022

Conversation

tcsc
Copy link
Contributor

@tcsc tcsc commented May 5, 2022

Adds tooling to locally run promotions to the terraform registry in the event of errors in the Drone process.

Expected process looks something like:

 $ (cd tooling; go build -o .. ./cmd/with-secrets)
 $ sops -d $plugins_secrets | ./with-secrets ./tooling/bin/release $terraform_provider_version

...substituting the appropriate $plugins_secrets Drone secrets file and $terrafom_provider_version tag (e.g. terraform-provider-teleport-v9.1.3)

See tooling/cmd/with-secrets/main.go for more info on why with-secrets is required.

Adds tooling to locally run promotions to the terraform registry in the
event of errors in the Drone process.

Expected process is expected to look something like:

```
 $ (cd tooling; go build -o .. ./cmd/with-secrets)
 $ sops -d $plugins_secrets | ./with-secrets ./tooling/bin/release $terraform_provider_version
```

...substituting the appropriate `$plugins_secrets` Drone secrets file
and `$terrafom_provider_version` tag (e.g. `terraform-provider-teleport-v9.1.3`)
@tcsc tcsc requested a review from wadells May 5, 2022 03:05
Copy link
Contributor

@wadells wadells left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected process looks something like:

 $ (cd tooling; go build -o .. ./cmd/with-secrets)
 $ sops -d $plugins_secrets | ./with-secrets ./tooling/bin/release $terraform_provider_version

...substituting the appropriate $plugins_secrets Drone secrets file and $terrafom_provider_version tag (e.g. terraform-provider-teleport-v9.1.3)

Could we capture these instructions in a README.md, or maybe RELEASE.md? I see value in the scripts as a template for how things happen, but I'm concerned without context, only you or I would know what to do with them.

Also: It'd be worth having pointers in both these scripts and the drone logic to each other, to help increase the chance they stay in sync. I expect updates will get made to one, but not the other eventually. Any quick edits to decrease that likelihood is time well invested IMO.

@tcsc
Copy link
Contributor Author

tcsc commented May 5, 2022

Could we capture these instructions in a README.md, or maybe RELEASE.md

That was my plan. I was trying to figure out where best to put them while this PR was under review. @Joerger - would me just adding a top-level RELEASE.md conflict with your #440 at all?

@wadells
Copy link
Contributor

wadells commented May 6, 2022

Could we capture these instructions in a README.md, or maybe RELEASE.md

That was my plan. I was trying to figure out where best to put them while this PR was under review. @Joerger - would me just adding a top-level RELEASE.md conflict with your #440 at all?

Or maybe chuck a commit into joerger/teleport-plugins-release-strategy. I'm ok if the docs are added in a separate PR.

Copy link
Contributor

@Joerger Joerger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding a RELEASE.md seems useful, and I can just add additional notes to it in #440. Up to you!

@tcsc
Copy link
Contributor Author

tcsc commented May 6, 2022

I've created a bare-bones RELEASE.md for now.

@tcsc tcsc merged commit c773042 into master May 6, 2022
@tcsc tcsc deleted the tcsc/release-tooling branch May 6, 2022 02:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants