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

Add airnode-deployer rollback command #1170

Merged
merged 2 commits into from
Jan 24, 2023
Merged

Add airnode-deployer rollback command #1170

merged 2 commits into from
Jan 24, 2023

Conversation

amarthadan
Copy link
Contributor

Close #1167

@amarthadan amarthadan requested review from wkande and dcroote January 23, 2023 13:09
@amarthadan amarthadan self-assigned this Jan 23, 2023
@github-actions
Copy link

github-actions bot commented Jan 23, 2023

Visit the preview URL for this PR (updated for commit 3125ea1):

https://api3-docs--pr1170-deployer-rollback-u511ckx0.web.app

(expires Tue, 31 Jan 2023 07:14:11 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d319eb7cbe610768342d5dc1b5cd1f3aed912886

@wkande wkande added ★ migrate Need to be added to VitePress-docs v0.10 labels Jan 23, 2023
Copy link
Contributor

@dcroote dcroote left a comment

Choose a reason for hiding this comment

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

Tested and it works. Two small nits

docs/airnode/v0.10/reference/packages/deployer.md Outdated Show resolved Hide resolved

Positionals:
deployment-id ID of the deployment (from 'list' command) [string] [required]
version-id ID of the deployment version (from 'info' command) [string] [required]
Copy link
Contributor

Choose a reason for hiding this comment

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

I wouldn't say it's critical, but the version-id argument would be clearer if it were slightly more explicit:

ID of the deployment version to rollback to

(Applies to api3dao/airnode as well...)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, I'll add it 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated the help messages for all the commands and added a commit to this PR. Hope you don't mind.

Copy link
Contributor

@dcroote dcroote left a comment

Choose a reason for hiding this comment

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

Quick turnaround from the Airnode PR! LGTM

@amarthadan amarthadan merged commit 7e6fb06 into main Jan 24, 2023
@amarthadan amarthadan deleted the deployer-rollback branch January 24, 2023 11:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
★ migrate Need to be added to VitePress-docs ✔︎ mIgrated v0.10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add rollback functionality for Airnode deployments
3 participants