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

Grid Stack docker compose update for dashboard suite flist #59

Closed
wants to merge 11 commits into from

Conversation

mik-tf
Copy link
Contributor

@mik-tf mik-tf commented Jul 24, 2024

Work Done

  • adjusted the deployment process so it works with micro VM for flist on tfgrid
  • Set tar with pv to get a status of the progress
    • Then we get status from rsync (already happening) and from tar (this PR)
  • we print the process to log file, this is used on flist to show a wepage of the progress
  • Detail: links for networks are e.g. mainnet, but for qanet it was qa_net, so I updated this too

Status

Detail

  • The files are in xz not gzip, despite the gz file extension

Related Work

References

@mik-tf mik-tf marked this pull request as draft July 24, 2024 16:09
@mik-tf mik-tf marked this pull request as ready for review July 24, 2024 19:45
@coesensbert
Copy link
Contributor

Thanks a lot for the effort. I went over the commits which look fine only that once merged would break the current flow. We have currently no grid backend stacks running on the grid or in a vm, all are bare metal machines setup exactly the same. We want to keep it this way since only changing a path for example, involves a lot of work making changes on all the live deployments. What is in this repo is also live in production.

Since also changes like using zinit are in commits, I would suggest to make a separate branch (or some other git magic) to keep the code for our bare metal deployments separate from flist compatible code. Although I'm not sure how to do that.

Since this pull request we also made changes to extracting commands, so we would need to go over integrating pv again

@mik-tf
Copy link
Contributor Author

mik-tf commented Aug 5, 2024

Thanks for the feedback Bert.

So finally we won't use the flist images + dashboard apps, thus this PR is not necessary. We will do everything with docker compose, as we have already started to work on.

I will update the grid deployment project specs with this so it's clear. So I will close this PR and others that are not relevant anymore.

@mik-tf mik-tf closed this Aug 5, 2024
@mik-tf mik-tf deleted the development_pv_2 branch August 5, 2024 16:07
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