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.

Sign up for GitHub

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

Jump to bottom

feat: docker support for mainnet validator #197

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat: docker support for mainnet validator #197

wants to merge 6 commits into from

Conversation

nghuyenthevinh2000
Copy link
Contributor

@nghuyenthevinh2000 nghuyenthevinh2000 commented Mar 30, 2023 •
edited
Loading

Summary of changes

Add docker image to run on mainnet

My node has been running for 2 days without problem

Screenshot 2023-04-04 at 11 17 09

Report of required housekeeping

  • Github issue OR spec proposal link
  • Wrote tests
  • Updated API documentation (client/lcd/swagger-ui/swagger.yaml)
  • Added a relevant changelog entry: clog add [section] [stanza] [message]

(FOR ADMIN) Before merging

  • Added appropriate labels to PR
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • Confirm added tests are consistent with the intended behavior of changes
  • Ensure all tests pass

@nghuyenthevinh2000 nghuyenthevinh2000 marked this pull request as ready for review April 4, 2023 04:16
@inon-man inon-man added the enhancement New feature or request label Jun 21, 2023
@inon-man inon-man changed the title Mainnet docker feat: docker support for terra operator Jun 21, 2023
@nghuyenthevinh2000
Copy link
Contributor Author

please leave this here as it needs to update images to latest core

@nghuyenthevinh2000 nghuyenthevinh2000 changed the title feat: docker support for terra operator feat: docker support for mainnet validator Jul 7, 2023
@weegerry
Copy link

Does anyone have any idea how the containerised terrad handled the IBC attacks? Was there any noticable downside to running the container over a server install?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inon-man inon-man inon-man approved these changes

@fragwuerdig fragwuerdig Awaiting requested review from fragwuerdig

@alchemist-ti alchemist-ti Awaiting requested review from alchemist-ti

Assignees

@nghuyenthevinh2000 nghuyenthevinh2000

Labels
enhancement New feature or request
Projects
No open projects
L1 Q3 Sprint
Status: 👀 In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nghuyenthevinh2000 @weegerry @inon-man