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

add a make target to retrieve the blockstore and prometheus data store #11

Merged
merged 7 commits into from
Sep 8, 2022

Conversation

williambanfield
Copy link
Contributor

@williambanfield williambanfield commented Sep 7, 2022

places the blockstore.db and the prometheus data into a directory in a new experiments directory. Each new fetch generates a new directory in the experiments directory of the current time, e.g. experiments/2022-09-07-13_21_59595212278.

Makefile Outdated Show resolved Hide resolved
@@ -45,6 +45,14 @@ runload:
--ip-list `grep ' name=' ./ansible/hosts | cut -d' ' -f1 | paste -s -d, -` \
--seed-delta $(shell echo $$RANDOM)

retrieve-data:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Question:
IIUC, we are retrieving the experiment data into our laptop.
Then, we would have to transfer it to the DO "space", where we decided we'd keep it.
Looks suboptimal to me, but, on the other hand, as we have two different DO accounts (Informal and IG), I don't see a more efficient way of doing this.
So, just a reflection

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At the moment, I'm not sure the easiest way to get the data to the DO storage. IMO, we should store it in a digital ocean block store, which they call spaces. Spaces allow upload of data, but I'm not currently seeing an easy way to upload to them using ansible. You need credentials to upload to them as well, so the credentials need to be present on the machine where the data is generated. Perhaps just ssh, scp the credentials onto the machine and run doctl ...?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah, that's what I was thinking, on top of the fact that we have two DO accounts, and probably we want to have all data in both for convenience, so I don't see how we would do that other than manually

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Created an issue in this repo to track
#14

Makefile Outdated
retrieve-data:
@DIR=`date "+%Y-%m-%d-%H_%M_%S%N"`; \
mkdir -p "./experiments/$${DIR}"; \
cd ansible && ansible-playbook -i hosts -u root retrieve-blockstore.yaml --limit `ansible -i hosts --list-hosts validators | tail -1| sed 's/ //g'` -e "dir=../experiments/$${DIR}/blockstore.db.zip"; \
Copy link
Collaborator

@sergio-mena sergio-mena Sep 7, 2022

Choose a reason for hiding this comment

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

I'm testing the value of the --limit argument, and it outputs the last of the validators. Is this what you want to do?
If yes, I'd retrieve everything, in case the chosen node was late... AFAIU, disk space in "DO spaces" is cheap

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated to fetch all. It now places each in a directory under the experiments dir, of the format ./experiments/<date>/seed01/blockstore.db.zip.

@williambanfield williambanfield merged commit 6fce11a into main Sep 8, 2022
@williambanfield williambanfield deleted the wb/retrieve-blockstore branch September 8, 2022 13:47
@williambanfield williambanfield restored the wb/retrieve-blockstore branch September 9, 2022 16:12
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