Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial action to build containers for tag pushes #256

Merged
merged 2 commits into from
Feb 17, 2022
Merged

Conversation

MCrank
Copy link
Contributor

@MCrank MCrank commented Feb 17, 2022

Soundtrack of this PR: Fantasy Ambient

Motivation

Currently there is no automated build of Docker images for Full-Service so updates lag behind the code. This adds a github action to build and publish docker images when a tag is pushed

In this PR

  • Adds a github action workflow to build docker images for testnet and mainnet.
  • Currently only builds images when a tag is pushed.
  • Added the -testnet and -mainnet suffixes to the images for better clarification.
  • Commented out a tests job placeholder for Mikey who is currently working on some stress tests.
    • Defined the develop and feature/** filters so when the tests are put in they should run for any push to those branches. This is also just a placeholder and should be adjusted when the test action is created if need be for any additional branches (main?)

Future Work

  • Would like to actually perform the build of the binaries as well in the future to apply more automation around CI/CD

@MCrank
Copy link
Contributor Author

MCrank commented Feb 17, 2022

@briancorbin Just double checking some things and realized I put the wrong secret name. Just pushed an update to fix that error.

LIne 64

@MCrank MCrank merged commit 2c96964 into develop Feb 17, 2022
@MCrank MCrank deleted the action-docker-ci branch February 17, 2022 19:56
briancorbin added a commit that referenced this pull request Mar 4, 2022
* adding max limit to some API endpoints (#248)

* Remove unnecessary foreign keys from gift_codes table. (#249)

* Update readme installation instructions (#250)

* Enable foreign key constraints. Fix transient FK errors when deleting an account. (#251)

* Remove foreign key check before running migrations. This allows databases with existing foreign key errors to be fixed by the migrations. (#252)

* Fix a bug causing sync to create many tiny chunks. (#253)

* Remove target block arg from manually sync account fn (#255)

* Initial action to build containers for tag pushes (#256)

* Supporting FOG enabled accounts (#254)

* FOG Creds default to empty string if not provided from API (#257)

* update readme (#259)

* Api key guard (#205)

* fixing issue with ledger not syncing automatically (#261)

* sync up to the last block instead of excluding it (#262)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants