Skip to content

[LAB-679] trying to fix compute in docker issue with CI#703

Merged
alabdao merged 35 commits intomainfrom
ops/LAB-679-fix-compute-in-docker
Oct 19, 2023
Merged

[LAB-679] trying to fix compute in docker issue with CI#703
alabdao merged 35 commits intomainfrom
ops/LAB-679-fix-compute-in-docker

Conversation

@alabdao
Copy link
Copy Markdown
Contributor

@alabdao alabdao commented Oct 16, 2023

No description provided.

@linear
Copy link
Copy Markdown

linear bot commented Oct 16, 2023

LAB-679 Fix compute in docker issue in CI

If we mount the /root/ directly inside compute we can potentially get around this issue in CI.

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Oct 19, 2023 6:39pm

@alabdao alabdao temporarily deployed to ci October 16, 2023 14:18 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:18 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:18 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:18 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:18 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:21 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:33 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:33 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:34 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:34 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:34 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:38 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:58 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:58 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:58 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:58 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 14:58 — with GitHub Actions Inactive
@alabdao alabdao temporarily deployed to ci October 16, 2023 15:01 — with GitHub Actions Inactive
@alabdao
Copy link
Copy Markdown
Contributor Author

alabdao commented Oct 19, 2023

Currently private part fails cuz of code changes.
This is ready to be merged.

Copy link
Copy Markdown

@thetechnocrat-dev thetechnocrat-dev left a comment

Choose a reason for hiding this comment

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

Is this suppose to point to local host?

Screenshot 2023-10-19 at 2 23 54 PM


ipfs:
image: "ipfs/kubo"
image: "ipfs/kubo:v0.18.0"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💯

name: docker-compose-plugin-${{matrix.os}}
path: ./docker-compose
# # Only downloading cli for macos
# ci-setup-compose:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

How is the CI using docker compose now that this commented out?

Also, why did you comment out this code instead of just removing it?

Copy link
Copy Markdown
Contributor Author

@alabdao alabdao Oct 19, 2023

Choose a reason for hiding this comment

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

How is the CI using docker compose now that this commented out?

ubuntu nodes already have docker compose I was mistaken installing it when it wasn't needed. So removed it as it was redundant.

Also, why did you comment out this code instead of just removing it?

I'm thinking in the future there maybe testing on macos-13, if I can get it to work. So I left it in commented out for macos-13 future testing as its not needed for linux host.

Copy link
Copy Markdown

@thetechnocrat-dev thetechnocrat-dev left a comment

Choose a reason for hiding this comment

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

Feel free to merge so you are not blocked, but had some questions above.

@alabdao
Copy link
Copy Markdown
Contributor Author

alabdao commented Oct 19, 2023

Is this suppose to point to local host?
Screenshot 2023-10-19 at 2 23 54 PM

That's meant to print which BACALHAU_API_HOST is pointing to just as additional debug setup/

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