Conversation
chore: create Makefile and script to build and push Docker image refactor: change name of some env vars
luislhl
force-pushed
the
chore/pre-deploy-adjustments
branch
from
9fee1c3 to
048a7d0
Compare
msbrogli
approved these changes
Jul 9, 2021
| fi; | ||
|
|
||
| echo $DOCKER_IMAGE_TAG; | ||
| aws ecr get-login-password --region eu-central-1 | docker login --username AWS --password-stdin $AWS_ACCOUNT_ID.dkr.ecr.eu-central-1.amazonaws.com; |
Member
There was a problem hiding this comment.
Should we exit in case one of these commands fail? We can use something like this:
{ cmd } || { echo "error"; exit -1; }}
Collaborator
Author
There was a problem hiding this comment.
Good point.
I've added this to the script, to assure that:
set -e
set -o pipefail
Do you think it's enough?
andreabadesso
pushed a commit
that referenced
this pull request
Sep 30, 2021
* fix: broken Dockerfile chore: create Makefile and script to build and push Docker image refactor: change name of some env vars * refactor: Make sure the script fails if a command fails
andreabadesso
added a commit
that referenced
this pull request
Sep 30, 2021
* feat: initial commit * chore: removed .env from git * chore: removed size.yml github action (came from tsdx template) * fix: added missing attributes to PreparedTx type * feat: typed yield result from generator * fix: token creation tx was not sending token_name and token_symbol * fix: send transactions in topological order (#9) * fix: send transactions in topological order (using the timestamp to order) * tests: removed extra spentBy on block api response * feat: handle reorg on the daemon (#7) * chore: empty commit * chore: empty commit * feat: invoking onHandleReorgRequest on reorg state * fix: possible infinite loop on circular parents on transactions * feat: handling reorg on generator yield * chore: logging errors properly * chore: passing eslint * refactor: logging reorg fail as error * docs: added comment explaining the seen list on the recursivelyDownloadTxs method * tests: fixed tests * refactor: setting state as reorg when our best block is not found on the wallet-service * refactor: returning the lambda invoke on invokeReorg and sendTx methods * chore: added more details to error log on lambda call * refactor: sending transactions before blocks (#12) * feat: sync mempool (#11) feat: sync mempool * fix: using decoded to get token uid (#14) * chore: pre-deploy adjustments (#13) * fix: broken Dockerfile chore: create Makefile and script to build and push Docker image refactor: change name of some env vars * refactor: Make sure the script fails if a command fails * fix: ignoring NFT transaction outputs with undecoded scripts (#15) * feat: validating tx outputs before sending transactions * refactor: ignoring output instead of whole transaction * chore: logging ignored tx output index * feat: websocket conn error log (#17) * chore: upgraded hathor-wallet-lib to 0.20.3 * feat: logging error messages on websocket connection * refactor: logging connection attempts as INFO instead of ERROR * fix: send height on transactions (#18) * chore: updated lodash * chore: improved logs * fix: sending height on every block transaction * refactor: send NFT outputs to wallet-service (#20) * refactor: invalid nft output is now being validated on the wallet-service * fix: lint passing (#21) * chore: passing lint * chore: added eslintrc to automatically detect carriage returns * chore: passing NODE_OPTIONS as env variable globally on workflow * chore: using generated eslintrc.js * fix: validating decoded data before trying to get the token data (#22) * fix: validating decoded data before trying to get the token data * chore: lint * ci: configure automated deployment (#19) * ci: configure automated deploy * send message to Slack * send message only in case of deploy * feat: added ALERT string to critical failure logs (#23) * fix: deploy of docker images (#24) * chore: added log on connection error (to fullnode) (#25) * chore: bumped to v1.0.0-alpha (#28) Co-authored-by: André Carneiro <andreluizmrcarneiro@gmail.com> Co-authored-by: Luis Helder <luislhl@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was done